Improved tests.

Author: ezzatron

.gitignore

@@ -1,3 +1,4 @@
 /build/
 /coverage/
 /node_modules/
+/test/fixture/tmp

test/fixture/handler/coffee.coffee

@@ -1,2 +1,2 @@
 test = 'It works.'
-module.exports = -> [test, arguments]
+module.exports = -> [test, Array.prototype.slice.call(arguments)]

test/fixture/handler/js.js

@@ -1,2 +1,2 @@
 var test = 'It works.';
-module.exports = function () { return [test, arguments] };
+module.exports = function () { return [test, Array.prototype.slice.call(arguments)] };

test/fixture/handler/nested/coffee.coffee

@@ -1,2 +1,2 @@
 test = 'It works.'
-module.exports = -> [test, arguments]
+module.exports = -> [test, Array.prototype.slice.call(arguments)]

test/fixture/handler/nested/js.js

@@ -1,2 +1,2 @@
 var test = 'It works.';
-module.exports = function () { return [test, arguments] };
+module.exports = function () { return [test, Array.prototype.slice.call(arguments)] };

test/suite/handler/CommonCoffeeHandler.spec.coffee

@@ -1,3 +1,5 @@
+fs = require 'fs'
+
 CommonCoffeeHandler = require '../../../src/handler/CommonCoffeeHandler'
 HandlerError = require '../../../src/handler/error/HandlerError'
 
@@ -6,8 +8,10 @@ describe 'CommonCoffeeHandler', ->
     beforeEach ->
         @subject = new CommonCoffeeHandler()
 
+        @filePath = "#{__dirname}/../../fixture/handler/coffee.coffee"
+        @tmpPath = "#{__dirname}/../../fixture/tmp"
+
     it 'resolves to a compiled module wrapper for CoffeeScript files', ->
-        path = "#{__dirname}/../../fixture/handler/coffee.coffee"
         expected = [
             'coffee'
             '''
@@ -20,7 +24,7 @@ describe 'CommonCoffeeHandler', ->
                   test = 'It works.';
 
                   module.exports = function() {
-                    return [test, arguments];
+                    return [test, Array.prototype.slice.call(arguments)];
                   };
 
                 }).call(this);
@@ -30,10 +34,18 @@ describe 'CommonCoffeeHandler', ->
             '''
         ]
 
-        return @subject.handleFile path
+        return @subject.handleFile @filePath
         .then (actual) ->
             assert.deepEqual actual, expected
 
+    it 'produces code that will work with CouchDB', ->
+        return @subject.handleFile @filePath
+        .then (actual) =>
+            fs.writeFileSync @tmpPath, "module.exports = #{actual[1]};"
+            actual = (require @tmpPath) 'a', 'b'
+
+            assert.deepEqual actual, ['It works.', ['a', 'b']]
+
     it 'resolves to null for non-CoffeeScript files', ->
         path = "#{__dirname}/../../fixture/handler/other.other"
 
@@ -54,3 +66,4 @@ describe 'CommonCoffeeHandler', ->
         return @subject.handleFile path
         .catch (actual) ->
             assert.instanceOf actual, HandlerError
+

test/suite/handler/CommonJsHandler.spec.coffee

@@ -1,3 +1,5 @@
+fs = require 'fs'
+
 CommonJsHandler = require '../../../src/handler/CommonJsHandler'
 HandlerError = require '../../../src/handler/error/HandlerError'
 
@@ -6,8 +8,10 @@ describe 'CommonJsHandler', ->
     beforeEach ->
         @subject = new CommonJsHandler()
 
+        @filePath = "#{__dirname}/../../fixture/handler/js.js"
+        @tmpPath = "#{__dirname}/../../fixture/tmp"
+
     it 'resolves to a module wrapper for JavaScript files', ->
-        path = "#{__dirname}/../../fixture/handler/js.js"
         expected = [
             'js'
             '''
@@ -16,18 +20,26 @@ describe 'CommonJsHandler', ->
                 (function () {
 
                 var test = 'It works.';
-                module.exports = function () { return [test, arguments] };
+                module.exports = function () { return [test, Array.prototype.slice.call(arguments)] };
 
                 }).call(this);
                 return module.exports.apply(this, arguments);
                 }
             '''
         ]
 
-        return @subject.handleFile path
+        return @subject.handleFile @filePath
         .then (actual) ->
             assert.deepEqual actual, expected
 
+    it 'produces code that will work with CouchDB', ->
+        return @subject.handleFile @filePath
+        .then (actual) =>
+            fs.writeFileSync @tmpPath, "module.exports = #{actual[1]};"
+            actual = (require @tmpPath) 'a', 'b'
+
+            assert.deepEqual actual, ['It works.', ['a', 'b']]
+
     it 'resolves to null for non-JavaScript files', ->
         path = "#{__dirname}/../../fixture/handler/other.other"
 

test/suite/index.spec.coffee

@@ -16,7 +16,7 @@ describe 'Module', ->
               test = 'It works.';
 
               module.exports = function() {
-                return [test, arguments];
+                return [test, Array.prototype.slice.call(arguments)];
               };
 
             }).call(this);
@@ -30,7 +30,7 @@ describe 'Module', ->
             (function () {
 
             var test = 'It works.';
-            module.exports = function () { return [test, arguments] };
+            module.exports = function () { return [test, Array.prototype.slice.call(arguments)] };
 
             }).call(this);
             return module.exports.apply(this, arguments);