test sdk: fix typing and codestyle errors

Author: hpoeche

sdk/test/adapter/aasx/test_aasx.py

@@ -106,11 +106,12 @@ def test_write_missing_aas_objects(self):
                         "/aasx/selection.xml",
                         ["https://acplt.org/Test_AssetAdministrationShell",
                          "http://false-identifier.org/",
-                        "http://acplt.org/Submodels/Assets/TestAsset/Identification"],
+                         "http://acplt.org/Submodels/Assets/TestAsset/Identification"],
                         data, aasx.DictSupplementaryFileContainer()
                     )
 
-            self.assertIn("Could not find identifiable http://false-identifier.org/ in IdentifiableStore", log.output[0])
+            self.assertIn("Could not find identifiable http://false-identifier.org/ in IdentifiableStore",
+                          log.output[0])
 
             # assert only the two existing objects have been written to aasx file
             object_store = model.DictIdentifiableStore()
@@ -142,7 +143,7 @@ def test_writing_with_missing_file(self) -> None:
             self.assertIn("Could not find file", cm.exception.args[0])
 
     def test_writing_file_twice(self) -> None:
-        with tempfile.TemporaryDirectory() as tmpdir:
+        with (tempfile.TemporaryDirectory() as tmpdir):
             tmpdir_path = Path(tmpdir)
 
             # ---- Arange ----
@@ -167,7 +168,8 @@ def test_writing_file_twice(self) -> None:
                     value=resulting_file_name
                 )]
             )
-            data = model.DictIdentifiableStore([first_submodel, second_submodel])
+            data: model.DictIdentifiableStore[model.Identifiable] \
+                = model.DictIdentifiableStore([first_submodel, second_submodel])
 
             # ---- Act & Assert ----
             with self.assertNoLogs(level="WARNING"):
@@ -184,7 +186,6 @@ def test_write_non_aas(self) -> None:
             with open(Path(__file__).parent / "TestFile.pdf", "rb") as pdf:
                 file_store.add_file("/TestFile.pdf", pdf, "application/pdf")
 
-
             # ---- Act & Assert ----
             # assert warning is present in failsafe mode
             with self.assertLogs(level="WARNING") as log:
@@ -199,7 +200,7 @@ def test_write_non_aas(self) -> None:
                     # try to write a non AAS object
                     writer.write_aas("https://acplt.org/Test_Submodel", data, file_store)
             self.assertIn("Identifier https://acplt.org/Test_Submodel does not belong "
-                          "to an AssetAdministrationShell",cm.exception.args[0])
+                          "to an AssetAdministrationShell", cm.exception.args[0])
 
     def test_write_aas_missing_submodel(self) -> None:
         with tempfile.TemporaryDirectory() as tmpdir:
@@ -315,7 +316,6 @@ def test_write_core_properties_twice(self) -> None:
 
             self.assertIn("Core Properties have already been written", cm.exception.args[0])
 
-
     def test_write_thumbnail_twice(self) -> None:
         with tempfile.TemporaryDirectory() as tmpdir:
             tmpdir_path = Path(tmpdir)
@@ -334,7 +334,6 @@ def test_write_thumbnail_twice(self) -> None:
 
             self.assertIn("package thumbnail has already been written", cm.exception.args[0])
 
-
     def test_writing_reading_example_aas(self) -> None:
         # Create example data and file_store
         data = example_aas.create_full_example()    # creates a complete, valid example AAS
@@ -457,7 +456,7 @@ def test_get_thumbnail(self) -> None:
             # ---- Arange ----
             tmpdir_path = Path(tmpdir)
 
-            data = model.DictIdentifiableStore([
+            data: model.DictIdentifiableStore[model.Identifiable] = model.DictIdentifiableStore([
                 model.AssetAdministrationShell(
                     id_="http://example.org/Test_AAS",
                     asset_information=model.AssetInformation(

sdk/test/adapter/test_load_directory.py

@@ -67,7 +67,6 @@ def test_reading_all_files(self):
                     file_store=file_container
                 )
 
-
             # ---- Act ----
             new_object_store, new_file_store = adapter.load_directory(temp_dir_path)
 
@@ -80,7 +79,6 @@ def test_reading_all_files(self):
             # check pdf is loaded
             self.assertIn(resulting_file_name, new_file_store)
 
-
     def test_skipping_other_files(self):
         with tempfile.TemporaryDirectory() as tmp_dir:
             # ---- Arange ----
@@ -99,4 +97,4 @@ def test_skipping_other_files(self):
             # ---- Assert ----
             # check stores are empty
             self.assertEqual(len(object_store), 0)
-            self.assertEqual(len(file_store), 0)
+            self.assertEqual(len(file_store), 0)