I think this is better:

@Ignore("TODO fix LOG4J2-1313")

Than:

@Ignore() // TODO fix LOG4J2-1313

So I've change it.

Presumably some tooling could pick this up.

Gary

On Sun, Sep 11, 2016 at 10:06 AM, Remko Popma <remko.po...@gmail.com> wrote:

> This test is for LOG4J2-1313
> <https://issues.apache.org/jira/browse/LOG4J2-1313>.
> I've @ignored it for now.
>
> On Mon, Sep 12, 2016 at 1:41 AM, Gary Gregory <garydgreg...@gmail.com>
> wrote:
>
>> With git master, I get:
>>
>> Failed tests:
>>   PropertyTest.testEmptyAttribute:60 expected:<1=elementValue,2=[,3
>> =attributeValue,4=attributeValue2,5=attribute]Value3,m=msg> but
>> was:<1=elementValue,2=[${sys:emptyElementKey},3=attributeVal
>> ue,4=attributeValue2,5=element]Value3,m=msg>
>>
>> Tests run: 1334, Failures: 1, Errors: 0, Skipped: 32
>>
>> [INFO] ------------------------------------------------------------
>> ------------
>> [INFO] Reactor Summary:
>> [INFO]
>> [INFO] Apache Log4j 2 ..................................... SUCCESS [
>>  6.073 s]
>> [INFO] Apache Log4j API ................................... SUCCESS [
>> 17.259 s]
>> [INFO] Apache Log4j Core .................................. FAILURE
>> [10:29 min]
>> [INFO] Apache Log4j 1.x Compatibility API ................. SKIPPED
>> [INFO] Apache Log4j SLF4J Binding ......................... SKIPPED
>> [INFO] Apache Log4j to SLF4J Adapter ...................... SKIPPED
>> [INFO] Apache Log4j Commons Logging Bridge ................ SKIPPED
>> [INFO] Apache Log4j Flume Bridge .......................... SKIPPED
>> [INFO] Apache Log4j Web ................................... SKIPPED
>> [INFO] Apache Log4j Tag Library ........................... SKIPPED
>> [INFO] Apache Log4j JMX GUI ............................... SKIPPED
>> [INFO] Apache Log4j Samples ............................... SKIPPED
>> [INFO] Apache Log4j Samples: Flume - Common ............... SKIPPED
>> [INFO] Apache Log4j Samples: Flume - Remote ............... SKIPPED
>> [INFO] Apache Log4j Samples: Flume - Embedded ............. SKIPPED
>> [INFO] Apache Log4j Samples: Configuration ................ SKIPPED
>> [INFO] Apache Log4j Samples: LoggerProperties ............. SKIPPED
>> [INFO] Apache Log4j Scala 2.11 wrapper for Log4j API ...... SKIPPED
>> [INFO] Apache Log4j Samples: Scala API .................... SKIPPED
>> [INFO] Apache Log4j BOM ................................... SKIPPED
>> [INFO] Apache Log4j NoSQL ................................. SKIPPED
>> [INFO] Apache Log4J Performance Tests ..................... SKIPPED
>> [INFO] Apache Log4j Streaming Interface ................... SKIPPED
>> [INFO] Apache Log4j JUL Adapter ........................... SKIPPED
>> [INFO] Apache Log4j Liquibase Binding ..................... SKIPPED
>> [INFO] Apache Log4j Scala 2.10 wrapper for Log4j API ...... SKIPPED
>> [INFO] ------------------------------------------------------------
>> ------------
>> [INFO] BUILD FAILURE
>> [INFO] ------------------------------------------------------------
>> ------------
>> [INFO] Total time: 10:55 min
>> [INFO] Finished at: 2016-09-11T09:36:23-07:00
>> [INFO] Final Memory: 33M/308M
>> [INFO] ------------------------------------------------------------
>> ------------
>>
>> --
>> E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
>> Java Persistence with Hibernate, Second Edition
>> <http://www.manning.com/bauer3/>
>> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
>> Spring Batch in Action <http://www.manning.com/templier/>
>> Blog: http://garygregory.wordpress.com
>> Home: http://garygregory.com/
>> Tweet! http://twitter.com/GaryGregory
>>
>
>


-- 
E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
Java Persistence with Hibernate, Second Edition
<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Reply via email to