Fix enum conditional for env parsing (#137)

* Refactoring changed file names

* Roll version

* Update src/unstract/sdk/utils/tool_utils.py

Co-authored-by: Chandrasekharan M <[email protected]>
Signed-off-by: Gayathri <[email protected]>

* Update tests/test_fs_permanent.py

Co-authored-by: Chandrasekharan M <[email protected]>
Signed-off-by: Gayathri <[email protected]>

* Update tests/test_fs_permanent.py

Co-authored-by: Chandrasekharan M <[email protected]>
Signed-off-by: Gayathri <[email protected]>

* Update src/unstract/sdk/utils/tool_utils.py

Co-authored-by: Chandrasekharan M <[email protected]>
Signed-off-by: Gayathri <[email protected]>

* Address review comments

* Add support for passing length to mime_type

* Add recursive and fix mypy issue

* CHange test case with new behavior to return FileNotFound in read()

* Remove typing kwargs.

* Resolve mypy issues

* Resolve mypy issues

* Remove unwanted conditionals/vars

* Remove pandoc and tessaract.

* Details of provider added to error message

* fixed enum conditional matching value

---------

Signed-off-by: Gayathri <[email protected]>
Co-authored-by: Chandrasekharan M <[email protected]>

Author: gaya3-zipstack

src/unstract/sdk/file_storage/env_helper.py

@@ -37,9 +37,9 @@ def get_storage(storage_type: StorageType, env_name: str) -> FileStorage:
                 file_storage_creds[CredentialKeyword.PROVIDER]
             )
             credentials = file_storage_creds.get(CredentialKeyword.CREDENTIALS, "{}")
-            if storage_type == StorageType.PERMANENT.value:
+            if storage_type == StorageType.PERMANENT:
                 file_storage = PermanentFileStorage(provider=provider, **credentials)
-            elif storage_type == StorageType.TEMPORARY.value:
+            elif storage_type == StorageType.TEMPORARY:
                 file_storage = SharedTemporaryFileStorage(
                     provider=provider, **credentials
                 )

tests/test_file_storage.py

@@ -756,12 +756,12 @@ def test_glob(file_storage, folder_path, expected_result):
     "storage_type, env_name, expected",
     [
         (
-            StorageType.PERMANENT.value,
+            StorageType.PERMANENT,
             "TEST_PERMANENT_STORAGE",
             FileStorageProvider.GCS,
         ),
         (
-            StorageType.TEMPORARY.value,
+            StorageType.TEMPORARY,
             "TEST_TEMPORARY_STORAGE",
             FileStorageProvider.MINIO,
         ),