Merge pull request #74 from supabase-community/j0_fix_test_instance_settings

Update Test instance settings

Author: J0

Diff for: supabase/lib/storage/storage_bucket_api.py

@@ -3,13 +3,13 @@
 from collections.abc import Awaitable
 from dataclasses import dataclass
 from datetime import datetime
-from typing import Any, Literal, Optional, Type, Union
+from typing import Any, Dict, List, Optional, Type, Union
 
 from httpx import AsyncClient, Client, HTTPError
 
 __all__ = ["Bucket", "StorageBucketAPI"]
 
-_RequestMethod = Literal["GET", "POST", "PUT", "DELETE", "PATCH"]
+_RequestMethod = str
 
 
 class StorageException(Exception):
@@ -33,10 +33,10 @@ def __post_init__(self) -> None:
 
 
 ResponseType = Union[
-    dict[
+    Dict[
         str, str
     ],  # response from an endpoint without a custom response_class, example: create_bucket
-    list[
+    List[
         Bucket
     ],  # response from an endpoint which returns a list of objects, example: list_buckets
     Bucket,  # response from an endpoint which returns a single object, example: get_bucket
@@ -80,7 +80,7 @@ def _sync_request(
         response_class: Optional[Type] = None,
     ) -> ResponseType:
         if isinstance(self._client, AsyncClient):  # only to appease the type checker
-            return
+            return None
 
         response = self._client.request(method, url, json=json)
         try:

Diff for: test.sh

@@ -1,4 +1,4 @@
 #!/bin/sh
-SUPABASE_TEST_KEY="eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJyb2xlIjoiYW5vbiIsImlhdCI6MTYxMjYwOTMyMiwiZXhwIjoxOTI4MTg1MzIyfQ.XL9W5I_VRQ4iyQHVQmjG0BkwRfx6eVyYB3uAKcesukg" \
-SUPABASE_TEST_URL="https://tfsatoopsijgjhrqplra.supabase.co" \
+SUPABASE_TEST_KEY="eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJyb2xlIjoiYW5vbiIsImlhdCI6MTYzNTAwODQ4NywiZXhwIjoxOTUwNTg0NDg3fQ.l8IgkO7TQokGSc9OJoobXIVXsOXkilXl4Ak6SCX5qI8" \
+SUPABASE_TEST_URL="https://ibrydvrsxoapzgtnhpso.supabase.co" \
 poetry run pytest

Diff for: tests/test_client.py

@@ -83,17 +83,3 @@ def test_client_insert(supabase: Client) -> None:
     assert current_length == previous_length + 1
     # Check returned result for insert was valid.
     assert result.get("status_code", 400) == 201
-
-
-def test_client_bucket(supabase: Client) -> None:
-
-    """Ensure that the storage bucket operations work"""
-    TEST_BUCKET_NAME = "atestbucket"
-    # TODO[Joel] - Reinstate once permissions on test instance are updated
-    # storage = supabase.storage()
-    # storage_bucket = storage.StorageBucketAPI()
-    # storage_bucket.create_bucket(TEST_BUCKET_NAME)
-    # storage_bucket.list_buckets()
-    # storage_bucket.get_bucket(TEST_BUCKET_NAME)
-    # storage_bucket.empty_bucket(TEST_BUCKET_NAME)
-    # storage_bucket.delete_bucket(TEST_BUCKET_NAME)