-
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
[e2e] Update remaining tests and add missing comparison coverage #7363
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
8953b32
remove remaining page.locators
unlikelyzero d4310d3
Merge branch 'master' of https://github.com/nasa/openmct into fix-mor…
unlikelyzero 18899bc
remove icon
unlikelyzero 121e0ed
update test to remove bad locators
unlikelyzero 746e8fa
Update tests to match aria changes
unlikelyzero 774bbfc
duplicate aria-label from title
unlikelyzero c9bae39
Update Edit action to be more explicit
unlikelyzero f041845
update the watch file to better suite long runs
unlikelyzero d2155c8
fix locators
unlikelyzero a37642e
fix locator
unlikelyzero 8884ae0
revert to action.name for test
unlikelyzero 49b4f1e
PER JESSE'S INTENT
unlikelyzero f9122d9
review comments
unlikelyzero 4dd515f
fix tests
unlikelyzero 5affd36
more fixes
unlikelyzero d6d0739
update text
unlikelyzero 29244c0
review comment
unlikelyzero d846c1f
locator fixes
unlikelyzero 7c96dcd
oops
unlikelyzero b75e333
oops
unlikelyzero e04caef
all fixes
unlikelyzero a178533
more fixes
unlikelyzero 1ac2c8a
last changes before bed
unlikelyzero ca2e7c5
remove deploysentienl
unlikelyzero 316fb00
remove deploysentinel
unlikelyzero d15a6f1
remove deploysentinel
unlikelyzero 090bba2
fix generatelocal storage
unlikelyzero 9dcf95a
lint fix
unlikelyzero a1c2844
new locator
unlikelyzero 9bad40a
header component test added
unlikelyzero b3a5cf2
telemetryView tests
unlikelyzero 26c0f81
Merge branch 'master' of https://github.com/nasa/openmct into fix-mor…
unlikelyzero 003d7c2
2024
unlikelyzero 4b27e09
driveby: fix from missing pr review
unlikelyzero 72fe139
add delay for known issue
unlikelyzero 1d5c287
feedback
unlikelyzero 89b5bab
comments
unlikelyzero File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -43,7 +43,6 @@ | |
"sharded", | ||
"perfromance", | ||
"MMOC", | ||
"deploysentinel", | ||
"codegen", | ||
"Unfortuantely", | ||
"viewports", | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,6 +15,9 @@ | |
*.idea | ||
*.iml | ||
|
||
# VSCode | ||
.vscode/settings.json | ||
|
||
# Build output | ||
target | ||
dist | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,9 @@ | ||
// playwright.config.js | ||
// @ts-check | ||
|
||
// eslint-disable-next-line no-unused-vars | ||
import { devices } from '@playwright/test'; | ||
const MAX_FAILURES = 5; | ||
const NUM_WORKERS = 2; | ||
|
||
/** @type {import('@playwright/test').PlaywrightTestConfig} */ | ||
const config = { | ||
retries: 0, //Retries 2 times for a total of 3 runs. When running sharded and with max-failures=5, this should ensure that flake is managed without failing the full suite | ||
retries: 0, //Retries are not needed with watch mode | ||
testDir: 'tests', | ||
timeout: 60 * 1000, | ||
webServer: { | ||
|
@@ -17,8 +12,7 @@ const config = { | |
timeout: 200 * 1000, | ||
reuseExistingServer: true //This was originally disabled to prevent differences in local debugging vs. CI. However, it significantly speeds up local debugging. | ||
}, | ||
maxFailures: MAX_FAILURES, //Limits failures to 5 to reduce CI Waste | ||
workers: NUM_WORKERS, //Limit to 2 for CircleCI Agent | ||
workers: '75%', //Limit to 75% of the CPU to support running with dev server | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. rad |
||
use: { | ||
baseURL: 'http://localhost:8080/', | ||
headless: true, | ||
|
@@ -45,8 +39,7 @@ const config = { | |
outputFolder: '../html-test-results' //Must be in different location due to https://github.com/microsoft/playwright/issues/12840 | ||
} | ||
], | ||
['junit', { outputFile: '../test-results/results.xml' }], | ||
['@deploysentinel/playwright'] | ||
['junit', { outputFile: '../test-results/results.xml' }] | ||
] | ||
}; | ||
|
||
|
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
😞