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

Integration tests for the assumptions of our social network #753

Closed
iislucas opened this issue Dec 30, 2014 · 3 comments
Closed

Integration tests for the assumptions of our social network #753

iislucas opened this issue Dec 30, 2014 · 3 comments
Labels

Comments

@iislucas
Copy link
Contributor

Different social networks throttle messages in different ways. We should have some integration tests that allow us to check our assumptions. e.g. that 100ms gap is enough for the google-talk xmpp servers not to throttle uProxy messages. These should be run periodically on our continuos integration servers. This will also let us avoid server-side changes breaking us and us not knowing what happened. These tests should tell us that the social network gives errors when we expect it to also.

@iislucas iislucas added this to the v0.9 Allardice (Web Store Feature Complete) milestone Dec 30, 2014
@dborkan dborkan modified the milestones: v0.7 Allardice (sprint), v0.9 Allardice (Web Store Feature Complete) Jan 9, 2015
@dborkan dborkan self-assigned this Jan 9, 2015
@dborkan
Copy link
Contributor

dborkan commented Jan 28, 2015

Tests are added in freedomjs/freedom-social-xmpp#85. We now have real GTalk tests in a Chrome app with these cases:

  • Can login and logout
  • Peers can detect each other
  • Can send messages
  • Can send 8 messages per second

Currently these tests must be run with "grunt jasmine_chromeapp". We are working to make them a part of "grunt test", but ran into an issue: freedomjs/freedom-social-xmpp#87

@dborkan
Copy link
Contributor

dborkan commented Jan 28, 2015

Closing this issue - we can add these tests to be run along with "grunt test" when we have freedomjs/freedom-social-xmpp#87 fixed

@ryscheng
Copy link
Member

This should be subsumed by freedomjs/freedom-social-xmpp#90

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

No branches or pull requests

3 participants