Skip to content
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.

Sign up for GitHub

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

Jump to bottom

update acceptance test workflow name #779

Merged
merged 1 commit into from
May 13, 2021
Merged

update acceptance test workflow name #779

merged 1 commit into from
May 13, 2021

Conversation

oshi97
Copy link
Contributor

@oshi97 oshi97 commented May 12, 2021 •
edited
Loading

I think initially the workflow only ran tests in browserstack,
but the headless tests don't use browserstack.

@oshi97
update acceptance test workflow name
76c72ab 
I think initially the workflow only ran
tests in browserstack, but the headless
tests don't use browserstack.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 4.724% when pulling 76c72ab on dev/workflow-rename-nits into 1e7549c on feature/test-infra-fixes.

@oshi97 oshi97 merged commit d97720a into feature/test-infra-fixes May 13, 2021
@oshi97 oshi97 deleted the dev/workflow-rename-nits branch May 13, 2021 14:29
@oshi97 oshi97 mentioned this pull request May 13, 2021
Merged
oshi97 added a commit that referenced this pull request May 13, 2021
@oshi97
Merge pull request #777 from yext/feature/test-infra-fixes
ffdb70b 
use express instead of serve-handler for snapshots (#766)
remove + signs from percy snapshot query params (#765)
fix test-site's mock date (#764)
fix acceptance tests false positives (#778)
update acceptance test workflow name (#779)

J=SLAP-1313
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tmeyer2115 tmeyer2115 tmeyer2115 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@oshi97 @coveralls @tmeyer2115