Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: The test coverage does NOT show the correct statistics #413

Closed
sanG-github opened this issue Jun 16, 2023 · 4 comments · Fixed by #452
Closed

Fix: The test coverage does NOT show the correct statistics #413

sanG-github opened this issue Jun 16, 2023 · 4 comments · Fixed by #452
Assignees

Comments

@sanG-github
Copy link
Contributor

sanG-github commented Jun 16, 2023

Issue

The code coverage always shows 0.00%, which means it does not reflect the real code coverage of the project. It was caused by the SimpleCov not starting before the Rails was loaded.

image

Expected

Show accurate statistics.
image

Steps to reproduce

  1. Generate the project from the template
  2. Push the code to the remote repository
  3. Add codes, and create a PR
  4. 💥
@sanG-github sanG-github self-assigned this Jun 16, 2023
@andyduong1920
Copy link
Member

@htoo-eain-lwin as you had some work for this topic on your IC, you could colobrate with @sanG-github on this topic 👌

@sanG-github
Copy link
Contributor Author

@htoo-eain-lwin
If you're interested in this matter, I can re-assign it to you. 😉
By the way, I've attached a PR in the description that outlines two ways to address this issue. Feel free to take a look and refer to it.

@htoo-eain-lwin
Copy link
Contributor

Hello @sanG-github ,
Thanks for the consideration. Although, I am interested, I am currently busy with CX. 😅
And I don't have access on the EwaPayroll.

@sanG-github
Copy link
Contributor Author

Thank @htoo-eain-lwin. 😜

I've updated the sample fix, please take a look at it again. Also, thanks for noticing, I will take care of resolving the issue myself. 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants