-
Notifications
You must be signed in to change notification settings - Fork 155
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.
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
Phpunit tests for Symfony read listener #782
Phpunit tests for Symfony read listener #782
Conversation
4ab1e45
to
b545768
Compare
b545768
to
f98f14b
Compare
|
||
$this->operationInitiator->initializeOperation($request)->willReturn($operation); | ||
|
||
$request->attributes = $attributes; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible or makes sense to prophesize attributes directly to $request->attributes
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nope, we cannot do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We also did like that on the old spec file.
f98f14b
to
3163781
Compare
This PR was merged into the 1.11 branch. Discussion ---------- | Q | A | --------------- | ----- | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Related tickets | | License | MIT Based on #782 Commits ------- 45d5b25 Phpunit tests for Symfony read listener 3163781 Improve test of read listener 345475e Init read provider
Based on #780