Skip to content

Conversation

@effie-ms
Copy link
Contributor

@effie-ms effie-ms commented Dec 2, 2025

Which Jira task is linked to this PR?

https://lifi.atlassian.net/browse/LF-16706

Why was it implemented this way?

This PR:

  • adds tests to 2 examples: connectkit and vite
  • adds data-testid to widget
  • NB: locators in the tests need to be replaced once data-testid are released.

Checklist before requesting a review

  • I have performed a self-review and testing of my code.
  • This pull request is focused and addresses a single problem.

@effie-ms effie-ms self-assigned this Dec 2, 2025

export default defineConfig({
testDir: './tests',
fullyParallel: false,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

To discuss: parallelism - pnpm level or within Playwright

/**
* See https://playwright.dev/docs/test-configuration.
*/
const port = 4173
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ports should be different for each example if we run tests in parallel (pnpm level)

},
],
webServer: {
command: `NPM_CONFIG_LOGLEVEL=error npm run dev -- --port ${port}`,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

(pnpm just hangs on some reason, used npm)

@effie-ms
Copy link
Contributor Author

effie-ms commented Dec 2, 2025

@chybisov can you check this setup, what do you think?

@effie-ms effie-ms requested a review from chybisov December 2, 2025 15:38
@effie-ms effie-ms marked this pull request as ready for review December 3, 2025 10:34
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