Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mark DocCommentHighlightingTest as a flaky test #268

Merged
merged 4 commits into from
Jan 30, 2023

Conversation

jonahgraham
Copy link
Member

This test has been failing a lot recently on GitHub CI, but not on Jenkins or local dev. Therefore mark this test as flaky so that PRs don't fail due to unrelated test failures.

Fixes #259

The field implies that the fSourceViewer has a longer life
than it does. But reality is that the viewer is only
valid until the editor gets reopened while the test is
running.
This is the analogous class to BaseUITestCase, but for JUnit5.
This is a pre-requisite to be able to use flaky test tag
This test regularly fails on CI builds, but the test passes ok.

Fixes eclipse-cdt#259
@github-actions
Copy link

Test Results

   539 files     539 suites   15m 37s ⏱️
9 634 tests 9 612 ✔️ 22 💤 0
9 645 runs  9 623 ✔️ 22 💤 0

Results for commit e550b48.

@jonahgraham jonahgraham merged commit a4ba717 into eclipse-cdt:main Jan 30, 2023
@jonahgraham jonahgraham deleted the fixes_259 branch January 30, 2023 19:59
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.

testDCOM_BDFG (org.eclipse.cdt.ui.tests.text.doctools.DocCommentHighlightingTest) flaky
1 participant