-
Notifications
You must be signed in to change notification settings - Fork 26
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
AwsIteratorITCase.java:51-53: Integration test fail when adding... #64
Comments
@olegsimanov can you please help? Keep in mind this. If you have any technical questions, don't hesitate to ask right here... Budget here is 30 mins (keep this principle in mind) |
@olegsimanov 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... do not forget that you should remove the |
@olegsimanov it takes too long. Usually, we expect any task to be finished in less than a week. I'll assign someone else. Please stop working with it right now. See our no obligations principle. -60 added to your rating, at the moment it is: -180 |
@jrdalpra the task is for you now, follow these guidelines. Don't hesitate to ask any technical questions right here The budget of this issue is 30 mins, which is exactly how much will be paid when the task is done (see this for explanation) |
@dmarkov there's no need to have a AWS Credential to run the tests, right? Everything is mocked? |
@jrdalpra please, don't ask technical questions here, check this: http://www.yegor256.com/2015/02/16/it-is-not-a-school.html |
@dmarkov @yegor256 @dmzaytsev PR merged and task still open at netbout. How can we solve it? |
@dmarkov you say from javadoc? |
@dmarkov If it's about the TODO at javadoc I made a PR to solved it. |
@dmarkov PR merge by rultor |
@jrdalpra you have to remove the puzzle from the source code, it's important |
@dmarkov I think I'm not getting the point of "solve the puzzle". I solved it (problem is not at our tool) and removed the TODO at source code. There's more to do to solve it? Thanks. |
@dmarkov ok. If you can close all tasks at netbout I'll be thankful. |
@dmarkov rultor deployed the project. :) Now you can close the issue. Thanks. |
@yegor256 Closing this issue now... |
Puzzle
56-be65f863
insrc/test/java/com/jcabi/dynamo/AwsIteratorITCase.java:51-53
has to be resolved: Integration test fail when adding @NotNull constraint this should be fixed in methods AwsIteratorITCase.iteratesItems() and AwsIteratorITCase.iteratesItemsAndDeletes(). Also see RegionITCase... The puzzle was created by tkjee on 5-Oct-2015.If you have any technical questions, don't ask me, submit new tickets instead
The text was updated successfully, but these errors were encountered: