willisapi-client 1.5__tar.gz → 1.7__tar.gz

{willisapi_client-1.5 → willisapi_client-1.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: willisapi_client
-Version: 1.5
+Version: 1.7
 Summary: A Python client for willisapi
 Home-page: https://github.com/bklynhlth/willsiapi_client
 Author: bklynhlth

{willisapi_client-1.5 → willisapi_client-1.7}/requirements.txt

@@ -8,16 +8,16 @@ charset-normalizer==3.2.0
 frozenlist==1.4.0
 idna==3.4
 multidict==6.0.4
+nest-asyncio==1.5.7
 numpy==1.25.2
 pandas==2.0.3
+pytest==7.4.0
+pytest-cov==4.1.0
 python-dateutil==2.8.2
 pytz==2023.3
 requests==2.31.0
 six==1.16.0
+tqdm==4.66.1
 tzdata==2023.3
 urllib3==2.0.7
 yarl==1.9.2
-tqdm==4.66.1
-nest-asyncio==1.5.7
-pytest==7.4.0
-pytest-cov==4.1.0

willisapi_client-1.7/tests/test_auth.py

@@ -0,0 +1,69 @@
+from willisapi_client.services.auth.login_manager import login
+
+from unittest.mock import patch
+from datetime import timedelta, datetime
+
+
+class TestLoginFunction:
+    def setup_method(self):
+        self.dt = datetime.now()
+        self.username = "dummy"
+        self.password = "password"
+        self.id_token = "dummy_token"
+        self.client_email = "dummy@gmail.com"
+        self.client_name = "dummy_group"
+        self.account = "Group1"
+        self.expires_in = 100
+        self.expires_in_date = str(
+            self.dt.replace(hour=0, minute=0, second=0, microsecond=0)
+            + timedelta(seconds=self.expires_in)
+        )
+
+    @patch("willisapi_client.services.auth.login_manager.AuthUtils.login")
+    def test_login_failed(self, mocked_login):
+        mocked_login.return_value = {}
+        key, expire_in = login("", "")
+        assert key == None
+        assert expire_in == None
+
+    @patch("willisapi_client.services.auth.login_manager.AuthUtils.login")
+    def test_login_success(self, mocked_login):
+        mocked_login.return_value = {
+            "status_code": 200,
+            "result": {"id_token": self.id_token, "expires_in": self.expires_in},
+        }
+
+        key, expire_in = login(self.username, self.password)
+        expire_in = str(self.dt.replace(hour=0, minute=1, second=40, microsecond=0))
+        assert key == self.id_token
+        assert expire_in == self.expires_in_date
+
+    @patch("willisapi_client.services.auth.login_manager.AuthUtils.login")
+    def test_login_400_status(self, mocked_login):
+        mocked_login.return_value = {
+            "status_code": 400,
+            "result": {"id_token": self.id_token, "expires_in": self.expires_in},
+        }
+        key, expire_in = login(self.username, self.password)
+        assert key == None
+        assert expire_in == None
+
+    @patch("willisapi_client.services.auth.login_manager.AuthUtils.login")
+    def test_login_403_status(self, mocked_login):
+        mocked_login.return_value = {
+            "status_code": 403,
+            "result": {"id_token": self.id_token, "expires_in": self.expires_in},
+        }
+        key, expire_in = login(self.username, self.password)
+        assert key == None
+        assert expire_in == None
+
+    @patch("willisapi_client.services.auth.login_manager.AuthUtils.login")
+    def test_login_500_status(self, mocked_login):
+        mocked_login.return_value = {
+            "status_code": 500,
+            "result": {"id_token": self.id_token, "expires_in": self.expires_in},
+        }
+        key, expire_in = login(self.username, self.password)
+        assert key == None
+        assert expire_in == None

willisapi_client-1.7/tests/test_csv_validation.py

@@ -0,0 +1,556 @@
+from willisapi_client.services.upload.csv_validation import CSVValidation
+
+from unittest.mock import patch
+
+
+class TestCSVValidation:
+    def test_directory(self):
+        csv = CSVValidation(file_path="/")
+        assert csv._is_valid() == False
+
+    def test_incorrect_mp4_file(self):
+        csv = CSVValidation(file_path="/video.mp4")
+        assert csv._is_valid() == False
+
+    def test_incorrect_csv_file(self):
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    def test_correct_csv_file(self, mocked_headers, mocked_file):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        assert csv._is_file() == True
+        assert csv._is_valid_file_ext() == True
+        assert csv.get_filename() == "metadata.csv"
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_success(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/data.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "Study Arm",
+            "clinical_score_a": "Score A",
+            "clinical_score_b": "Score B",
+            "clinical_score_c": "Score C",
+            "clinical_score_d": "Score D",
+            "clinical_score_e": "Score E",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == True
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_project(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_failed_incorrect_file(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (False, "Error")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_failed_incorrect_wfts(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "wrong_tag",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_dynamic_wfts(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "scale_abc,speech_transcription_aws_dummy",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "Study Arm",
+            "clinical_score_a": "Score A",
+            "clinical_score_b": "Score B",
+            "clinical_score_c": "Score C",
+            "clinical_score_d": "Score D",
+            "clinical_score_e": "Score E",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == True
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_failed_empty_pt_id(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "",
+            "time_collected": "2023-02-02",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_failed_incorrect_time_collected(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "",
+            "time_collected": "20-02-02",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_failed_empty_language_code(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "",
+            "time_collected": "20-02-02",
+            "language": None,
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_passed_non_empty_language_code(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "",
+            "time_collected": "20-02-02",
+            "language": "random",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_age(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_sex_field(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, None)
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "file.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_race(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_study_arm(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_clinical_score_a(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "Study Arm",
+            "clinical_score_a": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_clinical_score_b(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "Study Arm",
+            "clinical_score_a": "Score A",
+            "clinical_score_b": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_clinical_score_c(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "Study Arm",
+            "clinical_score_a": "Score A",
+            "clinical_score_b": "Score B",
+            "clinical_score_c": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_clinical_score_d(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "Study Arm",
+            "clinical_score_a": "Score A",
+            "clinical_score_b": "Score B",
+            "clinical_score_c": "Score C",
+            "clinical_score_d": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False
+
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_valid_headers"
+    )
+    @patch("willisapi_client.services.upload.csv_validation.CSVValidation._is_file")
+    @patch(
+        "willisapi_client.services.upload.csv_validation.CSVValidation._is_file_path_valid"
+    )
+    def test_csv_row_validation_fail_empty_clinical_score_e(
+        self, mocked_upload_file, mocked_file, mocked_headers
+    ):
+        mocked_headers.return_value = True
+        mocked_file.return_value = True
+        mocked_upload_file.return_value = (True, "")
+        csv = CSVValidation(file_path="/metadata.csv")
+        assert csv._is_valid() == True
+        row = {
+            "project_name": "project_name",
+            "file_path": "/video.mp4",
+            "workflow_tags": "speech_characteristics",
+            "pt_id_external": "qwerty",
+            "time_collected": "2023-02-02",
+            "language": "en-US",
+            "age": 30,
+            "sex": "M",
+            "race": "Asian",
+            "study_arm": "Study Arm",
+            "clinical_score_a": "Score A",
+            "clinical_score_b": "Score B",
+            "clinical_score_c": "Score C",
+            "clinical_score_d": "Score C",
+            "clinical_score_e": "",
+        }
+        is_valid, _ = csv.validate_row(row)
+        assert is_valid == False

{willisapi_client-1.5 → willisapi_client-1.7}/tests/test_download.py

@@ -38,16 +38,23 @@ class TestDownloadFunction:
     def test_download_no_items_from_api(self, mocked_data):
         mocked_data.return_value = {
             "status_code": 200,
-            "items": {},
+            "presigned_url": None,
         }
         data = download(self.key, self.project_name)
         assert data.empty == True
 
     @patch("willisapi_client.services.download.download_utils.DownloadUtils.request")
-    def test_download_success(self, mocked_data):
+    @patch(
+        "willisapi_client.services.download.download_utils.DownloadUtils.get_data_from_presigned_url"
+    )
+    def test_download_success(self, mocked_data, mocked_response):
         with open("tests/test.json") as json_file:
             data = json.load(json_file)
         mocked_data.return_value = data
+        mocked_response.return_value = {
+            "status_code": 200,
+            "presigned_url": "https://google.com",
+        }
         data = download(self.key, self.project_name)
         assert data.empty == False
         assert data.filename.tolist()[0] == "test_video.mp4"