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

Awesome Endeavour: Circuit Relay #830

Merged
merged 19 commits into from
Nov 8, 2017
Merged

Awesome Endeavour: Circuit Relay #830

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9aef6b7
feat: enable circuit relay
dryajov Apr 13, 2017
e3957d2
chore: update deps
daviddias Nov 8, 2017
72b6fb6
chore: clean the house a bit
daviddias Nov 8, 2017
492fa16
test: avoid spawning nodes that not used
daviddias Nov 8, 2017
c7fd99a
test: clean up bitswap tests
daviddias Nov 8, 2017
1742398
test: move timeouts
daviddias Nov 8, 2017
50b29e4
fix(test): move timeouts to the top
dryajov Nov 8, 2017
aa98ec0
feat: run interop tests with new aegir
dryajov Nov 8, 2017
c5c88aa
onto interop tests
daviddias Nov 8, 2017
18f5a9a
chore
daviddias Nov 8, 2017
983bfd9
abrvs
daviddias Nov 8, 2017
61e1266
chore: more clean up
daviddias Nov 8, 2017
19d8555
interop exchange files is all good
daviddias Nov 8, 2017
36afc59
update deps
daviddias Nov 8, 2017
bad8232
only missing circuit relay interop tests
daviddias Nov 8, 2017
05e19a0
skip circuit-relay interop for now
daviddias Nov 8, 2017
cba1be2
adjust timeouts on sharding tests
daviddias Nov 8, 2017
409a523
fix timouts on bitswap
daviddias Nov 8, 2017
ddcca8b
moar time
daviddias Nov 8, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
chore
  • Loading branch information
@daviddias
daviddias committed Nov 8, 2017
commit 18f5a9a7f51ef79fb51feb59f273218f863e434c
12 changes: 6 additions & 6 deletions .aegir.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
const parallel = require('async/parallel')
const spawnTools = require('./test/utils/spawn-tools')
const js = spawnTools.spawnJsNode
const go = spawnTools.spawnGoNode
// const go = spawnTools.spawnGoNode
const stop = spawnTools.stopNodes

/*
Expand All @@ -18,11 +18,11 @@ function pre (done) {
(cb) => js([`${base}/10012`, `${base}/20012/ws`], true, 31012, 32012, cb),
(cb) => js([`${base}/10013`, `${base}/20013/ws`], true, 31013, 32013, cb),
(cb) => js([`${base}/10014`, `${base}/20014/ws`], true, 31014, 32014, cb),
(cb) => js([`${base}/10015`, `${base}/20015/ws`], true, 31015, 32015, cb),
(cb) => go([`${base}/10027`, `${base}/20027/ws`], true, 33027, 44027, cb),
(cb) => go([`${base}/10028`, `${base}/20028/ws`], true, 33028, 44028, cb),
(cb) => go([`${base}/10031`, `${base}/20031/ws`], true, 33031, 44031, cb),
(cb) => go([`${base}/10032`, `${base}/20032/ws`], true, 33032, 44032, cb)
(cb) => js([`${base}/10015`, `${base}/20015/ws`], true, 31015, 32015, cb)
// (cb) => go([`${base}/10027`, `${base}/20027/ws`], true, 33027, 44027, cb),
// (cb) => go([`${base}/10028`, `${base}/20028/ws`], true, 33028, 44028, cb),
// (cb) => go([`${base}/10031`, `${base}/20031/ws`], true, 33031, 44031, cb),
// (cb) => go([`${base}/10032`, `${base}/20032/ws`], true, 33032, 44032, cb)
], done)
}
Copy link
Member

@daviddias daviddias Oct 31, 2017

Choose a reason for hiding this comment

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

could you make these spawn daemon tools part of test/utils? This file should have a very declarative feel and just inform the dev what is going to be spawned for testing

Copy link
Member Author

@dryajov dryajov Oct 31, 2017

Choose a reason for hiding this comment

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

sure, I'll move them over.

Copy link
Member

@daviddias daviddias Nov 3, 2017

Choose a reason for hiding this comment

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

Why can't the "Spawner" use IPFS Factory?

Copy link
Member

@daviddias daviddias Nov 3, 2017

Choose a reason for hiding this comment

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

Also, what are the magic numbers?

Copy link
Member Author

@dryajov dryajov Nov 3, 2017

Choose a reason for hiding this comment

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

The numbers are the preexisting port numbers that this daemons were being started with - I kept them because I believe some other tests might rely on them.


Expand Down