[Pkg-javascript-commits] [node-coveralls] 331/332: merge patched into master

Bastien Roucariès rouca at moszumanska.debian.org
Thu Nov 9 13:54:17 UTC 2017


This is an automated email from the git hooks/post-receive script.

rouca pushed a commit to branch master
in repository node-coveralls.

commit ee799a2821b96408b2ea6982e24b0d581465f6b1
Merge: cff85c8 2297ed8
Author: Bastien ROUCARIÈS <roucaries.bastien at gmail.com>
Date:   Thu Nov 9 14:42:33 2017 +0100

    merge patched into master

 debian/.git-dpm                                        |  4 ++--
 .../0002-Do-not-use-sinon-restore-but-sinon.patch      | 18 ++++++++++++++++--
 .../patches/0003-Fix-path-with-version-in-test.patch   |  2 +-
 ...0004-Disable-a-test-that-need-a-full-git-tree.patch |  2 +-
 package.json                                           |  2 +-
 5 files changed, 21 insertions(+), 7 deletions(-)

diff --cc debian/.git-dpm
index c88b4b1,0000000..b438fb9
mode 100644,000000..100644
--- a/debian/.git-dpm
+++ b/debian/.git-dpm
@@@ -1,8 -1,0 +1,8 @@@
 +# see git-dpm(1) from git-dpm package
- 6c494f62c66b6f55ec9a76abe85f7a05d7ae31ad
- 6c494f62c66b6f55ec9a76abe85f7a05d7ae31ad
++2297ed854a5fd26ff119376fbaaf4a19d3c38bc3
++2297ed854a5fd26ff119376fbaaf4a19d3c38bc3
 +0fd1604e6190e44c4b6c78bdc9bc7024a4269642
 +0fd1604e6190e44c4b6c78bdc9bc7024a4269642
 +node-coveralls_3.0.0.orig.tar.gz
 +22ef730330538080d29b8c151dc9146afde88a99
 +16987
diff --cc debian/patches/0002-Do-not-use-sinon-restore-but-sinon.patch
index e795198,0000000..aad6677
mode 100644,000000..100644
--- a/debian/patches/0002-Do-not-use-sinon-restore-but-sinon.patch
+++ b/debian/patches/0002-Do-not-use-sinon-restore-but-sinon.patch
@@@ -1,65 -1,0 +1,79 @@@
- From 5c1bcb89048f6f7465dba47f686cecbb5c969a79 Mon Sep 17 00:00:00 2001
++From 907985f066c8b6ff040e21d645b5cc35e59bb3cb Mon Sep 17 00:00:00 2001
 +From: =?UTF-8?q?Bastien=20ROUCARI=C3=88S?= <roucaries.bastien at gmail.com>
 +Date: Thu, 7 Sep 2017 22:04:18 +0200
 +Subject: Do not use sinon-restore but sinon
 +
 +Forwarded: no
 +---
++ package.json            | 2 +-
 + test/handleInput.js     | 6 +++---
 + test/logger.js          | 2 +-
 + test/sendToCoveralls.js | 4 ++--
-  3 files changed, 6 insertions(+), 6 deletions(-)
++ 4 files changed, 7 insertions(+), 7 deletions(-)
 +
++diff --git a/package.json b/package.json
++index 25e840d..45b2d34 100644
++--- a/package.json
+++++ b/package.json
++@@ -42,7 +42,7 @@
++     "mocha": "^3.2.0",
++     "mocha-lcov-reporter": "^1.2.0",
++     "should": "^9.0.2",
++-    "sinon-restore": "^1.0.1",
+++    "sinon": "^1.0.0",
++     "snyk": "^1.23.3"
++   },
++   "engines": {
 +diff --git a/test/handleInput.js b/test/handleInput.js
 +index dc88902..cc51e7a 100644
 +--- a/test/handleInput.js
 ++++ b/test/handleInput.js
 +@@ -1,13 +1,13 @@
 + var should = require('should');
 +-var sinon = require('sinon-restore');
 ++var sinon = require('sinon').sandbox.create();
 + var index = require('../index');
 + var fs = require('fs');
 + logger = require('log-driver')({level : false});
 + 
 + describe("handleInput", function(){
 +    afterEach(function() {
 +-        sinon.restoreAll();
 +-      });
 ++        sinon.restore();
 ++  });
 +   it ("returns an error when there's an error getting options", function(done){
 +     sinon.stub(index, 'getOptions', function(cb){
 +       return cb("some error", {}); 
 +diff --git a/test/logger.js b/test/logger.js
 +index 19767df..5430829 100644
 +--- a/test/logger.js
 ++++ b/test/logger.js
 +@@ -1,5 +1,5 @@
 + var should = require('should');
 +-var sinon = require('sinon-restore');
 ++var sinon = require('sinon');
 + var index = require('../index');
 + 
 + describe("logger", function(){
 +diff --git a/test/sendToCoveralls.js b/test/sendToCoveralls.js
 +index 6121bbb..3589bd8 100644
 +--- a/test/sendToCoveralls.js
 ++++ b/test/sendToCoveralls.js
 +@@ -1,6 +1,6 @@
 + var should = require('should');
 + var request = require('request');
 +-var sinon = require('sinon-restore');
 ++var sinon = require('sinon').sandbox.create();
 + var index = require('../index');
 + logger = require('log-driver')({level : false});
 + 
 +@@ -11,7 +11,7 @@ describe("sendToCoveralls", function(){
 +   });
 + 
 +   afterEach(function() {
 +-    sinon.restoreAll();
 ++    sinon.restore();
 +     if (realCoverallsHost !== undefined) {
 +       process.env.COVERALLS_ENDPOINT = realCoverallsHost;
 +     } else {
diff --cc debian/patches/0003-Fix-path-with-version-in-test.patch
index 9a0ab49,0000000..a460d85
mode 100644,000000..100644
--- a/debian/patches/0003-Fix-path-with-version-in-test.patch
+++ b/debian/patches/0003-Fix-path-with-version-in-test.patch
@@@ -1,23 -1,0 +1,23 @@@
- From b1ee5034aeffaf8bcf087f44405fccdc32e5385e Mon Sep 17 00:00:00 2001
++From 4e86ed530e913d7b9652507784d850e1301d313a Mon Sep 17 00:00:00 2001
 +From: =?UTF-8?q?Bastien=20ROUCARI=C3=88S?= <roucaries.bastien at gmail.com>
 +Date: Thu, 7 Sep 2017 22:54:41 +0200
 +Subject: Fix path with version in test
 +
 +Forwarded: no
 +---
 + test/detectLocalGit.js | 2 +-
 + 1 file changed, 1 insertion(+), 1 deletion(-)
 +
 +diff --git a/test/detectLocalGit.js b/test/detectLocalGit.js
 +index 64da493..f47163f 100644
 +--- a/test/detectLocalGit.js
 ++++ b/test/detectLocalGit.js
 +@@ -57,7 +57,7 @@ function _cleanTempGitDir() {
 + 
 + function _deleteFolderRecursive(dir) {
 + 
 +-  if (!dir.match('node-coveralls/test')) {
 ++  if (!dir.match('node-coveralls[^/]*/test')) {
 +     throw new Error('Tried to clean a temp git directory that did not match path: node-coveralls/test');
 +   }
 + 
diff --cc debian/patches/0004-Disable-a-test-that-need-a-full-git-tree.patch
index 94230ff,0000000..bf3fb80
mode 100644,000000..100644
--- a/debian/patches/0004-Disable-a-test-that-need-a-full-git-tree.patch
+++ b/debian/patches/0004-Disable-a-test-that-need-a-full-git-tree.patch
@@@ -1,75 -1,0 +1,75 @@@
- From 6c494f62c66b6f55ec9a76abe85f7a05d7ae31ad Mon Sep 17 00:00:00 2001
++From 2297ed854a5fd26ff119376fbaaf4a19d3c38bc3 Mon Sep 17 00:00:00 2001
 +From: =?UTF-8?q?Bastien=20ROUCARI=C3=88S?= <roucaries.bastien at gmail.com>
 +Date: Thu, 7 Sep 2017 23:06:35 +0200
 +Subject: Disable a test that need a full git tree
 +
 +Forwarded; no
 +---
 + test/fetchGitData.js | 3 ++-
 + test/getOptions.js   | 7 ++++++-
 + 2 files changed, 8 insertions(+), 2 deletions(-)
 +
 +diff --git a/test/fetchGitData.js b/test/fetchGitData.js
 +index 012e129..cd42605 100644
 +--- a/test/fetchGitData.js
 ++++ b/test/fetchGitData.js
 +@@ -163,6 +163,7 @@ describe("fetchGitData", function(){
 +       done();
 +     });
 +   });
 ++  /*
 +   it("should execute git commands when a valid commit hash is given", function(done) {
 +     process.env.COVERALLS_GIT_COMMIT = "HEAD";
 +     process.env.COVERALLS_GIT_BRANCH = "master";
 +@@ -178,5 +179,5 @@ describe("fetchGitData", function(){
 +       options.remotes.length.should.be.above(0);
 +       done();
 +     });
 +-  });
 ++  });*/
 + });
 +diff --git a/test/getOptions.js b/test/getOptions.js
 +index 51e4fa0..f03ca94 100644
 +--- a/test/getOptions.js
 ++++ b/test/getOptions.js
 +@@ -25,9 +25,11 @@ describe("getBaseOptions", function(){
 +   it ("should detect detached git head if no hash passed in", function(done) {
 +     testGitDetachedHeadDetection(getBaseOptions, done);
 +   });
 ++  /*
 +   it ("should fail local Git detection if no .git directory", function(done) {
 +     testNoLocalGit(getBaseOptions, done);
 +   });
 ++  */
 +   it ("should set repo_token if it exists", function(done){
 +     testRepoToken(getBaseOptions, done);
 +   });
 +@@ -101,9 +103,11 @@ describe("getOptions", function(){
 +   it ("should detect detached git head if no hash passed in", function(done) {
 +     testGitDetachedHeadDetection(getOptions, done);
 +   });
 ++  /*
 +   it ("should fail local Git detection if no .git directory", function(done) {
 +     testNoLocalGit(getOptions, done);
 +   });
 ++  */
 +   it ("should set repo_token if it exists", function(done){
 +     testRepoToken(getOptions, done);
 +   });
 +@@ -143,6 +147,7 @@ describe("getOptions", function(){
 +   it ("should set service_name and service_job_id if it's running via Surf", function(done){
 +     testSurf(getOptions, done);
 +   });
 ++  /*
 +   it ("should override set options with user options", function(done){
 +     var userOptions = {service_name: 'OVERRIDDEN_SERVICE_NAME'};
 +     process.env.COVERALLS_SERVICE_NAME = "SERVICE_NAME";
 +@@ -150,7 +155,7 @@ describe("getOptions", function(){
 +       options.service_name.should.equal("OVERRIDDEN_SERVICE_NAME");
 +       done();
 +     }, userOptions);
 +-  });
 ++  });*/
 + });
 + 
 + var testServiceJobId = function(sut, done){

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/node-coveralls.git



More information about the Pkg-javascript-commits mailing list