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

BkBasicTest.java:67-68: It will be nice to... #533

Closed
davvd opened this issue Jan 22, 2016 · 21 comments
Closed

BkBasicTest.java:67-68: It will be nice to... #533

davvd opened this issue Jan 22, 2016 · 21 comments

Comments

@davvd
Copy link

davvd commented Jan 22, 2016

Puzzle 516-c8aa6c8f in src/test/java/org/takes/http/BkBasicTest.java:67-68 has to be resolved: It will be nice to refactor tests with Socket usage and replace them to real statements. See usage of BkBasicTest.createMockSocket.... The puzzle was created by lexx on 21-Jan-2016.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code.

@davvd
Copy link
Author

davvd commented Jan 22, 2016

@exper0 please proceed, it's yours

@davvd
Copy link
Author

davvd commented Jan 22, 2016

@yegor256 since the ticket has no milestone I set it to 1.0

@davvd davvd added this to the 1.0 milestone Jan 22, 2016
@exper0
Copy link
Contributor

exper0 commented Jan 22, 2016

@yegor256 @dmzaytsev @lexx what should I do here? I do not understand the task. What does "real statements" means?

exper0 pushed a commit to exper0/takes that referenced this issue Jan 22, 2016
@yegor256
Copy link
Owner

@exper0 I think that we should get rid of mocking here, since it's a bad thing in general, see http://www.yegor256.com/2014/09/23/built-in-fake-objects.html

@exper0
Copy link
Contributor

exper0 commented Jan 31, 2016

@davvd need more time here.

@davvd
Copy link
Author

davvd commented Jan 31, 2016

@davvd need more time here.

@exper0 no problem, thanks for letting me know

exper0 pushed a commit to exper0/takes that referenced this issue Feb 4, 2016
exper0 pushed a commit to exper0/takes that referenced this issue Feb 4, 2016
@exper0
Copy link
Contributor

exper0 commented Feb 19, 2016

@davvd please assign to someone else.

@davvd
Copy link
Author

davvd commented Feb 27, 2016

@davvd please assign to someone else.

@exper0 30 points deducted from your rating

@davvd
Copy link
Author

davvd commented Feb 27, 2016

@davvd please assign to someone else.

@exper0 OK, I will try to assign someone else

@davvd davvd removed the @exper0 label Feb 27, 2016
@davvd
Copy link
Author

davvd commented Feb 28, 2016

@essobedo the task is yours plz proceed

@essobedo
Copy link
Contributor

@davvd the PR is ready to be reviewed thx in advance

@davvd
Copy link
Author

davvd commented Mar 1, 2016

@davvd the PR is ready to be reviewed thx in advance

@essobedo OK

@essobedo
Copy link
Contributor

essobedo commented Mar 2, 2016

@davvd the PR has been merged, so could you please close the ticket and ask the QA to proceed. Thx in advance.

@essobedo
Copy link
Contributor

essobedo commented Mar 3, 2016

@davvd please close the ticket

1 similar comment
@essobedo
Copy link
Contributor

essobedo commented Mar 3, 2016

@davvd please close the ticket

@essobedo
Copy link
Contributor

essobedo commented Mar 4, 2016

@davvd the PR has been merged, so could you please close the ticket and ask the QA to proceed. Thx in advance.

@davvd davvd closed this as completed Mar 4, 2016
@davvd
Copy link
Author

davvd commented Mar 4, 2016

@yegor256 I'm closing the issue...

@davvd
Copy link
Author

davvd commented Mar 4, 2016

@elenavolokhova please, check this issue for QA compliance, as per par.24

@elenavolokhova
Copy link

@davvd Looks good!

@davvd
Copy link
Author

davvd commented Mar 5, 2016

@davvd Looks good!

@elenavolokhova many thanks

@davvd
Copy link
Author

davvd commented Mar 5, 2016

@essobedo I added 10 mins to @elenavolokhova (for QA review) in transaction 79352198. done, I added 30 mins in payment 79352218, 100 hours and 1 min spent to complete this. +30 added to your rating, current score is: +315

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

5 participants