aplos-nca-saas-sdk 0.0.12__py3-none-any.whl → 0.0.13__py3-none-any.whl

aplos_nca_saas_sdk/integration_testing/configs/_config_base.py

@@ -1,5 +1,10 @@
-from typing import Dict
-from typing import Any
+"""
+Copyright 2024-2025 Aplos Analytics
+All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
+Property of Aplos Analytics, Utah, USA
+"""
+
+from typing import Dict, Any
 
 
 class ConfigBase:

aplos_nca_saas_sdk/integration_testing/configs/app_settings_config.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
@@ -8,26 +8,26 @@ from typing import List, Dict, Any
 from aplos_nca_saas_sdk.integration_testing.configs._config_base import ConfigBase
 
 
-class ApplicationDomainConfig(ConfigBase):
+class ApplicationHostConfig(ConfigBase):
     """
-    Application Domain: Defines the domains that the application configuration tests will check against
+    Application host: Defines the domains that the application configuration tests will check against
 
     """
 
-    def __init__(self, domain: str | None = None):
+    def __init__(self, host: str | None = None):
         super().__init__()
-        self.__domain: str | None = domain
+        self.__host: str | None = host
 
     @property
-    def domain(self) -> str:
-        """The domain to validate"""
-        if self.__domain is None:
-            raise RuntimeError("Domain is not set")
-        return self.__domain
+    def host(self) -> str:
+        """The host to validate"""
+        if self.__host is None:
+            raise RuntimeError("host is not set")
+        return self.__host
 
-    @domain.setter
-    def domain(self, value: str):
-        self.__domain = value
+    @host.setter
+    def host(self, value: str):
+        self.__host = value
 
 
 class ApplicationDomainConfigs(ConfigBase):
@@ -38,19 +38,19 @@ class ApplicationDomainConfigs(ConfigBase):
 
     def __init__(self):
         super().__init__()
-        self.__domains: List[ApplicationDomainConfig] = []
+        self.__hosts: List[ApplicationHostConfig] = []
 
     @property
-    def list(self) -> List[ApplicationDomainConfig]:
+    def list(self) -> List[ApplicationHostConfig]:
         """List the logins"""
-        return self.__domains
+        return self.__hosts
 
-    def add(self, *, domain: str, enabled: bool = True):
+    def add(self, *, host: str, enabled: bool = True):
         """Add a loging"""
-        app_domain = ApplicationDomainConfig()
-        app_domain.domain = domain
+        app_domain = ApplicationHostConfig()
+        app_domain.host = host
         app_domain.enabled = enabled
-        self.__domains.append(app_domain)
+        self.__hosts.append(app_domain)
 
     def load(self, test_config: Dict[str, Any]):
         """Load the logins from a list of dictionaries"""
@@ -58,13 +58,13 @@ class ApplicationDomainConfigs(ConfigBase):
         super().load(test_config)
         domains: List[Dict[str, Any]] = test_config.get("domains", [])
 
-        domain: Dict[str, Any]
-        for domain in domains:
-            app_domain = ApplicationDomainConfig()
-            app_domain.domain = domain.get("domain", None)
-            app_domain.enabled = bool(domain.get("enabled", True))
+        host: Dict[str, Any]
+        for host in domains:
+            app_domain = ApplicationHostConfig()
+            app_domain.host = host.get("host", None)
+            app_domain.enabled = bool(host.get("enabled", True))
 
-            self.__domains.append(app_domain)
+            self.__hosts.append(app_domain)
 
 
 class ApplicationSettings(ConfigBase):
@@ -75,12 +75,12 @@ class ApplicationSettings(ConfigBase):
 
     def __init__(self):
         super().__init__()
-        self.__domains: ApplicationDomainConfigs = ApplicationDomainConfigs()
+        self.__hosts: ApplicationDomainConfigs = ApplicationDomainConfigs()
 
     @property
     def domains(self) -> ApplicationDomainConfigs:
-        """List of the domain"""
-        return self.__domains
+        """List of the host"""
+        return self.__hosts
 
     def load(self, test_config: Dict[str, Any]):
         """Load the domains from the config"""

aplos_nca_saas_sdk/integration_testing/configs/config_sample.json

@@ -3,14 +3,14 @@
         "purpose": "Tests the application configuration endpoints",
         "domains": [
             {
-                "domain": "api.example.com",
+                "host": "api.example.com",
                 "expected_results": {
                     "status_code": 200
                 },
                 "enabled": true
             },
             {
-                "domain": "XXXXXXXXXXXXXXXXXXXXX",
+                "host": "XXXXXXXXXXXXXXXXXXXXX",
                 "expected_results": {
                     "status_code": 403
                 },
@@ -24,13 +24,13 @@
             {
                 "username": "foo",
                 "password": "barr",
-                "domain": "api.example.com",
+                "host": "api.example.com",
                 "roles": []
             },
             {
                 "username": "XXXXXXXXXXXXXXXXXXXXX",
                 "password": "XXXXXXXXXXXXXXXXXXXXX",
-                "domain": "XXXXXXXXXXXXXXXXXXXXX",
+                "host": "XXXXXXXXXXXXXXXXXXXXX",
                 "roles": [
                     "XXXXXXXXXXXXXXXXXXXXX"
                 ],
@@ -48,7 +48,7 @@
             "purpose": "optional: if present this login is used, unless a specific login is defined for the test",
             "username": "foo",
             "password": "bar",
-            "domain": "api.example.com"
+            "host": "api.example.com"
         },
         "files": [
             {
@@ -60,7 +60,7 @@
                     "purpose": "optional: if present tests an upload for a specific user",
                     "username": "XXXXXXXXXXXXXXXXXXXXX",
                     "password": "XXXXXXXXXXXXXXXXXXXXX",
-                    "domain": "XXXXXXXXXXXXXXXXXXXXX"
+                    "host": "XXXXXXXXXXXXXXXXXXXXX"
                 }
             }
         ]
@@ -70,7 +70,7 @@
         "login": {
             "username": "XXXXXXXXXXXXXXXXXXXXX",
             "password": "XXXXXXXXXXXXXXXXXXXXX",
-            "domain": "XXXXXXXXXXXXXXXXXXXXX"
+            "host": "XXXXXXXXXXXXXXXXXXXXX"
         },
         "output_dir": "XXXXXXXXXXXXXXXXXXXXX",
         "analyses": [
@@ -90,7 +90,7 @@
                 "login": {
                     "username": "XXXXXXXXXXXXXXXXXXXXX",
                     "password": "XXXXXXXXXXXXXXXXXXXXX",
-                    "domain": "XXXXXXXXXXXXXXXXXXXXX"
+                    "host": "XXXXXXXXXXXXXXXXXXXXX"
                 },
                 "expected_results": {
                     "status_code": 200
@@ -103,7 +103,7 @@
         "login": {
             "username": "XXXXXXXXXXXXXXXXXXXXX",
             "password": "XXXXXXXXXXXXXXXXXXXXX",
-            "domain": "XXXXXXXXXXXXXXXXXXXXX"
+            "host": "XXXXXXXXXXXXXXXXXXXXX"
         },
         "expected_results": {
             "status_code": 200

aplos_nca_saas_sdk/integration_testing/configs/file_upload_config.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """

aplos_nca_saas_sdk/integration_testing/configs/login_config.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
@@ -18,13 +18,13 @@ class LoginConfig(ConfigBase):
         self,
         username: Optional[str] = None,
         passord: Optional[str] = None,
-        domain: Optional[str] = None,
+        host: Optional[str] = None,
         roles: Optional[List[str]] = None,
     ):
         super().__init__()
         self.__username: Optional[str] = username
         self.__password: Optional[str] = passord
-        self.__domain: Optional[str] = domain
+        self.__host: Optional[str] = host
         self.__roles: List[str] = roles if roles is not None else []
 
     @property
@@ -48,14 +48,15 @@ class LoginConfig(ConfigBase):
         self.__password = value
 
     @property
-    def domain(self) -> str:
-        if self.__domain is None:
-            raise RuntimeError("Domain is not set")
-        return self.__domain
+    def host(self) -> str:
+        """API Host"""
+        if self.__host is None:
+            raise RuntimeError("Host is not set")
+        return self.__host
 
-    @domain.setter
-    def domain(self, value: str):
-        self.__domain = value
+    @host.setter
+    def host(self, value: str):
+        self.__host = value
 
     @property
     def roles(self) -> List[str]:
@@ -84,12 +85,12 @@ class LoginConfigs(ConfigBase):
         """List the logins"""
         return self.__logins
 
-    def add(self, *, username: str, password: str, domain: str, enabled: bool = True):
+    def add(self, *, username: str, password: str, host: str, enabled: bool = True):
         """Add a loging"""
         login = LoginConfig()
         login.username = username
         login.password = password
-        login.domain = domain
+        login.host = host
         login.enabled = enabled
         self.__logins.append(login)
 
@@ -111,8 +112,8 @@ class LoginConfigs(ConfigBase):
         if login_config is not None:
             username = login_config.get("username", None)
             password = login_config.get("password", None)
-            domain = login_config.get("domain", None)
+            host = login_config.get("host", None)
             enabled = login_config.get("enabled", True)
-            login = LoginConfig(username, password, domain)
+            login = LoginConfig(username, password, host)
             login.enabled = enabled
         return login

aplos_nca_saas_sdk/integration_testing/configs/nca_execution_config.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """

aplos_nca_saas_sdk/integration_testing/integration_test_base.py

@@ -1,11 +1,11 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
 
-from typing import Dict, Any, List
 from abc import ABC, abstractmethod
+from typing import List
 
 from aplos_nca_saas_sdk.integration_testing.integration_test_configurations import (
     TestConfiguration,

aplos_nca_saas_sdk/integration_testing/integration_test_configurations.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
@@ -9,9 +9,13 @@ from typing import Any, Dict
 from aplos_nca_saas_sdk.integration_testing.configs.app_settings_config import (
     ApplicationSettings,
 )
-from aplos_nca_saas_sdk.integration_testing.configs.file_upload_config import FileUploadConfigs
+from aplos_nca_saas_sdk.integration_testing.configs.file_upload_config import (
+    FileUploadConfigs,
+)
 from aplos_nca_saas_sdk.integration_testing.configs.login_config import LoginConfigs
-from aplos_nca_saas_sdk.integration_testing.configs.nca_execution_config import NCAExecutionConfigs
+from aplos_nca_saas_sdk.integration_testing.configs.nca_execution_config import (
+    NCAExecutionConfigs,
+)
 
 
 class TestConfiguration:
@@ -25,7 +29,7 @@ class TestConfiguration:
         self.logins: LoginConfigs = LoginConfigs()
         self.file_uploads: FileUploadConfigs = FileUploadConfigs()
         self.nca_executions: NCAExecutionConfigs = NCAExecutionConfigs()
-        
+
     def load(self, file_path: str):
         """
         Loads the configuration from a file
@@ -41,4 +45,4 @@ class TestConfiguration:
         self.logins.load(config.get("login_test", {}))
         self.app_config.load(config.get("application_config_test", {}))
         self.file_uploads.load(config.get("file_upload_test", {}))
-        self.nca_executions.load(config.get("analysis_execution_test", {}))
+        self.nca_executions.load(config.get("analysis_execution_test", {}))

aplos_nca_saas_sdk/integration_testing/integration_test_factory.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """

aplos_nca_saas_sdk/integration_testing/integration_test_response.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """

aplos_nca_saas_sdk/integration_testing/integration_test_suite.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
@@ -93,6 +93,8 @@ class IntegrationTestSuite:
         return len(failures) == 0
 
     def __print_results(self, start_time: datetime, failures: List[Dict[str, Any]]):
+        print("")
+        print("--------------------------------")
         print("Test Results:")
         skipped = sum([test["skipped_count"] for test in self.test_results])
 
@@ -101,8 +103,6 @@ class IntegrationTestSuite:
             print(
                 f"  {test_result['test_name']} {'succeeded' if test_result['success'] else 'failed'} duration: {duration}"
             )
-            if not test_result["success"]:
-                print(f"    Errors: {test_result['errors']}")
 
         print(f"Test Suite completed in {datetime.now(UTC) - start_time}")
 
@@ -110,3 +110,10 @@ class IntegrationTestSuite:
         print(f"  Successful: {len(self.test_results) - len(failures) - skipped}")
         print(f"  Skipped: {skipped}")
         print(f"  Failed: {len(failures)}")
+
+        if len(failures) > 0:
+            print("--------------------------------")
+
+        for test_result in self.test_results:
+            if not test_result["success"]:
+                print(f"Errors: {test_result['errors']}")

aplos_nca_saas_sdk/integration_testing/main.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
@@ -36,28 +36,29 @@ def main():
     )
     # load it so we can see what it looks like
     config.load(file_path=config_file)
-    
+
     # override the configuration
     # override_config(config)
 
     its.test(test_config=config)
 
+
 def override_config(config: TestConfiguration):
     """Override the configuration for the tests"""
     username = os.getenv("TEST_USERNAME")
     password = os.getenv("TEST_PASSWORD")
-    domain = os.getenv("TEST_DOMAIN")
+    host = os.getenv("TEST_HOST")
 
-    if not username or not password or not domain:
+    if not username or not password or not host:
         raise RuntimeError(
-            "TEST_USERNAME, TEST_PASSWORD, and TEST_DOMAIN must be set in the environment"
+            "TEST_USERNAME, TEST_PASSWORD, and TEST_HOST must be set in the environment"
         )
 
     config.logins.list.clear()
-    config.logins.add(username=username, password=password, domain=domain)
+    config.logins.add(username=username, password=password, host=host)
 
     config.app_config.domains.list.clear()
-    config.app_config.domains.add(domain=domain)
+    config.app_config.domains.add(host=host)
 
 
 if __name__ == "__main__":

aplos_nca_saas_sdk/integration_testing/readme.md

@@ -9,7 +9,7 @@ The integration tests will require the following:
 ### Environment Vars
 |Variable Name|Description|
 |--|--|
-|APLOS_API_DOMAIN|The full domain. e.g. app.aplos-nca.com| 
+|APLOS_host|The full host name. e.g. app.aplos-nca.com| 
 
 
 ### Users

aplos_nca_saas_sdk/integration_testing/tests/app_configuration_test.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
@@ -27,10 +27,10 @@ class TestAppConfiguration(IntegrationTestBase):
 
         self.results.clear()
         for domain_config in self.config.app_config.domains.list:
-            config: NCAAppConfiguration = NCAAppConfiguration(domain_config.domain)
+            config: NCAAppConfiguration = NCAAppConfiguration(domain_config.host)
 
             test_response: IntegrationTestResponse = IntegrationTestResponse()
-            test_response.meta = {"domain": domain_config}
+            test_response.meta = {"host": domain_config}
 
             if not domain_config.enabled or not self.config.app_config.enabled:
                 test_response.skipped = True

aplos_nca_saas_sdk/integration_testing/tests/app_login_test.py

@@ -1,10 +1,10 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
 
-from aplos_nca_saas_sdk.nca_resources.nca_login import NCALogin
+from aplos_nca_saas_sdk.nca_resources.nca_authenticator import NCAAuthenticator
 
 
 from aplos_nca_saas_sdk.integration_testing.integration_test_base import (
@@ -35,7 +35,7 @@ class TestAppLogin(IntegrationTestBase):
                 self.results.append(test_response)
                 continue
             try:
-                nca_login = NCALogin(aplos_saas_domain=login.domain)
+                nca_login = NCAAuthenticator(host=login.host)
                 token = nca_login.authenticate(
                     username=login.username, password=login.password
                 )

aplos_nca_saas_sdk/integration_testing/tests/file_upload_test.py

@@ -1,5 +1,5 @@
 """
-Copyright 2024 Aplos Analytics
+Copyright 2024-2025 Aplos Analytics
 All Rights Reserved.   www.aplosanalytics.com   LICENSED MATERIALS
 Property of Aplos Analytics, Utah, USA
 """
@@ -18,8 +18,12 @@ from aplos_nca_saas_sdk.integration_testing.integration_test_response import (
     IntegrationTestResponse,
 )
 from aplos_nca_saas_sdk.nca_resources.nca_file_upload import NCAFileUpload
-from aplos_nca_saas_sdk.nca_resources.nca_login import NCALogin
+from aplos_nca_saas_sdk.nca_resources.nca_file_download import NCAFileDownload
+from aplos_nca_saas_sdk.nca_resources.nca_authenticator import NCAAuthenticator
 from aplos_nca_saas_sdk.utilities.http_utility import HttpUtilities
+from aplos_nca_saas_sdk.integration_testing.configs.file_upload_config import (
+    FileUploadConfig,
+)
 
 logger = Logger()
 
@@ -35,31 +39,30 @@ class FileUploadTest(IntegrationTestBase):
 
         self.results.clear()
 
+        file_upload: FileUploadConfig
         for file_upload in self.config.file_uploads.list:
             test_response: IntegrationTestResponse = IntegrationTestResponse()
             test_response.name = self.name
             try:
                 # Confirm Login
                 nca_login = self.__login(file_upload.login)
-                if not nca_login.jwt:
-                    test_response.error = "Failed to authenticate"
+                # if not nca_login.cognito.jwt:
+                #     test_response.error = "Failed to authenticate"
+                # else:
+                # Confirm Upload
+                upload_response: Dict[str, Any] = self.__upload(
+                    nca_login, file_upload.file_path
+                )
+                if upload_response is None:
+                    test_response.error = "Failed to upload"
                 else:
-                    # Confirm Upload
-                    upload_response: Dict[str, Any] = self.__upload(
-                        nca_login, file_upload.file_path
-                    )
-                    if upload_response is None:
-                        test_response.error = "Failed to upload"
-                    else:
-                        # Confirm conversion and download
-                        # Allow time buffer so file data is available
-                        file_id: str = upload_response.get("file_id", "")
-                        if not file_id:
-                            raise RuntimeError(
-                                "Failed to get a file_id from the upload"
-                            )
-                        time.sleep(3)
-                        self.__download(nca_login, file_id, test_response)
+                    # Confirm conversion and download
+                    # Allow time buffer so file data is available
+                    file_id: str = upload_response.get("file_id", "")
+                    if not file_id:
+                        raise RuntimeError("Failed to get a file_id from the upload")
+                    time.sleep(3)
+                    self.__download(nca_login, file_id, test_response)
 
             except Exception as e:  # pylint: disable=w0718
                 test_response.error = str(e)
@@ -68,91 +71,37 @@ class FileUploadTest(IntegrationTestBase):
 
         return self.success()
 
-    def __login(self, login: LoginConfig) -> NCALogin:
-        nca_login = NCALogin(aplos_saas_domain=login.domain)
+    def __login(self, login: LoginConfig) -> NCAAuthenticator:
+        nca_login = NCAAuthenticator(host=login.host)
         nca_login.authenticate(username=login.username, password=login.password)
         return nca_login
 
-    def __upload(self, nca_login: NCALogin, upload_file_path: str) -> Dict[str, Any]:
+    def __upload(self, auth: NCAAuthenticator, upload_file_path: str) -> Dict[str, Any]:
         logger.info({"message": "Uploading file", "file_path": upload_file_path})
 
-        nca_file_upload = NCAFileUpload(nca_login)
+        nca_file_upload = NCAFileUpload(auth.host)
+        nca_file_upload.authenticator = auth
         upload_response: Dict[str, Any] = nca_file_upload.upload(upload_file_path)
         return upload_response
 
     def __download(
         self,
-        nca_login: NCALogin,
+        nca_login: NCAAuthenticator,
         file_id: str,
         test_response: IntegrationTestResponse,
     ):
         logger.info({"message": "Downloading file", "file_id": file_id})
-        file_data_endpoint = nca_login.config.endpoints.file_data(
-            nca_login.cognito.tenant_id,
-            nca_login.cognito.user_id,
-            file_id,
-        )
-        file_info_endpoint = nca_login.config.endpoints.file(
-            nca_login.cognito.tenant_id,
-            nca_login.cognito.user_id,
-            file_id,
-        )
-
-        max_wait_in_minutes: int = 3
-        headers = HttpUtilities.get_headers(nca_login.jwt)
-        current_time = datetime.now()
-
-        # Create a timedelta object representing 3 minutes
-        time_delta = timedelta(minutes=max_wait_in_minutes)
-        # Add the timedelta to the current time
-        max_time = current_time + time_delta
-
-        complete = False
-        while not complete:
-            response = requests.get(file_info_endpoint, headers=headers, timeout=60)
-            json_response: dict = response.json()
-            errors: List[Dict[str, Any]] = []
-            errors.extend(json_response.get("errors") or [])
-            status = json_response.get("workable_state")
-            complete = status == "ready"
-
-            if status == "invalid" or len(errors) > 0:
-                test_response.success = False
-                test_response.error = (
-                    "File conversion failed."
-                    if len(errors) < 0
-                    else f"File conversion failed with errors ${errors}."
-                )
-                break
-            if complete:
-                break
-            if not complete:
-                time.sleep(5)
-            if datetime.now() > max_time:
-                test_response.success = False
-                test_response.error = (
-                    "Timeout attempting to get conversion file status. "
-                    f"The current timeout limit is {max_wait_in_minutes} minutes. "
-                    "You may need to up the timeout period, or check for errors. "
-                )
-                break
-
-        if test_response.error is not None:
-            return
+        downloader: NCAFileDownload = NCAFileDownload(nca_login.host)
+        downloader.authenticator = nca_login
+        response = downloader.download(file_id)
 
-        file_response = requests.get(file_data_endpoint, headers=headers, timeout=30)
-
-        json_file_response: dict = file_response.json()
-        data = json_file_response.get("data", None)
-        error = json_file_response.get("error", None)
-
-        if data is None:
+        status = response.get("workable_state")
+        if status != "ready":
             test_response.success = False
-            test_response.error = "File download missing expected data. "
-        if error is not None:
-            test_response.success = False
-            test_response.error = (
-                test_response.error or ""
-            ) + f"File download contained error ${error}"
+            test_response.error = f"File conversion failed. Status: {status}"
+        else:
+            test_response.success = True
+
+        test_response.response = response
 
-        test_response.success = True
+        return response