Skip to content

added wdio-video-reporter for failed spec video recording#1059

Merged
tomholub merged 1 commit intomasterfrom
tests/issue-1052
Nov 22, 2021
Merged

added wdio-video-reporter for failed spec video recording#1059
tomholub merged 1 commit intomasterfrom
tests/issue-1052

Conversation

@fcvakintos
Copy link
Contributor

@fcvakintos fcvakintos commented Nov 22, 2021

This PR contains changes wdio-video-reporter - > the failed spec video will attach to artifacts

close #1052


Tests

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

Copy link
Collaborator

@tomholub tomholub left a comment

Choose a reason for hiding this comment

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

👍

@tomholub tomholub merged commit d142c8f into master Nov 22, 2021
@tomholub tomholub deleted the tests/issue-1052 branch November 22, 2021 11:32
@tomholub
Copy link
Collaborator

Whoops - I was meaning to just enable auto-merge. Hopefully it doesn't break ci.

tomholub added a commit that referenced this pull request Nov 22, 2021
tomholub added a commit that referenced this pull request Nov 22, 2021
@tomholub tomholub restored the tests/issue-1052 branch November 22, 2021 19:04
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.

failed appium tests to put video in artifacts?

3 participants