-
Notifications
You must be signed in to change notification settings - Fork 70
feat(ccwidgets): add e2e tests for real time transcript and real time assist #729
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
base: next
Are you sure you want to change the base?
Changes from 6 commits
8d70374
4a01bae
98e7afe
0b146b6
09aa387
6619af0
633ecfd
7760066
80db8ab
1969bba
7c5c768
da690b2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -46,7 +46,11 @@ const RealTimeTranscriptComponent: React.FC<RealTimeTranscriptComponentProps> = | |
| ) : null} | ||
| </Text> | ||
| ) : null} | ||
| <div className="real-time-transcript__item" data-testid="real-time-transcript:item"> | ||
| <div | ||
| className="real-time-transcript__item" | ||
| data-testid="real-time-transcript:item" | ||
| data-speaker-role={entry.isCustomer ? 'customer' : 'agent'} | ||
|
Comment on lines
+51
to
+52
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The new E2E helpers depend on AGENTS.md reference: AGENTS.md:L68-L68 Useful? React with 👍 / 👎. |
||
| > | ||
| <div className="real-time-transcript__avatar-wrap"> | ||
| <Avatar | ||
| className="real-time-transcript__avatar-fallback" | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replacing
test:e2ewith only the two suffixed scripts leaves no root script namedtest:e2e, while the modified Playwright README still usesyarn test:e2efor every all-suite, single-suite, project, UI, debug, and headed invocation. Those documented local commands now fail during script resolution before Playwright starts; retain a compatible alias or update the documented callers to usetest:e2e:cc.AGENTS.md reference: AGENTS.md:L78-L78
Useful? React with 👍 / 👎.