-
Notifications
You must be signed in to change notification settings - Fork 8
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
Test to check that bar graph makes requests with size 1 (limit=1). #470
Conversation
Test to check that bar graph makes requests with size 1 (limit=1).
Move waitForResponse to just before telemetry is added to the graph
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.
let's modify the tests a bit
…o bar-graph-historical-requests
package.json
Outdated
@@ -20,7 +20,7 @@ | |||
"lint:fix": "eslint src example --fix", | |||
"build:dist": "webpack --config ./.webpack/webpack.prod.mjs", | |||
"build:example": "npm install openmct@unstable --no-save", | |||
"build:example:master": "npm install nasa/openmct --no-save", | |||
"build:example:master": "npm install nasa/openmct#bar-graph-historical-requests --no-save", |
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.
here
|
||
// Create the Bar Graph | ||
barGraph = await createDomainObjectWithDefaults(page, { type: 'Graph', name: 'Bar Graph' }); | ||
// Enter edit mode for the overlay plot |
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.
Link 'Magnetometer' to barGraph
tests/e2e/yamcsAppActions.mjs
Outdated
/** | ||
* Search for telemetry and link it to an object. objectName should come from the domainObject.name function. | ||
* @param {import('@playwright/test').Page} page | ||
* @param {string} telemetryText |
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.
rename to parameterNameText?
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.
rename to parameterName
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.
Need another reviewer
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.
mostly LGTM just have a couple requests
tests/e2e/yamcsAppActions.mjs
Outdated
/** | ||
* Search for telemetry and link it to an object. objectName should come from the domainObject.name function. | ||
* @param {import('@playwright/test').Page} page | ||
* @param {string} telemetryText |
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.
rename to parameterName
Closes nasa/openmct#7811
Describe your changes:
Include Graphs in default plugins installed.
Test to check that bar graph makes requests with size 1 (limit=1)
All Submissions:
Author Checklist
type:
label? Note: this is not necessarily the same as the original issue.Reviewer Checklist