-
Notifications
You must be signed in to change notification settings - Fork 38
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
[MM-57421] Live captions e2e tests #665
Conversation
@streamer45 This won't work yet of course, but it works locally, so it should work once everything is updated. |
02a607f
to
9232c0a
Compare
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.
Nice. Now it's a proper monster test but sadly it can't be avoided.
9232c0a
to
ff7764d
Compare
Yah, I can't see another way around it other than multiple test servers, or some kind of lock (for the duration of updating settings and starting a call)... |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #665 +/- ##
=======================================
Coverage 11.16% 11.16%
=======================================
Files 26 26
Lines 5606 5606
=======================================
Hits 626 626
Misses 4923 4923
Partials 57 57 ☔ View full report in Codecov by Sentry. |
@cpoile Can be merged. |
@streamer45 Was looking at making a lock for concurrent tests, but it looks like they purposefully make that kind of thing very difficult. Will try again later. |
Yeah good luck with that microsoft/playwright#21484 |
* Translations update from Mattermost Weblate (#661) * Translated using Weblate (Chinese (Simplified)) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/zh_Hans/ * Translated using Weblate (Dutch) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/nl/ --------- Co-authored-by: ThrRip <coding@thrrip.space> Co-authored-by: Tom De Moor <tom@controlaltdieliet.be> * [MM-57483] Add job config validation (#667) * Add job config validation * Update calls-offloader * go mod tidy * MM-56540 - Live captions 🎥 (#633) * implement live captions server side and frontend in expanded_view * linting; PR comments * i18n * clarify an unrelated component's naming: expanded_incoming_call_container * live captions should not overlap rhs * add EnableLiveCaptions and LiveCaptionsModelSize as console settings * upgrade @calls/common for standalone * add LiveCaptionsAudioLen as a new histogram metric; fix config passing * remove debugging statement and go mod tidy * update transcriber; fix defaults * rename live captions metrics * live captions metrics: buckets * live captions metrics: tweak buckets * add a metrics ws endpoint; move over struct defs from transcriber * add a metrics for transcriber buffer full * add a metrics for tick rate by transcriberID * clean up gauges after transcription ended * remove tick rate gauges -- not useful. * remove tick rate gauges -- not useful. * clean up naming * only show CC button once recording has started * move CC button * tweak captions width; update transcriber dependency, default numTranscribers -> 1 * new cc icon; 32px * add TranscriberNumThreads and LiveCaptionsNumThreadsPerTranscriber * update calls transcriber version * revert update calls transcriber version * update calls_transcriber_version to v0.2.2-dev * update calls/common; LiveCaptionsData uses channel_id now * send the channel_id with LiveCaptionsData for simplicity * and simplify the receipt of LiveCaptionData * add LiveCaptionsPktPayloadChBufFullCounter to metrics * add live captions state; recState -> jobState * combine job state messages under `call_job_state` * moved type to jobStateClient * add JobTypeCaptioning; use public.JobType as JobStateClient.Type * better comment * fix captions covering screensharing expand button * add LiveCaptionsLanguage system console setting * comment cleanup, useless code cleanup * bump deps for transcriber and rtcd * bumps for everyone except you rtcd * [MM-57421] Live captions e2e tests (#665) * MM-54243 - Slash command for host change (#668) * host change control PR squashed changes needed after rebase put behind enterprise for now (for community) -- will change on release blank commit hostlock if newHost is already the host rearrange order simplify PR comment PR comments use TrimPrefix implement host change slash command * removeUserSession doesn't need userID anymore * host controls are professional licensed * PR comments * PR comments * always have a host, but return it to the HostLockedID when they return * host should be allowed to change host even if they aren't admin * Translations update from Mattermost Weblate (#669) * Translated using Weblate (Chinese (Simplified)) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/zh_Hans/ * Translated using Weblate (Dutch) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/nl/ * Translated using Weblate (Japanese) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/ja/ * Translated using Weblate (Dutch) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/nl/ * Translated using Weblate (Swedish) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/sv/ * Translated using Weblate (Japanese) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/ja/ * Translated using Weblate (Chinese (Simplified)) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/zh_Hans/ --------- Co-authored-by: ThrRip <coding@thrrip.space> Co-authored-by: Tom De Moor <tom@controlaltdieliet.be> Co-authored-by: kaakaa <stooner.hoe@gmail.com> Co-authored-by: MArtin Johnson <martinjohnson@bahnhof.se> * [MM-57529] e2e tests for host change slash command (#671) * host change e2e tests * extend timeout --------- Co-authored-by: Weblate (bot) <hosted@weblate.org> Co-authored-by: ThrRip <coding@thrrip.space> Co-authored-by: Tom De Moor <tom@controlaltdieliet.be> Co-authored-by: Christopher Poile <cpoile@gmail.com> Co-authored-by: kaakaa <stooner.hoe@gmail.com> Co-authored-by: MArtin Johnson <martinjohnson@bahnhof.se>
* Improve store consistency * API router * Fix JSON values binary conversion * temp * temp2 * temp3 * Return expected client state * Remove channelState * Fix tests * Cleanup state * calls_jobs store * Jobs business logic * Fix tests * Fix linter * Include jobs in returned client state * More fixes and tests * Link to Jira ticket * GetActiveCallJobs * Rename lockCall functions * Make JSON columns not NULL * [MM-42464] Merge in latest changes (#670) * Translations update from Mattermost Weblate (#661) * Translated using Weblate (Chinese (Simplified)) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/zh_Hans/ * Translated using Weblate (Dutch) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/nl/ --------- Co-authored-by: ThrRip <coding@thrrip.space> Co-authored-by: Tom De Moor <tom@controlaltdieliet.be> * [MM-57483] Add job config validation (#667) * Add job config validation * Update calls-offloader * go mod tidy * MM-56540 - Live captions 🎥 (#633) * implement live captions server side and frontend in expanded_view * linting; PR comments * i18n * clarify an unrelated component's naming: expanded_incoming_call_container * live captions should not overlap rhs * add EnableLiveCaptions and LiveCaptionsModelSize as console settings * upgrade @calls/common for standalone * add LiveCaptionsAudioLen as a new histogram metric; fix config passing * remove debugging statement and go mod tidy * update transcriber; fix defaults * rename live captions metrics * live captions metrics: buckets * live captions metrics: tweak buckets * add a metrics ws endpoint; move over struct defs from transcriber * add a metrics for transcriber buffer full * add a metrics for tick rate by transcriberID * clean up gauges after transcription ended * remove tick rate gauges -- not useful. * remove tick rate gauges -- not useful. * clean up naming * only show CC button once recording has started * move CC button * tweak captions width; update transcriber dependency, default numTranscribers -> 1 * new cc icon; 32px * add TranscriberNumThreads and LiveCaptionsNumThreadsPerTranscriber * update calls transcriber version * revert update calls transcriber version * update calls_transcriber_version to v0.2.2-dev * update calls/common; LiveCaptionsData uses channel_id now * send the channel_id with LiveCaptionsData for simplicity * and simplify the receipt of LiveCaptionData * add LiveCaptionsPktPayloadChBufFullCounter to metrics * add live captions state; recState -> jobState * combine job state messages under `call_job_state` * moved type to jobStateClient * add JobTypeCaptioning; use public.JobType as JobStateClient.Type * better comment * fix captions covering screensharing expand button * add LiveCaptionsLanguage system console setting * comment cleanup, useless code cleanup * bump deps for transcriber and rtcd * bumps for everyone except you rtcd * [MM-57421] Live captions e2e tests (#665) * MM-54243 - Slash command for host change (#668) * host change control PR squashed changes needed after rebase put behind enterprise for now (for community) -- will change on release blank commit hostlock if newHost is already the host rearrange order simplify PR comment PR comments use TrimPrefix implement host change slash command * removeUserSession doesn't need userID anymore * host controls are professional licensed * PR comments * PR comments * always have a host, but return it to the HostLockedID when they return * host should be allowed to change host even if they aren't admin * Translations update from Mattermost Weblate (#669) * Translated using Weblate (Chinese (Simplified)) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/zh_Hans/ * Translated using Weblate (Dutch) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/nl/ * Translated using Weblate (Japanese) Currently translated at 100.0% (185 of 185 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/ja/ * Translated using Weblate (Dutch) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/nl/ * Translated using Weblate (Swedish) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/sv/ * Translated using Weblate (Japanese) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/ja/ * Translated using Weblate (Chinese (Simplified)) Currently translated at 100.0% (187 of 187 strings) Translation: Calls/webapp Translate-URL: https://translate.mattermost.com/projects/calls/webapp/zh_Hans/ --------- Co-authored-by: ThrRip <coding@thrrip.space> Co-authored-by: Tom De Moor <tom@controlaltdieliet.be> Co-authored-by: kaakaa <stooner.hoe@gmail.com> Co-authored-by: MArtin Johnson <martinjohnson@bahnhof.se> * [MM-57529] e2e tests for host change slash command (#671) * host change e2e tests * extend timeout --------- Co-authored-by: Weblate (bot) <hosted@weblate.org> Co-authored-by: ThrRip <coding@thrrip.space> Co-authored-by: Tom De Moor <tom@controlaltdieliet.be> Co-authored-by: Christopher Poile <cpoile@gmail.com> Co-authored-by: kaakaa <stooner.hoe@gmail.com> Co-authored-by: MArtin Johnson <martinjohnson@bahnhof.se> --------- Co-authored-by: Weblate (bot) <hosted@weblate.org> Co-authored-by: ThrRip <coding@thrrip.space> Co-authored-by: Tom De Moor <tom@controlaltdieliet.be> Co-authored-by: Christopher Poile <cpoile@gmail.com> Co-authored-by: kaakaa <stooner.hoe@gmail.com> Co-authored-by: MArtin Johnson <martinjohnson@bahnhof.se>
Summary
Ticket Link