Skip to content

[O2B-1603] Fix node detached with multiple attempts instead - #2201

Closed
graduta wants to merge 3 commits into
mainfrom
bug/O2B-1603-fix-test-with-detached-node
Closed

[O2B-1603] Fix node detached with multiple attempts instead#2201
graduta wants to merge 3 commits into
mainfrom
bug/O2B-1603-fix-test-with-detached-node

Conversation

@graduta

@graduta graduta commented Jul 31, 2026

Copy link
Copy Markdown
Member

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • N/A

Notable changes for developers:

  • the pressElement function was refactored into attempting multiple times to click in case the node-detached in the mean time due to multiple re-renders on the page

Changes made to the database:

@graduta graduta self-assigned this Jul 31, 2026
@graduta
graduta requested a review from isaachilly as a code owner July 31, 2026 07:39
@graduta graduta added javascript Pull requests that update Javascript code CI test Label to be used when PR is only about tests and removed CI labels Jul 31, 2026
@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.01%. Comparing base (bb254ed) to head (86fd04e).

Files with missing lines Patch % Lines
lib/public/components/common/table/SortModel.js 0.00% 6 Missing ⚠️
lib/public/models/OverviewModel.js 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2201      +/-   ##
==========================================
- Coverage   46.20%   46.01%   -0.19%     
==========================================
  Files        1039     1039              
  Lines       17131    17138       +7     
  Branches     3123     3123              
==========================================
- Hits         7915     7886      -29     
- Misses       9216     9252      +36     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

@graduta graduta closed this Jul 31, 2026
@graduta
graduta deleted the bug/O2B-1603-fix-test-with-detached-node branch July 31, 2026 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update Javascript code test Label to be used when PR is only about tests

Development

Successfully merging this pull request may close these issues.

1 participant