test: reinstate skipped research.routes test

[thisislawatts]

PR Checklist

• - Commit messages are descriptive, it will be used in our Release Notes

PR Type

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Developer experience (improves developer workflows for contributing to the project)

Description

Adds back a test suite which had previously been disabled.

[cypress]

1 flaky tests on run #2914 ↗︎

0

52

0

0

1

Details:

fix: test craco config and usecommonstores import
Project: onearmy-community-platform	Commit: aea15e43a1
Status: Passed	Duration: 02:57 💡
Started: Feb 14, 2023 6:47 PM	Ended: Feb 14, 2023 6:50 PM

  src/integration/common.spec.ts • 1 flaky test • ci-chrome

View Output Video

Test
[Common] > [User Menu] > [By Authenticated]		Screenshot

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[thisislawatts]

@ONEARMY/maintainers can I get some eyes on this? Would be happy to see it merged or instead close it.

[chrismclarke]

Thanks @thisislawatts

For whatever reason the tests weren't happy with files that imported directly from src, I can see you mocked imports from src/index so instead of changing all other files updated the jest config to provide a mapping in aea15e4 which looks happier.

At some point it might be good to tidy things up to make it easier to consistently mock the useCommonStores method that you created (super handy btw), but for now I'd say good to go, thanks again.

[onearmy-bot]

🎉 This PR is included in version 1.38.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀