ifmeorg / ifme

Free, open source mental health communication web app to share experiences with loved ones
http://www.if-me.org/
GNU Affero General Public License v3.0
1.49k stars 744 forks source link

Fix issue with tests in utils/add additional tests #2293

Closed vivek-p-a closed 5 months ago

vivek-p-a commented 5 months ago

Description

In 'client/app/utils' the tests were not being run by jest due to naming convention. For fixing this i have changed the filename of the test file.

After changing the file name, have fixed the issues with the existing test file and have also added additional tests to cover all the methods defined in 'client/app/utils/index.js'

Reviewing this pull request? Check out our Code Review Practices guide if you haven't already!

welcome[bot] commented 5 months ago

Thank you for opening this pull request with us! Be sure to follow our Pull Request Practices. Let us know if you have any questions on Slack.

julianguyen commented 5 months ago

Thanks for fixing this!! And adding more coverage 🎉

welcome[bot] commented 5 months ago

Thank you for merging this pull request with us! If you haven't already, in another pull request, please add yourself to our About page.