Closed mjac0bs closed 1 week ago
Coverage Report: โ
Base Coverage: 87.33%
Current Coverage: 87.47%
:x: Failing | :white_check_mark: Passing | :arrow_right_hook: Skipped | :clock1: Duration |
0 Failing | 445 Passing | 2 Skipped | 86m 12s |
Passed #6790
•
4ec07b0eb4: fix, test: [M3-8830] - Use unit tested function for Pendo url transformation (#1...
Project |
Cloud Manager E2E
|
Branch Review |
develop
|
Run status |
Passed #6790
|
Run duration | 26m 49s |
Commit |
4ec07b0eb4: fix, test: [M3-8830] - Use unit tested function for Pendo url transformation (#1...
|
Committer | Mariah Jacobs |
View all properties for this run โ๏ธ |
Test results | |
---|---|
Failures |
0
|
Flaky |
3
|
Pending |
2
|
Skipped |
0
|
Passing |
445
|
View all changes introduced in this branch โ๏ธ |
Description ๐
This ticket provides test coverage and corrects two discovered issues with the original URL transforms for Pendo page views:
id
s were replaced once, not globally, which would not account for urls with multiple ids (this is rare, but does exist in CM)Changes ๐
transformUrl
function and better documented the purpose of transformsusePendo.test.tsx
file to testtransformUrl()
Target release date ๐๏ธ
11/12/24
How to test ๐งช
Verification steps
(How to verify changes) Run
and confirm test cases provide full coverage of different possible urls and that the tests pass reliably.
As an Author I have considered ๐ค
Check all that apply