diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2021-11-13 19:23:24 -0500 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2021-11-14 08:12:49 -0500 |
commit | b82e9fd8766a77c2a275bde7b574f3e8cb529f8f (patch) | |
tree | f310ea38a19f2b4e8c93ee3e3a5b8a99aa43351a /coverage/summary.py | |
parent | 342e7da2941ae5291f1a94b6ad66ce489f6985fe (diff) | |
download | python-coveragepy-git-b82e9fd8766a77c2a275bde7b574f3e8cb529f8f.tar.gz |
refactor: specialize exceptionsnedbat/exceptions
CoverageException is fine as a base class, but not good to use for
raising (and catching sometimes). Introduce specialized exceptions that
allow third-party tools to integrate better.
Diffstat (limited to 'coverage/summary.py')
-rw-r--r-- | coverage/summary.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/coverage/summary.py b/coverage/summary.py index 0b54a05b..467c601d 100644 --- a/coverage/summary.py +++ b/coverage/summary.py @@ -5,7 +5,7 @@ import sys -from coverage.exceptions import CoverageException +from coverage.exceptions import ConfigError, NoDataError from coverage.misc import human_sorted_items from coverage.report import get_analysis_to_report from coverage.results import Numbers @@ -103,7 +103,7 @@ class SummaryReporter: else: position = column_order.get(sort_option) if position is None: - raise CoverageException(f"Invalid sorting option: {self.config.sort!r}") + raise ConfigError(f"Invalid sorting option: {self.config.sort!r}") lines.sort(key=lambda l: (l[1][position], l[0]), reverse=reverse) for line in lines: @@ -122,7 +122,7 @@ class SummaryReporter: # Write other final lines. if not self.total.n_files and not self.skipped_count: - raise CoverageException("No data to report.") + raise NoDataError("No data to report.") if self.config.skip_covered and self.skipped_count: self.writeout( |