vocabulary-legacy
vocabulary-legacy copied to clipboard
Fix broken tests and re-enable them
Description
Presently the card and header tests have been de-activated by changing the name from .test.js
to .test.js.x
. This is because the card tests are flaky (#327) and because there is an inexplicable error in the header test. The goal is to fix both test suites, update code, styling and snapshots as required and make them pass.
Reproduction Re-enable the tests and run on GitHub Actions.
Expected behavior Tests should pass.
Can i take this?
How to run on actions? how do i test them?
@av-00 GitHub Actions will automatically run on your PR. You can however run the tests by following the contribution instructions in the Storybook.
Can you please explain the difference between tests passed and snapshots passed as mentioned?
Each test suite contains multiple tests. Each of these tests is associated with a single snapshot. So the number of passing tests will equal the number of passing snapshots.
@dhruvkb Shall I implement this one?
@Cronus1007 we are currently not enabling visual regression testing on our apps because of large dev overhead.
@dhruvkb So shall I close this PR #847
Stale issue message
I would love this to be assigned to me please
Thanks, @Lovelynchristian! I've assigned you to the issue :smile:
Great! Thanks
Good evening @brylie , I'm having issues installing Docker on my mac and windows. Please i need your assistance. Thanks.
@Lovelynchristian, Docker isn't required for Vocabulary development. All that is needed are Node and npm. Please follow the Vocabulary installation instructions to get started with development.
@brylie Thank you.
Hi @brylie , trust you're doing great. In trying to replicate this issue on my local machine, I've been able to install all necessary dependencies and run the unit test within the unit directory. It took me sometime to locate the header.test.js file eventually i found out that the current version of the main branch does not have this file. How do i get a version of a branch that has all the necessary files for me to fix this issue?
@dhruvkb, are you able to assist @Lovelynchristian with their question?
@Lovelynchristian this file was likely removed during the restructuring. You can get that file from an older commit. Here it is.
@dhruvkb Thank you.
@dhruvkb, Thanks once again.
However, I still am not able to replicate the broken issue, on the latest version of the vocabulary repo. All test seems to pass with no failures. I was also able to get the test.js.x file as mentioned. It does not seem to impact the current version from my findings.
Kindly assist. I will like to first replicate the issue.
@brylie Please i would appreciate your assistance .I would need response to my question. Thankyou.
@Lovelynchristian it would help if I could try out the changes you have made. Would you open a pull request with the changes you have tried so I can review them?