Working common CS handler.

Author: ezzatron

src/handler/CommonCoffeeHandler.coffee

@@ -0,0 +1,36 @@
+coffee = require 'coffee-script'
+fs = require 'fs'
+path = require 'path'
+Promise = require 'bluebird'
+util = require 'util'
+
+module.exports = class CommonCoffeeHandler
+
+    constructor: (@template) ->
+        @template ?= '''
+            function () {
+            var module = {};
+
+            %s
+            return module.exports.apply(this, arguments);
+            }
+        '''
+
+    handle: (filePath) -> new Promise (resolve, reject) =>
+        if path.extname(filePath) isnt '.coffee'
+            resolve null
+
+            return
+
+        fs.readFile filePath, (error, data) =>
+            if error
+                reject error
+
+                return
+
+            resolve [
+                path.basename filePath, '.coffee'
+                util.format @template, coffee.compile data.toString()
+            ]
+
+            return

src/handler/CommonJsHandler.coffee

@@ -8,14 +8,12 @@ module.exports = class CommonJsHandler
     constructor: (@template) ->
         @template ?= '''
             function () {
-                var module = {};
-
-                (function () {
+            var module = {};
+            (function () {
 
             %s
-                })();
-
-                return module.exports.apply(this, arguments);
+            }).call(this);
+            return module.exports.apply(this, arguments);
             }
         '''
 

test/fixture/valid/coffee.coffee

@@ -0,0 +1,2 @@
+test = 'It works.'
+module.exports = -> [test, arguments]

test/suite/CouchBuilder.spec.coffee

@@ -32,6 +32,7 @@ describe 'CouchBuilder', ->
                 'directory-b':
                     'file-b-a': "b-a\n"
                     'file-b-b': "b-b\n"
+                'coffee.coffee': "test = 'It works.'\nmodule.exports = -> [test, arguments]\n"
                 'file': ''
                 'json.json': '{"a":1,"b":2}\n'
                 'js.js': "var test = 'It works.';\nmodule.exports = function () { return [test, arguments] };\n"

test/suite/handler/CommonCoffeeHandler.spec.coffee

@@ -0,0 +1,50 @@
+CommonCoffeeHandler = require '../../../src/handler/CommonCoffeeHandler'
+
+describe 'CommonCoffeeHandler', ->
+
+    beforeEach ->
+        @subject = new CommonCoffeeHandler()
+
+    describe 'handle', ->
+
+        it 'resolves to a compiled module wrapper for CoffeeScript files', ->
+            path = "#{__dirname}/../../fixture/valid/coffee.coffee"
+            expected = [
+                'coffee'
+                '''
+                    function () {
+                    var module = {};
+
+                    (function() {
+                      var test;
+
+                      test = 'It works.';
+
+                      module.exports = function() {
+                        return [test, arguments];
+                      };
+
+                    }).call(this);
+
+                    return module.exports.apply(this, arguments);
+                    }
+                '''
+            ]
+
+            return @subject.handle path
+            .then (actual) ->
+                assert.deepEqual actual, expected
+
+        it 'resolves to null for non-CoffeeScript files', ->
+            path = "#{__dirname}/../../fixture/valid/file"
+
+            return @subject.handle path
+            .then (actual) ->
+                assert.isNull actual
+
+        it 'handles file system errors', ->
+            path = "#{__dirname}/../../fixture/invalid/nonexistent.js"
+
+            return @subject.handle path
+            .catch (actual) ->
+                assert.instanceOf actual, Error

test/suite/handler/CommonJsHandler.spec.coffee

@@ -13,16 +13,14 @@ describe 'CommonJsHandler', ->
                 'js'
                 '''
                     function () {
-                        var module = {};
-
-                        (function () {
+                    var module = {};
+                    (function () {
 
                     var test = 'It works.';
                     module.exports = function () { return [test, arguments] };
 
-                        })();
-
-                        return module.exports.apply(this, arguments);
+                    }).call(this);
+                    return module.exports.apply(this, arguments);
                     }
                 '''
             ]