commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thomas Neidhart <thomas.neidh...@gmail.com>
Subject Re: [email] Java 8 test errors
Date Mon, 30 Jun 2014 19:44:16 GMT
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.

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


Mime
View raw message