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

re-enable client integration tests #301

Merged
merged 1 commit into from Jun 2, 2016
Merged

re-enable client integration tests #301

merged 1 commit into from Jun 2, 2016

Conversation

sonnyp
Copy link
Member

@sonnyp sonnyp commented Jun 2, 2016

This moves /packages/*/test/integration to /test/integration

Benefits:

  • Makes requires in unit tests cleaner (no unit sub-directory)
  • I don't think integration tests should be "bound" to packages, specially when we'll split them into smaller pieces
  • Makes setup easier, let's have one prosody config for all integration tests
  • We can easily reuse the same tests against prosody and node-xmpp-server (and potentially other servers)
  • Allows us to remove duplicate integration tests (c2s node-xmpp-server and node-xmpp-client)

@sonnyp sonnyp force-pushed the integration-tests branch 4 times, most recently from 568e5b9 to fa74362 Compare June 2, 2016 11:32
@@ -25,7 +25,7 @@ describe('TCP client/server', function () {
cl = new xmpp.Client({
jid: 'test@localhost',
password: 'test',
host: '::1',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for some reason I'd get ENETUNREACH on travis

@sonnyp sonnyp changed the title re-enable integration tests re-enable client integration tests Jun 2, 2016
@@ -21,7 +21,7 @@ server.on('connection', function (connection) {
})
})

describe.skip('C2S WebSocket server client', function () {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was temporary disabled during the move to WebSocket final RFC and we forgot to re-enable it

@sonnyp sonnyp mentioned this pull request Jun 2, 2016
44 tasks
@coveralls
Copy link

coveralls commented Jun 2, 2016

Coverage Status

Coverage increased (+0.06%) to 78.769% when pulling eee91b6 on integration-tests into 999acbb on master.

@sonnyp sonnyp merged commit 21b53a9 into master Jun 2, 2016
@sonnyp sonnyp deleted the integration-tests branch June 2, 2016 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants