Skip to content

Fix FileMoverTest flaky test#3590

Merged
kikoveiga merged 1 commit into
developfrom
kikoveiga/fix-FileMoverTest-flaky-test
Jun 30, 2026
Merged

Fix FileMoverTest flaky test#3590
kikoveiga merged 1 commit into
developfrom
kikoveiga/fix-FileMoverTest-flaky-test

Conversation

@kikoveiga

Copy link
Copy Markdown
Contributor

This was something on my todo list from a failed pipeline I saw some weeks ago. It just aligns with the other test which was already removing duplicates.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@kikoveiga
kikoveiga requested review from a team as code owners June 30, 2026 10:51
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.77%. Comparing base (9e60d05) to head (0831c1d).
⚠️ Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3590      +/-   ##
===========================================
- Coverage    72.80%   72.77%   -0.03%     
===========================================
  Files          974      974              
  Lines        35228    35225       -3     
  Branches      5965     5968       +3     
===========================================
- Hits         25645    25634      -11     
- Misses        7915     7919       +4     
- Partials      1668     1672       +4     

see 37 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kikoveiga
kikoveiga merged commit 3b2aa61 into develop Jun 30, 2026
27 checks passed
@kikoveiga
kikoveiga deleted the kikoveiga/fix-FileMoverTest-flaky-test branch June 30, 2026 15:28
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