Change TimeStamp from DateTime to Millisecond Instead of Ticks (#127)

* Change TimeStamp from DateTime to Millisecond Instead of Ticks

* Assert.Equal the Values

* Update src/NRedisStack/TimeSeries/DataTypes/TimeStamp.cs

Co-authored-by: Steve Lorello <[email protected]>

* update tests

---------

Co-authored-by: Steve Lorello <[email protected]>

Author: shacharPash

src/NRedisStack/TimeSeries/DataTypes/TimeStamp.cs

@@ -23,7 +23,7 @@ public class TimeStamp
         /// Build a TimeStamp from DateTime.
         /// </summary>
         /// <param name="dateTime">DateTime value</param>
-        public TimeStamp(DateTime dateTime) => Value = dateTime.Ticks;
+        public TimeStamp(DateTime dateTime) => Value = new DateTimeOffset(dateTime).ToUnixTimeMilliseconds();
 
         /// <summary>
         /// Build a TimeStamp from one of the strings "-", "+", "*".

tests/NRedisStack.Tests/TimeSeries/TestAPI/TestAdd.cs

@@ -210,7 +210,8 @@ public void TestOldAdd()
             ts.Create(key);
             ts.Add(key, new_dt, 1.1);
             // Adding old event
-            Assert.Equal(old_dt, ts.Add(key, old_dt, 1.1));
+            var res = ts.Add(key, old_dt, 1.1);
+            Assert.Equal(old_dt.Value, res.Value);
         }
 
         [Fact]

tests/NRedisStack.Tests/TimeSeries/TestAPI/TestAddAsync.cs

@@ -228,7 +228,8 @@ public async Task TestOldAdd()
             await ts.CreateAsync(key);
             await ts.AddAsync(key, newTimeStamp, 1.1);
             // Adding old event
-            Assert.Equal(oldTimeStamp, await ts.AddAsync(key, oldTimeStamp, 1.1));
+            var res = await ts.AddAsync(key, oldTimeStamp, 1.1);
+            Assert.Equal(oldTimeStamp.Value, res.Value);
         }
 
         [Fact]

tests/NRedisStack.Tests/TimeSeries/TestAPI/TestMADD.cs

@@ -76,7 +76,7 @@ public void TestSuccessfulMADD()
             Assert.Equal(timestamps.Count, response.Count);
             for (int i = 0; i < response.Count; i++)
             {
-                Assert.Equal<DateTime>(timestamps[i], response[i]);
+                Assert.Equal(new DateTimeOffset(timestamps[i]).ToUnixTimeMilliseconds(), response[i].Value);
             }
         }
 
@@ -117,7 +117,7 @@ public void TestOverrideMADD()
             Assert.Equal(oldTimeStamps.Count, response.Count);
             for (int i = 0; i < response.Count; i++)
             {
-                Assert.Equal<DateTime>(oldTimeStamps[i], response[i]);
+                Assert.Equal(new DateTimeOffset(oldTimeStamps[i]).ToUnixTimeMilliseconds(), response[i].Value);
             }
         }
     }

tests/NRedisStack.Tests/TimeSeries/TestAPI/TestMAddAsync.cs

@@ -69,7 +69,7 @@ public async Task TestSuccessfulMAdd()
             Assert.Equal(timestamps.Count, response.Count);
             for (var i = 0; i < response.Count; i++)
             {
-                Assert.Equal<DateTime>(timestamps[i], response[i]);
+                Assert.Equal(new DateTimeOffset(timestamps[i]).ToUnixTimeMilliseconds(), response[i].Value);
             }
         }
 
@@ -112,7 +112,7 @@ public async Task TestOverrideMAdd()
             Assert.Equal(oldTimeStamps.Count, response.Count);
             for (int i = 0; i < response.Count; i++)
             {
-                Assert.Equal<DateTime>(oldTimeStamps[i], response[i]);
+                Assert.Equal(new DateTimeOffset(oldTimeStamps[i]).ToUnixTimeMilliseconds(), response[i].Value);
             }
         }
     }

tests/NRedisStack.Tests/TimeSeries/TestDataTypes/TestTimeStamp.cs

@@ -25,7 +25,7 @@ public void TestTimeStampImplicitCast()
 
             DateTime now = DateTime.UtcNow;
             ts = now;
-            Assert.Equal<DateTime>(now, ts);
+            Assert.Equal(new DateTimeOffset(now).ToUnixTimeMilliseconds(), ts.Value);
 
         }
     }