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 aggregates section #1626

Merged
merged 4 commits into from
Oct 7, 2024

Conversation

Piotrk39
Copy link
Contributor

@Piotrk39 Piotrk39 commented Oct 4, 2024

Describe your changes

Adds assertions for all elements in Aggregates section.

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

@Piotrk39 Piotrk39 added the test This involves testing label Oct 4, 2024
@Piotrk39 Piotrk39 marked this pull request as ready for review October 7, 2024 12:57
@gulfaraz gulfaraz changed the base branch from master to feat.add-e2e-test-for-chat-section-no-trigger October 7, 2024 16:06
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.

Nice work @Piotrk39 !

const layerCount = await this.aggregatesLayerRow.count();
const iconLayerCount = await this.aggregatesInfoIcon.count();
// Basic Assertions
expect(headerTextModified).toBe('National View 0 Predicted Flood(s)');
Copy link
Member

Choose a reason for hiding this comment

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

This will be different for other countries and hazards. It's good for now. We should run these tests for other countries in the next month to see what is different.

@@ -3,22 +3,73 @@ import { Locator, Page } from 'playwright';

import DashboardPage from './DashboardPage';

const expectedLayersNames = [
Copy link
Member

Choose a reason for hiding this comment

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

It's good that this list is outside the logic. This list varies depending on country and hazard.

Base automatically changed from feat.add-e2e-test-for-chat-section-no-trigger to master October 7, 2024 18:20
@Piotrk39 Piotrk39 merged commit 096e7d9 into master Oct 7, 2024
8 checks passed
@Piotrk39 Piotrk39 deleted the feat.add-e2e-test-for-aggregates-section branch October 7, 2024 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test This involves testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants