These tests have been removed.

On Wed, Mar 21, 2018 at 7:39 PM, Remko Popma <remko.po...@gmail.com> wrote:

> We will probably just remove these tests; there is no need to bring the
> picocli tests in to the Log4j2 project.
>
> On Wed, Mar 21, 2018 at 3:20 PM, Atle Tokle <ato...@gmail.com> wrote:
>
>> I made a fresh clone from master today, and looked into it myself.
>> In CommandLineHelpTest.java all the tests that is failing is instanciating
>> a Help class. In all cases it have an alternative constructor where you
>> can
>> specify Help.Ansi.OFF to avoid that the test depends on the console it run
>> from.
>>
>> CommandLineTest did also have 3 faults, and here it is CommandLine.run()
>> that needs Help.Ansi.OFF as it's third argument. I can send you patch if
>> you tell me how you want it.
>>
>> Atle
>>
>>
>> Den søn. 11. mar. 2018 kl. 11:04 skrev Remko Popma <remko.po...@gmail.com
>> >:
>>
>> > Thanks for reporting this. I’ll take a look.
>> >
>> > Remko
>> >
>> > > On Mar 11, 2018, at 17:32, Atle Tokle <ato...@gmail.com> wrote:
>> > >
>> > > I tried to build from GIT bash on windows 10, and first test that
>> failed
>> > > was
>> > >
>> > > testSynopsisOrderCorrectWhenParametersDeclaredOutOfOrder in
>> > >
>> > > org.apache.logging.log4j.core.tools.picocli.
>> > >
>> > > CommandLineHelpTest:2008
>> > >
>> > >
>> > > It seems like the expected result
>> > >
>> > > format("<main class> <param0> <param1>%n") on line 2014 always assumes
>> > > that no ANSI color codes is output, while the actual results have ANSI
>> > > codes.
>> > >
>> > > I looked into the code, and saw that one of the things that was check
>> > > was if TERM environment variable started with xterm, and did a test
>> > > where I first did
>> > >
>> > > export TERM=zzzzz
>> > >
>> > > and did then build it again. Now without faults.
>> > >
>> > > The test should have more control on what environment it is running
>> > > on. It is several other tests in the same testclass that fails because
>> > > of same issue.
>> > >
>> > >
>> > > I guess this is the same fault that Gary Gregory experienced here
>> > > https://www.mail-archive.com/dev@logging.apache.org/msg05223.html
>> > >
>> > > in message from Nov 19, 2017, at 6:32 PM
>> > >
>> > >
>> > > Best regards
>> > >
>> > > Atle
>> >
>>
>
>

Reply via email to