bug - options can now be passed in. fixes #7

jgallen23 · Nov 12, 2013 · 70ac413 · 70ac413
1 parent 1ff3a33
commit 70ac413
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -30,11 +30,12 @@ module.exports = function(grunt) {
 
     // Configuration to be run (and then tested).
     inlinecss: {
-      default_options: {
+      basic: {
         options: {
+          extraCss: 'body { background: green; }'
         },
         files: {
-          'examples/out.html': 'examples/in.html',
+          'tmp/out.html': 'test/fixtures/in.html'
         },
       },
     },

diff --git a/tasks/inline_css.js b/tasks/inline_css.js
@@ -17,7 +17,7 @@ module.exports = function(grunt) {
 
   grunt.registerMultiTask('inlinecss', 'Takes an html file with css link and turns inline.  Great for emails.', function() {
     // Merge task-specific and/or target-specific options with these defaults.
-    var options = this.data.options;
+    var options = this.options();
     var done = this.async();
     var index = 0;
     var count = this.files.length;
@@ -26,7 +26,7 @@ module.exports = function(grunt) {
     this.files.forEach(function(f) {
 
       var filepath = f.src.toString();
-      if (typeof filepath !== 'string') {
+      if (typeof filepath !== 'string' || filepath === '') {
         grunt.log.error('src must be a single string');
         return false;
       }