Page MenuHomePhabricator

Create a generic NPM package for recording videos of test runs
Closed, DuplicatePublic

Description

It should use xvfb and ffmpeg, like webdriverio tests used to have. See 604635.

The only argument should be the file name, and the only commands are start and stop recording.

Links

TODO

Event Timeline

Restricted Application added a subscriber: Aklapper. ยท View Herald TranscriptJul 31 2020, 11:24 AM
zeljkofilipin moved this task from In Progress to Backlog on the User-Soham board.
zeljkofilipin added a subscriber: Soham.

@jaredblumer I've tried following the readme, but I get this:

$ npm install test-video-recorder
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/test-video-recorder - Not found
npm ERR! 404 
npm ERR! 404  'test-video-recorder@latest' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404 
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/z/.npm/_logs/2021-01-07T18_30_11_650Z-debug.log

I also don't see the package at NPM: https://www.npmjs.com/package/test-video-recorder (it returns 404). Did you forget to publish the package?

A great test for the package would be to use it in mediawiki/core. Similar to See 604635, but replace wdio-video-reporter with test-video-recorder.

Let me know if you have any questions or need any help.

Change 673828 had a related patch set uploaded (by Jared Blumer; owner: Jared Blumer):
[mediawiki/core@master] Add test-video-recorder package to selenium tests

https://gerrit.wikimedia.org/r/673828

Change 679554 had a related patch set uploaded (by Jared Blumer; author: Jared Blumer):

[mediawiki/core@master] Replace wdio-video-reporter package with test-video-recorder

https://gerrit.wikimedia.org/r/679554

The simplest solution is to move video recording to wdio-mediawiki's wdio.conf.js file. See T281484.

Change 679554 abandoned by Zfilipin:

[mediawiki/core@master] Replace wdio-video-reporter package with test-video-recorder

Reason:

Thanks for the patch. This will be implemented in a different way. See T253113.

https://gerrit.wikimedia.org/r/679554

Change 673828 abandoned by Zfilipin:

[mediawiki/core@master] Add test-video-recorder package to selenium tests

Reason:

Thanks for the patch. This will be implemented in a different way. See T253113.

https://gerrit.wikimedia.org/r/673828