-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
fix: cleanup from #7029 #7328
fix: cleanup from #7029 #7328
Conversation
Current Playwright Test Results Summary✅ 166 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 12/28/2023 06:34:02pm UTC)
|
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Example Imagery Object Can use independent time conductor to change time
Retry 2 • Retry 1 • Initial Attempt |
0% (0)0 / 46 runsfailed over last 7 days |
4.35% (2)2 / 46 runsflaked over last 7 days |
📄 functional/plugins/tabs/tabs.e2e.spec.js • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Tabs View Renders tabbed elements
Retry 1 • Initial Attempt |
10.81% (4)4 / 37 runsfailed over last 7 days |
24.32% (9)9 / 37 runsflaked over last 7 days |
📄 functional/plugins/notebook/restrictedNotebook.e2e.spec.js • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Restricted Notebook with a page locked and with an embed @addinit Allows embeds to be deleted if page unlocked @addinit
Retry 1 • Initial Attempt |
0% (0)0 / 39 runsfailed over last 7 days |
56.41% (22)22 / 39 runsflaked over last 7 days |
📄 functional/plugins/conditionSet/conditionSet.e2e.spec.js • 3 Flakes
Top 1 Common Error Messages
|
3 Test Cases Affected |
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Condition Set CRUD Operations on @localStorage Condition set object properties persist in main view and inspector @localStorage
Retry 1 • Initial Attempt |
5.41% (2)2 / 37 runsfailed over last 7 days |
2.70% (1)1 / 37 runflaked over last 7 days |
Condition Set CRUD Operations on @localStorage condition set object can be modified on @localStorage
Retry 1 • Initial Attempt |
0% (0)0 / 34 runsfailed over last 7 days |
2.94% (1)1 / 34 runflaked over last 7 days |
Condition Set CRUD Operations on @localStorage condition set object can be deleted by Search Tree Actions menu on @localStorage
Retry 1 • Initial Attempt |
-3.03% (-1)-1 / 33 runsfailed over last 7 days |
3.03% (1)1 / 33 runflaked over last 7 days |
📄 functional/plugins/timeConductor/timeConductor.e2e.spec.js • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Time conductor input fields real-time mode preserve offsets and url params when switching between fixed and real-time mode
Retry 1 • Initial Attempt |
0% (0)0 / 34 runsfailed over last 7 days |
2.94% (1)1 / 34 runflaked over last 7 days |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #7328 +/- ##
==========================================
- Coverage 55.69% 55.65% -0.05%
==========================================
Files 656 656
Lines 26203 26194 -9
Branches 2546 2546
==========================================
- Hits 14594 14577 -17
+ Misses 10911 10909 -2
- Partials 698 708 +10
... and 10 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Thanks for fixing those. Looks like this just needs a npm run lint:fix
and should be good to go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! LGTM
Describe your changes:
Cleanup PR from #7029 to convert a few more modules to ESM, and to stop using
require
.All Submissions:
Author Checklist
type:
label? Note: this is not necessarily the same as the original issue.Reviewer Checklist