search

exercism / java-analyzer

GNU Affero General Public License v3.0
8 stars 14 forks source link

Rework unit tests #124

Closed sanderploegsma closed 7 months ago

sanderploegsma commented 8 months ago

Reworked the unit tests to use the Approvals library for some additional coverage in the JUnit tests. Now each test scenario acts more or less like an integration test for each exercise instead of unit test for a specific analyzer. There is some overlap with the smoke tests this way, but the way I see it we can use these JUnit integration-style tests to cover (most of) all branches and possible approaches, and keep the number of smoke tests a bit smaller.

sanderploegsma commented 8 months ago

This is going to conflict with #122 by the way, I would suggest merging that one first, and I'll fix the damage here afterwards.

sanderploegsma commented 7 months ago

@ErikSchierboom could you review this one please?