#8 coveralls is now optional

biojs · Oct 5, 2014 · f2b3969 · f2b3969
1 parent 2e34ce9
commit f2b3969
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "slush-biojs",
-  "version": "0.2.3",
+  "version": "0.2.4",
   "description": "A slush generator for BioJS modules",
   "keywords": [
     "slushgenerator",

diff --git a/slushfile.js b/slushfile.js
@@ -40,6 +40,11 @@ gulp.task('default', function(done) {
       type: "confirm",
       default: true,
       message: 'A visualization lib?',
+   }, {
+     name: 'coverage',
+     type: "confirm",
+     default: true,
+     message: 'Check code coverage?',
     },{
         type: 'list',
         name: 'license',
@@ -77,19 +82,28 @@ gulp.task('default', function(done) {
             var files = [__dirname + '/templates/**'];
             if (answers.license === 'MIT') {
                 files.push('!' + __dirname + '/templates/LICENSE_BSD');
-                files.push('!' + __dirname + '/templates/LICENSE_APACHE2');
+                files.push('!' + __dirname + '/templates/LICENSE_APACHE');
             }else if (answers.license === 'Apache 2') {
                 files.push('!' + __dirname + '/templates/LICENSE_BSD');
                 files.push('!' + __dirname + '/templates/LICENSE_MIT');
             } else {
                 files.push('!' + __dirname + '/templates/LICENSE_MIT');
-                files.push('!' + __dirname + '/templates/LICENSE_APACHE2');
+                files.push('!' + __dirname + '/templates/LICENSE_APACHE');
             }
+
             if (!answers.vis) {
               files.push('!' + __dirname + '/snippets/**');
               files.push('!' + __dirname + '/test/index.html');
               files.push('!' + __dirname + '/test/dom/**');
             }
+
+            //TODO
+            answers.coverage = false;
+
+            if (!answers.coverage) {
+              files.push('!' + __dirname + '/coverage/**');
+            }
+
             answers.keywords = answers.keywords.split(",");
             // toLower
             answers.keywords = answers.keywords.join('|').toLowerCase().split('|');

diff --git a/templates/gulpfile.js b/templates/gulpfile.js
@@ -49,7 +49,8 @@ var packageConfig = require('./package.json');
 
 // a failing test breaks the whole build chain
 gulp.task('build', ['build-browser', 'build-browser-gzip']);
-gulp.task('default', ['lint', 'test', 'coveralls', 'build']);
+gulp.task('default', ['lint', 'test', <% if(coverage){ %>'coveralls',<% } %> 'build']);
+
 
 gulp.task('lint', function() {
   return gulp.src('./lib/*.js')
@@ -88,10 +89,12 @@ gulp.task('build-test',['init'], function() {
 
 <% } %>
 
+<% if (coverage){ %>
 gulp.task('coveralls', function () {
     return gulp.src('coverage/lcov.info')
 	.pipe(coveralls());
 });
+<% } %>
 
 gulp.task('test-watch', function() {
    gulp.watch(['./src/**/*.js','./lib/**/*.js', './test/**/*.js'], function() {

diff --git a/templates/package.json b/templates/package.json
@@ -22,7 +22,7 @@
   ],
   "main": "lib/<%= appNameSlug %>",
   "scripts": {
-    "test": "mocha --require blanket --reporter mocha-lcov-reporter > coverage/lcov.info && gulp test && gulp coveralls",
+    "test": "gulp",
 	"build": "gulp build",
     "build-browser": "gulp build-browser",
 	"build-browser-min": "gulp build-browser-gzip",