fix(fixers): count fixed violations correctly

guilatrova · Sep 12, 2021 · 825794c · 825794c
1 parent ee708a4
commit 825794c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/tryceratops/interfaces.py b/src/tryceratops/interfaces.py
@@ -54,8 +54,8 @@ def _present_status(self):
         else:
             print("Nothing to check!")
 
-        if self.runner.fixes_made:
-            print(f"Fixed {self.runner.fixes_made} violations")
+        if self.runner.fixed_violations:
+            print(f"Fixed {self.runner.fixed_violations} violations")
 
         if self.discovery.had_issues:
             print(

diff --git a/src/tryceratops/runners.py b/src/tryceratops/runners.py
@@ -26,7 +26,7 @@ def __init__(self):
         self.violations: List[Violation] = []
         self.analyzed_files = 0
         self.excluded_files = 0
-        self.fixes_made = 0
+        self.fixed_violations = 0
 
     def _clear(self):
         self.violations = []
@@ -60,7 +60,7 @@ def _run_fixers(self, fixers: Set[BaseFixer]):
                 logger.exception(f"Exception raised when running Fixer: {type(fixer)}")
                 self.runtime_errors.append(RuntimeError("unknown", type(fixer), ex))
             else:
-                self.fixes_made += 1
+                self.fixed_violations += fixer.fixes_made
 
     def analyze(self, trees: ParsedFilesType, global_filter: GlobalFilter) -> List[Violation]:
         analyzers = get_analyzer_chain(global_filter)