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.

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

chore(surveys): add basic survey e2e tests #948

Merged
merged 5 commits into from
Jan 8, 2024
Merged

Conversation

liyiy
Copy link
Contributor

@liyiy liyiy commented Jan 4, 2024

Changes

Adds some basic survey e2e tests for some form of visual testing

To test:

run cypress tests

Checklist

@liyiy liyiy changed the title Add basic survey e2e tests chore(surveys): add basic survey e2e tests Jan 4, 2024
Copy link

github-actions bot commented Jan 4, 2024

Size Change: 0 B

Total Size: 752 kB

ℹ️ View Unchanged
Filename Size
dist/array.full.js 177 kB
dist/array.js 118 kB
dist/es.js 118 kB
dist/exception-autocapture.js 12 kB
dist/module.js 118 kB
dist/recorder-v2.js 104 kB
dist/recorder.js 58.4 kB
dist/surveys.js 46.8 kB

compressed-size-action

@liyiy liyiy requested a review from neilkakkar January 4, 2024 22:49
Copy link
Contributor

@neilkakkar neilkakkar left a comment

Choose a reason for hiding this comment

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

some event assertions failing but otherwise LGTM!

@liyiy liyiy merged commit ea7b84a into master Jan 8, 2024
13 checks passed
@liyiy liyiy deleted the surveys-e2e-tests branch January 8, 2024 20:22
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.

2 participants