Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"-type_mapping" not recognized in 3.1.6 #143

Closed
felipebz opened this issue Apr 5, 2019 · 2 comments

Comments

@felipebz
Copy link

commented Apr 5, 2019

Hi everyone.

I updated the utPLSQL-cli from 3.1.2 to 3.1.6 and the option "-type_mapping" isn't recognized anymore, for example:

utplsql-cli run (database) -p=(schema) -f=ut_sonar_test_reporter -o=tests.xml -source_path=src -test_path=test -type_mapping="sql=PACKAGE BODY"

If I use utplsql-cli 3.1.2 then the file "tests.xml" is generated correctly, but if I try to use the version 3.1.6 I get this error:

Unknown option: -type_mapping
run tests
Usage: (...)

Is this expected?

@pesse

This comment has been minimized.

Copy link
Member

commented Apr 5, 2019

No, thanks for finding!
Time to create an integration test for this behaviour, too.

@pesse

This comment has been minimized.

Copy link
Member

commented Jun 4, 2019

The issue is actually caused by a bug in JCommander, which uses startsWith at several places to match VariableArity parameters. This is a problem because our new -t, --timeout matches anything which starts with -t...

After investigation, I fear fixing the bug in JCommander might have unwanted side-effects, therefore I think about switching to a different CLI library like https://github.com/remkop/picocli/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.