improve ParallelExecutor::generateOptions to manage all types of InputOption

[rvitaliy]

Q	A
Bug fix?	Yes
New feature?	Yes
BC breaks?	No
Deprecations?	No
Fixed tickets	#1721 (comment)

[kick-the-bucket]

Just by looking at the code it seems like it now handles NO_VALUE options wrong by always adding them no matter if they are true or false (set or not).

I'll try to test it more thoroughly later today.

[antonmedv]

@kick-the-bucket Will be cool if you can add test for this case too.

[kick-the-bucket]

The previous fatals would've been caught just by adding something like this to any recipe test:

option('value-none', null, InputOption::VALUE_NONE);
option('value-optional', null, InputOption::VALUE_OPTIONAL, 'blah');
option('value-required', null, InputOption::VALUE_REQUIRED);
option('value-array-required', null, InputOption::VALUE_IS_ARRAY | InputOption::VALUE_REQUIRED);
option('value-array-optional', null, InputOption::VALUE_IS_ARRAY | InputOption::VALUE_OPTIONAL, ['a' => 'b']);

But since the code has now moved to separate classes it will be way easier write some unit tests.