Basic implementation and test

Author: tomasz-oponowicz

Gruntfile.js

@@ -34,16 +34,7 @@ module.exports = function(grunt) {
         options: {
         },
         files: {
-          'tmp/default_options': ['test/fixtures/testing', 'test/fixtures/123']
-        }
-      },
-      custom_options: {
-        options: {
-          separator: ': ',
-          punctuation: ' !!!'
-        },
-        files: {
-          'tmp/custom_options': ['test/fixtures/testing', 'test/fixtures/123']
+          'tmp/default_options': ['test/fixtures/sample1.js']
         }
       }
     },

package.json

@@ -31,7 +31,11 @@
     "grunt-cli": "^1.2.0",
     "grunt-contrib-clean": "^0.5.0",
     "grunt-contrib-jshint": "^0.9.2",
-    "grunt-contrib-nodeunit": "^0.3.3"
+    "grunt-contrib-nodeunit": "^0.3.3",
+    "javascript-obfuscator": "0.7.2"
+  },
+  "peerDependencies": {
+    "javascript-obfuscator": ">=0.7.2"
   },
   "keywords": [
     "gruntplugin"

tasks/javascript_obfuscator.js

@@ -8,36 +8,44 @@
 
 'use strict';
 
+var JavaScriptObfuscator = require('javascript-obfuscator');
+
+function concat(grunt, paths) {
+  if (paths.length === 0) {
+    grunt.log.warn('Source files don\'t exist');
+    return '';
+  }
+
+  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) {
+    // 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({
-      punctuation: '.',
-      separator: ', '
-    });
+    var options = this.options({});
 
     // Iterate over all specified file groups.
     this.files.forEach(function(f) {
       // Concat specified files.
-      var src = f.src.filter(function(filepath) {
-        // Warn on and remove invalid source files (if nonull was set).
-        if (!grunt.file.exists(filepath)) {
-          grunt.log.warn('Source file "' + filepath + '" not found.');
-          return false;
-        } else {
-          return true;
-        }
-      }).map(function(filepath) {
-        // Read file source.
-        return grunt.file.read(filepath);
-      }).join(grunt.util.normalizelf(options.separator));
-
-      // Handle options.
-      src += options.punctuation;
+
+      var src = concat(grunt, f.src);
+
+      src = JavaScriptObfuscator.obfuscate(src, options).getObfuscatedCode();
 
       // Write the destination file.
       grunt.file.write(f.dest, src);

test/expected/custom_options

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

test/expected/default_options

@@ -24,25 +24,21 @@ var grunt = require('grunt');
 
 exports.javascript_obfuscator = {
   setUp: function(done) {
-    // setup here if necessary
     done();
   },
   default_options: function(test) {
-    test.expect(1);
+    test.expect(4);
 
-    var actual = grunt.file.read('tmp/default_options');
-    var expected = grunt.file.read('test/expected/default_options');
-    test.equal(actual, expected, 'should describe what the default behavior is.');
+    var obfuscated = grunt.file.read('tmp/default_options');
 
-    test.done();
-  },
-  custom_options: function(test) {
-    test.expect(1);
+    // should NOT be obfuscated
+    test.ok(obfuscated.indexOf('console') > -1, '`console` shouldn\'t be obfuscated');
+    test.ok(obfuscated.indexOf('TEST_LOCAL_VARIABLE') > -1, '`TEST_LOCAL_VARIABLE` shouldn\'t be obfuscated');
 
-    var actual = grunt.file.read('tmp/custom_options');
-    var expected = grunt.file.read('test/expected/custom_options');
-    test.equal(actual, expected, 'should describe what the custom option(s) behavior is.');
+    // should be obfuscated
+    test.ok(obfuscated.indexOf('.log') > -1, '`.log` shouldn\'t be obfuscated');
+    test.ok(obfuscated.indexOf('TEST_STRING_LITERAL') === -1, '`TEST_STRING_LITERAL` should be obfuscated');
 
     test.done();
-  },
+  }
 };