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.

Sign up for GitHub

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

Jump to bottom

RqAuth is not tested #37

Closed
yegor256 opened this issue Mar 12, 2015 · 7 comments
Closed

RqAuth is not tested #37

yegor256 opened this issue Mar 12, 2015 · 7 comments
Labels
30 mins bug DEV @dmzaytsev

Comments

@yegor256
Copy link
Owner

There is no unit test for RqAuth class, let's create one
@davvd
Copy link

davvd commented Mar 12, 2015

@yegor256 got it, tag bug is here

@davvd
Copy link

davvd commented Mar 14, 2015

@yegor256 thanks for tis bug, I topped your account for 15 mins, transaction 000-5aef22f8

@davvd
Copy link

davvd commented Mar 26, 2015

@dmzaytsev the task is for you now, follow these guidelines. Don't hesitate to ask any technical questions right here... The budget here is 30 mins, which is exactly how much time will be paid for, when the task is completed

dmzaytsev added a commit to dmzaytsev/takes that referenced this issue Mar 27, 2015
@dmzaytsev
yegor256#37 added unit test for RqAuth
2e98f44
dmzaytsev added a commit to dmzaytsev/takes that referenced this issue Mar 27, 2015
@dmzaytsev
yegor256#37 added unit test for RqAuth
8c6e917
dmzaytsev added a commit to dmzaytsev/takes that referenced this issue Mar 27, 2015
@dmzaytsev
yegor256#37 add newline
7901181
dmzaytsev added a commit to dmzaytsev/takes that referenced this issue Mar 27, 2015
@dmzaytsev
yegor256#37 added unit test for RqAuth
3c27e9a
@dmzaytsev
Copy link
Contributor

@yegor256 the PR has been merged
could you close the issue please

@yegor256
Copy link
Owner Author

@dmzaytsev thanks!

dmzaytsev added a commit to dmzaytsev/takes that referenced this issue Mar 29, 2015
@dmzaytsev
yegor256#37 change XeFlash to accept encoded cookie
379a1ff 
rewrite XeFlashTest  to test RsFlash and XeFlash together
@dmzaytsev
Copy link
Contributor

@davvd I've created the PR #84 which has been merged

@davvd
Copy link

davvd commented Mar 29, 2015

@dmzaytsev Thanks a lot, I just topped your account for 30 mins, transaction ID 54285554, total time was 42 hours and 39 mins; +30 added to your rating, current score is: +195

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
30 mins bug DEV @dmzaytsev
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@yegor256 @davvd @dmzaytsev