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

test(e2e): added new cases #6

Merged
merged 4 commits into from
Sep 2, 2024
Merged

Conversation

volodymyr-luzmo
Copy link
Collaborator

No description provided.

Copy link

Test Summary - Run #47

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
4 3 1 0 0 0 1 00:00:39

Some tests failed!

Name Status Failure Message
should have working initial screen failed ❌ Error: �[31mScreenshot comparison failed:�[39m

217717 pixels (ratio 0.07 of all image pixels) are different.

Expected: �[33m/home/runner/work/flex-showcases/flex-showcases/e2e/tests/wearables-dashboard/wearables-dashboard.spec.ts-snapshots/init-page-chromium-linux.png�[39m
Received: �[33m/home/runner/work/flex-showcases/flex-showcases/e2e/test-results/wearables-dashboard-wearab-e696a-have-working-initial-screen-chromium-retry2/init-page-actual.png�[39m
Diff: �[33m/home/runner/work/flex-showcases/flex-showcases/e2e/test-results/wearables-dashboard-wearab-e696a-have-working-initial-screen-chromium-retry2/init-page-diff.png�[39m

Call log:
�[2m- expect.toHaveScreenshot(init-page.png) with timeout 5000ms�[22m
�[2m - verifying given screenshot expectation�[22m
�[2m - taking page screenshot�[22m
�[2m - disabled all CSS animations�[22m
�[2m - waiting for fonts to load...�[22m
�[2m - fonts loaded�[22m
�[2m - 303603 pixels (ratio 0.09 of all image pixels) are different.�[22m
�[2m - waiting 100ms before taking screenshot�[22m
�[2m - taking page screenshot�[22m
�[2m - disabled all CSS animations�[22m
�[2m - waiting for fonts to load...�[22m
�[2m - fonts loaded�[22m
�[2m - 199111 pixels (ratio 0.06 of all image pixels) are different.�[22m
�[2m - waiting 250ms before taking screenshot�[22m
�[2m - taking page screenshot�[22m
�[2m - disabled all CSS animations�[22m
�[2m - waiting for fonts to load...�[22m
�[2m - fonts loaded�[22m
�[2m - captured a stable screenshot�[22m
�[2m - 217717 pixels (ratio 0.07 of all image pixels) are different.�[22m
|

A ctrf plugin

Copy link

Test Summary - Run #48

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
4 4 0 0 0 0 0 00:00:12

🎉 All tests passed!

A ctrf plugin

import { WearablesDashboardPage } from '../wearables-dashboard/wearables-dashboard.po';

test.describe('Wearables dashboard', () => {
test('should have working initial screen', async ({ page }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

2 spaces indent

import { BasePage } from '../base/base.po';

export class WearablesDashboardPage extends BasePage {
public readonly pageAdress = '/wearables-dashboard';
Copy link
Collaborator

Choose a reason for hiding this comment

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

2 spaces indent


public gotoMainPage = async () => await this.goto(this.pageAdress);
public NumberOfStepsChart = () => this.page.getByText('Number of steps by minute', {exact: true});
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please add new line in EOF

import { BasePage } from '../base/base.po';

export class DashboartFilterpanelPage extends BasePage {
public readonly pageAdress = '/dashboard-filter-panel';
Copy link
Collaborator

Choose a reason for hiding this comment

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

2 spaces indent

import { DashboartFilterpanelPage } from './dashboard-filter-panel.po';

test.describe('Dashboard filter panel page', () => {
test('should have working initial screen', async ({ page }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

2 spaces indent

import { BasePage } from '../base/base.po';

export class DragNDropChartLibraryPage extends BasePage {
public readonly pageAdress = '/drag-n-drop-chart-library';
Copy link
Collaborator

Choose a reason for hiding this comment

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

2 spaces indent

import { DragNDropChartLibraryPage } from './drag-n-drop-chart-library.po';

test.describe('Drag and drop page', () => {
test('should have working initial screen', async ({ page }) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

2 spaces indent

constructor(protected page: Page) {}

public goto = (adress: string) => this.page.goto(adress);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please check each file and add. new line to the end of file

Copy link

github-actions bot commented Sep 2, 2024

Test Summary - Run #51

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
4 4 0 0 0 0 0 00:00:26

🎉 All tests passed!

A ctrf plugin

Copy link

github-actions bot commented Sep 2, 2024

Test Summary - Run #52

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Pending ⏳ Other ❓ Flaky 🍂 Duration ⏱️
4 4 0 0 0 0 0 00:00:25

🎉 All tests passed!

A ctrf plugin

@volodymyr-luzmo volodymyr-luzmo merged commit da25855 into main Sep 2, 2024
7 checks passed
@volodymyr-luzmo volodymyr-luzmo deleted the test/e2e-expand-coverage branch September 2, 2024 08:20
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