growlmock helper for testability, correct helper test

alextucker · Jul 15, 2012 · f75d012 · f75d012
1 parent 7cee76e
commit f75d012
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 8 deletions.
diff --git a/tasks/growl.js b/tasks/growl.js
@@ -49,7 +49,13 @@ module.exports = function(grunt) {
   // HELPERS
   // ==========================================================================
 
-  grunt.registerHelper('growl', growlMessage);
+  grunt.registerHelper('growl', function(config){
+    growlMessage(config);
+  });
+
+  grunt.registerHelper('growlmock', function(mock){
+    growlMessage = mock;
+  });
 
   // ==========================================================================
   // DEFAULT NOTIFICATIONS

diff --git a/test/growl_test.js b/test/growl_test.js
@@ -21,14 +21,14 @@ var grunt = require('grunt');
 */
 
 exports['growl'] = {
-  setUp: function(done) {
-    // setup here
-    done();
-  },
   'helper': function(test) {
     test.expect(1);
-    // tests here
-    test.equal(grunt.helper('growl'), 'growl!!!', 'should return the correct value.');
-    test.done();
+
+    grunt.helper('growlmock', function(config) {
+        test.equal(config.message, 'test message', 'should use message property.');
+        test.done();
+    });
+
+    grunt.helper('growl', {message:'test message'});
   }
 };