X-Git-Url: https://jxself.org/git/?a=blobdiff_plain;f=tests%2Fcoverage_dungeon.py;h=1f27838fbca6fcdd4c1cf10b25fcb4a6333d1f2b;hb=ca5b6975dcb7cc88e74ed85faecb4d792a2a4faf;hp=288b1c01a707d4f323ac35aa5647de71a3bad519;hpb=a01c08385c0ce32976b94a10019576c348aa53bc;p=open-adventure.git

diff --git a/tests/coverage_dungeon.py b/tests/coverage_dungeon.py
index 288b1c0..1f27838 100755
--- a/tests/coverage_dungeon.py
+++ b/tests/coverage_dungeon.py
@@ -7,7 +7,7 @@ various strings contained are present within the test check files.
 The default HTML output is appropriate for use with Gitlab CI.
 You can override it with a command-line argument.
 
-The DANGLING lists are for actions and messages that should be 
+The DANGLING lists are for actions and messages that should be
 considered always found even if the checkfile search doesn't find them.
 Typically this will because an action emit a templated message that
 can't be regression-tested by equality.
@@ -25,7 +25,7 @@ YAML_PATH = "../adventure.yaml"
 HTML_TEMPLATE_PATH = "../templates/coverage_dungeon.html.tpl"
 DEFAULT_HTML_OUTPUT_PATH = "../coverage/adventure.yaml.html"
 DANGLING_ACTIONS = ["ACT_VERSION"]
-DANGLING_MESSAGES = ["SAVERESUME_DISABLED"]
+DANGLING_MESSAGES = ["SAVERESUME_DISABLED", "SAVE_TAMPERING"]
 
 STDOUT_REPORT_CATEGORY = "  {name:.<19}: {percent:5.1f}% covered ({covered} of {total})\n"
 
@@ -147,7 +147,7 @@ def threshold_coverage(classes, text, report):
     # property
     for name, item in enumerate(classes):
         if name not in report["messages"]:
-            report["messages"][name] = {"covered" : "False"}
+            report["messages"][name] = {"covered" : False}
             report["total"] += 1
         if not report["messages"][name]["covered"] and search(item["message"], text):
             report["messages"][name]["covered"] = True
@@ -158,7 +158,7 @@ def arb_coverage(arb_msgs, text, report):
         if name not in report["messages"]:
             report["messages"][name] = {"covered" : False}
             report["total"] += 1
-        if not report["messages"][name]["covered"] and search(message, text) or name in DANGLING_MESSAGES:
+        if not report["messages"][name]["covered"] and (search(message, text) or name in DANGLING_MESSAGES):
             report["messages"][name]["covered"] = True
             report["covered"] += 1