cordova-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From agri...@apache.org
Subject cordova-cli git commit: CB-8129 Adds 'npm run cover' command to generate tests coverage report
Date Tue, 23 Dec 2014 16:43:41 GMT
Repository: cordova-cli
Updated Branches:
  refs/heads/master 17ce90623 -> 3afab318b


CB-8129 Adds 'npm run cover' command to generate tests coverage report


Project: http://git-wip-us.apache.org/repos/asf/cordova-cli/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-cli/commit/3afab318
Tree: http://git-wip-us.apache.org/repos/asf/cordova-cli/tree/3afab318
Diff: http://git-wip-us.apache.org/repos/asf/cordova-cli/diff/3afab318

Branch: refs/heads/master
Commit: 3afab318bfcf976b3add8db24cd1fb0fac7cb4d5
Parents: 17ce906
Author: Vladimir Kotikov <v-vlkoti@microsoft.com>
Authored: Mon Dec 15 10:54:05 2014 +0300
Committer: Vladimir Kotikov <v-vlkoti@microsoft.com>
Committed: Mon Dec 15 10:54:05 2014 +0300

----------------------------------------------------------------------
 .gitignore       | 1 +
 README.md        | 4 ++++
 package.json     | 4 +++-
 spec/cli.spec.js | 3 ---
 4 files changed, 8 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/3afab318/.gitignore
----------------------------------------------------------------------
diff --git a/.gitignore b/.gitignore
index 3e180c5..f2b11bc 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,4 @@
+coverage
 node_modules
 npm-debug.log
 temp

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/3afab318/README.md
----------------------------------------------------------------------
diff --git a/README.md b/README.md
index dffac20..14c5202 100644
--- a/README.md
+++ b/README.md
@@ -239,6 +239,10 @@ The directory structure of KewlApp now looks like this:
 
     npm test
 
+## Get test coverage reports
+
+    npm run cover
+
 ## TO-DO + Issues
 
 Please check [Cordova issues with the CLI Component](http://issues.cordova.io). If you find
issues with this tool, please be so kind as to include relevant information needed to debug
issues such as:

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/3afab318/package.json
----------------------------------------------------------------------
diff --git a/package.json b/package.json
index aadbf4a..61c0e17 100644
--- a/package.json
+++ b/package.json
@@ -12,7 +12,8 @@
     "cordova": "./bin/cordova"
   },
   "scripts": {
-    "test": "jasmine-node --captureExceptions --color spec"
+    "test": "node node_modules/jasmine-node/bin/jasmine-node --captureExceptions --color
spec",
+    "cover": "node node_modules/istanbul/lib/cli.js cover --root src --print detail node_modules/jasmine-node/bin/jasmine-node
-- spec"
   },
   "repository": {
     "type": "git",
@@ -34,6 +35,7 @@
     "underscore":"1.7.0"
   },
   "devDependencies": {
+    "istanbul": "^0.3.4",
     "jasmine-node": "1.14.5"
   },
   "author": "Anis Kadri",

http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/3afab318/spec/cli.spec.js
----------------------------------------------------------------------
diff --git a/spec/cli.spec.js b/spec/cli.spec.js
index c619617..81f6b6d 100644
--- a/spec/cli.spec.js
+++ b/spec/cli.spec.js
@@ -30,9 +30,6 @@ describe("cordova cli", function () {
         // logging events registered as a result of the "--verbose" flag in
         // CLI testing below would cause lots of logging messages printed out by other specs.
         spyOn(events, "on");
-        // Each call to cli() registers another listener for uncaughtException.
-        // This results in a warning when too many of them are registered.
-        process.removeAllListeners();
         spyOn(console, 'log');
     });
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org


Mime
View raw message