Thanks to visit codestin.com
Credit goes to github.com

Skip to content

"-type_mapping" not recognized in 3.1.6 #143

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.

Sign up for GitHub

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

Jump to bottom
Closed
felipebz opened this issue Apr 5, 2019 · 2 comments · Fixed by #154
Closed

"-type_mapping" not recognized in 3.1.6 #143

felipebz opened this issue Apr 5, 2019 · 2 comments · Fixed by #154

Comments

@felipebz
Copy link

felipebz 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
Copy link
Member

pesse commented Apr 5, 2019

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

@pesse
Copy link
Member

pesse 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/

pesse added a commit that referenced this issue Jun 11, 2019
@pesse
Failing test to catch and reproduce #143
6717057
@pesse pesse mentioned this issue Jun 13, 2019
Merged
@felipebz felipebz mentioned this issue Oct 16, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature/new cli library utPLSQL/utPLSQL-cli
2 participants
@felipebz @pesse