-
Notifications
You must be signed in to change notification settings - Fork 155
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: Add integ tests for single page funnels #1671
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1671 +/- ##
=======================================
Coverage 94.20% 94.20%
=======================================
Files 650 650
Lines 17539 17539
Branches 5776 5776
=======================================
Hits 16522 16522
Misses 948 948
Partials 69 69 ☔ View full report in Codecov by Sentry. |
dd58ea2
to
20b0765
Compare
20b0765
to
d893b11
Compare
d893b11
to
1f3428a
Compare
1f3428a
to
e43516b
Compare
e43516b
to
14f941f
Compare
This adds tests for the existing behaviour. It includes some fix mes that were found while writing and is intended as a start to testing.
14f941f
to
9236679
Compare
}, | ||
|
||
funnelSubStepError(props): void { | ||
const stepName = document.querySelector(props.stepNameSelector)!.innerHTML; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In some places you've used ?
and in some places you've used !
. I'd recommend to use ?
everywhere, so that we still have the event even when some data is missing
|
||
describe('Embedded Form', () => { | ||
test.skip( | ||
'Forms embedded in Modal unrelataed to the main Form', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo in unrelataed
Description
This adds tests for the existing behaviour. It includes some fix mes that were found while writing and is intended as a start to testing.
Related links, issue #, if available: n/a
How has this been tested?
Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness
CONTRIBUTING.md
.CONTRIBUTING.md
.Security
checkSafeUrl
function.Testing
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.