fix: Another round of JSON DateTime fixes

This commit is contained in:
Chris Cameron
2020-05-27 19:38:04 -04:00
parent 6240d3fccf
commit 25fb5a0ad8
3 changed files with 52 additions and 5 deletions

View File

@@ -0,0 +1,41 @@
using System;
using ICD.Common.Utils.Json;
using Newtonsoft.Json;
using NUnit.Framework;
namespace ICD.Common.Utils.Tests.Json
{
[TestFixture]
public sealed class DateTimeIsoConverterTest : AbstractGenericJsonConverterTest
{
[Test]
public override void WriteJsonTest()
{
JsonSerializerSettings settings = new JsonSerializerSettings
{
DateParseHandling = DateParseHandling.None
};
settings.Converters.Add(new DateTimeIsoConverter());
DateTime dateTime = new DateTime(2020, 1, 2, 3, 4, 5, DateTimeKind.Utc);
string serial = JsonConvert.SerializeObject(dateTime, settings);
Assert.AreEqual("\"2020-01-02T03:04:05Z\"", serial);
}
[Test]
public override void ReadJsonTest()
{
JsonSerializerSettings settings = new JsonSerializerSettings
{
DateParseHandling = DateParseHandling.None
};
settings.Converters.Add(new DateTimeIsoConverter());
string serial = "\"2020-01-02T03:04:05Z\"";
DateTime dateTime = JsonConvert.DeserializeObject<DateTime>(serial, settings);
Assert.AreEqual(new DateTime(2020, 1, 2, 3, 4, 5, DateTimeKind.Utc), dateTime);
}
}
}

View File

@@ -1,4 +1,5 @@
using System; using System;
using System.Globalization;
using ICD.Common.Properties; using ICD.Common.Properties;
using Newtonsoft.Json; using Newtonsoft.Json;
@@ -272,7 +273,7 @@ namespace ICD.Common.Utils.Extensions
#endif #endif
string stringValue = extends.GetValueAsString(); string stringValue = extends.GetValueAsString();
return DateTime.Parse(stringValue); return DateTime.Parse(stringValue, null, DateTimeStyles.RoundtripKind);
} }
/// <summary> /// <summary>

View File

@@ -1,4 +1,5 @@
using System; using System;
using System.Globalization;
using System.Text.RegularExpressions; using System.Text.RegularExpressions;
using ICD.Common.Utils.Extensions; using ICD.Common.Utils.Extensions;
using Newtonsoft.Json; using Newtonsoft.Json;
@@ -15,7 +16,12 @@ namespace ICD.Common.Utils.Json
/// <param name="serializer">The calling serializer.</param> /// <param name="serializer">The calling serializer.</param>
public override void WriteJson(JsonWriter writer, DateTime value, JsonSerializer serializer) public override void WriteJson(JsonWriter writer, DateTime value, JsonSerializer serializer)
{ {
writer.WriteValue(value.ToIso()); string iso = value.ToIso();
// Remove redundant ms
iso = iso.Replace(".0000000", "");
writer.WriteValue(iso);
} }
/// <summary> /// <summary>
@@ -48,11 +54,10 @@ namespace ICD.Common.Utils.Json
// No TimeZoneInfo in CF, so now things get gross // No TimeZoneInfo in CF, so now things get gross
dateTime = DateTime.SpecifyKind(dateTime, DateTimeKind.Unspecified); dateTime = DateTime.SpecifyKind(dateTime, DateTimeKind.Unspecified);
string iso = dateTime.ToIso() + match.Groups["zone"].Value; serial = dateTime.ToIso() + match.Groups["zone"].Value;
return DateTime.Parse(iso);
} }
return DateTime.Parse(serial); return DateTime.Parse(serial, null, DateTimeStyles.RoundtripKind);
} }
} }
} }