Skip to content

Commit

Permalink
Ignore flaky test on Linux, too
Browse files Browse the repository at this point in the history
The GitHub Actions runners are apparently slow to deliver filesystem
notifications, which is exactly the scenario that makes this test
useless. It's already disabled on Windows; there's no reason for it to
cause the CI build to fail all the time when it's the only failing test.
  • Loading branch information
rmunn committed Mar 14, 2024
1 parent 341dd6b commit ba7db05
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/LibChorusTests/notes/AnnotationRepositoryTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ public void AddAnnotation_NotifiesIndices()
// the setup of the watcher or the code in Save that tries to prevent the notifications at all,
// which is more than half the code this test wants to exercise.
[Test]
[Platform(Exclude = "Win", Reason = "flaky (on both platforms)")]
[Ignore("flaky (on both platforms)")]
public void ExternalFileModification_NotifiesIndices_ButSaveDoesNot()
{
const int SleepTime = 25; // milliseconds
Expand Down

0 comments on commit ba7db05

Please sign in to comment.