diff options
| author | loic@dachary.org <loic@dachary.org> | 2016-12-16 13:29:18 +0100 |
|---|---|---|
| committer | loic@dachary.org <loic@dachary.org> | 2016-12-16 13:29:18 +0100 |
| commit | fe5739b703c4f4171e998bee9ce2ad3dfd7959a2 (patch) | |
| tree | c3196d831452a96c23c59b4914831c7631e29a54 /coverage/config.py | |
| parent | 878363970556406d1eed322f6bd8747af1145462 (diff) | |
| download | python-coveragepy-git-fe5739b703c4f4171e998bee9ce2ad3dfd7959a2.tar.gz | |
--source and --include are mutually exclusive #265
Add a sanity_check function to CoverageConfig to perform sanity checks
after configuration values have been collected from files, arguments
etc.
Raise an error if --source and --include are both set.
The config tests are amended because some of them have both source and
include set.
Based on the original patch from Nathan Land
close #265
--HG--
branch : issue-265
extra : amend_source : 2c0e1173285d07174bc632fb3a8df437201607a6
Diffstat (limited to 'coverage/config.py')
| -rw-r--r-- | coverage/config.py | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/coverage/config.py b/coverage/config.py index ad3efa91..f3e296c3 100644 --- a/coverage/config.py +++ b/coverage/config.py @@ -383,6 +383,12 @@ class CoverageConfig(object): # If we get here, we didn't find the option. raise CoverageException("No such option: %r" % option_name) + def sanity_check(self): + if ((self.source is not None) and + (self.include is not None)): + raise CoverageException( + "--include and --source are mutually exclusive") + def read_coverage_config(config_file, **kwargs): """Read the coverage.py configuration. @@ -439,4 +445,6 @@ def read_coverage_config(config_file, **kwargs): # 4) from constructor arguments: config.from_args(**kwargs) + config.sanity_check() + return config_file, config |