Fix JS and CS for use as an actual module.

Author: ezzatron

.gitignore

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

lib/handler/CommonCoffeeHandler.js

@@ -10,7 +10,7 @@ module.exports = class CommonCoffeeHandler
     constructor: (@template) ->
         @template ?= '''
             (function () {
-            if (!module) { var module = {}; }
+            if (!module) { module = {}; }
 
             %s
             return module.exports;

lib/handler/CommonJsHandler.js

@@ -10,7 +10,7 @@ module.exports = class CommonJsHandler
     constructor: (@template) ->
         @template ?= '''
             (function () {
-            if (!module) { var module = {}; }
+            if (!module) { module = {}; }
 
             %s
             return module.exports;

src/handler/CommonCoffeeHandler.coffee

@@ -1,3 +1,5 @@
+fs = require 'fs'
+
 CommonCoffeeHandler = require '../../../src/handler/CommonCoffeeHandler'
 HandlerError = require '../../../src/handler/error/HandlerError'
 
@@ -7,14 +9,14 @@ describe 'CommonCoffeeHandler', ->
         @subject = new CommonCoffeeHandler()
 
         @filePath = "#{__dirname}/../../fixture/handler/coffee.coffee"
-        @tmpPath = "#{__dirname}/../../fixture/tmp"
+        @tmpPath = "#{__dirname}/../../fixture/tmp.js"
 
     it 'resolves to a compiled module wrapper for CoffeeScript files', ->
         expected = [
             'coffee'
             '''
                 (function () {
-                if (!module) { var module = {}; }
+                if (!module) { module = {}; }
 
                 var test;
 
@@ -33,12 +35,22 @@ describe 'CommonCoffeeHandler', ->
         .then (actual) ->
             assert.deepEqual actual, expected
 
-    it 'produces code that will work with CouchDB', ->
+    it 'produces code that will work as a CouchDB view function', ->
+        return @subject.handleFile @filePath
+        .then (actual) =>
+            actual = eval actual[1]
+
+            assert.isFunction actual
+            assert.deepEqual actual('a', 'b'), ['It works.', ['a', 'b']]
+
+    it 'produces code that will work as a CommonJS module', ->
         return @subject.handleFile @filePath
         .then (actual) =>
-            actual = (eval actual[1])('a', 'b');
+            fs.writeFileSync @tmpPath, actual[1]
+            actual = require @tmpPath
 
-            assert.deepEqual actual, ['It works.', ['a', 'b']]
+            assert.isFunction actual
+            assert.deepEqual actual('a', 'b'), ['It works.', ['a', 'b']]
 
     it 'resolves to null for non-CoffeeScript files', ->
         path = "#{__dirname}/../../fixture/handler/other.other"

src/handler/CommonJsHandler.coffee

@@ -1,3 +1,5 @@
+fs = require 'fs'
+
 CommonJsHandler = require '../../../src/handler/CommonJsHandler'
 HandlerError = require '../../../src/handler/error/HandlerError'
 
@@ -7,14 +9,14 @@ describe 'CommonJsHandler', ->
         @subject = new CommonJsHandler()
 
         @filePath = "#{__dirname}/../../fixture/handler/js.js"
-        @tmpPath = "#{__dirname}/../../fixture/tmp"
+        @tmpPath = "#{__dirname}/../../fixture/tmp.js"
 
     it 'resolves to a module wrapper for JavaScript files', ->
         expected = [
             'js'
             '''
                 (function () {
-                if (!module) { var module = {}; }
+                if (!module) { module = {}; }
 
                 var test = 'It works.';
                 module.exports = function () { return [test, Array.prototype.slice.call(arguments)] };
@@ -28,12 +30,22 @@ describe 'CommonJsHandler', ->
         .then (actual) ->
             assert.deepEqual actual, expected
 
-    it 'produces code that will work with CouchDB', ->
+    it 'produces code that will work as a CouchDB view function', ->
+        return @subject.handleFile @filePath
+        .then (actual) =>
+            actual = eval actual[1]
+
+            assert.isFunction actual
+            assert.deepEqual actual('a', 'b'), ['It works.', ['a', 'b']]
+
+    it 'produces code that will work as a CommonJS module', ->
         return @subject.handleFile @filePath
         .then (actual) =>
-            actual = (eval actual[1])('a', 'b');
+            fs.writeFileSync @tmpPath, actual[1]
+            actual = require @tmpPath
 
-            assert.deepEqual actual, ['It works.', ['a', 'b']]
+            assert.isFunction actual
+            assert.deepEqual actual('a', 'b'), ['It works.', ['a', 'b']]
 
     it 'resolves to null for non-JavaScript files', ->
         path = "#{__dirname}/../../fixture/handler/other.other"

test/suite/handler/CommonCoffeeHandler.spec.coffee

@@ -8,7 +8,7 @@ describe 'Module', ->
     it 'builds a result using the correct handlers', ->
         expectedCoffee = '''
             (function () {
-            if (!module) { var module = {}; }
+            if (!module) { module = {}; }
 
             var test;
 
@@ -23,7 +23,7 @@ describe 'Module', ->
         '''
         expectedJs = '''
             (function () {
-            if (!module) { var module = {}; }
+            if (!module) { module = {}; }
 
             var test = 'It works.';
             module.exports = function () { return [test, Array.prototype.slice.call(arguments)] };