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

Repackage test to match src #710

Open
jbassbusinessolver opened this issue Oct 31, 2024 · 0 comments
Open

Repackage test to match src #710

jbassbusinessolver opened this issue Oct 31, 2024 · 0 comments

Comments

@jbassbusinessolver
Copy link

jbassbusinessolver commented Oct 31, 2024

Brian S pointed out this horrendous flaw:

Main test is under: https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition/blob/uinverse/src/test/java/FizzBuzzTest.java

Main source is under: https://github.com/EnterpriseQualityCoding/FizzBuzzEnterpriseEdition/blob/uinverse/src/main/java/com/seriouscompany/business/java/fizzbuzz/packagenamingpackage/impl/Main.java

This is improper, tests should be under the same directory as source so that testing frameworks can properly generate code coverage. While the tests are at a higher level, it's improper and inefficient as we might not care about every package. I've had to reconfigure my code coverage settings every single time I change a build setting and it's driving me nuts, it's including too much potentially!

I'm going on strike and refusing to do any useful work until this is fixed, I honestly might quit if we don't prioritize this.

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

No branches or pull requests

1 participant