Skip to content

Do not cast integer as string ids + Fix Zero ID + PHP Coduo #93

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
May 6, 2023
Merged

Do not cast integer as string ids + Fix Zero ID + PHP Coduo #93

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
fbd0ad8
Do not cast integer as string ids
yoanm May 5, 2023
7149d10
Replace PHPUnit exact match with PHP-coduo
yoanm May 5, 2023
3b0fd12
Allow for zero IDs; notifications are only NULL
bradjones1 Jul 2, 2021
3b594ab
Fix https://github.com/yoanm/php-jsonrpc-server-sdk/issues/94
yoanm May 6, 2023
2955c9a
Remove useless empty lines
yoanm May 6, 2023
746ae8d
Fix lint
yoanm May 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Replace PHPUnit exact match with PHP-coduo
  • Loading branch information
@yoanm
yoanm committed May 5, 2023
commit 7149d10b8d0c5cb3db2503d8869cda6f1ad7af9c
1 change: 1 addition & 0 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
},
"require-dev": {
"behat/behat": "^3.9.0",
"coduo/php-matcher": "^6.0",
"dvdoug/behat-code-coverage": "^5.0",
"phpspec/prophecy": "^1.15",
"phpspec/prophecy-phpunit": "^2.0",
Expand Down
7 changes: 4 additions & 3 deletions features/bootstrap/FeatureContext.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
use Behat\Behat\Context\Environment\InitializedContextEnvironment;
use Behat\Behat\Hook\Scope\BeforeScenarioScope;
use Behat\Gherkin\Node\PyStringNode;
use DemoApp\Dispatcher\BehatRequestLifecycleDispatcher;
use Coduo\PHPMatcher\PHPUnit\PHPMatcherAssertions;
use PHPUnit\Framework\Assert;
use Tests\Functional\BehatContext\Helper\FakeEndpointCreator;

Expand All @@ -13,6 +13,8 @@
*/
class FeatureContext extends AbstractContext
{
use PHPMatcherAssertions;

const KEY_JSON_RPC = 'jsonrpc';
const KEY_ID = 'id';
const KEY_RESULT = 'result';
Expand Down Expand Up @@ -75,8 +77,7 @@ public function thenLastResponseShouldBeAValidJsonRpcError()
*/
public function thenIShouldHaveTheFollowingResponse(PyStringNode $expectedResult)
{
// Decode content to get rid of any indentation/spacing/... issues
Assert::assertEquals(
$this->assertMatchesPattern(
$this->jsonDecode($expectedResult->getRaw()),
$this->getLastResponseDecoded()
);
Expand Down