Merge pull request #2 from mrourke/testing

Presentation

Author: mrourke

Gruntfile.js

@@ -47,17 +47,19 @@ module.exports = function(grunt) {
         },
       },
       test: {
-
         src: ['test/fixtures/*.html'],
         css: ['test/fixtures/styles.css'],
-        dest: 'tmp/purestyles.css',
-        hello: 'hello world'
+        dest: 'tmp/purestyles.css'
+      },
+      goose: {
+        src: [
+          'Goose-Windmill/public/app/**/*.html', 
+          'Goose-Windmill/public/app/**/*.js',
+          'Goose-Windmill/public/index.html'
+          ],
+        css: ['Goose-Windmill/public/styles/*.css'],
+        dest: 'tmp/gooseCSS.css'
       }
-    },
-
-    // Unit tests.grunt
-    nodeunit: {
-      tests: ['test/*_test.js']
     }
 
   });
@@ -68,11 +70,10 @@ module.exports = function(grunt) {
   // 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', 'purifycss', 'nodeunit']);
+  grunt.registerTask('test', ['clean', 'purifycss:test', 'nodeunit']);
 
   // By default, lint and run all tests.
   grunt.registerTask('default', ['purifycss:test']);

README.md

@@ -19,72 +19,22 @@ grunt.loadNpmTasks('grunt-purifycss');
 
 ## The "purifycss" task
 
-### Overview
-In your project's Gruntfile, add a section named `purifycss` to the data object passed into `grunt.initConfig()`.
-
-```js
-grunt.initConfig({
-  purifycss: {
-    options: {
-      // Task-specific options go here.
-    },
-    your_target: {
-      // Target-specific file lists and/or options go here.
-    },
-  },
-});
-```
-
-### Options
-
-#### options.separator
-Type: `String`
-Default value: `',  '`
-
-A string value that is used to do something with whatever.
-
-#### options.punctuation
-Type: `String`
-Default value: `'.'`
-
-A string value that is used to do something else with whatever else.
-
 ### Usage Examples
 
-#### Default Options
-In this example, the default options are used to do something with whatever. So if the `testing` file has the content `Testing` and the `123` file had the content `1 2 3`, the generated result would be `Testing, 1 2 3.`
+In this example, the default options are to specify a target with src files, style files, and an output destination. The plugin will search for style selectors used in the source files, and then output a trimmed down style sheet.
 
 ```js
 grunt.initConfig({
   purifycss: {
     options: {},
-    files: {
-      'dest/default_options': ['src/testing', 'src/123'],
-    },
-  },
-});
-```
-
-#### Custom Options
-In this example, custom options are used to do something else with whatever else. So if the `testing` file has the content `Testing` and the `123` file had the content `1 2 3`, the generated result in this case would be `Testing: 1 2 3 !!!`
-
-```js
-grunt.initConfig({
-  purifycss: {
-    options: {
-      separator: ': ',
-      punctuation: ' !!!',
-    },
-    files: {
-      'dest/default_options': ['src/testing', 'src/123'],
+    target: {
+      src: ['test/fixtures/*.html', 'test/fixtures/*.js'],
+      css: ['test/fixtures/*.css'],
+      dest: 'tmp/purestyles.css'
     },
   },
 });
 ```
 
 ## Contributing
-In lieu of a formal styleguide, take care to maintain the existing coding style. Add unit tests for any new or changed functionality. Lint and test your code using [Grunt](http://gruntjs.com/).
-
-## Release History
-_(Nothing yet)_
-# grunt-purify-css
+In lieu of a formal styleguide, take care to maintain the existing coding style. Add unit tests for any new or changed functionality. Lint and test your code using [Grunt](http://gruntjs.com/).

package.json

@@ -39,6 +39,6 @@
   ],
   "dependencies": {
     "glob": "^5.0.10",
-    "purify-css": "^1.0.3"
+    "purify-css": "^1.0.x"
   }
 }

tasks/purifycss.js

@@ -13,9 +13,6 @@ var purify = require('purify-css');
 
 module.exports = function(grunt) {
 
-  // Please see the Grunt documentation for more information regarding task
-  // creation: http://gruntjs.com/creating-tasks
-
   grunt.registerMultiTask('purifycss', 'Clean unnecessary CSS', function() {
     // Merge task-specific and/or target-specific options with these defaults.
     var options = this.options({
@@ -24,19 +21,16 @@ module.exports = function(grunt) {
     var src = [];
     this.data.src.forEach(function(pathPattern) {
       var files = glob.sync(pathPattern);
-      console.log("glob files: ", files);
+      console.log("Source Files: ", files);
       src = src.concat(files);
     });
 
     var styles = [];
     this.data.css.forEach(function(pathPattern) {
-      var files = glob.sync(pathPattern);
-      console.log("glob styles: ", files);
-      styles = styles.concat(files);
-    })
-
-    console.log(src);
-    console.log(styles);
+      var style = glob.sync(pathPattern);
+      console.log("Style Files: ", style);
+      styles = styles.concat(style);
+    });
 
     var pure = purify(src, styles, {write: false, info: true});