commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benedikt Ritter <brit...@apache.org>
Subject Re: [email] Java 8 test errors
Date Tue, 01 Jul 2014 06:30:41 GMT
2014-06-30 21:44 GMT+02:00 Thomas Neidhart <thomas.neidhart@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-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>


-- 
http://people.apache.org/~britter/
http://www.systemoutprintln.de/
http://twitter.com/BenediktRitter
http://github.com/britter

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message