Skip to content
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

PsGithub.java:54-56: There is currently no unit... #243

Closed
davvd opened this issue May 2, 2015 · 14 comments
Closed

PsGithub.java:54-56: There is currently no unit... #243

davvd opened this issue May 2, 2015 · 14 comments

Comments

@davvd
Copy link

davvd commented May 2, 2015

Puzzle 229-401562da in src/main/java/org/takes/facets/auth/social/PsGithub.java:54-56 has to be resolved: There is currently no unit test in PsGithubTest that checks that the request parameters are passed to the request body, instead of the URI. Let's add this test.... The puzzle was created by cyberone on 1-May-2015.

If you have any technical questions, don't ask me, submit new tickets instead

@davvd
Copy link
Author

davvd commented May 2, 2015

@dmzaytsev the task is yours,please go ahead

@dmzaytsev
Copy link
Contributor

@davvd I will resolve the puzzle in the PR #228

@davvd davvd added this to the 1.0 milestone May 2, 2015
@davvd
Copy link
Author

davvd commented May 2, 2015

@yegor256 since there is no milestone yet I set it to "1.0"

@dmzaytsev
Copy link
Contributor

@davvd we are wating for #253 here

@yegor256
Copy link
Owner

yegor256 commented May 6, 2015

@davvd waiting for #253

@dmzaytsev
Copy link
Contributor

@davvd #228 has been merged. Please close the task

@davvd
Copy link
Author

davvd commented May 7, 2015

@davvd waiting for #253

@yegor256 right, let's wait for #253

@dmzaytsev
Copy link
Contributor

@davvd you can see pull request #228 that is fixing this issue, if that's all here please close.

@dmzaytsev
Copy link
Contributor

@davvd please close the task

@davvd
Copy link
Author

davvd commented May 10, 2015

@yegor256 Closing..

@davvd davvd closed this as completed May 10, 2015
@davvd
Copy link
Author

davvd commented May 10, 2015

@elenavolokhova please, let us know what do you think about this ticket, according to our QA rules

@elenavolokhova
Copy link

@davvd Quality is good.

@davvd
Copy link
Author

davvd commented May 11, 2015

@davvd Quality is good.

@elenavolokhova many thanks

@davvd
Copy link
Author

davvd commented May 11, 2015

@dmzaytsev just added 10 mins to @elenavolokhova (for QA), payment ID is 57168002. thank you, added 30 mins to your acc, payment num is 57168022, 169 hours and 42 mins spent in total to complete. +30 to your rating, your total score is +1460

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants