From 742a588083e8de11097cd5c780267bf478741126 Mon Sep 17 00:00:00 2001 From: Daniel Kraus Date: Wed, 5 Aug 2015 20:02:02 +0200 Subject: [PATCH] Make tests pass again (except not implemented ones). --- Tests/Controller/Elements/ItemsTest.cs | 10 +++++----- Tests/Controller/SettingsRepositoryTest.cs | 6 ++++-- zaaReloaded2/Controller/Settings.cs | 6 +++++- zaaReloaded2/Importer/ZaaImporter/LaurisTimePoint.cs | 8 +++++++- 4 files changed, 21 insertions(+), 9 deletions(-) diff --git a/Tests/Controller/Elements/ItemsTest.cs b/Tests/Controller/Elements/ItemsTest.cs index 7fa49d3..46c02b6 100755 --- a/Tests/Controller/Elements/ItemsTest.cs +++ b/Tests/Controller/Elements/ItemsTest.cs @@ -61,7 +61,7 @@ namespace Tests.Controller.Elements _formatter.Laboratory = lab; _formatter.Settings.Elements.Add(new zaa.Items("Na, K, Cl")); _formatter.Run(); - Assert.AreEqual("Na 133, K 6 (5)\r", _document.Range().Text); + Assert.AreEqual("\r13.07.2015 13:31:00:\rNa 133, K 6 (5)\r\r", _document.Range().Text); } [Test] @@ -78,7 +78,7 @@ namespace Tests.Controller.Elements _formatter.Laboratory = lab; _formatter.Settings.Elements.Add(new zaa.Items("Klinische Chemie: Na, K, Cl")); _formatter.Run(); - Assert.AreEqual("Klinische Chemie: Na 133, K 6 (5)\r", _document.Range().Text); + Assert.AreEqual("\r13.07.2015 13:31:00:\rKlinische Chemie: Na 133, K 6 (5)\r\r", _document.Range().Text); } [Test] @@ -113,7 +113,7 @@ namespace Tests.Controller.Elements _formatter.Laboratory = lab; _formatter.Settings.Elements.Add(new zaa.Items("Klinische Chemie: Na, *")); _formatter.Run(); - Assert.AreEqual("Klinische Chemie: Na 133, Cl 110, K 6\r", + Assert.AreEqual("\r13.07.2015 13:31:00:\rKlinische Chemie: Na 133, Cl 110, K 6\r\r", _document.Range().Text); } @@ -133,7 +133,7 @@ namespace Tests.Controller.Elements _formatter.Laboratory = lab; _formatter.Settings.Elements.Add(new zaa.Items("Klinische Chemie: Na, SU-*")); _formatter.Run(); - Assert.AreEqual("Klinische Chemie: Na 133, SU-Protein 2,8\r", + Assert.AreEqual("\r13.07.2015 13:31:00:\rKlinische Chemie: Na 133, SU-Protein 2,8\r\r", _document.Range().Text); } @@ -153,7 +153,7 @@ namespace Tests.Controller.Elements _formatter.Laboratory = lab; _formatter.Settings.Elements.Add(new zaa.Items("Klinische Chemie: Na, SU-*, *")); _formatter.Run(); - Assert.AreEqual("Klinische Chemie: Na 133, SU-Protein 2,8, Cl 110, U-Na 99\r", + Assert.AreEqual("\r13.07.2015 13:31:00:\rKlinische Chemie: Na 133, SU-Protein 2,8, Cl 110, U-Na 99\r\r", _document.Range().Text); } } diff --git a/Tests/Controller/SettingsRepositoryTest.cs b/Tests/Controller/SettingsRepositoryTest.cs index e6b789a..39f04d1 100755 --- a/Tests/Controller/SettingsRepositoryTest.cs +++ b/Tests/Controller/SettingsRepositoryTest.cs @@ -50,11 +50,13 @@ namespace Tests.Controller string testName = "test"; s.Name = testName; sr.SettingsList.Add(s); + int expected = sr.SettingsList.Count; + int index = sr.SettingsList.IndexOf(s); sr.Store(); sr = null; sr = SettingsRepository.Load(); - Assert.AreEqual(1, sr.SettingsList.Count); - Assert.AreEqual(testName, sr.SettingsList[0].Name); + Assert.AreEqual(expected, sr.SettingsList.Count); + Assert.AreEqual(testName, sr.SettingsList[index].Name); } } } diff --git a/zaaReloaded2/Controller/Settings.cs b/zaaReloaded2/Controller/Settings.cs index 5691988..cb81027 100755 --- a/zaaReloaded2/Controller/Settings.cs +++ b/zaaReloaded2/Controller/Settings.cs @@ -58,7 +58,7 @@ namespace zaaReloaded2.Controller #region Constructors public Settings() - : this(string.Empty, new List()) + : this(string.Empty, null) { } public Settings(string name) @@ -87,6 +87,10 @@ namespace zaaReloaded2.Controller Uid = Guid.NewGuid(); Name = name; Elements = initialElements; + if (Elements == null) + { + Elements = new List(); + } } #endregion diff --git a/zaaReloaded2/Importer/ZaaImporter/LaurisTimePoint.cs b/zaaReloaded2/Importer/ZaaImporter/LaurisTimePoint.cs index c293938..18c751d 100755 --- a/zaaReloaded2/Importer/ZaaImporter/LaurisTimePoint.cs +++ b/zaaReloaded2/Importer/ZaaImporter/LaurisTimePoint.cs @@ -87,7 +87,13 @@ namespace zaaReloaded2.Importer.ZaaImporter /// Is true if the LaurisText has time stamp in the first /// paragraph and s in the others. /// - public bool IsValidTimePoint { get; private set; } + public bool IsValidTimePoint + { + get + { + return Items.Count > 0; + } + } /// /// Gets or sets the original Lauris text for this timepoint.