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

test: use pin tests #1405

Merged
merged 3 commits into from
Jun 22, 2018
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
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
5 changes: 5 additions & 0 deletions test/core/node.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
'use strict'

require('./pin')
require('./pin-set')
// require('./key-exchange')
Copy link
Member Author

Choose a reason for hiding this comment

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

Does anyone know why these tests were never being run? @richardschneider ??

Copy link
Contributor

Choose a reason for hiding this comment

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

@diasdavid No idea, the tests should be run.

Copy link
Member Author

Choose a reason for hiding this comment

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

They weren't being picked up anywhere. I'll add them as a separate PR then.

2 changes: 1 addition & 1 deletion test/core/pin-set.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ const parallelLimit = require('async/parallelLimit')
const series = require('async/series')
const { fromB58String } = require('multihashes')
const { DAGNode } = require('ipld-dag-pb')
const CID = require('CIDs')
const CID = require('cids')

const IPFS = require('../../src/core')
const createPinSet = require('../../src/core/components/pin-set')
Expand Down
2 changes: 1 addition & 1 deletion test/node.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
require('./cli')
require('./http-api')
require('./gateway')
// require('./core') // get automatically picked up
require('./core/node.js')