Re: Test failure in PropertyTest.testEmptyAttribute:

2016-09-11 Thread Gary Gregory
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 wrote: > This test is for LOG4J2-1313 >

Re: Test failure in PropertyTest.testEmptyAttribute:

2016-09-11 Thread Remko Popma
This test is for LOG4J2-1313 . I've @ignored it for now. On Mon, Sep 12, 2016 at 1:41 AM, Gary Gregory wrote: > With git master, I get: > > Failed tests: > PropertyTest.testEmptyAttribute:60 expected:<1=elementValue,2=[, > 3=attributeValue,4=a

Test failure in PropertyTest.testEmptyAttribute:

2016-09-11 Thread Gary Gregory
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=attributeValue,4=attributeValue2,5=element]Value3,m=msg> Tests run: 1334, Failur