Skip to content

Commit eec1b8c

Browse files
committed
WaitForDelay timing assertions widen from 100/500ms to 2000ms: the original timings were unnecessarily aggressive and not reliable on slow/busy test runner systems.
1 parent 7789a10 commit eec1b8c

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

SqlServerSimulator.Tests/WaitForDelayTests.cs

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@ public void Delay_50ms_ActuallySleeps()
5656
var elapsed = Stopwatch.GetElapsedTime(start);
5757
IsGreaterThanOrEqualTo(40, elapsed.TotalMilliseconds,
5858
$"Expected ≥40ms sleep, got {elapsed.TotalMilliseconds}ms");
59-
IsLessThan(500, elapsed.TotalMilliseconds,
60-
$"Expected <500ms sleep, got {elapsed.TotalMilliseconds}ms");
59+
IsLessThan(2000, elapsed.TotalMilliseconds,
60+
$"Expected well under a misparsed-magnitude sleep, got {elapsed.TotalMilliseconds}ms");
6161
}
6262

6363
[TestMethod]
@@ -68,8 +68,8 @@ public void Delay_InUntakenIf_DoesNotSleep()
6868
var start = Stopwatch.GetTimestamp();
6969
_ = new Simulation().ExecuteNonQuery("if 1=0 waitfor delay '00:00:10'");
7070
var elapsed = Stopwatch.GetElapsedTime(start);
71-
IsLessThan(100, elapsed.TotalMilliseconds,
72-
$"Expected near-instant skip, got {elapsed.TotalMilliseconds}ms");
71+
IsLessThan(2000, elapsed.TotalMilliseconds,
72+
$"Expected skip well under the guarded 10s sleep, got {elapsed.TotalMilliseconds}ms");
7373
}
7474

7575
[TestMethod]
@@ -78,8 +78,8 @@ public void Delay_AfterReturn_DoesNotSleep()
7878
var start = Stopwatch.GetTimestamp();
7979
_ = new Simulation().ExecuteNonQuery("return; waitfor delay '00:00:10'");
8080
var elapsed = Stopwatch.GetElapsedTime(start);
81-
IsLessThan(100, elapsed.TotalMilliseconds,
82-
$"Expected near-instant skip, got {elapsed.TotalMilliseconds}ms");
81+
IsLessThan(2000, elapsed.TotalMilliseconds,
82+
$"Expected skip well under the guarded 10s sleep, got {elapsed.TotalMilliseconds}ms");
8383
}
8484

8585
[TestMethod]
@@ -94,8 +94,8 @@ public void Delay_AfterBreak_DoesNotSleep()
9494
end
9595
""");
9696
var elapsed = Stopwatch.GetElapsedTime(start);
97-
IsLessThan(100, elapsed.TotalMilliseconds,
98-
$"Expected near-instant skip, got {elapsed.TotalMilliseconds}ms");
97+
IsLessThan(2000, elapsed.TotalMilliseconds,
98+
$"Expected skip well under the guarded 10s sleep, got {elapsed.TotalMilliseconds}ms");
9999
}
100100

101101
[TestMethod]
@@ -204,7 +204,7 @@ public void Delay_InWhile_RunsEachIteration()
204204
end
205205
""");
206206
var elapsed = Stopwatch.GetElapsedTime(start);
207-
IsLessThan(500, elapsed.TotalMilliseconds,
207+
IsLessThan(2000, elapsed.TotalMilliseconds,
208208
$"Expected fast loop, got {elapsed.TotalMilliseconds}ms");
209209
}
210210
}

0 commit comments

Comments
 (0)