added option for testing (read only property); failure robustness

Gruntfile.js

@@ -17,8 +17,6 @@
 
 "use strict";
 
-var path = require("path");
-
 module.exports = function(grunt) {
 
   // load local tasks
@@ -41,6 +39,14 @@ module.exports = function(grunt) {
       build: {
         src: "test/source.js",
         dest: "test/result.js"
+      },
+
+      // set "readOnly" flag for testing
+      test: {
+        options: {
+          readOnly: true
+        },
+        src: "test/src/*.js"
       }
     }
   });
@@ -49,5 +55,5 @@ module.exports = function(grunt) {
   grunt.loadNpmTasks("grunt-contrib-nodeunit");
 
   // default task will be used for 'frontend sync'
-  grunt.registerTask("default", ["clean", "es3ify", "nodeunit"]);
+  grunt.registerTask("default", ["clean", "es3ify:build", "nodeunit"]);
 };

README.md

@@ -29,6 +29,10 @@ grunt.initConfig({
     },
 
     fooBar: {
+      // use "readOnly" flag for testing your configuration
+      options: {
+        readOnly: true
+      },
       cwd: "js/app/",
       src: "**/*.js",
       dest: "target"
@@ -37,4 +41,8 @@ grunt.initConfig({
 });
 ```
 
+### Options
+
+{{options.readOnly}} (Boolean, default: false)
+Shows in the console what will be transformed and where is the destination without any write operations.
 

package.json

@@ -6,10 +6,12 @@
     "name": "pro!vision GmbH",
     "url": "http://www.pro-vision.de"
   },
-  "contributors": [{
-    "name": "Matthias Hauch",
-    "email": "[email protected]"
-  }],
+  "contributors": [
+    {
+      "name": "Matthias Hauch",
+      "email": "[email protected]"
+    }
+  ],
   "homepage": "https://github.com/wcm-io-frontend/grunt-es3ify",
   "bugs": {
     "url": "https://github.com/wcm-io-frontend/grunt-es3ify/issues"
@@ -39,6 +41,7 @@
     "grunt-contrib-nodeunit": "^1.0.0"
   },
   "dependencies": {
+    "chalk": "^1.1.3",
     "es3ify": "^0.2.2"
   }
 }

tasks/es3ify.js

@@ -21,17 +21,38 @@
 
 "use strict";
 
+var path = require("path");
+var chalk = require("chalk");
 var transform = require("es3ify").transform;
 
 module.exports = function(grunt) {
 
   grunt.registerMultiTask("es3ify", "Transforms ES5 to ES3", function() {
 
+    var options = this.options({
+      readOnly: false
+    });
+
     this.files.forEach(function(filePair) {
-      
+
       filePair.src.forEach(function(src) {
-        var content = grunt.file.read(src);
-        grunt.file.write(filePair.dest, transform(content));
+
+        var dest = filePair.dest;
+        if (!dest) {
+          dest = src;
+        }
+        else if (grunt.util._.endsWith(dest, "/")) {
+          dest = path.join(dest, src);
+        }
+
+        if (options.readOnly) {
+          grunt.log.writeln("Will transform " + chalk.cyan(src) + " -> " + chalk.cyan(dest));
+        }
+        else {
+          grunt.verbose.writeln("Transforming " + chalk.cyan(src) + " -> " + chalk.cyan(dest));
+          var content = grunt.file.read(src);
+          grunt.file.write(dest, transform(content));
+        }
       });
     });
   });

test/source.js

@@ -2,12 +2,12 @@
 
 var x = {
   dynamic: 0,
-  static: 17,
-  throws: "forbidden",
-  class: "foo"
+  "static": 17,
+  "throws": "forbidden",
+  "class": "foo"
 };
 
 x.dynamic++;
-x.static++;
-x.throws = "test";
-x.class += "bar";
+x["static"]++;
+x["throws"] = "test";
+x["class"] += "bar";