remove comments

Author: andrewelamb

tests/unit/synapseclient/models/async/unit_test_manifest_async.py

@@ -71,8 +71,6 @@ async def test_missing_parent_id_column_raises(self, tmp_path: Path) -> None:
         with pytest.raises(ValueError, match="'parentId'"):
             await read_manifest_for_upload(str(manifest), self.syn, True, False)
 
-    # -- error column handling ---------------------------------------------
-
     async def test_rows_with_error_are_skipped(self, tmp_path: Path) -> None:
         """Rows with a non-empty 'error' cell are excluded from the returned items list."""
         f = tmp_path / "file.txt"
@@ -97,8 +95,6 @@ async def test_all_rows_have_errors_returns_empty(self, tmp_path: Path) -> None:
         assert items == []
         assert total == 0
 
-    # -- path validation ---------------------------------------------------
-
     async def test_duplicate_paths_raise(self, tmp_path: Path) -> None:
         """Two rows referencing the same file path raise ValueError about unique file paths."""
         f = tmp_path / "file.txt"
@@ -125,8 +121,6 @@ async def test_empty_file_raises(self, tmp_path: Path) -> None:
         with pytest.raises(ValueError, match="empty"):
             await read_manifest_for_upload(str(manifest), self.syn, True, False)
 
-    # -- successful upload items -------------------------------------------
-
     async def test_valid_manifest_returns_items_and_size(self, tmp_path: Path) -> None:
         """A valid manifest returns one upload item and the correct total file size."""
         f = tmp_path / "file.txt"
@@ -552,7 +546,7 @@ def test_valid_names_pass(self) -> None:
                 "parentId": ["syn1", "syn1"],
             }
         )
-        _check_file_names(df)  # should not raise
+        _check_file_names(df)
 
     def test_invalid_name_raises(self) -> None:
         """A file name containing characters not permitted by Synapse raises ValueError."""
@@ -587,7 +581,7 @@ def test_same_name_different_parent_is_ok(self) -> None:
                 "parentId": ["syn1", "syn2"],
             }
         )
-        _check_file_names(df)  # should not raise
+        _check_file_names(df)
 
     @pytest.mark.parametrize(
         "name",
@@ -608,7 +602,7 @@ def test_special_characters_in_name_accepted(self, name: str) -> None:
                 "parentId": ["syn1"],
             }
         )
-        _check_file_names(df)  # should not raise
+        _check_file_names(df)
 
 
 class TestSortAndFixProvenance: