Skip to content

Add extra assertions to SortingTest.testSorted#3988

Merged
trancexpress merged 1 commit intoeclipse-platform:masterfrom
trancexpress:gh3987
May 8, 2026
Merged

Add extra assertions to SortingTest.testSorted#3988
trancexpress merged 1 commit intoeclipse-platform:masterfrom
trancexpress:gh3987

Conversation

@trancexpress
Copy link
Copy Markdown
Contributor

SortingTest.testSorted failed in an I-build, indicating fewer matches than expected in AbstractTextSearchResult. This change adds extra assertions, to fail with the match that was not added to the result.

See: #3987

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Test Results

   867 files  ±0     867 suites  ±0   52m 27s ⏱️ - 4m 29s
 7 986 tests ±0   7 743 ✅ ±0  243 💤 ±0  0 ❌ ±0 
20 460 runs  ±0  19 805 ✅ ±0  655 💤 ±0  0 ❌ ±0 

Results for commit cc0a180. ± Comparison against base commit d9b3d56.

♻️ This comment has been updated with latest results.

@trancexpress trancexpress force-pushed the gh3987 branch 3 times, most recently from 448e52d to 409fa6c Compare May 8, 2026 08:58
SortingTest.testSorted failed in an I-build, indicating fewer matches
than expected in AbstractTextSearchResult. This change adds extra assertions,
to fail with the match that was not added to the result.

See: eclipse-platform#3987
@trancexpress trancexpress merged commit d29b936 into eclipse-platform:master May 8, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant