Fix constructor bug by no longer using class

Instead the new implementation will return a test double function and
place named properties on its prototype. This will allow folks to
verify constructor calls just as they would for imitated classes

Author: searls

regression/src/constructor-test.coffee

@@ -62,14 +62,17 @@ describe 'td.constructor', ->
 
   describe 'being given an array of function names', ->
     Given -> @fakeConstructor = td.constructor(['foo', 'bar'])
-    Given -> @fakeInstance = new @fakeConstructor()
+    Given -> @fakeInstance = new @fakeConstructor('biz')
     Then -> @fakeConstructor.prototype.foo == @fakeInstance.foo
+    And -> td.verify(new @fakeConstructor('biz'))
     And -> td.explain(@fakeInstance.foo).isTestDouble == true
     And -> td.explain(@fakeInstance.bar).isTestDouble == true
-    And -> @fakeConstructor.toString() == '[test double constructor]'
+    And -> @fakeConstructor.toString() == '[test double for "(unnamed constructor)"]'
     And -> @fakeInstance.toString() == '[test double instance of constructor]'
     And -> @fakeInstance.foo.toString() == '[test double for "#foo"]'
 
+
+
   describe 'edge case: being given a function without prototypal methods', ->
     Given -> @boringFunc = ->
     Given -> @boringFunc.foo = ->

src/constructor.js

@@ -8,19 +8,12 @@ export default (typeOrNames) =>
     : fakeConstructorFromNames(typeOrNames)
 
 var fakeConstructorFromNames = (funcNames) => {
-  return _.tap(class TestDoubleConstructor {}, (cls) => {
+  return _.tap(td.function('(unnamed constructor)'), (fakeConstructor) => {
+    fakeConstructor.prototype.toString = () =>
+      '[test double instance of constructor]'
+
     _.each(funcNames, (funcName) => {
-      cls.prototype[funcName] = tdFunction(`#${funcName}`)
+      fakeConstructor.prototype[funcName] = tdFunction(`#${funcName}`)
     })
-
-    addToStringMethodsToFakeType(cls)
   })
 }
-
-var addToStringMethodsToFakeType = (fakeType, name) => {
-  fakeType.toString = () =>
-    `[test double constructor${name ? ` for "${name}"` : ''}]`
-
-  fakeType.prototype.toString = () =>
-    `[test double instance of constructor${name ? ` "${name}"` : ''}]`
-}