Skip to content

Commit a5d15bc

Browse files
committed
Sort before asserting with SequenceEqual.
1 parent 17f4334 commit a5d15bc

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

RubberduckTests/Settings/InspectionSettingsTests.cs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,8 @@ public void SaveConfigWorks()
6060
var config = GetDefaultConfig();
6161
viewModel.UpdateConfig(config);
6262

63-
Assert.IsTrue(config.UserSettings.CodeInspectionSettings.CodeInspections.SequenceEqual(
64-
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>()));
63+
Assert.IsTrue(config.UserSettings.CodeInspectionSettings.CodeInspections.OrderBy(setting => setting.InspectionType).SequenceEqual(
64+
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>().OrderBy(setting => setting.InspectionType)));
6565
}
6666

6767
[Category("Settings")]
@@ -73,8 +73,8 @@ public void SetDefaultsWorks()
7373
var defaultConfig = GetDefaultConfig();
7474
viewModel.SetToDefaults(defaultConfig);
7575

76-
Assert.IsTrue(defaultConfig.UserSettings.CodeInspectionSettings.CodeInspections.SequenceEqual(
77-
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>()));
76+
Assert.IsTrue(defaultConfig.UserSettings.CodeInspectionSettings.CodeInspections.OrderBy(setting => setting.InspectionType).SequenceEqual(
77+
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>().OrderBy(setting => setting.InspectionType)));
7878
}
7979

8080
[Category("Settings")]
@@ -84,8 +84,8 @@ public void InspectionsAreSetInCtor()
8484
var defaultConfig = GetDefaultConfig();
8585
var viewModel = new InspectionSettingsViewModel(defaultConfig);
8686

87-
Assert.IsTrue(defaultConfig.UserSettings.CodeInspectionSettings.CodeInspections.SequenceEqual(
88-
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>()));
87+
Assert.IsTrue(defaultConfig.UserSettings.CodeInspectionSettings.CodeInspections.OrderBy(setting => setting.InspectionType).SequenceEqual(
88+
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>().OrderBy(setting => setting.InspectionType)));
8989
}
9090

9191
[Category("Settings")]
@@ -102,8 +102,8 @@ public void InspectionSeveritiesAreUpdated()
102102
updatedConfig.UserSettings.CodeInspectionSettings.CodeInspections.First().Severity =
103103
GetNondefaultConfig().UserSettings.CodeInspectionSettings.CodeInspections.First().Severity;
104104

105-
Assert.IsTrue(updatedConfig.UserSettings.CodeInspectionSettings.CodeInspections.SequenceEqual(
106-
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>()));
105+
Assert.IsTrue(updatedConfig.UserSettings.CodeInspectionSettings.CodeInspections.OrderBy(setting => setting.InspectionType).SequenceEqual(
106+
viewModel.InspectionSettings.SourceCollection.OfType<CodeInspectionSetting>().OrderBy(setting => setting.InspectionType)));
107107
}
108108
}
109109
}

0 commit comments

Comments
 (0)