Skip to content

Commit b449aeb

Browse files
committed
Fix test timing bug due to double current time retrieval.
1 parent 744bd82 commit b449aeb

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

test/MsgPack.UnitTest/MessagePackConvertTest.cs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#region -- License Terms --
1+
#region -- License Terms --
22
//
33
// MessagePack for CLI
44
//
@@ -317,19 +317,21 @@ public void TestToDateTime_MinimumMinusOne_IsUtcEpoc()
317317
[Test]
318318
public void TestFromDateTimeOffset_UtcNow_AsUnixEpoc()
319319
{
320+
var utcNow = DateTimeOffset.UtcNow;
320321
Assert.AreEqual(
321-
checked( DateTime.UtcNow.Subtract( UtcEpoc ).Ticks / TicksToMilliseconds ),
322-
MessagePackConvert.FromDateTimeOffset( DateTimeOffset.UtcNow )
322+
checked( utcNow.DateTime.Subtract( UtcEpoc ).Ticks / TicksToMilliseconds ),
323+
MessagePackConvert.FromDateTimeOffset( utcNow )
323324
);
324325
}
325326

326327
[Test]
327328
public void TestFromDateTimeOffset_Now_AsUtcUnixEpoc()
328329
{
330+
var utcNow = DateTimeOffset.UtcNow;
329331
// LocalTime will be converted to UtcTime
330332
Assert.AreEqual(
331-
checked( DateTime.UtcNow.Subtract( UtcEpoc ).Ticks / TicksToMilliseconds ),
332-
MessagePackConvert.FromDateTimeOffset( DateTimeOffset.Now )
333+
checked( utcNow.DateTime.Subtract( UtcEpoc ).Ticks / TicksToMilliseconds ),
334+
MessagePackConvert.FromDateTimeOffset( utcNow.ToLocalTime() )
333335
);
334336
}
335337

0 commit comments

Comments
 (0)