Skip to content
This repository has been archived by the owner on Feb 11, 2020. It is now read-only.

Pre-Node4 #394

Closed
wants to merge 7 commits into from
Closed

Pre-Node4 #394

wants to merge 7 commits into from

Conversation

psorowka
Copy link
Contributor

This already builds correctly on node4 but still has the ascoltatori@2.0.0-pre1 dependency.

Only few laps left :)

Peter Sorowka added 4 commits January 16, 2016 02:04
- added ascoltatori-2.0.0pre1 as a dependency
- bumped almost all other dependencies
- minor changes needed for new version of `memdown` packet
- locally building and passing on node 4, thumbs down!
Ascoltatori does not support iojs v3
@psorowka
Copy link
Contributor Author

  1) mosca.Server with mongo persistence should publish each subscribe to '$SYS/{broker-id}/new/subscribes':
      Uncaught AssertionError: expected '$SYS/NkZZciSfBOe/new/unsubscribes' to equal '$SYS/NkZZciSfBOe/new/subscribes'
      + expected - actual
      -$SYS/NkZZciSfBOe/new/unsubscribes
      +$SYS/NkZZciSfBOe/new/subscribes

this test is failing or passing randomly... looking into it.

@psorowka
Copy link
Contributor Author

!"§$"§"!$%#

It builds (repeatedly) without issues here https://travis-ci.org/cybusio/mosca/builds/103011310

And that's exactly the same version. @mcollina have you experienced such behavior before?

update: just failed there.. with another race condition..

@psorowka
Copy link
Contributor Author

there we go! Let's hope it was just the tests...I couldn't reproduce any of these failures locally, though

@mcollina ready for mosca 0.4-pre1

@psorowka psorowka closed this Jan 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant