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

Commit

Permalink
fix: fix name-pubsub test (#108)
Browse files Browse the repository at this point in the history
Doing a name.resolve to subscribe should not be expected to reject in the second test because both tests use the same node(which should be changed) and in the second test name.resolve will resolve because the nodes already now about each other.

Co-authored-by: achingbrain <alex@achingbrain.net>
  • Loading branch information
hugomrdias and achingbrain committed Apr 20, 2020
1 parent 91ebc09 commit 1a33bf9
Show file tree
Hide file tree
Showing 5 changed files with 11,604 additions and 28,238 deletions.
9 changes: 3 additions & 6 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,13 @@ stages:
- cov

node_js:
- '10'
- '12'

os:
- linux
- osx

script:
- npm install ipfs@0.41.0-rc.1 go-ipfs-dep ipfs-http-client@42.0.0
- npx nyc -s npm run test:node -- --bail
after_success: npx nyc report --reporter=text-lcov > coverage.lcov && npx codecov

Expand All @@ -39,16 +38,14 @@ jobs:
addons:
chrome: stable
script:
- npm install ipfs@0.41.0-rc.1 go-ipfs-dep ipfs-http-client@42.0.0
- npx aegir test -t browser
- npx aegir test -t browser --bail

- stage: test
name: firefox
addons:
firefox: latest
script:
- npm install ipfs@0.41.0-rc.1 go-ipfs-dep ipfs-http-client@42.0.0
- npx aegir test -t browser -- --browsers FirefoxHeadless
- npx aegir test -t browser --bail -- --browsers FirefoxHeadless

notifications:
email: false
Loading

0 comments on commit 1a33bf9

Please sign in to comment.