-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
When searching, build the path objects asynchronously while returning the results #7265
Conversation
Current Playwright Test Results Summary✅ 162 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 12/04/2023 09:40:08pm UTC)
|
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Tabs View Renders tabbed elements
Retry 1 • Initial Attempt |
0% (0)0 / 42 runsfailed over last 7 days |
4.76% (2)2 / 42 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 Disallows embeds to be deleted if page locked @addinit
Retry 1 • Initial Attempt |
2.33% (1)1 / 43 runfailed over last 7 days |
37.21% (16)16 / 43 runsflaked over last 7 days |
Current Playwright Test Results Summary
✅ 14 Passing -
Run may still be in progress, this comment will be updated as current testing workflow or job completes...
(Last updated on 12/04/2023 09:40:08pm UTC)
Run Details
Running Workflow e2e-couchdb on Github Actions
Commit: f573aa8
Started: 12/04/2023 09:34:07pm UTC
⚠️ Flakes
📄 functional/couchdb.e2e.spec.js • 1 Flake
Test Case Results
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
CouchDB Status Indicator with mocked responses @couchdb Shows unknown if it receives an unexpected response code
Retry 1 • Initial Attempt |
0% (0)0 / 39 runsfailed over last 7 days |
15.38% (6)6 / 39 runsflaked over last 7 days |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #7265 +/- ##
==========================================
+ Coverage 54.76% 55.16% +0.39%
==========================================
Files 654 654
Lines 26256 26238 -18
Branches 2536 2530 -6
==========================================
+ Hits 14380 14474 +94
+ Misses 11171 11059 -112
Partials 705 705
... and 15 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Before: before.movNote all the results, object paths, and annotations come at once. After: after.movNote you get results relatively quickly that you can click on. |
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.
LGTM! A legit use of for await
!
async getOriginalPath(identifier, path = []) { | ||
const domainObject = await this.get(identifier); | ||
async getOriginalPath(identifier, path = [], abortSignal = null) { | ||
const domainObject = await this.get(identifier, abortSignal); |
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.
awesome
}, | ||
async searchObjects(abortSignal) { | ||
const objectSearchPromises = this.openmct.objects.search(this.searchValue, abortSignal); | ||
for await (const objectSearchResult of objectSearchPromises) { |
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.
brilliant stuff!
Closes #7264
Describe your changes:
All Submissions:
Author Checklist
Reviewer Checklist