This PR brings in messy stuff that would be a pain to remove because doing so would cause lots of merge conflicts with future work (e.g. the entity-creation-this branch) even though the latter mostly doesn't depend on it:
Also add most of our own packages to ignorePackages to help against this class of problems.
Renaming the package nz.ac.wgtn.veracity.provenance.injector.instrumentation to nz.ac.wgtn.veracity.provenance.injector.testinstrumentation (which isn't in ignorePackages) to enable unit tests to still work despite the above. This also forced many things to become public, since the tests are now in a different package and thus lose their special access.
Lots of debug logging.
Some of these changes may be removed in a separate PR later.
This PR brings in messy stuff that would be a pain to remove because doing so would cause lots of merge conflicts with future work (e.g. the
entity-creation-this
branch) even though the latter mostly doesn't depend on it:java.net.URLClassLoader
specifically to get around theNoClassDefFoundError
problem of https://github.com/veracitylab/provenance-injector/issues/25#issue-2318159921.ignorePackages
to help against this class of problems.nz.ac.wgtn.veracity.provenance.injector.instrumentation
tonz.ac.wgtn.veracity.provenance.injector.testinstrumentation
(which isn't inignorePackages
) to enable unit tests to still work despite the above. This also forced many things to becomepublic
, since the tests are now in a different package and thus lose their special access.Some of these changes may be removed in a separate PR later.