add front end build test

leoliew · Sep 9, 2016 · 6897380 · 6897380
1 parent 414e9e0
commit 6897380
Show file tree

Hide file tree

Showing 7 changed files with 75 additions and 87 deletions.
diff --git a/src/app/steps/writing.js b/src/app/steps/writing.js
@@ -18,9 +18,16 @@ module.exports = function(){
   this.copy('README.md', 'README.md');
   var sails_config = {
     grunt: false,
-    pm2:false
+    gulp: false,
+    pm2: false
   };
-  if(this.answers['grunt:enabled']){
+  // if(this.answers['grunt:enabled']){
+  //   sails_config.grunt = true
+  // }
+  if (this.answers['febuild:chosen'] == 'Gulp') {
+    sails_config.gulp = true
+  }
+  if (this.answers['febuild:chosen'] == 'Grunt'){
     sails_config.grunt = true
   }
   if(this.answers['pm2:enabled']){

diff --git a/src/app/templates/sailsrc b/src/app/templates/sailsrc
@@ -1,6 +1,9 @@
 {
   "generators": {
     "modules": {
+      <%_ if (sails_config.gulp){_%>
+        "gulp": "sails-generate-gulp"
+      <%_}_%>
     }
   },
   "hooks": {

diff --git a/src/febuild/templates/gulp/tasks/config/cssmin.js b/src/febuild/templates/gulp/tasks/config/cssmin.js
diff --git a/src/febuild/templates/gulp/tasks/config/uglify.js b/src/febuild/templates/gulp/tasks/config/uglify.js
diff --git a/test/unit/app.test.js b/test/unit/app.test.js
@@ -13,7 +13,7 @@ describe('sails-diy:app', function(){
           [generators.test.createDummyGenerator(), 'sails-diy:config'],
           [generators.test.createDummyGenerator(), 'sails-diy:controller'],
           [generators.test.createDummyGenerator(), 'sails-diy:cron'],
-          [generators.test.createDummyGenerator(), 'sails-diy:grunt'],
+          [generators.test.createDummyGenerator(), 'sails-diy:febuild'],
           [generators.test.createDummyGenerator(), 'sails-diy:hook'],
           [generators.test.createDummyGenerator(), 'sails-diy:logger'],
           [generators.test.createDummyGenerator(), 'sails-diy:model'],
@@ -79,7 +79,7 @@ describe('sails-diy:app', function(){
           [generators.test.createDummyGenerator(), 'sails-diy:config'],
           [generators.test.createDummyGenerator(), 'sails-diy:controller'],
           [generators.test.createDummyGenerator(), 'sails-diy:cron'],
-          [generators.test.createDummyGenerator(), 'sails-diy:grunt'],
+          [generators.test.createDummyGenerator(), 'sails-diy:febuild'],
           [generators.test.createDummyGenerator(), 'sails-diy:hook'],
           [generators.test.createDummyGenerator(), 'sails-diy:logger'],
           [generators.test.createDummyGenerator(), 'sails-diy:model'],
@@ -94,7 +94,7 @@ describe('sails-diy:app', function(){
         })
         .withPrompts({
           'cron:enabled': false,
-          'grunt:enabled': true,
+          'febuild:chosen': 'Grunt',
           'views:chosen': 'Jade'
         })
         .on('end', done);

diff --git a/test/unit/febuild.test.js b/test/unit/febuild.test.js
@@ -0,0 +1,60 @@
+var path = require('path');
+var generators = require('yeoman-generator');
+
+describe('sails-diy:febuild', function() {
+
+  describe('Should properly scaffold default grunt tool', function () {
+    before(function (done) {
+      generators.test
+        .run(path.join(__dirname, '../../src/febuild'))
+        .withArguments(['Grunt'])
+        .on('end', done)
+    });
+
+    it('Should properly create configuration files', function(){
+      generators.assert.file([
+        'Gruntfile.js',
+        'tasks/config/clean.js',
+        'tasks/config/copy.js',
+        'tasks/config/less.js',
+        'tasks/config/sync.js',
+        'tasks/config/watch.js',
+        'tasks/register/compileAssets.js',
+        'tasks/register/default.js',
+        'tasks/register/prod.js',
+        'tasks/register/syncAssets.js',
+        'tasks/pipeline.js',
+        'tasks/README.md'
+      ]);
+    });
+  });
+
+
+  describe('Should properly scaffold default gulp tool', function () {
+    before(function (done) {
+      generators.test
+        .run(path.join(__dirname, '../../src/febuild'))
+        .withArguments(['Gulp'])
+        .on('end', done);
+    });
+
+    it('Should properly create configuration files', function(){
+      generators.assert.file([
+        'gulpfile.js',
+        'tasks/config/clean.js',
+        'tasks/config/copy.js',
+        'tasks/config/less.js',
+        'tasks/config/sync.js',
+        'tasks/config/watcher.js',
+        'tasks/config/filerev.js',
+        'tasks/config/replacerev.js',
+        'tasks/register/compileAssets.js',
+        'tasks/register/default.js',
+        'tasks/register/prod.js',
+        'tasks/register/syncAssets.js',
+        'tasks/pipeline.js',
+        'tasks/README.md'
+      ]);
+    });
+  });
+});
diff --git a/test/unit/grunt.test.js b/test/unit/grunt.test.js