The -o option can be used multiple times No.2

[milo]

Alternative approach to #310.

[dg]

stdout -> stdoutFormat 👍

options['-o'] -> $this->outputs hmmm, perhaps unnecessary internal dependency…

I compared it on GitHub in phone and it is quite possible that I missed the other differences.

[dg]

What about -o <console|tap|junit|log|none> (e.g. -o junit:output.xml) to save one row?

[milo]

I'll try that.

[milo]

IMHO, it looks strange when not aligned.

[milo]

options['-o'] -> $this->outputs hmmm, perhaps unnecessary internal dependency…

Yes. That's part I don't like too but I cannot figure how to solve it in other way. Imagine I will call -o console -o tap:output.tap -o junit:output.xml, I would like to:

• keep ENUM check in CommandLine. So value tap:file.name has to be modified before this check (NORMALIZER do it) and file.name has to be kept somewhere for OutputHandler's constructor. The $this->outputs is used for this. I cannot figure how to keep file.name as a part of $options['-o'] without hidden functionality.
• prevent more outputs into one file

[milo]

@dg Updated

[TomasVotruba]

What needs to be done here?

[milo]

Not sure. But I'm not satisfied with an implementation.

[TomasVotruba]

What could be changed to make you more satisfied?

[milo]

That's what I'm nor sure about. The part with NORMALIZER probably. The main purpose for this PR is to allow pass multiple output handlers by CLI arguments. Propose is -o format:file.ext. Maybe another syntax would be better and more straightfoward for implementation.