From bb743fffd83d7c99748769dd8b73b35bb7f8dc96 Mon Sep 17 00:00:00 2001 From: "greenkeeper[bot]" Date: Fri, 2 Dec 2016 21:56:18 -0800 Subject: [PATCH] =?UTF-8?q?Update=20dependencies=20to=20enable=20Greenkeep?= =?UTF-8?q?er=20=F0=9F=8C=B4=20(#2621)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * update most dependencies (less `lodash.create` and `phantomjs`) * lint fixes for ESLint * sanity test uses npm v1.4.x under Node.js v0.10 only * only run node-specific tests on appveyor; add node v7 --- appveyor.yml | 3 ++- lib/runner.js | 4 ++-- lib/utils.js | 6 ++++-- package.json | 28 +++++++++++++++++----------- scripts/travis-before-install.sh | 20 +++++++++++++++++--- test/acceptance/misc/many.spec.js | 1 + test/reporters/base.spec.js | 16 ++++++++-------- 7 files changed, 51 insertions(+), 27 deletions(-) diff --git a/appveyor.yml b/appveyor.yml index 539b8b073d..6a2a6e1811 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -1,5 +1,6 @@ environment: matrix: + - nodejs_version: '7' - nodejs_version: '6' - nodejs_version: '4' - nodejs_version: '0.12' @@ -20,7 +21,7 @@ clone_depth: 1 test_script: - node --version - npm --version - - npm test + - make test-node skip_commits: message: /\[ci\s+skip\]/ diff --git a/lib/runner.js b/lib/runner.js index 697ffb65ac..1d3dd959da 100644 --- a/lib/runner.js +++ b/lib/runner.js @@ -681,9 +681,9 @@ Runner.prototype.runSuite = function (suite, fn) { */ Runner.prototype.uncaught = function (err) { if (err) { - debug('uncaught exception %s', err !== function () { + debug('uncaught exception %s', err === (function () { return this; - }.call(err) ? err : (err.message || err)); + }.call(err)) ? (err.message || err) : err); } else { debug('uncaught undefined exception'); err = undefinedError(); diff --git a/lib/utils.js b/lib/utils.js index bf430b9168..657c75838c 100644 --- a/lib/utils.js +++ b/lib/utils.js @@ -416,7 +416,7 @@ var type = exports.type = function type (value) { return 'buffer'; } return Object.prototype.toString.call(value) - .replace(/^\[.+\s(.+?)\]$/, '$1') + .replace(/^\[.+\s(.+?)]$/, '$1') .toLowerCase(); }; @@ -741,7 +741,9 @@ exports.stackTraceFilter = function () { if (is.node) { cwd = process.cwd() + slash; } else { - cwd = (typeof location === 'undefined' ? window.location : location).href.replace(/\/[^\/]*$/, '/'); + cwd = (typeof location === 'undefined' + ? window.location + : location).href.replace(/\/[^/]*$/, '/'); slash = '/'; } diff --git a/package.json b/package.json index f1afe9e6e8..71bfad2882 100644 --- a/package.json +++ b/package.json @@ -304,10 +304,10 @@ "dependencies": { "browser-stdout": "1.3.0", "commander": "2.9.0", - "debug": "2.2.0", - "diff": "1.4.0", + "debug": "2.3.3", + "diff": "3.1.0", "escape-string-regexp": "1.0.5", - "glob": "7.0.5", + "glob": "7.1.1", "growl": "1.9.2", "json3": "3.3.2", "lodash.create": "3.1.1", @@ -318,24 +318,24 @@ "assert": "^1.4.1", "browserify": "^13.0.0", "coffee-script": "^1.10.0", - "eslint": "^2.13.1", - "eslint-config-semistandard": "^6.0.2", - "eslint-config-standard": "^5.0.0", - "eslint-plugin-promise": "^2.0.1", - "eslint-plugin-standard": "1.3.2", + "eslint": "^3.11.1", + "eslint-config-semistandard": "^7.0.0", + "eslint-config-standard": "^6.2.1", + "eslint-plugin-promise": "^3.4.0", + "eslint-plugin-standard": "2.0.1", "expect.js": "^0.3.1", "karma": "^1.1.0", "karma-browserify": "^5.0.5", "karma-chrome-launcher": "^2.0.0", "karma-expect": "^1.1.2", "karma-mocha": "^1.3.0", - "karma-phantomjs-launcher": "^0.2.3", + "karma-phantomjs-launcher": "^1.0.2", "karma-sauce-launcher": "^1.0.0", "karma-spec-reporter": "0.0.26", "os-name": "^2.0.1", "phantomjs": "1.9.8", "rimraf": "^2.5.2", - "should": "^9.0.2", + "should": "^11.1.1", "through2": "^2.0.1", "watchify": "^3.7.0" }, @@ -361,5 +361,11 @@ "supports-color": false }, "homepage": "https://mochajs.org", - "logo": "https://cldup.com/S9uQ-cOLYz.svg" + "logo": "https://cldup.com/S9uQ-cOLYz.svg", + "greenkeeper": { + "ignore": [ + "phantomjs", + "lodash.create" + ] + } } diff --git a/scripts/travis-before-install.sh b/scripts/travis-before-install.sh index 9913070ae9..7d2f303007 100755 --- a/scripts/travis-before-install.sh +++ b/scripts/travis-before-install.sh @@ -1,5 +1,19 @@ #!/usr/bin/env bash -npm install --production -# this avoids our mocha.opts (and thus devDependencies) in a roundabout way -./bin/mocha --opts /dev/null --reporter spec test/sanity/sanity.spec.js +runSanityTest () { + # avoids our mocha.opts (and thus devDependencies) in a roundabout way + ./bin/mocha --opts /dev/null --reporter spec test/sanity/sanity.spec.js +} + +# install npm@1.4 locally, and try to install production dependencies w/ it. +if [[ ${TRAVIS_NODE_VERSION} =~ 'v0.10' ]] +then + echo "Downgrading to npm v1.4.x..." + npm install npm@~1.4 + ./node_modules/.bin/npm install --production + runSanityTest + rm -rf node_modules/ +else + npm install --production + runSanityTest +fi diff --git a/test/acceptance/misc/many.spec.js b/test/acceptance/misc/many.spec.js index d4e4a1e806..d2297767dd 100644 --- a/test/acceptance/misc/many.spec.js +++ b/test/acceptance/misc/many.spec.js @@ -14,6 +14,7 @@ describe('a load of tests', function () { for (var i = 0; i < iterations; ++i) { a += i; } + return a; } function addTest () { diff --git a/test/reporters/base.spec.js b/test/reporters/base.spec.js index 6282d882ff..43d45e4b7d 100644 --- a/test/reporters/base.spec.js +++ b/test/reporters/base.spec.js @@ -44,7 +44,7 @@ describe('Base reporter', function () { Base.list([test]); errOut = stdout.join('\n'); - errOut.should.match(/\- actual/); + errOut.should.match(/- actual/); errOut.should.match(/\+ expected/); }); @@ -58,7 +58,7 @@ describe('Base reporter', function () { Base.list([test]); errOut = stdout.join('\n'); - errOut.should.match(/\- actual/); + errOut.should.match(/- actual/); errOut.should.match(/\+ expected/); }); @@ -72,7 +72,7 @@ describe('Base reporter', function () { Base.list([test]); errOut = stdout.join('\n'); - errOut.should.not.match(/\- actual/); + errOut.should.not.match(/- actual/); errOut.should.not.match(/\+ expected/); }); @@ -85,7 +85,7 @@ describe('Base reporter', function () { Base.list([test]); errOut = stdout.join('\n'); - errOut.should.not.match(/\- actual/); + errOut.should.not.match(/- actual/); errOut.should.not.match(/\+ expected/); }); }); @@ -127,7 +127,7 @@ describe('Base reporter', function () { errOut = stdout.join('\n'); errOut.should.match(/"key"/); errOut.should.match(/test/); - errOut.should.match(/\- actual/); + errOut.should.match(/- actual/); errOut.should.match(/\+ expected/); }); @@ -147,7 +147,7 @@ describe('Base reporter', function () { errOut = stdout.join('\n'); errOut.should.match(/"hasOwnProperty"/); errOut.should.match(/test/); - errOut.should.match(/\- actual/); + errOut.should.match(/- actual/); errOut.should.match(/\+ expected/); }); @@ -166,8 +166,8 @@ describe('Base reporter', function () { errOut = stdout.join('\n'); errOut.should.match(/\+true/); - errOut.should.match(/\-false/); - errOut.should.match(/\- actual/); + errOut.should.match(/-false/); + errOut.should.match(/- actual/); errOut.should.match(/\+ expected/); } });