-
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
fix: Gantt Chart displays draft status of Plan in composition #6642
Conversation
Current Playwright Test Results Summary✅ 12 Passing - Run may still be in progress, this comment will be updated as current testing workflow or job completes... (Last updated on 05/05/2023 11:48:21pm UTC) Run DetailsRunning Workflow e2e-couchdb on Github Actions Commit: 8392d3a Started: 05/05/2023 11:45:20pm UTC
|
Test Case | Last 7 days Failures | Last 7 days Flakes |
---|---|---|
Notebook Tests with CouchDB @couchdb Inspect Notebook Entry Network Requests
Retry 1 • Initial Attempt |
0% (0)0 / 27 runsfailed over last 7 days |
44.44% (12)12 / 27 runsflaked over last 7 days |
@@ -232,6 +233,7 @@ export default { | |||
this.planObject = domainObject; | |||
this.swimlaneVisibility = this.configuration.swimlaneVisibility; | |||
this.planData = getValidatedData(domainObject); | |||
this.setStatus(this.openmct.status.get(domainObject.identifier)); |
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.
is there a simple way to add coverage for this line
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.
This line should be covered by the e2e test I added? Wonder why that's not being picked up... 🤔
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. I had a simple question on how to add coverage to the new line.
Also, if you have some cycles, it might be worth making a visual test for draft
Good call. This really is more of a visual test anyway. |
37b8342
to
1aacccf
Compare
Codecov Report
@@ Coverage Diff @@
## master #6642 +/- ##
==========================================
- Coverage 54.16% 53.73% -0.43%
==========================================
Files 623 623
Lines 24374 24376 +2
Branches 2412 2412
==========================================
- Hits 13202 13099 -103
- Misses 10512 10618 +106
+ Partials 660 659 -1
*This pull request uses carry forward flags. Click here to find out more.
... and 22 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Closes #6641 VIPEROMCT-303
Describe your changes:
The Gantt Chart view was not respecting / displaying the draft status of a Plan if it was in its composition. This updates the logic to set the status when a domainObject is added in composition.
All Submissions:
Author Checklist
Reviewer Checklist