Skip to content
This repository has been archived by the owner on Aug 1, 2023. It is now read-only.

[WIP] test: Private network tests #24

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[WIP] test: Private network tests #24

wants to merge 2 commits into from

Conversation

jacobheun
Copy link
Contributor

This depends on libp2p/js-libp2p-pnet#1 and its subsequent release into libp2p.

I created this to provide visibility into the existing interop tests while those all get merged and released.

Currently the only running tests are for js to js and js to go on the same private network. There is currently an issue in pull-reader with a private node attempts to talk with an invalid node (public or with a different key) that causes the initial connection to hang until it times out. I've added comments above the skips pointing to the issue. Rather than leave in tests that timeout, I've skipped them until we're able to error the connection out timely.

@ghost ghost assigned jacobheun Jun 26, 2018
@ghost ghost added the status/in-progress In progress label Jun 26, 2018
@jacobheun jacobheun force-pushed the test/pnet branch 2 times, most recently from b7f5906 to ab26a4e Compare July 5, 2018 10:03
test: add skips for tests that timeout
test: add additional pnet tests
temporarily use a temp branch of pull-reader for testing
test: add test for go getting data from js
test: add tests for go to go
test: check pnet peers after attempted connections
test: update order of operations to avoid timeouts from go
@daviddias
Copy link
Member

@jacobheun can you take these to the finish line in the foreseeable future?

@daviddias daviddias changed the title WIP: Private network tests [WIP] test: Private network tests Feb 13, 2019
@jacobheun
Copy link
Contributor Author

@daviddias yes! I updated the blocking PR in ipfs-repo if you can take a look ipfs/js-ipfs-repo#172. There were concerns there about not using the keychain, but the spec isn't written to support using the keychain.

@daviddias daviddias added the status/blocked Unable to be worked further until needs are met label Sep 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status/blocked Unable to be worked further until needs are met status/in-progress In progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants