Merge pull request #7598 from apolcyn/fix_sanity_test_diff_ga

compare test config as objects instead of strings
8 years ago · cb21e8be4e
parent f45dd74b01 b4280fa084
commit cb21e8be4e
1 changed files with 4 additions and 3 deletions
--- a/src/csharp/Grpc.Core.Tests/SanityTest.cs
+++ b/src/csharp/Grpc.Core.Tests/SanityTest.cs
@ -58,10 +58,11 @@ namespace Grpc.Core.Tests
        [Test]
        public void TestsJsonUpToDate()
        {
-            var discoveredTests = DiscoverAllTestClasses();
-            string discoveredTestsJson = JsonConvert.SerializeObject(discoveredTests, Formatting.Indented);
+            Dictionary<string, List<string>> discoveredTests = DiscoverAllTestClasses();
+			Dictionary<string, List<string>> testsFromFile 
+			    = JsonConvert.DeserializeObject<Dictionary<string, List<string>>>(ReadTestsJson());

-            Assert.AreEqual(discoveredTestsJson, ReadTestsJson());
+            Assert.AreEqual(discoveredTests, testsFromFile);
        }

        /// <summary>