ruff 🐶

Author: soustruh

tests/test_dao.py

@@ -88,7 +88,7 @@ def test_datatype_accepts_enum_for_manifest_valid(self):
 
     def test_invalid_datatype_fails(self):
         tmetadata = TableMetadata()
-        with self.assertRaises(ValueError) as ctx:
+        with self.assertRaises(ValueError):
             tmetadata.add_column_data_type("col", "invalid type")
 
     def test_table_description_metadata_for_legacy_manifest_is_valid(self):
@@ -259,7 +259,7 @@ def test_out_old_to_new_has_headers_columns(self):
         self.assertEqual(manifest["has_header"], False)
 
     def test_out_pkey_and_no_columns_incompatible(self):
-        with self.assertRaises(UserException) as e:
+        with self.assertRaises(UserException):
             TableDefinition("testDef", "somepath", primary_key=["foo"])
 
     def test_out_legacy_to_new_compatible(self):
@@ -307,7 +307,7 @@ def test_table_manifest_minimal(self):
 
     def test_table_manifest_missing_key(self):
         with self.assertRaises(UserException) as e:
-            table_def = TableDefinition(
+            TableDefinition(
                 "testDef", "somepath", is_sliced=False, primary_key=["foo", "bar"]
             )
 
@@ -605,9 +605,7 @@ def test_unsupported_legacy_queue_properties_log(self):
                 log.output[0],
             )
 
-    def test_unsupported_legacy_queue_properties_excluded(self):
         td = TableDefinition("testDef", "somepath", write_always=True, stage="out")
-        manifest = td.get_manifest_dictionary(legacy_queue=True)
         self.assertTrue("write_always" not in manifest)
 
         manifest = td.get_manifest_dictionary(legacy_queue=False)