Fix integration test licensing through test rewrites

gselzer commented 2 years ago

This PR rewrites NoSubpackageDependenceRuleIntegrationTest and NoPackageCyclesRuleIntegrationTest to remove the vending of FitNesse and JUnit. Instead, dummy projects were created to test both rules. Specifically, we add:

A control project that breaks neither rule
A project that contains a package cycle
A project that contains a subpackage dependency

Closes #30

Questions:

Should we remove all .java files, just keeping the needed .class files? We might as well keep them in the case of updates.

ctrueden commented 2 years ago

Wow, thanks for working on this, @gselzer! I see you have acclimated to the Pythonic four space indentation... 😉

gselzer commented 2 years ago

Dang, I could have sworn I formatted before each commit. Maybe I didn't have the SciJava profile active in IntelliJ?

scijava / scijava-maven-plugin

Fix integration test licensing through test rewrites #31