From d59a03bc1960df95db685799ada51081ac8cf7e3 Mon Sep 17 00:00:00 2001 From: kim pham Date: Wed, 18 Dec 2024 10:20:16 +0100 Subject: [PATCH] clean up some flake8 linting flags --- src/jp2_remediator/processor.py | 1 - src/jp2_remediator/tests/unit/test_processor.py | 5 ++--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/jp2_remediator/processor.py b/src/jp2_remediator/processor.py index 95b9040..7690838 100644 --- a/src/jp2_remediator/processor.py +++ b/src/jp2_remediator/processor.py @@ -26,7 +26,6 @@ def process_directory(self, directory_path): reader = self.box_reader_factory.get_reader(file_path) reader.read_jp2_file() - def process_s3_bucket(self, bucket_name, prefix="", output_bucket_name=None, output_prefix=""): """Process all JP2 files in a given S3 bucket.""" s3 = boto3.client("s3") diff --git a/src/jp2_remediator/tests/unit/test_processor.py b/src/jp2_remediator/tests/unit/test_processor.py index 3190761..34d890f 100644 --- a/src/jp2_remediator/tests/unit/test_processor.py +++ b/src/jp2_remediator/tests/unit/test_processor.py @@ -57,8 +57,7 @@ def test_process_directory_with_multiple_files( @patch("jp2_remediator.processor.boto3.client", autospec=True) @patch("builtins.print", autospec=True) def test_process_s3_bucket_with_output_options( - self, mock_print, mock_boto3_client, processor, mock_box_reader_factory - ): + self, mock_print, mock_boto3_client, processor, mock_box_reader_factory): # Ensure processor is the actual Processor object assert hasattr(processor, "process_s3_bucket"), "Processor object expected" # Set up the mock S3 client @@ -130,4 +129,4 @@ def test_process_s3_bucket_with_output_options( unittest.mock.call(f"Processing file: file1.jp2 from bucket {bucket_name}"), unittest.mock.call(f"Processing file: file2.jp2 from bucket {bucket_name}"), ] - mock_print.assert_has_calls(expected_print_calls, any_order=True) \ No newline at end of file + mock_print.assert_has_calls(expected_print_calls, any_order=True)