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

Bootstrapers tests, for @lgierth #899

Merged
merged 2 commits into from
Nov 17, 2017
Merged

Bootstrapers tests, for @lgierth #899

merged 2 commits into from
Nov 17, 2017

Conversation

daviddias
Copy link
Member

@daviddias daviddias commented Jul 5, 2017

Blocked by ipfs/aegir#113

@daviddias daviddias added the status/in-progress In progress label Jul 5, 2017
* the WebSockets Bootstrappers easily :)
*/
console.log('=>', process.env.TEST_BOOTSTRAPERS)
if (process.env.TEST_BOOTSTRAPERS) {
Copy link
Member Author

Choose a reason for hiding this comment

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

@dignifiedquire how can I get env vars to be 'browserified' to the browser context?

Copy link
Member Author

Choose a reason for hiding this comment

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

Will wait for the new aegir ipfs/aegir#113

@daviddias
Copy link
Member Author

@lgierth I'll get you this once #961 is done which is blocked until this is figured out ipfs-inactive/aegir-test-repo#1

@daviddias daviddias self-assigned this Oct 17, 2017
@daviddias daviddias added status/ready Ready to be worked P0 Critical: Tackled by core team ASAP and removed status/in-progress In progress status/ready Ready to be worked labels Oct 17, 2017
@daviddias daviddias added status/in-progress In progress and removed status/ready Ready to be worked labels Nov 8, 2017
@daviddias
Copy link
Member Author

@lgierth just finished this one :)

Now all you need to test the WebSockets bootstrapers is to get js-ipfs master, npm install and npm run test:bootstrapers, it should give you something like:

image

@daviddias daviddias merged commit b2106c1 into master Nov 17, 2017
@daviddias daviddias deleted the feat/tests-for-lgierth branch November 17, 2017 10:19
@ghost ghost removed the status/in-progress In progress label Nov 17, 2017
@daviddias daviddias mentioned this pull request Nov 20, 2017
24 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P0 Critical: Tackled by core team ASAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant