From 63fee4f2752daef93d42ce1778d2d886c75e9b66 Mon Sep 17 00:00:00 2001
From: Amar Zavery <amzavery@microsoft.com>
Date: Fri, 29 Apr 2016 14:00:49 -0700
Subject: [PATCH] UnixTime tests

---
 .../NodeJS/NodeJS.Tests/AcceptanceTests/acceptanceTests.ts  | 4 ++--
 ClientRuntimes/NodeJS/ms-rest/README.md                     | 3 ++-
 ClientRuntimes/NodeJS/ms-rest/lib/serialization.js          | 1 -
 ClientRuntimes/NodeJS/ms-rest/test/serializationTests.js    | 6 ++++++
 4 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/AutoRest/Generators/NodeJS/NodeJS.Tests/AcceptanceTests/acceptanceTests.ts b/AutoRest/Generators/NodeJS/NodeJS.Tests/AcceptanceTests/acceptanceTests.ts
index 53e027b7e9a65..db5042f716ed3 100644
--- a/AutoRest/Generators/NodeJS/NodeJS.Tests/AcceptanceTests/acceptanceTests.ts
+++ b/AutoRest/Generators/NodeJS/NodeJS.Tests/AcceptanceTests/acceptanceTests.ts
@@ -201,7 +201,7 @@ describe('nodejs', function () {
       });
 
       it('should put and get UnixTime date correctly', function (done) {
-        var d = new Date(2016, 3, 12, 17, 0, 0);
+        var d = new Date('2016-04-13T00:00:00.000Z');
         testClient.intModel.putUnixTimeDate(d, function (error, result) {
           should.not.exist(error);
           testClient.intModel.getUnixTime(function (error, result) {
@@ -1915,7 +1915,7 @@ describe('nodejs', function () {
       });
 
       it('should work when path has a paramaeter in UnixTime format', function (done) {
-        testClient.paths.unixTimeUrl(new Date(2016, 3, 12, 17, 0, 0), function (error, result) {
+        testClient.paths.unixTimeUrl(new Date('2016-04-13T00:00:00.000Z'), function (error, result) {
           should.not.exist(error);
           done();
         });
diff --git a/ClientRuntimes/NodeJS/ms-rest/README.md b/ClientRuntimes/NodeJS/ms-rest/README.md
index cb4aca73ab405..1da52bff1ac92 100644
--- a/ClientRuntimes/NodeJS/ms-rest/README.md
+++ b/ClientRuntimes/NodeJS/ms-rest/README.md
@@ -18,7 +18,7 @@ var msrest = require('ms-rest');
 ## Serialization/Deserialization
 Features
 - Type checking
-  - (String, Number, Boolean, ByteArray, Date, DateTime, Enum, TimeSpan, DateTimeRfc1123, Object, Stream, Sequence, Dictionary, Composite, Uuid(as a string))
+  - (String, Number, Boolean, ByteArray, Base64Url, Date, DateTime, Enum, TimeSpan, DateTimeRfc1123, UnixTime, Object, Stream, Sequence, Dictionary, Composite, Uuid(as a string))
 - Validation of specified constraints
   - ExclusiveMaximum, ExclusiveMinimum, InclusiveMaximum, InclusiveMinimum, MaxItems, MaxLength, MinItems, MinLength, MultipleOf, Pattern, UniqueItems
 - Flattening/Unflattening properties
@@ -26,6 +26,7 @@ Features
 - Model Properties marked as constant are set during serialization, irrespective of they being provided or not
 - Required check (If a model or property is marked required and is not provided in the object then an error is thrown)
 - Readonly check (If a model or property is marked readonly then it is not sent on the wire during, serialization)
+- Serializing Constant values
 
 - serialize an array of dictionary of primitive values
 ```javascript
diff --git a/ClientRuntimes/NodeJS/ms-rest/lib/serialization.js b/ClientRuntimes/NodeJS/ms-rest/lib/serialization.js
index d71e1ab54a1e4..671292b9f43ba 100644
--- a/ClientRuntimes/NodeJS/ms-rest/lib/serialization.js
+++ b/ClientRuntimes/NodeJS/ms-rest/lib/serialization.js
@@ -608,7 +608,6 @@ function dateToUnixTime(d) {
   if (!d) {
     return null;
   }
-  // A Date can be converted to string in JSON-clone operations, so let's tolerate.
   if (typeof d.valueOf() === 'string') {
     d = new Date(d);
   }
diff --git a/ClientRuntimes/NodeJS/ms-rest/test/serializationTests.js b/ClientRuntimes/NodeJS/ms-rest/test/serializationTests.js
index d9be4bd39fba9..32c89671d37cf 100644
--- a/ClientRuntimes/NodeJS/ms-rest/test/serializationTests.js
+++ b/ClientRuntimes/NodeJS/ms-rest/test/serializationTests.js
@@ -195,6 +195,12 @@ describe('msrest', function () {
       serializedDateString.should.equal('+010000-01-01T11:59:59.000Z');
       done();
     });
+    it('should correctly serialize a Date object with max value and format UnixTime', function (done) {
+      mapper = { type : { name: 'UnixTime' } };
+      var serializedDate = msRest.serialize(mapper, new Date('9999-12-31T23:59:59-12:00'), 'dateTimeObj');
+      serializedDate.should.equal(253402343999);
+      done();
+    });
     it('should correctly serialize a string in DateTimeRfc1123', function (done) {
       mapper = { type : { name: 'DateTimeRfc1123' } };
       var rfc = new Date('Mon, 01 Jan 0001 00:00:00 GMT');