test: Eliminate flake8 errors in tests (#2353)

Eliminate some occurrences of the following flake8 errors in tests:

* E265 block comment should start with '# '
* E266 too many leading '#' for block comment
* E402 module level import not at top of file
* E712 comparison to False should be 'if cond is False:' or 'if not cond:'
* E712 comparison to True should be 'if cond is True:' or 'if cond:'
* E722 do not use bare 'except'
* F401 ... imported but unused
* F403 ... used; unable to detect undefined names
* F405 ... may be undefined, or defined from star imports: ...
* F541 f-string is missing placeholders
* F841 local variable 'result' is assigned to but never used
* N806 variable 'TEST_DIR' in function should be lowercase

This is pursuant to eliminating
[flakeheaven](https://github.com/flakeheaven/flakeheaven), as it no longer
supports the latest version of flake8
[[ref](flakeheaven/flakeheaven#132)].

Author: aucampia

test/jsonld/__init__.py

@@ -5,6 +5,5 @@
 assert plugin
 assert serializer
 assert parser
-import json
 
 __all__: List[str] = []

test/jsonld/test_api.py

@@ -1,11 +1,4 @@
 # -*- coding: UTF-8 -*-
-from rdflib.parser import Parser
-from rdflib.plugin import register
-from rdflib.serializer import Serializer
-
-register("json-ld", Parser, "rdflib.plugins.parsers.jsonld", "JsonLDParser")
-register("json-ld", Serializer, "rdflib.plugins.serializers.jsonld", "JsonLDSerializer")
-
 from rdflib import Graph, Literal, URIRef
 
 

test/jsonld/test_named_graphs.py

@@ -1,10 +1,5 @@
 # -*- coding: UTF-8 -*-
-from rdflib import *
-from rdflib.parser import Parser
-from rdflib.plugin import register
-
-register("json-ld", Parser, "rdflib.plugins.parsers.jsonld", "JsonLDParser")
-register("application/ld+json", Parser, "rdflib.plugins.parsers.jsonld", "JsonLDParser")
+from rdflib import ConjunctiveGraph, Dataset, Graph, URIRef
 
 data = """
 {

test/test_dataset/test_dataset.py

@@ -105,7 +105,7 @@ def get_dataset(request):
         else:
             try:
                 os.remove(path)
-            except:
+            except Exception:
                 pass
 
 

test/test_graph/test_graph_formula.py

@@ -115,7 +115,7 @@ def checkFormulaStore(store="default", configString=None):
             os.unlink(path)
         else:
             g.store.destroy(configString)
-    except:
+    except Exception:
         g.close()
         if store == "SQLite":
             os.unlink(path)

test/test_issues/test_issue492.py

@@ -1,7 +1,4 @@
 # test for https://github.com/RDFLib/rdflib/issues/492
-
-#!/usr/bin/env python3
-
 import rdflib
 
 

test/test_issues/test_issue604.py

@@ -1,4 +1,4 @@
-from rdflib import *
+from rdflib import RDF, BNode, Graph, Literal, Namespace
 from rdflib.collection import Collection
 
 

test/test_literal/test_term.py

@@ -253,7 +253,7 @@ def isclose(a, b, rel_tol=1e-09, abs_tol=0.0):
             if not case_passed:
                 try:
                     case_passed = isclose((case[1] + case[2].value), case[3].value)
-                except:
+                except Exception:
                     pass
 
             if not case_passed:

test/test_misc/test_events.py

@@ -60,5 +60,5 @@ def testEvents(self):
         assert c2["bob"] == "uncle"
         assert c3["bob"] == "uncle"
         del c3["bob"]
-        assert ("bob" in c1) == False
-        assert ("bob" in c2) == False
+        assert ("bob" in c1) is False
+        assert ("bob" in c2) is False

test/test_namespace/test_namespace.py

@@ -284,10 +284,10 @@ def test_expand_curie_exception_messages(self) -> None:
         ["curie", "expected_result"],
         [
             ("ex:tarek", URIRef("urn:example:tarek")),
-            ("ex:", URIRef(f"urn:example:")),
-            ("ex:a", URIRef(f"urn:example:a")),
-            ("ex:a:b", URIRef(f"urn:example:a:b")),
-            ("ex:a:b:c", URIRef(f"urn:example:a:b:c")),
+            ("ex:", URIRef("urn:example:")),
+            ("ex:a", URIRef("urn:example:a")),
+            ("ex:a:b", URIRef("urn:example:a:b")),
+            ("ex:a:b:c", URIRef("urn:example:a:b:c")),
             ("ex", ValueError),
             ("em:tarek", ValueError),
             ("em:", ValueError),