Change proposed solution

The proposed solution didn't work, because the `arguments.length` was
always 2 since it was an internal private method. Since this is about to
be rewritten anyway I just tacked on a stupid third arg that measures 
arg length on the public function.

I also removed the extra error message since the original message is 
actually probably clearer. I highly doubt anyone is intentionally 
relying on this, so a compat notice isn't very necessary IMO

Author: searls

regression/src/object-test.coffee

@@ -34,8 +34,12 @@ describe 'td.object', ->
     When -> @testDouble = td.object(Object.create(respond: -> 'no'))
     Then -> td.explain(@testDouble.respond).isTestDouble == true
 
+  describe 'passing undefined raises error / compatibility note', ->
+    When -> try td.object(undefined) catch error then @error = error
+    Then -> /pass it a plain object/.test(@error.message)
+
   if global.Proxy?
-    describe 'creating a proxy object (ES2015; only supported in FF + Edge atm)', ->
+    describe 'creating a ES Proxy object', ->
       Given -> @testDouble = td.object('thing')
       Given -> @testDouble.magic('sauce')
       When -> td.when(@testDouble.whateverYouWant()).thenReturn('YESS')

src/object.js

@@ -5,20 +5,19 @@ import imitate from './imitate'
 
 const DEFAULT_OPTIONS = {excludeMethods: ['then']}
 
-export default (nameOrType, config) =>
-  _.tap(fakeObject(nameOrType, config), (obj) => {
+export default function object (nameOrType, config) {
+  return _.tap(fakeObject(nameOrType, config, arguments.length), (obj) => {
     addToStringToDouble(obj, nameOrType)
   })
+}
 
-var fakeObject = (nameOrType, config) => {
+var fakeObject = function (nameOrType, config, argCount) {
   if (_.isArray(nameOrType)) {
     return createTestDoublesForFunctionNames(nameOrType)
   } else if (_.isObjectLike(nameOrType)) {
     return imitate(nameOrType)
-  } else if (_.isString(nameOrType) || arguments.length === 0) {
+  } else if (_.isString(nameOrType) || argCount === 0) {
     return createTestDoubleViaProxy(nameOrType, withDefaults(config))
-  } else if (nameOrType === undefined) {
-    ensureUndifinedVariableNotPassed()
   } else if (_.isFunction(nameOrType)) {
     ensureFunctionIsNotPassed()
   } else {
@@ -58,9 +57,6 @@ Did you mean \`td.object(['${name}'])\`?\
   }
 }
 
-var ensureUndifinedVariableNotPassed = () => 
-  log.error('td.object', `undefined variable was passed to \`td.object\` (as of [email protected]). Please use \`td.object()\` or \`td.object('someString')\` to create a test doulbe via proxy.`)
-
 var ensureFunctionIsNotPassed = () =>
   log.error('td.object', `Functions are not valid arguments to \`td.object\` (as of [email protected]). Please use \`td.function()\` or \`td.constructor()\` instead for creating fake functions.`)