Skip to content

Add --overwrite-ini ININAME=INIVALUE cli option#1667

Merged
hpk42 merged 1 commit intopytest-dev:featuresfrom
fengxx:feature/override_ini_option
Jun 25, 2016
Merged

Add --overwrite-ini ININAME=INIVALUE cli option#1667
hpk42 merged 1 commit intopytest-dev:featuresfrom
fengxx:feature/override_ini_option

Conversation

@fengxx
Copy link
Contributor

@fengxx fengxx commented Jun 25, 2016

support for "--overwrite-ini ININAME=INIVALUE" to overrides ini values from the command line
so that one can do e.g. "-o xfail_strict=True".

@fengxx
Copy link
Contributor Author

fengxx commented Jun 25, 2016

@blueyed please help review

@The-Compiler
Copy link
Member

The-Compiler commented Jun 25, 2016

Did you check if there are existing commandline options which we now could maybe deprecate?

hpk42
hpk42 reviewed Jun 25, 2016
View reviewed changes
CHANGELOG.rst Outdated
* New cli flag ``--override-ini`` or ``-o`` that overrides ini values from the
command line so that one can do e.g. "-o xfail_strict=True". This is useful for
specifying options for which there is no separate command-line flag, a complete
ini-options can be showed by py.test --help
Copy link
Contributor

@hpk42 hpk42 Jun 25, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'd make a "." after flag. and then say "The complete list of ini-options can be shown by py.test --help". Mind the escape of the command which is important for rendering the docs.

@coveralls
Copy link

coveralls commented Jun 25, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.009%) to 92.398% when pulling f5a90ab on fengxx:feature/override_ini_option into f2bb3df on pytest-dev:features.

omarkohl
omarkohl reviewed Jun 25, 2016
View reviewed changes
_pytest/config.py Outdated
value = None
if self.getoption("override_ini", None):
for ini_config in self.option.override_ini:
(key, user_ini_value) = ini_config.split("=", 2)
Copy link
Contributor

@omarkohl omarkohl Jun 25, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think maxsplit needs to be 1

Copy link
Contributor Author

@fengxx fengxx Jun 25, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot. Good catch!

@fengxx fengxx force-pushed the feature/override_ini_option branch from f5a90ab to 77104d6 Compare June 25, 2016 14:08
@fengxx
Copy link
Contributor Author

fengxx commented Jun 25, 2016

amended the commit to fix issues

@fengxx fengxx force-pushed the feature/override_ini_option branch from 77104d6 to 18725e6 Compare June 25, 2016 14:13
hpk42
hpk42 reviewed Jun 25, 2016
View reviewed changes
testing/test_config.py
"ini2:url=/tmp/user2?a=b&d=e",
"ini3:True",
"ini4:False"
]) No newline at end of file
Copy link
Contributor

@hpk42 hpk42 Jun 25, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think we also need a test for multiple -o options -- they all should get applied. So e.g.
-o custom_option1=yes -o custom_option2=no.

Copy link
Contributor Author

@fengxx fengxx Jun 25, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rewrited the testcase to use multiple -o

@fengxx fengxx force-pushed the feature/override_ini_option branch 3 times, most recently from 27c77d6 to 9c48bb7 Compare June 25, 2016 14:58
@fengxx
Copy link
Contributor Author

fengxx commented Jun 25, 2016

will rebase on features branch

@fengxx fengxx force-pushed the feature/override_ini_option branch from 9c48bb7 to 3f623e4 Compare June 25, 2016 15:04
@aostr aostr mentioned this pull request Jun 25, 2016
Closed
@fengxx fengxx force-pushed the feature/override_ini_option branch from 3f623e4 to 7bc4cc8 Compare June 25, 2016 15:18
@coveralls
Copy link

coveralls commented Jun 25, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.01%) to 92.429% when pulling 7bc4cc8 on fengxx:feature/override_ini_option into 13a188f on pytest-dev:features.

@coveralls
Copy link

coveralls commented Jun 25, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.01%) to 92.429% when pulling 7bc4cc8 on fengxx:feature/override_ini_option into 13a188f on pytest-dev:features.

@fengxx
add --override-ini option to overrides ini values
856e6ca 
Signed-off-by: Ted Xiao <xiao.xj@gmail.com>
@fengxx fengxx force-pushed the feature/override_ini_option branch from 7bc4cc8 to 856e6ca Compare June 25, 2016 15:45
@coveralls
Copy link

coveralls commented Jun 25, 2016 •
edited
Loading

Coverage Status

Coverage increased (+0.01%) to 92.429% when pulling 856e6ca on fengxx:feature/override_ini_option into 13a188f on pytest-dev:features.

@hpk42 hpk42 merged commit 68bed00 into pytest-dev:features Jun 25, 2016
@hpk42
Copy link
Contributor

hpk42 commented Jun 25, 2016

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@fengxx @The-Compiler @coveralls @hpk42 @omarkohl