Shift the Spanner.date check to multiple arguments.

Luke Sneeringer · Luke Sneeringer · commit 326c64ed5a02 · 2017-03-22T09:08:36.000-07:00
diff --git a/packages/spanner/src/codec.js b/packages/spanner/src/codec.js
@@ -26,9 +26,11 @@ var commonGrpc = require('@google-cloud/common-grpc');
 var is = require('is');
 
 function SpannerDate(value) {
-  if (is.number(value)) {
-    throw new TypeError('The spanner.date function accepts a Date object, ' +
-                        'or a string parseable by Date\'s constructor.');
+  if (arguments.length > 1) {
+    throw new TypeError(
+      'The spanner.date function accepts a Date object, ' +
+      'or a single argument parseable by Date\'s constructor.'
+    );
   }
   if (is.undefined(value)) {
     value = new Date();
diff --git a/packages/spanner/test/codec.js b/packages/spanner/test/codec.js
@@ -54,9 +54,9 @@ describe('codec', function() {
       assert.strictEqual(spannerDate.value, date.toJSON().replace(/T.+/, ''));
     });
 
-    it('should choke on an integer', function() {
+    it('should choke on multiple arguments', function() {
       assert.throws(function() {
-        new codec.SpannerDate(2017);
+        new codec.SpannerDate(2012, 3, 21);
       }, /The spanner\.date function accepts a Date object,/);
     });
   });
