From c1b27579ca791d3728f211c862448c4398f3e44f Mon Sep 17 00:00:00 2001 From: Daniel Ang Date: Fri, 18 Jun 2021 12:43:17 +0200 Subject: [PATCH 1/7] Method json() to parse JSON strings - Parse a JSON string into array - Test --- src/Expectation.php | 11 +++++++++++ tests/Features/Expect/json.php | 14 ++++++++++++++ 2 files changed, 25 insertions(+) create mode 100644 tests/Features/Expect/json.php diff --git a/src/Expectation.php b/src/Expectation.php index 45b52891..3c238805 100644 --- a/src/Expectation.php +++ b/src/Expectation.php @@ -60,6 +60,17 @@ final class Expectation return new self($value); } + /** + * Parses Json String to Array. + */ + public function json(): Expectation + { + Assert::assertIsString($this->value); + Assert::assertJson($this->value); + + return new self(json_decode($this->value, true)); + } + /** * Dump the expectation value and end the script. * diff --git a/tests/Features/Expect/json.php b/tests/Features/Expect/json.php new file mode 100644 index 00000000..2228fd56 --- /dev/null +++ b/tests/Features/Expect/json.php @@ -0,0 +1,14 @@ +json() + ->name + ->toBe('Nuno'); +}); + +test('fails with broken json string', function () { + expect('{":"Nuno"}')->json(); +})->throws(ExpectationFailedException::class); From ecb37fce91d6ddcf55bf9d5d6c4176127340ad35 Mon Sep 17 00:00:00 2001 From: Daniel Ang Date: Fri, 18 Jun 2021 13:04:31 +0200 Subject: [PATCH 2/7] Refactoring to use toBeJson() Refactoring according to @lukeraymonddowning suggestion --- src/Expectation.php | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/Expectation.php b/src/Expectation.php index 3c238805..e2d6369b 100644 --- a/src/Expectation.php +++ b/src/Expectation.php @@ -65,10 +65,7 @@ final class Expectation */ public function json(): Expectation { - Assert::assertIsString($this->value); - Assert::assertJson($this->value); - - return new self(json_decode($this->value, true)); + return $this->toBeJson()->and(json_decode($this->value, true)); } /** From 4bf69b97bdb20844b36f902535ba9a8eb2771f19 Mon Sep 17 00:00:00 2001 From: Nuno Maduro Date: Fri, 18 Jun 2021 12:06:41 +0100 Subject: [PATCH 3/7] feat(json): updates docs --- src/Expectation.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Expectation.php b/src/Expectation.php index e2d6369b..85cb0094 100644 --- a/src/Expectation.php +++ b/src/Expectation.php @@ -61,7 +61,7 @@ final class Expectation } /** - * Parses Json String to Array. + * Creates a new expectation with the decoded JSON value. */ public function json(): Expectation { From 3afdedbd3f3e486483073fc0cacf87fb3ada454a Mon Sep 17 00:00:00 2001 From: Daniel Ang Date: Fri, 18 Jun 2021 13:11:37 +0200 Subject: [PATCH 4/7] Snapshots --- tests/.snapshots/success.txt | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/.snapshots/success.txt b/tests/.snapshots/success.txt index 05d2cc3a..422bfc2f 100644 --- a/tests/.snapshots/success.txt +++ b/tests/.snapshots/success.txt @@ -140,6 +140,10 @@ ✓ it macros true is true with argument ✓ it macros false is not true with argument + PASS Tests\Features\Expect\json + ✓ it properly parses json string + ✓ fails with broken json string + PASS Tests\Features\Expect\not ✓ not property calls @@ -569,5 +573,5 @@ ✓ it is a test ✓ it uses correct parent class - Tests: 4 incompleted, 7 skipped, 355 passed + Tests: 4 incompleted, 7 skipped, 357 passed \ No newline at end of file From db9f1254b5f40d20525b3bef096fb4c03bd32c6e Mon Sep 17 00:00:00 2001 From: luke Date: Fri, 18 Jun 2021 12:30:57 +0100 Subject: [PATCH 5/7] Tags Pest 1.6 --- src/Pest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Pest.php b/src/Pest.php index 759cb913..847a15fc 100644 --- a/src/Pest.php +++ b/src/Pest.php @@ -6,7 +6,7 @@ namespace Pest; function version(): string { - return '1.5.0'; + return '1.6.0'; } function testDirectory(string $file = ''): string From 9b34650e7297c2c506bc88d7528e4e4aba132a77 Mon Sep 17 00:00:00 2001 From: luke Date: Fri, 18 Jun 2021 12:36:43 +0100 Subject: [PATCH 6/7] Adds changelog for 1.6.0 --- CHANGELOG.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index ee5a3af9..ab56a6b0 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file. The format is based on [Keep a Changelog](http://keepachangelog.com/) and this project adheres to [Semantic Versioning](http://semver.org/). +## [v1.6.0 (2021-06-18)](https://github.com/pestphp/pest/compare/v1.5.0...v1.6.0) +### Added +- Adds a new `json` expectation method to improve testing with JSON strings ([#325](https://github.com/pestphp/pest/pull/325)) +- Adds dot notation support to the `toHaveKey` and `toHaveKeys` expectations ([#322](https://github.com/pestphp/pest/pull/322)) + ## [v1.5.0 (2021-06-15)](https://github.com/pestphp/pest/compare/v1.4.0...v1.5.0) ### Changed - Moves plugins from the `require` section to the core itself ([#317](https://github.com/pestphp/pest/pull/317)), ([#318](https://github.com/pestphp/pest/pull/318)), ([#320](https://github.com/pestphp/pest/pull/320)) From b6f0496c3c071a897c054be449d98a74ef850dd4 Mon Sep 17 00:00:00 2001 From: Nuno Maduro Date: Fri, 18 Jun 2021 12:39:05 +0100 Subject: [PATCH 7/7] chore: updates funding --- .github/FUNDING.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/FUNDING.yml b/.github/FUNDING.yml index 341f601a..8b8b5939 100644 --- a/.github/FUNDING.yml +++ b/.github/FUNDING.yml @@ -2,4 +2,3 @@ github: [nunomaduro,owenvoke,olivernybroe,octoper] patreon: nunomaduro -custom: https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=66BYDWAT92N6L