Think it'd even be fine to simplify to modules- and not worry about the environment variable - the main concern was keeping IntelliJ injected classes off the classpath.
Coverage decreased (-0.4%) to 94.36% when pulling df750312c1679e43b73c01209a7d3097ea94d012 on fix-integrationTest-classpath-for-shared-cache into f1f0a025d12596a35d96dd50ef5f7e258c9d4e8a on master.
Think it'd even be fine to simplify to
modules-
and not worry about the environment variable - the main concern was keeping IntelliJ injected classes off the classpath.