Joe Littlejohn

Sorry @unkish I misunderstood. I didn't realise there were unit tests. I'm happy to use surefire and failsafe, each for the appropriate kind of test.

Ah, the code changes here were made by someone who wont be around to sign the contributor agreement for a few weeks. I'll update you when it's done. Thanks!

Hopefully a trivial one to merge. Cheers!

I think we can use an existing pull request for this. What annotation does JaCoCo expect?

The answer here probably depends on how you are referencing those other types. If you want to refer to something on the classpath, you can use `"existingJavaType"` to refer to...

Are you able to edit the schema like this: ```json { "existingJavaType" : "com.example.external_module.Person" } ``` ?

Closing this as it's a duplicate of #1402. I had a fix for this but I think I want to revisit and solve it more definitively. Just clearing the decks...

@UnasZole This change has never been a priority for me since (as I've mentioned on many issues in the past), we can't support every possible style of Java people prefer....

@Elewyth Yes, agreed, that makes a lot of sense.

Closing this as it is out of date, and we'll address these things individually. Thanks for raising though and showing the way forward in these areas.