Closed RainNapper closed 3 years ago
Nice! Thanks for contributing this feature.
Could we add a test case that actually records video for a test and checks that video was recorded? I think it could be wired in one of the tests that runs
BPSampleApp
.
Added a test in the last push.
Hi there, you may want to allow the user to change the codec option, h264 or hevc. The default is hevc but slack cannot play hevc video file.
Suggestion:
new option deleteOnTestPassed
, meaning delete video if the test passed.
It can reduce the number of Videos stored on the CI machine.
Cool feature by the way.
Hi there, you may want to allow the user to change the codec option, h264 or hevc. The default is hevc but slack cannot play hevc video file.
Suggestion: new option
deleteOnTestPassed
, meaning delete video if the test passed. It can reduce the number of Videos stored on the CI machine.Cool feature by the way.
Totally agree. bp is pretty heavy on the configs atm, mind adding this as a follow up PR?
Hi there, you may want to allow the user to change the codec option, h264 or hevc. The default is hevc but slack cannot play hevc video file.
Suggestion: new option
deleteOnTestPassed
, meaning delete video if the test passed. It can reduce the number of Videos stored on the CI machine.Cool feature by the way.
@RainNapper I like @kinwahlai's suggestion. The deleteOnTestPassed can either be a config, or the default behavior too. It would be great if you can create an issue to discuss and follow it up with a PR. Thanks for the contribution.
Use the XCTest listener protocol to add hooks for
xcrun simctl io [sim-id] recordVideo
.