search

SimoneCallegarin / ingsw2022-AM07

Progetto di Ingegneria del Software 2021/2022
2 stars 0 forks source link

Low testing coverage on Model #14

Closed PiroX4256 closed 2 years ago

PiroX4256 commented 2 years ago

Should be at least 80%.

SimoneCallegarin commented 2 years ago

Sorry, se checked before pushing and it results that the coverage was around about 97%. Is there something wrong maybe?

cerutigiacomo commented 2 years ago

mvn test see only 48 tests instead of 101 am07

SimoneCallegarin commented 2 years ago

How can we fix the fact that maven sees only 48 tests instead of 101?

cerutigiacomo commented 2 years ago

By default Maven uses the following naming conventions when looking for tests to run:

Test Test Tests (has been added in Maven Surefire Plugin 2.20) TestCase

If your test class doesn't follow these conventions you should rename it or configure Maven Surefire Plugin to use another pattern for test classes.

SimoneCallegarin commented 2 years ago

Ok, thanks for the advice, we will, as soon as possibile, solve the problem