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.

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

Feat.add e2e test for maps no trigger #1597

Merged
merged 5 commits into from
Oct 3, 2024

Conversation

Piotrk39
Copy link
Contributor

@Piotrk39 Piotrk39 commented Sep 30, 2024

Describe your changes

Adds E2E test for validating front-end elements of map component are loaded and presented properly.

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added tests wherever relevant
  • I have made sure that all automated checks pass before requesting a review

Notes for the reviewer

  1. assertAdminBoundariesVisible() function may need a closer look since there were some issue with asynchronous actions between map and FE.

Add flodds no-trigger reset flow

Add POM locators, testData & dev country switcher

Change test names & locators selection

Add waitForLoadState

Add component object, testid & assert header function

Add TopBarComponent object & expect assert method

Add ChatComponent POM & assertions fo no-trigger

Add AggregateComponent POM & assertions

Add MapComponent POM & assertions

Add comments and timeout

Move test and add force click

Add additional click

add timeout

test

add angualr attribute and change data test ids

remove redundant waits

Create a base for the test

Fix typos and add breadCrumb is visible function and testIds

Add legend open true or false assertion and data-testid for legend

Add layer-menu test id and assertion for Layers to be hidden by default
@Piotrk39 Piotrk39 force-pushed the feat.add-e2e-test-for-maps-no-trigger branch from 725b690 to 5843e08 Compare October 1, 2024 09:47
@Piotrk39 Piotrk39 requested a review from gulfaraz October 1, 2024 12:03
@Piotrk39 Piotrk39 marked this pull request as ready for review October 1, 2024 12:04
Copy link
Member

@gulfaraz gulfaraz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! No blocking comments. Great work @Piotrk39

> [email protected] test
> playwright test


Running 2 tests using 1 worker

  ✓  1 [chromium] › Dashboard/AllElementsPresentNoTrigger.spec.ts:40:5 › [30509] All Dashboard elements are present in no-trigger mode (9.8s)
  ✓  2 [chromium] › Map/AllMapComponentsPresentNotrigger.spec.ts:37:5 › [30538] All Map elements are present in no-trigger mode (9.4s)

  2 passed (20.4s)

@Piotrk39 Piotrk39 merged commit 473cdc3 into master Oct 3, 2024
8 checks passed
@Piotrk39 Piotrk39 deleted the feat.add-e2e-test-for-maps-no-trigger branch October 3, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants