clj-commons / pomegranate

A sane Clojure API for Maven Artifact Resolver + dynamic runtime modification of the classpath
507 stars 61 forks source link

add some classpath tests, fix a minor bug #161

Closed lread closed 1 year ago

lread commented 1 year ago

Reviewed/updated pomegranate ns tests.

Added some sanity testing for adding to the classpath. Since these modify classpaths they are run separately under a new :isolated test suite.

Added a warning to get-classpath docstring about JDK9+ behaviour.

Minor test fix: help for the cognitect runner is now invoked with old runner so it does not barf on old versions of Clojure.

Test reporter now also includes JDK version in addition to Clojure version. This might seem excessive, but because these are important for Pomegranate, I like a super-in-my-face reminder of what environment I am testing against.

Minor fix to Pomegranate exception messages now show simple classloader class name, for example:

Closes #153, closes #154