Skip to content

Nt 3466#318

Draft
David Nalchevanidze (nalchevanidze) wants to merge 2 commits into
mainfrom
NT-3466
Draft

Nt 3466#318
David Nalchevanidze (nalchevanidze) wants to merge 2 commits into
mainfrom
NT-3466

Conversation

@nalchevanidze

Copy link
Copy Markdown
Contributor

No description provided.

- Add playwright.config.mjs with webServer readiness gate, workers=1,
  and per-assertion timeout to fix headless vs UI mode parity
- Add e2e/ with all specs copied from web-sdk_react as a starting point
- Update serve:app to use ng serve via PM2 with log-based readiness wait
- Add test:e2e:setup shared between headless and UI commands
- Add allowedCommonJsDependencies for lodash to suppress build warning

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Align heading selectors with Angular template text ('Auto-observed', 'Manually-observed')
- Use data-testid instead of role+name for consent button ('Grant' vs 'Accept Consent')
- Normalize page event URL to pathname in tracking log so navigation spec matches React behavior
- Count all non-null stream events for rawEventsCount so identify events are included

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We should probably add this to the main workflow, so we can ensure it passes in CI.

@nalchevanidze David Nalchevanidze (nalchevanidze) marked this pull request as draft June 16, 2026 13:29
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