particle-api-js 9.1.1 → 9.3.0

package/package.json

@@ -1,17 +1,19 @@
 {
   "name": "particle-api-js",
-  "version": "9.1.1",
+  "version": "9.3.0",
   "description": "Particle API Client",
   "main": "lib/Particle.js",
   "scripts": {
     "babel-watch": "babel src -d lib --watch --source-maps",
     "prepublish": "npm run lint && npm run compile",
     "compile": "babel src -sd lib",
-    "test": "npm run lint && npm run test:node",
-    "test:node": "mocha test/ -R spec --compilers js:babel-register",
+    "test": "npm run lint && npm run test:unit",
+    "test:ci": "npm run lint && npm run test:unit -- --forbid-only && npm run coverage",
+    "test:unit": "mocha test/ -R spec --compilers js:babel-register",
+    "test:unit:silent": "npm run test:unit > tmp/test-unit-log.txt 2>&1",
     "test:browser": "karma start --single-run",
-    "test:watch": "npm run test:node -- --watch",
-    "cover": "istanbul cover ./node_modules/.bin/_mocha test/ -- -R spec --compilers js:babel-register",
+    "test:watch": "npm run test:unit -- --watch",
+    "coverage": "nyc --reporter=text --include='src/**/*.js' --temp-dir=./tmp/ --check-coverage --lines 50 npm run test:unit:silent",
     "lint": "eslint . --ext .js --format unix --ignore-path .gitignore --ignore-pattern \"dist/*\"",
     "lint:fix": "npm run lint -- --fix",
     "docs": "documentation build src/Particle.js --shallow -g -f md -o docs/api.md",
@@ -58,7 +60,6 @@
     "documentation": "^4.0.0-rc.1",
     "eslint": "^5.16.0",
     "eslint-config-particle": "^2.2.1",
-    "istanbul": "1.0.0-alpha.2",
     "karma": "^1.1.1",
     "karma-browserify": "^5.1.0",
     "karma-chai": "^0.1.0",
@@ -68,6 +69,7 @@
     "karma-mocha": "^1.1.1",
     "minifyify": "^7.3.1",
     "mocha": "^2.5.1",
+    "nyc": "^15.1.0",
     "should": "^9.0.0",
     "sinon": "^7.2.5",
     "sinon-as-promised": "^4.0.3",
@@ -84,5 +86,9 @@
   "browser": {
     "http": "stream-http",
     "https": "stream-http"
+  },
+  "engines": {
+    "node": ">=12.x",
+    "npm": "8.x"
   }
 }

package/test/Agent.spec.js

@@ -1,8 +1,30 @@
 import { sinon, expect } from './test-setup';
 import Agent from '../src/Agent.js';
 
-
 describe('Agent', () => {
+	beforeEach(() => {
+		sinon.restore();
+	});
+
+	describe('constructor', () => {
+		it('calls setBaseUrl', () => {
+			const baseUrl = 'https://foo.com';
+			sinon.stub(Agent.prototype, 'setBaseUrl');
+			const agent = new Agent(baseUrl);
+			expect(agent.setBaseUrl).to.have.property('callCount', 1);
+			expect(agent.setBaseUrl.firstCall.args).to.have.lengthOf(1);
+			expect(agent.setBaseUrl.firstCall.args[0]).to.eql(baseUrl);
+		});
+
+		it('creates a prefix function instance property from the supplied baseUrl (via setBaseUrl)', () => {
+			const agent = new Agent();
+			expect(agent.prefix).to.be.a('Function');
+			// Note: validating specific .prefix function behavior seems hard without proxyquire
+			// and this test seems sufficient
+			// and redundant with the 'build request uses prefix if provided' test below
+		});
+	});
+
 	describe('sanitize files', () => {
 		it('can call sanitize will falsy value', () => {
 			const agent = new Agent();

package/test/Client.spec.js

@@ -56,8 +56,8 @@ describe('Client', () => {
 
 	describe('downloadFile', () => {
 		it('delegates to api', () => {
-			api.downloadFile = () => Promise.resolve('delegated');
-			return expect(client.downloadFile('url')).to.eventually.equal('delegated');
+			api.downloadFile = ({ uri }) => Promise.resolve(`${uri} delegated`);
+			return expect(client.downloadFile('uri')).to.eventually.equal('uri delegated');
 		});
 	});
 

package/test/EventStream-e2e-node.js

@@ -29,6 +29,7 @@ particle.getEventStream({ deviceId: 'mine', auth })
 	})
 	.catch((err) => {
 		console.error(err);
+		process.exit(1);
 	});
 /* eslint-enable no-console */
 

package/test/Particle.spec.js

@@ -2701,4 +2701,19 @@ describe('ParticleAPI', () => {
 			});
 		});
 	});
+
+	describe('setBaseUrl(baseUrl)', () => {
+		afterEach(() => {
+			sinon.restore();
+		});
+
+		it('calls agent.setBaseUrl', () => {
+			const baseUrl = 'foo';
+			sinon.stub(api.agent, 'setBaseUrl');
+			api.setBaseUrl(baseUrl);
+			expect(api.agent.setBaseUrl).to.have.property('callCount', 1);
+			expect(api.agent.setBaseUrl.firstCall.args).to.have.lengthOf(1);
+			expect(api.agent.setBaseUrl.firstCall.args[0]).to.eql(baseUrl);
+		});
+	});
 });

package/.travis.yml

@@ -1,34 +0,0 @@
-language: node_js
-node_js:
-- 6
-- 8
-- 10
-cache:
-  directories:
-  - node_modules
-sudo: false
-services:
-  - xvfb
-install:
-- npm -g install npm@5.10
-- npm install
-- npm prune
-script: "./travis_tests.sh"
-after_success:
-- npm run coveralls
-jobs:
-  include:
-  - node_js: 8
-    env: TEST_SUITE=browser
-  - stage: npm release
-    script: echo "Publishing to npm..."
-    node_js: 8
-    if: tag =~ ^v\d+\.\d+\.\d+$
-    deploy:
-      provider: npm
-      email: engineering+npm@particle.io
-      api_key:
-        secure: JzhJNTYb5bZmETc+MN7ivqeXTSUPMJ+ImGtiSXMFPNGnZc8PSwp82Q9Yx+cvno/8sSGaisleIcWDJ/4d9Vq6fNVchftu2tjCIauRUaQUf07tvO2bXW9HWmTaYCpySmvHsZD+v3FFOPEyWwkHWBGO0pQnlWOlPeJrzrpyp006uoh7shefK147zKwwUmZKJC0/VDbGOyWXeDXe1C0AapwxFovTaINir4OROmPCrmUX0ZzWQqWn6edyYZHMDDLLrAIywqiuj1f7MU5M4y+0lILVm0SC79x7fIm6ZnIk+V1KxbNYN1zrKAFLkGSRpVXbNPUULT/jxVFn+ThcSNidQf3cI5kX6sXfQLN8ue7MfMAYCcL3JZ4W0LkrsYn4RqC5t8D4VATTqsNMUn+oUDzgDm8NL8hW+RaE9hLkXEGz9NjMpXie4JThcJt/To6NEGDMd0hvzBcvKTG34xgsitM1XBnOe6uk/L4lx058hRshnIxHkctyvVQ/dZfHrRfTz6CkfC3UhSaw7ufKxBCceDId/DM2WDL2eUyy+ruJTbWhjbJv/GUBhR69atYriEqGmRmZSiA5nvj2+vTT6yrD0RbHi/CqhiEMFcZf3DI+mIrjXjmbvdKdt0cDUHazxxGSmU3S0EV+Qdvbv1pZxU/049WAnJWiwxwXHdpl9T6jZLw2e8d/xYs=
-      skip_cleanup: true
-      on:
-        tags: true

package/travis_tests.sh

@@ -1,11 +0,0 @@
-#!/bin/bash
-# Run different test suites on Travis depending on environment variables
-set -ev
-if [ "${TEST_SUITE}" = "browser" ]; then
-  # Load display for browser tests
-  export DISPLAY=:99.0
-  npm run build
-  npm run test:browser
-else
-  npm run test
-fi