Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update playwright project #8

Merged
merged 10 commits into from
Jul 15, 2024
Merged

Update playwright project #8

merged 10 commits into from
Jul 15, 2024

Conversation

EugenePetrik
Copy link
Collaborator

@EugenePetrik EugenePetrik commented Jul 14, 2024

Changes:

  • updated playwright dependency
  • added @types/node dependency
  • updated playwright.config.js
  • added vscode/extensions.json
  • added GitHub Actions CI config
  • updated base fixture
  • updated page objects
  • added application manager
  • updated tests

Tests run - https://github.com/ovartem/playwright_tests_with_po/actions/runs/9927851556
Job run - https://github.com/ovartem/playwright_tests_with_po/actions/runs/9927851556/job/27423339094

@EugenePetrik EugenePetrik requested a review from ovartem July 14, 2024 12:08
@EugenePetrik EugenePetrik self-assigned this Jul 14, 2024
Copy link
Owner

@ovartem ovartem left a comment

Choose a reason for hiding this comment

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

2 small comments left. Thank you

.eslintrc.js Outdated
@@ -41,5 +34,6 @@ module.exports = {
ignoreTemplateLiterals: true,
ignoreRegExpLiterals: true,
}],
'import/prefer-default-export': 'off' // TODO: remove
Copy link
Owner

Choose a reason for hiding this comment

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

I think this should be removed, right?

import { test as base } from '@playwright/test';
import { Application } from '../pages/Application';

export const test = base.extend({
Copy link
Owner

Choose a reason for hiding this comment

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

can you please setup JSDoc so that intellisense (codecomplition in tests) works for JS code?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for noticing, autocomplete is working now

image

@ovartem ovartem merged commit c7c1b7c into master Jul 15, 2024
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants