search

Rebuild-Black-Business / RBB-Website

Website to help connect black-owned businesses with consumers and resources
https://www.rebuildblackbusiness.com/
MIT License
119 stars 72 forks source link

[Feature] Add a test for SetURLUtm utility function #310

Open mbifulco opened 4 years ago

mbifulco commented 4 years ago

User Story

n/a, really - this is more of a tech debt task

This issue is to track the work around having a test written for setUrlUtm in /src/utils/urlUtils.js. We had a production failure today due to some incomplete fallback cases written for that function

Acceptance Criteria

Is your feature request related to a problem? Please describe.

On 7/19/2020, we had reports of the /business page crashing for some users who were trying to load certain data in the Safari browser. This was hotfixed in #309, but no tests were written.

Describe the solution you'd like

A good test case (or cases) for the above function, and any needed code changes to the function itself!

juanri0s commented 4 years ago

Is this task open for me to pickup? I'd like to get some tests going here 😄

magnificode commented 4 years ago

All you @juanri0s!

magnificode commented 3 years ago

@juanri0s - were you able to make any headway here sir?