[MSHARED-1125] Require Maven args to be provided one by one

[slawekjaranowski]

No description provided.

[gnodet]

Looks good to me.
Fwiw, I have found 32 references to such pattern in maven-integration-testing by looking for \.addCliOption\( "[\S]+ .*". A PR should be raised asap to align those, even before the switch to verifier 2.0.
Maybe we could add a helper method to add several arguments:

    public void addCliOptions( String... options )
    {
        ...
    }

[slawekjaranowski]

I will look at integration tests.

[michael-o]

What about invoker.properties?

[michael-o]

Looks good to me. Fwiw, I have found 32 references to such pattern in maven-integration-testing by looking for \.addCliOption\( "[\S]+ .*". A PR should be raised asap to align those, even before the switch to verifier 2.0. Maybe we could add a helper method to add several arguments:

    public void addCliOptions( String... options )
    {
        ...
    }

This is a good idea with varargs...

[slawekjaranowski]

What about invoker.properties?

it is not used here ... it is for maven-invoker-plugin - I think.

[michael-o]

What about invoker.properties?

it is not used here ... it is for maven-invoker-plugin - I think.

Right, this is used by Invoker.

[michael-o]

@slawekjaranowski FYI: https://issues.apache.org/jira/browse/MSHARED-1129