Skip to content
This repository has been archived by the owner on May 25, 2021. It is now read-only.

Add path function to detect binary #40

Merged
merged 2 commits into from
Feb 10, 2020
Merged
Show file tree
Hide file tree
Changes from all 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
16 changes: 16 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,22 @@ This module downloads `go-ipfs` binaries from https://dist.ipfs.io into your pro

By default it will download the go-ipfs version that matches the npm version of this module. So depending on `[email protected]` will install `go-ipfs v0.4.19` for your current system architecture, in to your project at `node_modules/go-ipfs-dep/go-ipfs/ipfs`.

After downloading you can find out the path of the installed binary by calling the `path` function exported by this module:

```javascript
const { path } = require('go-ipfs-dep')

console.info('go-ipfs is installed at', path())
```

An error will be thrown if the path to the binary cannot be resolved - if you do not wish this to happen, call `path.silent()`:

```javascript
const { path: silent } = require('go-ipfs-dep')

console.info('go-ipfs may installed at', silent())
```

### Overriding the go-ipfs version

You can override the version of go-ipfs that gets downloaded by adding by adding a `go-ipfs.version` field to your `package.json`
Expand Down
8 changes: 8 additions & 0 deletions src/bin.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,14 @@ const success = (output) => {
process.exit(0)
}

const existingPath = download.path.silent()

if (existingPath) {
process.stdout.write(`Detected existing binary at ${existingPath}\n`)
process.stdout.write(`Skipping download\n`)
process.exit(0)
}

// First param is the target version
// Second param is the target platform
// Third param is the target architecture
Expand Down
24 changes: 24 additions & 0 deletions src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ const unzip = require('unzip-stream')
const fetch = require('node-fetch')
const pkgConf = require('pkg-conf')
const pkg = require('./../package.json')
const fs = require('fs')

function unpack ({ url, installPath, stream }) {
return new Promise((resolve, reject) => {
Expand Down Expand Up @@ -115,3 +116,26 @@ module.exports = async function () {
installPath: path.join(args.installPath, 'go-ipfs') + path.sep
}
}

module.exports.path = function () {
const paths = [
path.resolve(path.join(__dirname, '..', 'go-ipfs', 'ipfs')),
path.resolve(path.join(__dirname, '..', 'go-ipfs', 'ipfs.exe'))
]

for (const bin of paths) {
if (fs.existsSync(bin)) {
return bin
}
}

throw new Error('go-ipfs binary not found, it may not be installed or an error may have occured during installation')
}

module.exports.path.silent = function () {
try {
return module.exports.path()
} catch (err) {
// ignore
}
}
56 changes: 48 additions & 8 deletions test/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ function download (version, platform, arch, callback) {
}

test('Ensure ipfs gets downloaded (current version and platform)', (t) => {
t.plan(5)
t.plan(7)
const dir = path.resolve(__dirname, '../go-ipfs')
rimraf.sync(dir)

Expand All @@ -52,12 +52,17 @@ test('Ensure ipfs gets downloaded (current version and platform)', (t) => {
fs.stat(dir, (err, stats) => {
t.error(err, 'go-ipfs should stat without error')
t.ok(stats, 'go-ipfs was downloaded')
// Check detected path
fs.stat(Download.path(), (err3, stats3) => {
t.error(err3, 'detected binary path should stat without error')
t.ok(stats3.mode, 'downloaded binary was detected')
})
})
})
})

test('Ensure Windows version gets downloaded', (t) => {
t.plan(7)
t.plan(9)
const dir = path.resolve(__dirname, '../go-ipfs')
rimraf.sync(dir)
download(version, 'windows', (err, res) => {
Expand All @@ -72,13 +77,18 @@ test('Ensure Windows version gets downloaded', (t) => {
fs.stat(path.join(dir, 'ipfs.exe'), (err2, stats2) => {
t.error(err2, 'windows bin should stat without error')
t.ok(stats2, 'windows bin was downloaded')
// Check detected path
fs.stat(Download.path(), (err3, stats3) => {
t.error(err3, 'detected binary path should stat without error')
t.ok(stats3.mode, 'downloaded binary was detected')
})
})
})
})
})

test('Ensure Linux version gets downloaded', (t) => {
t.plan(7)
t.plan(9)
const dir = path.resolve(__dirname, '../go-ipfs')
rimraf.sync(dir)
download(version, 'linux', (err, res) => {
Expand All @@ -93,13 +103,18 @@ test('Ensure Linux version gets downloaded', (t) => {
fs.stat(path.join(dir, 'ipfs'), (err2, stats2) => {
t.error(err2, 'linux bin should stat without error')
t.ok(stats2, 'linux bin was downloaded')
// Check detected path
fs.stat(Download.path(), (err3, stats3) => {
t.error(err3, 'detected binary path should stat without error')
t.ok(stats3.mode, 'downloaded binary was detected')
})
})
})
})
})

test('Ensure OSX version gets downloaded', (t) => {
t.plan(7)
t.plan(9)
const dir = path.resolve(__dirname, '../go-ipfs')
rimraf.sync(dir)
download(version, 'darwin', (err, res) => {
Expand All @@ -114,13 +129,18 @@ test('Ensure OSX version gets downloaded', (t) => {
fs.stat(path.join(dir, 'ipfs'), (err2, stats2) => {
t.error(err2, 'OSX bin should stat without error')
t.ok(stats2, 'OSX bin was downloaded')
// Check detected path
fs.stat(Download.path(), (err3, stats3) => {
t.error(err3, 'detected binary path should stat without error')
t.ok(stats3.mode, 'downloaded binary was detected')
})
})
})
})
})

test('Ensure TARGET_OS, TARGET_VERSION and TARGET_ARCH version gets downloaded', (t) => {
t.plan(7)
t.plan(9)
const dir = path.resolve(__dirname, '../go-ipfs')
rimraf.sync(dir)
process.env.TARGET_OS = 'windows'
Expand All @@ -142,9 +162,14 @@ test('Ensure TARGET_OS, TARGET_VERSION and TARGET_ARCH version gets downloaded',
fs.stat(path.join(dir, 'ipfs.exe'), (err2, stats2) => {
t.error(err2, 'windows bin should stat without error')
t.ok(stats2, 'windows bin was downloaded')
delete process.env.TARGET_OS
delete process.env.TARGET_VERSION
delete process.env.TARGET_ARCH
// Check detected path
fs.stat(Download.path(), (err3, stats3) => {
t.error(err3, 'detected binary path should stat without error')
t.ok(stats3.mode, 'downloaded binary was detected')
delete process.env.TARGET_OS
delete process.env.TARGET_VERSION
delete process.env.TARGET_ARCH
})
})
})
})
Expand All @@ -171,3 +196,18 @@ test('Returns an error when dist url is 404', (t) => {
delete process.env.GO_IPFS_DIST_URL
})
})

test('Path returns undefined when no binary has been downloaded', (t) => {
t.plan(1)
const dir = path.resolve(__dirname, '../go-ipfs')
rimraf.sync(dir)
t.ok(Download.path.silent() === undefined, 'Path is undefined before installation')
})

test('Path returns undefined when no binary has been downloaded', (t) => {
t.plan(1)
const dir = path.resolve(__dirname, '../go-ipfs')
rimraf.sync(dir)

t.throws(Download.path, /not found/, 'Path throws if binary is not installed')
})