changeset 8816:c76638100ca0

deps: also report unseen known violations
author Mads Kiilerich <mads@kiilerich.com>
date Tue, 29 Dec 2020 22:30:48 +0100
parents 1b683a4eb9fc
children 233ba8198781
files scripts/deps.py
diffstat 1 files changed, 4 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/scripts/deps.py	Mon Dec 28 00:43:18 2020 +0100
+++ b/scripts/deps.py	Tue Dec 29 22:30:48 2020 +0100
@@ -153,7 +153,7 @@
 shown_modules = normal_modules | top_modules
 
 # break the chains somehow - this is a cleanup TODO list
-known_violations = [
+known_violations = set([
 ('kallithea.lib.auth_modules', 'kallithea.lib.auth'),  # needs base&facade
 ('kallithea.lib.utils', 'kallithea.model'),  # clean up utils
 ('kallithea.lib.utils', 'kallithea.model.db'),
@@ -166,7 +166,7 @@
 ('kallithea.model', 'kallithea.lib.hooks'),  # clean up hooks
 ('kallithea.model', 'kallithea.model.scm'),
 ('kallithea.model.scm', 'kallithea.lib.hooks'),
-]
+])
 
 extra_edges = [
 ('kallithea.config', 'kallithea.controllers'),  # through TG
@@ -271,6 +271,8 @@
 
     # verify dependencies by untangling dependency chain bottom-up:
     todo = set(normalized_dep_edges)
+    unseen_violations = known_violations.difference(todo)
+    assert not unseen_violations, unseen_violations
     for x in known_violations:
         todo.remove(x)