#2 Add test cases

Author: tomasz-oponowicz

Gruntfile.js

@@ -28,35 +28,21 @@ module.exports = function(grunt) {
       tests: ['tmp']
     },
 
-    // Configuration to be run (and then tested).
-    javascript_obfuscator: {
-      default_options: {
-        options: {
-        },
-        files: {
-          'tmp/default_options': ['test/fixtures/sample1.js']
-        }
-      }
-    },
-
     // Unit tests.
     nodeunit: {
       tests: ['test/*_test.js']
     }
 
   });
 
-  // Actually load this plugin's task(s).
-  grunt.loadTasks('tasks');
-
   // These plugins provide necessary tasks.
   grunt.loadNpmTasks('grunt-contrib-jshint');
   grunt.loadNpmTasks('grunt-contrib-clean');
   grunt.loadNpmTasks('grunt-contrib-nodeunit');
 
   // Whenever the "test" task is run, first clean the "tmp" dir, then run this
   // plugin's task(s), then test the result.
-  grunt.registerTask('test', ['clean', 'javascript_obfuscator', 'nodeunit']);
+  grunt.registerTask('test', ['clean', 'nodeunit']);
 
   // By default, lint and run all tests.
   grunt.registerTask('default', ['jshint', 'test']);

tasks/javascript_obfuscator.js

@@ -11,39 +11,33 @@
 var JavaScriptObfuscator = require('javascript-obfuscator');
 
 function concat(grunt, paths) {
+  console.log("paths", paths);
+
   if (paths.length === 0) {
-    grunt.log.warn('Source files don\'t exist');
-    return '';
+    throw new Error('Source files not found.');
   }
 
-  return paths.filter(function(path) {
-    // Warn on and remove invalid source files (if nonull was set).
-    if (!grunt.file.exists(path)) {
-      grunt.log.warn('Source file "' + path + '" not found.');
-      return false;
-    } else {
-      return true;
-    }
-  }).map(function(path) {
+  return paths.map(function(path) {
+
     // Read file source.
     return grunt.file.read(path);
   }).join(grunt.util.normalizelf(';'));
 }
 
 module.exports = function(grunt) {
-
-  // Please see the Grunt documentation for more information regarding task
-  // creation: http://gruntjs.com/creating-tasks
-
   grunt.registerMultiTask('javascript_obfuscator', 'Obfuscates JavaScript files.', function() {
+
     // Merge task-specific and/or target-specific options with these defaults.
     var options = this.options({});
 
-    // Iterate over all specified file groups.
     this.files.forEach(function(f) {
-      // Concat specified files.
+      var src;
 
-      var src = concat(grunt, f.src);
+      try {
+        src = concat(grunt, f.src);
+      } catch (e) {
+        return grunt.fail.warn(e);
+      }
 
       src = JavaScriptObfuscator.obfuscate(src, options).getObfuscatedCode();
 
@@ -54,5 +48,4 @@ module.exports = function(grunt) {
       grunt.log.writeln('File "' + f.dest + '" created.');
     });
   });
-
 };

test/fixtures/first_sample.js

@@ -0,0 +1,2 @@
+var FIRST_LOCAL_VARIABLE = 1;
+console.log(FIRST_LOCAL_VARIABLE + 'FIRST_STRING_LITERAL');

test/fixtures/gruntfile_generic.js

@@ -0,0 +1,12 @@
+exports.setup = function(configuration) {
+    return function (grunt) {
+        'use strict';
+
+        grunt.initConfig({
+            javascript_obfuscator: configuration
+        });
+
+        grunt.loadTasks('../../tasks');
+        grunt.registerTask('default', 'javascript_obfuscator');
+    };
+};

test/fixtures/gruntfile_multiple_files.js

@@ -0,0 +1,8 @@
+module.exports = require('./gruntfile_generic').setup({
+    options: {},
+    multiple_files: {
+        files: {
+            '../../tmp/multiple_files': ['first_sample.js', 'second_sample.js']
+        }
+    }
+});

test/fixtures/gruntfile_single_file.js

@@ -0,0 +1,8 @@
+module.exports = require('./gruntfile_generic').setup({
+    options: {},
+    single_file: {
+        files: {
+            '../../tmp/single_file': ['first_sample.js']
+        }
+    }
+});

test/fixtures/gruntfile_unknown_file.js

@@ -0,0 +1,8 @@
+module.exports = require('./gruntfile_generic').setup({
+    options: {},
+    unknown_file: {
+        files: {
+            '../../tmp/unknown_file': ['unknown.js']
+        }
+    }
+});

test/fixtures/sample1.js

@@ -0,0 +1,2 @@
+var SECOND_LOCAL_VARIABLE = 2;
+console.log(SECOND_LOCAL_VARIABLE + 'SECOND_STRING_LITERAL');

test/fixtures/second_sample.js

@@ -0,0 +1,8 @@
+var cp = require("child_process");
+
+exports.callGruntfile = function (filename, whenDoneCallback) {
+    var command, options;
+    command = "grunt --gruntfile " + filename + " --no-color";
+    options = {cwd: 'test/'};
+    cp.exec(command, options, whenDoneCallback);
+};