Implemented text handler.

Author: ezzatron

src/handler/CommonCoffeeHandler.coffee

@@ -28,9 +28,16 @@ module.exports = class CommonCoffeeHandler
 
                 return
 
+            try
+                js = coffee.compile data.toString()
+            catch error
+                reject error
+
+                return
+
             resolve [
                 path.basename filePath, '.coffee'
-                util.format @template, coffee.compile data.toString()
+                util.format @template, js
             ]
 
             return

src/handler/TextHandler.coffee

@@ -0,0 +1,28 @@
+fs = require 'fs'
+path = require 'path'
+Promise = require 'bluebird'
+
+module.exports = class TextHandler
+
+    constructor: (@extensions = ['', '.txt']) ->
+
+    handle: (filePath) -> new Promise (resolve, reject) =>
+        extension = path.extname filePath
+
+        unless extension in @extensions
+            resolve null
+
+            return
+
+        fs.readFile filePath, (error, data) ->
+            if error
+                reject error
+
+                return
+
+            resolve [
+                path.basename filePath, extension
+                data.toString().replace /(?:\r\n|\r|\n)$/, ''
+            ]
+
+            return

test/fixture/invalid/coffee.coffee

@@ -0,0 +1 @@
+.

test/fixture/valid/file

@@ -0,0 +1 @@
+no extension

test/fixture/valid/no-extension

@@ -0,0 +1 @@
+other

test/fixture/valid/other.other

@@ -0,0 +1 @@
+text

test/fixture/valid/txt.txt

@@ -33,9 +33,11 @@ describe 'CouchBuilder', ->
                     '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"
+                'json.json': '{"a":1,"b":2}\n'
+                'no-extension': 'no extension\n'
+                'other.other': 'other\n'
+                'txt.txt': 'text\n'
 
             return @subject.build filePath
             .then (actual) ->

test/suite/CouchBuilder.spec.coffee

@@ -5,46 +5,51 @@ 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 = {};
 
-        it 'resolves to a compiled module wrapper for CoffeeScript files', ->
-            path = "#{__dirname}/../../fixture/valid/coffee.coffee"
-            expected = [
-                'coffee'
-                '''
-                    function () {
-                    var module = {};
+                (function() {
+                  var test;
 
-                    (function() {
-                      var test;
+                  test = 'It works.';
 
-                      test = 'It works.';
+                  module.exports = function() {
+                    return [test, arguments];
+                  };
 
-                      module.exports = function() {
-                        return [test, arguments];
-                      };
+                }).call(this);
 
-                    }).call(this);
+                return module.exports.apply(this, arguments);
+                }
+            '''
+        ]
 
-                    return module.exports.apply(this, arguments);
-                    }
-                '''
-            ]
+        return @subject.handle path
+        .then (actual) ->
+            assert.deepEqual actual, expected
 
-            return @subject.handle path
-            .then (actual) ->
-                assert.deepEqual actual, expected
+    it 'resolves to null for non-CoffeeScript files', ->
+        path = "#{__dirname}/../../fixture/valid/other.other"
 
-        it 'resolves to null for non-CoffeeScript files', ->
-            path = "#{__dirname}/../../fixture/valid/file"
+        return @subject.handle path
+        .then (actual) ->
+            assert.isNull actual
 
-            return @subject.handle path
-            .then (actual) ->
-                assert.isNull actual
+    it 'handles invalid CoffeeScript data', ->
+        path = "#{__dirname}/../../fixture/invalid/coffee.coffee"
 
-        it 'handles file system errors', ->
-            path = "#{__dirname}/../../fixture/invalid/nonexistent.js"
+        return @subject.handle path
+        .catch (actual) ->
+            assert.instanceOf actual, SyntaxError
 
-            return @subject.handle path
-            .catch (actual) ->
-                assert.instanceOf actual, Error
+    it 'handles file system errors', ->
+        path = "#{__dirname}/../../fixture/invalid/nonexistent.js"
+
+        return @subject.handle path
+        .catch (actual) ->
+            assert.instanceOf actual, Error

test/suite/handler/CommonCoffeeHandler.spec.coffee

@@ -5,40 +5,38 @@ describe 'CommonJsHandler', ->
     beforeEach ->
         @subject = new CommonJsHandler()
 
-    describe 'handle', ->
-
-        it 'resolves to a module wrapper for JavaScript files', ->
-            path = "#{__dirname}/../../fixture/valid/js.js"
-            expected = [
-                'js'
-                '''
-                    function () {
-                    var module = {};
-                    (function () {
-
-                    var 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-JavaScript 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
+    it 'resolves to a module wrapper for JavaScript files', ->
+        path = "#{__dirname}/../../fixture/valid/js.js"
+        expected = [
+            'js'
+            '''
+                function () {
+                var module = {};
+                (function () {
+
+                var 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-JavaScript files', ->
+        path = "#{__dirname}/../../fixture/valid/other.other"
+
+        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