chore: make workspace ID required when deleting a workspace

Signed-off-by: Donnie Adams <[email protected]>

Author: thedadams

gptscript/gptscript.py

@@ -318,9 +318,9 @@ async def create_workspace(self, provider_type: str, from_workspaces: list[str]
             }
         )
 
-    async def delete_workspace(self, workspace_id: str = ""):
+    async def delete_workspace(self, workspace_id: str):
         if workspace_id == "":
-            workspace_id = os.environ["GPTSCRIPT_WORKSPACE_ID"]
+            raise ValueError("workspace_id cannot be empty")
 
         await self._run_basic_command(
             "workspaces/delete",

tests/test_gptscript.py

@@ -40,15 +40,15 @@ def gptscript():
 
 
 # Simple tool for testing
-@pytest.fixture
+@pytest.fixture(scope="function")
 def simple_tool():
     return ToolDef(
         instructions="What is the capital of the united states?"
     )
 
 
 # Complex tool for testing
-@pytest.fixture
+@pytest.fixture(scope="function")
 def complex_tool():
     return ToolDef(
         jsonResponse=True,
@@ -68,7 +68,7 @@ def complex_tool():
 
 
 # Fixture for a list of tools
-@pytest.fixture
+@pytest.fixture(scope="function")
 def tool_list():
     shebang = "#!/bin/bash"
     if platform.system().lower() == "windows":