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

fix: swarm connect to local servers #175

Merged
merged 1 commit into from
Nov 22, 2017
Merged

fix: swarm connect to local servers #175

merged 1 commit into from
Nov 22, 2017

Conversation

richardschneider
Copy link
Contributor

Prevent dialer timeouts and speed up the tests.

This gets all the pubsub tests working!

aegir test -t node --timeout 5000 -f test\interface\pubsub.spec.js

Test Node.js


  .pubsub
Secp256k1 bindings are not compiled. Pure JS implementation will be used.
Secp256k1 bindings are not compiled. Pure JS implementation will be used.
    single node
      .publish
        √ errors on string messags
        √ message from buffer
      .subscribe
        √ to one topic
        √ attaches multiple event listeners
        √ discover options
    multiple nodes connected
      .peers
        √ does not error when not subscribed to a topic
        √ doesn't return extra peers
        √ returns peers for a topic - one peer (529ms)
        √ lists peers for a topic - multiple peers (526ms)
      .ls
        √ empty() list when no topics are subscribed
        √ list with 1 subscribed topic
        √ list with 3 subscribed topics
      multiple nodes
        √ receive messages from different node (526ms)
        √ round trips a non-utf8 binary buffer correctly (529ms)
        √ receive multiple messages (540ms)
      load tests
        √ call publish 1k times (3447ms)
Send/Receive 10k messages took: 34059 ms, 293 ops / s

        √ send/receive 10k messages (34575ms)
        √ call subscribe/unsubscribe 1k times (46ms)


  18 passing (1m)

@ghost ghost added the in progress label Nov 21, 2017
@daviddias daviddias merged commit 09d9573 into master Nov 22, 2017
@daviddias daviddias deleted the pubsub branch November 22, 2017 16:36
@ghost ghost removed the in progress label Nov 22, 2017
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

2 participants