Skip to content

Stopping tests when Disqus API rate limit reached #34

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

Merged
merged 2 commits into from
Jun 9, 2016

Conversation

freskoulix
Copy link
Contributor

@freskoulix freskoulix commented Jun 9, 2016

  • Checking Disqus API rate limit and stopping the test before trying to comment.
    This way we prevent leaving libraries/examples without comments.
  • Minor fix in tic-toc intialization at utils.py .
  • Added check in seleniumbender.py before sending email with report.
    If a report doesn't exists, no email is send.
  • Removed unnecessary prints from utilis.py .

…to comment.

This way we prevent leaving libraries/examples without comments.

- Minor fix in tic-toc intialization at utils.py .
- Added check in seleniumbender.py before sending email with report.

If a report doesn't exists, no email is send.
@freskoulix freskoulix changed the title Stopping tests when Disqus APi rate limit reached Stopping tests when Disqus API rate limit reached Jun 9, 2016
@freskoulix freskoulix merged commit 04a9ffc into master Jun 9, 2016
@freskoulix freskoulix deleted the disqus-stop-when-api-rate-limit-reached branch June 9, 2016 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant