Add type signatures required by mypy --strict

One more `# type: ignore` was added while awaiting rdflib PR #1407.

These were all identified by adding the `--strict` flag to mypy for a
run.  I will leave it up for future discussion whether to use that flag,
especially to wait for PR 1407 and to see if too much work would be
induced versus runtime safety improvements.

References:
* [AC-211] Add static type checking to CASE-Utilities-Python
* RDFLib/rdflib#1407

Signed-off-by: Alex Nelson <[email protected]>

Author: ajnelson-nist

case_utils/__init__.py

@@ -26,4 +26,4 @@ def guess_format(
 ) -> typing.Optional[str]:
     warnings.warn("The functionality in case_utils.guess_format is now upstream.  Please revise your code to use rdflib.util.guess_format.  The function arguments remain the same.  case_utils.guess_format will be removed in case_utils 0.4.0.", DeprecationWarning)
 
-    return rdflib.util.guess_format(fpath, fmap)
+    return rdflib.util.guess_format(fpath, fmap)  # type: ignore

tests/case_file/test_case_file.py

@@ -38,7 +38,7 @@
 SRCDIR = os.path.dirname(__file__)
 
 def load_graph(
-  filename
+  filename : str
 ) -> rdflib.Graph:
     in_graph = rdflib.Graph()
     in_graph.parse(filename)
@@ -52,7 +52,9 @@ def graph_case_file() -> rdflib.Graph:
 def graph_case_file_disable_hashes() -> rdflib.Graph:
     return load_graph(os.path.join(SRCDIR, "sample.txt-disable_hashes.ttl"))
 
-def test_confirm_hashes(graph_case_file) -> None:
+def test_confirm_hashes(
+  graph_case_file : rdflib.Graph
+) -> None:
     expected = {
       "MD5": "098F6BCD4621D373CADE4E832627B4F6",
       "SHA1": "A94A8FE5CCB19BA61C4C0873D391E987982FBBD3",
@@ -93,7 +95,10 @@ def test_confirm_hashes(graph_case_file) -> None:
 
     assert expected == computed
 
-def test_confirm_mtime(graph_case_file, graph_case_file_disable_hashes) -> None:
+def test_confirm_mtime(
+  graph_case_file : rdflib.Graph,
+  graph_case_file_disable_hashes : rdflib.Graph
+) -> None:
     query_confirm_mtime = """
 SELECT ?nFile
 WHERE {

tests/case_sparql_construct/test_case_sparql_construct.py

@@ -17,7 +17,9 @@
 
 import case_utils
 
-def _test_templates_with_blank_nodes_result(filename) -> None:
+def _test_templates_with_blank_nodes_result(
+  filename : str
+) -> None:
     ground_truth_positive = {
       ("Alice", "Hacker"),
       ("Bob", "Hacker")

tests/hexbinary/test_hexbinary.py

@@ -263,7 +263,9 @@ def test_rdflib_literal_topython_hexbinarycanonical() -> None:
 
     assert l_hb_uppercase.toPython() == l_hbc_uppercase.toPython()
 
-def _query_all_value_matches(graph) -> typing.Set[str]:
+def _query_all_value_matches(
+  graph : rdflib.Graph
+) -> typing.Set[str]:
     """
     Return set of all node names (as strings) that have a matching value, where
     "matching" is determined by the SPARQL engine's type and data coercions.

tests/src/compact.py

@@ -23,6 +23,7 @@
 import logging
 import os
 import json
+import typing
 
 import pyld  # type: ignore
 
@@ -38,7 +39,9 @@ def main() -> None:
 
         # Grab the first occurrence of every key.
         total_context = dict()
-        def _accrue_local_context(doc_object):
+        def _accrue_local_context(
+          doc_object : typing.Dict[str, typing.Any]
+        ) -> None:
             local_context = doc_object.get("@context", dict())
             for key in local_context.keys():
                 if not key in total_context:

tests/src/isomorphic_diff.py

@@ -70,7 +70,10 @@ def main() -> None:
     if i1 == i2:
         sys.exit(0)
 
-    def _report(diff_symbol, graph):
+    def _report(
+      diff_symbol : str,
+      graph : rdflib.Graph
+    ) -> None:
         """
         This function copied in spirit from:
         https://rdflib.readthedocs.io/en/stable/apidocs/rdflib.html#module-rdflib.compare