--source and --include are mutually exclusive #265issue-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

1 files changed, 8 insertions, 0 deletions

diff --git a/coverage/config.py b/coverage/config.py
index ad3efa9..f3e296c 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