Hi Benedikt, thanks for the hint, using a profile to detect Java 8 and use a different easymock/powermock version worked.
Thomas On Tue, Jul 1, 2014 at 8:30 AM, Benedikt Ritter <brit...@apache.org> wrote: > 2014-06-30 21:44 GMT+02:00 Thomas Neidhart <thomas.neidh...@gmail.com>: > > > On 06/30/2014 04:27 PM, Thomas Neidhart wrote: > > > On 06/30/2014 02:47 PM, Gary Gregory wrote: > > >> If I build trunk with Java 8, I get: > > >> > > >> Tests in error: > > >> HtmlEmailTest.initializationError » IllegalState Failed to transform > > >> class wit... > > >> ImageHtmlEmailTest.initializationError » IllegalState Failed to > > transform > > >> clas... > > >> MultiPartEmailTest.initializationError » IllegalState Failed to > > transform > > >> clas... > > >> > > >> Tests run: 125, Failures: 0, Errors: 3, Skipped: 1 > > >> > > >> Apache Maven 3.2.2 (45f7c06d68e745d05611f7fd14efb6594181933e; > > >> 2014-06-17T09:51:42-04:00) > > >> Maven home: C:\Java\apache-maven-3.2.2 > > >> Java version: 1.8.0_05, vendor: Oracle Corporation > > >> Java home: C:\Program Files\Java\jdk1.8.0_05\jre > > >> Default locale: en_US, platform encoding: Cp1252 > > >> OS name: "windows 7", version: "6.1", arch: "amd64", family: "dos" > > >> > > >> With Java 7, all is well: > > >> > > >> Apache Maven 3.2.2 (45f7c06d68e745d05611f7fd14efb6594181933e; > > >> 2014-06-17T09:51:42-04:00) > > >> Maven home: C:\Java\apache-maven-3.2.2 > > >> Java version: 1.7.0_60, vendor: Oracle Corporation > > >> Java home: C:\Program Files\Java\jdk1.7.0_60\jre > > >> Default locale: en_US, platform encoding: Cp1252 > > >> OS name: "windows 7", version: "6.1", arch: "amd64", family: "windows" > > >> Mon 06/30/2014- 8:46:17.47 > > C:\vcs\svn\apache\commons\trunks-proper\email> > > >> > > >> Thoughts? > > > > > > has to do with powermock. Need to check if a more recent version solves > > > the issue. > > > > it looks like there is no way to support both, Java 5 and Java 8 with > > the same set of jars. > > > > How about introducing profiles that pull in different versions of > powermock, based on the jdk version we're building with? > > > > > > The reason powermock was added was to avoid a potential test failure in > > some environments when an invalid url was resolved to something valid > > (and to prevent -1 at release votes because of this). > > > > The test was adapted to mock an jdk class to always throw an exception > > for these tests, but afaik the providers where this happened have > > changed their setup and it should work fine now. Maybe we should remove > > the dependency again and note this potential problem somewhere. > > > > Thomas > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > -- > http://people.apache.org/~britter/ > http://www.systemoutprintln.de/ > http://twitter.com/BenediktRitter > http://github.com/britter >
