-
Notifications
You must be signed in to change notification settings - Fork 202
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
HmRsBody matcher for checking response body #485
Comments
@carlosmiranda @davvd ready to take this |
@yegor256 what do you think? |
@yegor256 ping |
@carlosmiranda attached this issue to milestone "1.0" (let me know if this is wrong) |
@carlosmiranda thanks for the report, I topped your acc for 15 mins, payment ID |
@rfqu this task is for you, keep these principles in mind, and don't hesitate to ask any technical questions; 30 mins is the budget of the task. This is exactly how much will be paid when the problem is solved (no matter how much time you will actually spend). See this for more information |
I am almost done, but one of new tests fails because of the issue #524. |
@rfqu keep in mind that you should address your messages to someone, by starting them with the name of the person you're talking to. otherwise, it's just noise, nobody will pay attention to what you're saying. check this out: http://www.yegor256.com/2014/11/24/principles-of-bug-tracking.html#4.-avoid-noise-—-address-your-comments |
@rfqu the ticket is with you for 15 days already. If it is not finished in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle, this article should help if you're stuck. added -30 to your rating, now it is equal to -30 |
@davvd @yegor256 @carlosmiranda fixed - see pull request #545 |
@rfqu let's wait for merge. |
@rfqu thanks, I'll take a look |
@rfqu you've been working with this task for too long. I'm sorry, but I'll assign someone else. Please stop working with it. See our no obligations principle... added -60 to your rating, now it is equal to -75 |
@danff please proceed, it's yours |
@davvd need more time |
@0crat in |
@caarlos0 this job was assigned to you 8 days ago. |
Oops! Job |
2 similar comments
Oops! Job |
Oops! Job |
@t-izbassar The impediment for #485 was registered successfully by @t-izbassar/z |
@carlosmiranda can you close this one? #785 was merged and puzzles created. |
@t-izbassar thanks! |
Order was successfully finished: +30 points just awarded to @t-izbassar/z, total is +225 |
The job #485 is now out of scope |
At the moment we have
HmRsHeader
andHmRsStatus
which allows us to use Hamcrest to match againstResponse
headers or status codes, respectively.It would be good if we have a class
HmRsBody
that will allow us to match theResponse
body as well. Let's create one, with public constructors that can acceptMatcher<String>
andMatcher<byte[]>
, to support both text and binary responses.The text was updated successfully, but these errors were encountered: