summaryrefslogtreecommitdiff
path: root/coverage/summary.py
diff options
context:
space:
mode:
authorNed Batchelder <ned@nedbatchelder.com>2021-11-13 19:23:24 -0500
committerNed Batchelder <ned@nedbatchelder.com>2021-11-14 11:41:23 -0500
commit1c29ef3e4b871508bb2defd6b47b9a745547b626 (patch)
tree931fd2d7175321fd1549fd32e378cddcc99d2537 /coverage/summary.py
parent342e7da2941ae5291f1a94b6ad66ce489f6985fe (diff)
downloadpython-coveragepy-git-1c29ef3e4b871508bb2defd6b47b9a745547b626.tar.gz
refactor: specialize 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.py6
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(