Skip to content

fix(tests): Allow a 10 seconds delta for lockedUntil in form tests#2891

Merged
Chartman123 merged 1 commit into
mainfrom
fix/integration-test-locking
Aug 30, 2025
Merged

fix(tests): Allow a 10 seconds delta for lockedUntil in form tests#2891
Chartman123 merged 1 commit into
mainfrom
fix/integration-test-locking

Conversation

@Chartman123
Copy link
Copy Markdown
Collaborator

Currently the integration tests fail sometimes when the form is locked. To fix this, the lockedUntil value is now checked against a delta of 10 secs.

Signed-off-by: Christian Hartmann chris-hartmann@gmx.de

@Chartman123 Chartman123 added bug Something isn't working 3. to review Waiting for reviews labels Aug 30, 2025
@Chartman123 Chartman123 requested a review from susnux August 30, 2025 08:24
@Chartman123 Chartman123 added papercut Annoying recurring issue with possibly simple fix. technical debt Technical issue labels Aug 30, 2025
@Chartman123 Chartman123 self-assigned this Aug 30, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 44.20%. Comparing base (7e1c1d1) to head (e14e881).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #2891   +/-   ##
=========================================
  Coverage     44.20%   44.20%           
  Complexity     1028     1028           
=========================================
  Files            80       80           
  Lines          3696     3696           
=========================================
  Hits           1634     1634           
  Misses         2062     2062           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
@Chartman123 Chartman123 force-pushed the fix/integration-test-locking branch from ee2b6f7 to e14e881 Compare August 30, 2025 08:31
@Chartman123 Chartman123 merged commit 5a4889d into main Aug 30, 2025
54 checks passed
@Chartman123 Chartman123 deleted the fix/integration-test-locking branch August 30, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working papercut Annoying recurring issue with possibly simple fix. technical debt Technical issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant