Skip to content
This repository was archived by the owner on Feb 12, 2024. It is now read-only.

Conversation

daviddias
Copy link
Member

@daviddias daviddias commented May 28, 2016

  • Add new dial calls to libp2p
  • Make more robust general connectivity tests
  • Make turbulence tests

@jbenet jbenet added the status/in-progress In progress label May 28, 2016
@daviddias daviddias force-pushed the test/network-turbulence branch 5 times, most recently from ca50885 to d8cd44d Compare May 29, 2016 11:35
@daviddias daviddias changed the title WIP Bubble upstream the latest changes on libp2p May 29, 2016
@daviddias
Copy link
Member Author

Node.js tests work like a charm, however, the browser tests fails on the dial (no available transports), this typically means that the listener didn't start or that the dialer doesn't have the required transport.

@haadcode mentioned that libp2p-ipfs-browser is not being injected properly and after a few attempts playing with .aegir.js, I didn't manage to have a success confirming this. @dignifiedquire can you shine some light of Webpack goodness into this PR? :)

@daviddias daviddias force-pushed the test/network-turbulence branch 3 times, most recently from 6b4854e to c3a543f Compare May 29, 2016 12:13
@daviddias
Copy link
Member Author

@dignifiedquire nevermind, all good now :)

"gulp": "^3.9.1",
"idb-plus-blob-store": "^1.1.2",
"interface-ipfs-core": "^0.1.5",
"left-pad": "^1.1.0",
Copy link
Member

Choose a reason for hiding this comment

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

are you sure you want to use this module :D

Copy link
Member Author

Choose a reason for hiding this comment

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

ahaha I was waiting for someone to catch this :D

@daviddias daviddias force-pushed the test/network-turbulence branch from c3a543f to 011ea47 Compare May 29, 2016 14:06
@daviddias
Copy link
Member Author

@dignifiedquire any other comments? Good to merge and continue tackling on more tests in more PR (this was about making libp2p robustness tests, but then it became more about making libp2p-ipfs more powerful and fix a bunch of bugs along the way)

@dignifiedquire
Copy link
Member

go for it, I think we ahoul encapaulate even more in libp2p that we are doing currently in js-ipfs but that can come later

@daviddias daviddias merged commit 0d94c8e into master May 29, 2016
@daviddias daviddias deleted the test/network-turbulence branch May 29, 2016 16:18
@jbenet jbenet removed the status/in-progress In progress label May 29, 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.

3 participants