Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Commit ea80b29

Browse files
committed
More informative failure from Queryable_Tests.MatchSequencePattern()
When experimenting with #2322 for a while I hit a snag with Queryable_Tests.MatchSequencePattern. While this does Console.Write() the error cases, that's not helpful if you don't hit the problem locally, but do on the CI build. This change makes the failure message more directly informative. It leaves the old assertion in place, so no flaw in this change should hide a bug that the test would previously have caught.
1 parent 18ff2b0 commit ea80b29

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/System.Linq.Expressions/tests/SequenceTests/SequenceTests.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1789,7 +1789,8 @@ public static void MatchSequencePattern()
17891789
Console.WriteLine(m);
17901790
}
17911791
}
1792-
1792+
Assert.Equal(Enumerable.Empty<string>(), list.Select(mi => mi.Name));
1793+
Assert.Equal(Enumerable.Empty<string>(), list2.Select(mi => mi.Name));
17931794
Assert.True(list.Count == 0 && list2.Count == 0);
17941795
}
17951796

0 commit comments

Comments
 (0)