search

gianvitocaleca / ingsw2022-AM44

1 stars 0 forks source link

Low testing coverage on Model. #20

Closed cerutigiacomo closed 2 years ago

cerutigiacomo commented 2 years ago

Should be at least to 80%.

SabrinAzzi commented 2 years ago

Improved coverage to 80%.

SabrinAzzi commented 2 years ago

Updated coverage to 87%.

cerutigiacomo commented 2 years ago

Some tests class are not even compiling. am44

gianvitocaleca commented 2 years ago

Cattura We run maven tests several times and everything is working, the code is the same in the master repository

cerutigiacomo commented 2 years ago

My bad, but exclude some package from the coverage report. devvv

gianvitocaleca commented 2 years ago

We should exclude everything that isn't model or controller, is it right?

cerutigiacomo commented 2 years ago

Correct

gianvitocaleca commented 2 years ago

Pom updated

gianvitocaleca commented 2 years ago

Do we have to do something else to solve this issue?