Skip to content
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.

Sign up for GitHub

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

Experience/15262/receiver status smoke #15824

Merged
merged 20 commits into from
Sep 10, 2024
Merged

Experience/15262/receiver status smoke #15824

Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
e749436
15262 - Fixed smoke tests
penny-lischer Sep 4, 2024
63a740a
Merge branch 'master' into experience/15262/receiver-status-smoke
penny-lischer Sep 5, 2024
844110d
Created receiver-status-page-user-flow.spec.ts and made tests as sani…
penny-lischer Sep 5, 2024
3192545
Fixed linting
penny-lischer Sep 5, 2024
44e220a
Fixed path
penny-lischer Sep 5, 2024
54a0670
Fixed path
penny-lischer Sep 5, 2024
d91c908
Removed commented code
penny-lischer Sep 6, 2024
6805a26
Fixed spacing issues
penny-lischer Sep 6, 2024
6d11d81
Fixed spacing issues
penny-lischer Sep 6, 2024
720cc5e
Fixed spacing issues
penny-lischer Sep 6, 2024
1682da1
Fixed failing tests
penny-lischer Sep 7, 2024
f8e04fe
Fixed failing tests
penny-lischer Sep 7, 2024
5b29b41
Added check if using mock
penny-lischer Sep 9, 2024
d5e9d76
Added check if using mock
penny-lischer Sep 9, 2024
33592be
Fixing receiver name failing test
penny-lischer Sep 9, 2024
4110c1f
Skipped tests until filters have been fixed per #15737
penny-lischer Sep 9, 2024
875b719
Skipped tests until filters have been fixed per #15737
penny-lischer Sep 9, 2024
c71bbcf
Merge branch 'master' into experience/15262/receiver-status-smoke
penny-lischer Sep 9, 2024
4533a36
Merge branch 'master' into experience/15262/receiver-status-smoke
penny-lischer Sep 9, 2024
dd33b0f
Merge branch 'master' into experience/15262/receiver-status-smoke
penny-lischer Sep 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions frontend-react/e2e/spec/chromium-only/authenticated/receiver-status-page-user-flow.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,8 @@
},
);

test("receiver name", async ({adminReceiverStatusPage, isMockDisabled}) => {
// TODO: revisit after filters have been fixed per ticket #15737
test.skip("receiver name", async ({adminReceiverStatusPage, isMockDisabled}) => {

Check warning on line 179 in frontend-react/e2e/spec/chromium-only/authenticated/receiver-status-page-user-flow.spec.ts

View workflow job for this annotation

GitHub Actions / Lint 

Unexpected use of the `.skip()` annotation
test.skip(!isMockDisabled, "Mocks are ENABLED, skipping 'receiver name' test");
const {organizationName, receiverName, successRate} =
adminReceiverStatusPage.timePeriodData[1];
Expand Down Expand Up @@ -206,7 +207,7 @@
expect(defaultReceiversStatusRowsCount).toBe(adminReceiverStatusPage.timePeriodData.length);
});

test("result message", async ({adminReceiverStatusPage, isMockDisabled}) => {
test.skip("result message", async ({adminReceiverStatusPage, isMockDisabled}) => {
test.skip(!isMockDisabled, "Mocks are ENABLED, skipping 'result message' test");
// get first entry's result from all-fail receiver's first day -> third time period
const receiverI = 0;
Expand Down Expand Up @@ -242,7 +243,7 @@
// await adminReceiverStatusPage.testReceiverStatusDisplay();
});

test("success type", async ({ adminReceiverStatusPage, isMockDisabled }) => {
test.skip("success type", async ({ adminReceiverStatusPage, isMockDisabled }) => {
test.skip(!isMockDisabled, "Mocks are ENABLED, skipping 'success type' test");
const [failRow, ,] = adminReceiverStatusPage.timePeriodData;
const failRowTitle = adminReceiverStatusPage.getExpectedReceiverStatusRowTitle(
Expand Down
Loading