Skip to content

Fix flacky test when using the searchbar in the ScanConfigEditDialog - #5584

Merged
greenbonebot merged 1 commit into
mainfrom
fix-flacky-scan-config-edit-dialog-test
Sep 7, 2026
Merged

Fix flacky test when using the searchbar in the ScanConfigEditDialog#5584
greenbonebot merged 1 commit into
mainfrom
fix-flacky-scan-config-edit-dialog-test

Conversation

@bjoernricks

Copy link
Copy Markdown
Contributor

What

Fix flacky test when using the searchbar in the ScanConfigEditDialog

Why

When running the tests in parallel using the searchbar in the ScanConfigEditDialog could return the wrong count of displayed rows. Because the timers run in real time and due to concurrency the number of rows might be different. Therefore use fake timers when interacting with the searchbar to always get the expected results.

Checklist

  • Tests

…ialog

When running the tests in parallel using the searchbar in the
ScanConfigEditDialog could return the wrong count of displayed rows.
Because the timers run in real time and due to concurrency the number of
rows might be different. Therefore use fake timers when interacting with
the searchbar to always get the expected results.
@bjoernricks
bjoernricks requested a review from a team as a code owner September 7, 2026 11:39
@greenbonebot
greenbonebot enabled auto-merge (rebase) September 7, 2026 11:39
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 3c499a2.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@greenbonebot
greenbonebot merged commit 3f9e10e into main Sep 7, 2026
34 checks passed
@greenbonebot
greenbonebot deleted the fix-flacky-scan-config-edit-dialog-test branch September 7, 2026 12:16
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.

3 participants