Skip to content

Commit ab798cc

Browse files
committed
Test tidy ups
1 parent b584ba1 commit ab798cc

3 files changed

Lines changed: 12 additions & 5 deletions

File tree

EDDNResponder/Toolkit/LocationAugmenter.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -293,8 +293,8 @@ internal bool ConfirmName()
293293
}
294294
else
295295
{
296-
system = EDDI.Instance.DataProvider.GetOrFetchQuickStarSystemAsync( systemAddress )
297-
.GetResultOrTimeout( TimeSpan.FromSeconds( 5 ) );
296+
system = EDDI.Instance.DataProvider.GetOrFetchQuickStarSystemAsync( systemAddress, true )
297+
.GetResultOrTimeout( TimeSpan.FromSeconds( 10 ) );
298298
}
299299

300300
if ( systemName == system?.systemname )

Tests/AppTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ public void CrashLogger_SuppressesRollbarInternalHttpExceptions ()
141141
crashLogger.Invoke( null, new object[] { agg } );
142142
}
143143

144-
[TestMethod]
144+
[TestMethod, DoNotParallelize]
145145
public void CrashLogger_LogsUnhandledException ()
146146
{
147147
// Ensure we have a fresh log file to inspect

Tests/CottleHighlightingTests.cs

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,16 @@ public void TestSetSupportedColorKey()
3535
[TestMethod]
3636
public void TestSetUnsupportedColorKey()
3737
{
38-
Assert.ThrowsExactly<KeyNotFoundException>(
39-
() => cottleHighlighting.SetBackgroundColor("NotInTheGrammar", Colors.BlanchedAlmond)
38+
try
39+
{
40+
Assert.ThrowsExactly<KeyNotFoundException>( () =>
41+
cottleHighlighting.SetBackgroundColor( "NotInTheGrammar", Colors.BlanchedAlmond )
4042
);
43+
}
44+
catch ( KeyNotFoundException )
45+
{
46+
// This is expected.
47+
}
4148
}
4249
}
4350
}

0 commit comments

Comments
 (0)