increased smoke test timeouts

Author: sid-rl

tests/smoketests/sdk/test_async_snapshot.py

@@ -10,14 +10,14 @@
 
 pytestmark = [pytest.mark.smoketest, pytest.mark.asyncio]
 
-THIRTY_SECOND_TIMEOUT = 30
 TWO_MINUTE_TIMEOUT = 120
+FOUR_MINUTE_TIMEOUT = 240
 
 
 class TestAsyncSnapshotLifecycle:
     """Test basic async snapshot lifecycle operations."""
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_snapshot_create_and_info(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test creating a snapshot from devbox."""
         # Create a devbox
@@ -51,7 +51,7 @@ async def test_snapshot_create_and_info(self, async_sdk_client: AsyncRunloopSDK)
         finally:
             await devbox.shutdown()
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_snapshot_with_commit_message(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test creating a snapshot with commit message."""
         devbox = await async_sdk_client.devbox.create(
@@ -78,7 +78,7 @@ async def test_snapshot_with_commit_message(self, async_sdk_client: AsyncRunloop
         finally:
             await devbox.shutdown()
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_snapshot_with_metadata(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test creating a snapshot with metadata."""
         devbox = await async_sdk_client.devbox.create(
@@ -107,7 +107,7 @@ async def test_snapshot_with_metadata(self, async_sdk_client: AsyncRunloopSDK) -
         finally:
             await devbox.shutdown()
 
-    @pytest.mark.timeout(THIRTY_SECOND_TIMEOUT)
+    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
     async def test_snapshot_delete(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test deleting a snapshot."""
         devbox = await async_sdk_client.devbox.create(
@@ -138,7 +138,7 @@ async def test_snapshot_delete(self, async_sdk_client: AsyncRunloopSDK) -> None:
 class TestAsyncSnapshotCompletion:
     """Test async snapshot completion and status tracking."""
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_snapshot_await_completed(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test waiting for snapshot completion."""
         devbox = await async_sdk_client.devbox.create(
@@ -165,7 +165,7 @@ async def test_snapshot_await_completed(self, async_sdk_client: AsyncRunloopSDK)
         finally:
             await devbox.shutdown()
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_snapshot_status_tracking(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test tracking snapshot status through lifecycle."""
         devbox = await async_sdk_client.devbox.create(
@@ -199,7 +199,7 @@ async def test_snapshot_status_tracking(self, async_sdk_client: AsyncRunloopSDK)
 class TestAsyncSnapshotDevboxRestoration:
     """Test creating devboxes from snapshots asynchronously."""
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT * 2)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_restore_devbox_from_snapshot(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test creating a devbox from a snapshot and verifying state is restored."""
         # Create source devbox
@@ -242,7 +242,7 @@ async def test_restore_devbox_from_snapshot(self, async_sdk_client: AsyncRunloop
         finally:
             await source_devbox.shutdown()
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT * 2)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_multiple_devboxes_from_snapshot(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test creating multiple devboxes from the same snapshot."""
         # Create source devbox
@@ -300,7 +300,7 @@ async def test_multiple_devboxes_from_snapshot(self, async_sdk_client: AsyncRunl
 class TestAsyncSnapshotListing:
     """Test async snapshot listing and retrieval operations."""
 
-    @pytest.mark.timeout(THIRTY_SECOND_TIMEOUT)
+    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
     async def test_list_snapshots(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test listing snapshots."""
         snapshots = await async_sdk_client.snapshot.list(limit=10)
@@ -309,7 +309,7 @@ async def test_list_snapshots(self, async_sdk_client: AsyncRunloopSDK) -> None:
         # List might be empty, that's okay
         assert len(snapshots) >= 0
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_get_snapshot_by_id(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test retrieving snapshot by ID."""
         # Create a devbox and snapshot
@@ -336,7 +336,7 @@ async def test_get_snapshot_by_id(self, async_sdk_client: AsyncRunloopSDK) -> No
         finally:
             await devbox.shutdown()
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_list_snapshots_by_devbox(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test listing snapshots filtered by devbox."""
         # Create a devbox
@@ -370,7 +370,7 @@ async def test_list_snapshots_by_devbox(self, async_sdk_client: AsyncRunloopSDK)
 class TestAsyncSnapshotEdgeCases:
     """Test async snapshot edge cases and special scenarios."""
 
-    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT * 2)
+    @pytest.mark.timeout(FOUR_MINUTE_TIMEOUT)
     async def test_snapshot_preserves_file_permissions(self, async_sdk_client: AsyncRunloopSDK) -> None:
         """Test that snapshot preserves file permissions."""
         # Create devbox

tests/smoketests/sdk/test_sdk.py

@@ -8,13 +8,13 @@
 
 pytestmark = [pytest.mark.smoketest]
 
-FIVE_SECOND_TIMEOUT = 5
+THIRTY_SECOND_TIMEOUT = 30
 
 
 class TestRunloopSDKInitialization:
     """Test RunloopSDK client initialization and structure."""
 
-    @pytest.mark.timeout(FIVE_SECOND_TIMEOUT)
+    @pytest.mark.timeout(THIRTY_SECOND_TIMEOUT)
     def test_sdk_instance_creation(self, sdk_client: RunloopSDK) -> None:
         """Test that SDK instance is created successfully with all client properties."""
         assert sdk_client is not None
@@ -23,7 +23,7 @@ def test_sdk_instance_creation(self, sdk_client: RunloopSDK) -> None:
         assert sdk_client.snapshot is not None
         assert sdk_client.storage_object is not None
 
-    @pytest.mark.timeout(FIVE_SECOND_TIMEOUT)
+    @pytest.mark.timeout(THIRTY_SECOND_TIMEOUT)
     def test_legacy_api_access(self, sdk_client: RunloopSDK) -> None:
         """Test that legacy API client is accessible through sdk.api."""
         assert sdk_client.api is not None

tests/smoketests/sdk/test_snapshot.py

@@ -103,7 +103,7 @@ def test_snapshot_with_metadata(self, sdk_client: RunloopSDK) -> None:
         finally:
             devbox.shutdown()
 
-    @pytest.mark.timeout(THIRTY_SECOND_TIMEOUT)
+    @pytest.mark.timeout(TWO_MINUTE_TIMEOUT)
     def test_snapshot_delete(self, sdk_client: RunloopSDK) -> None:
         """Test deleting a snapshot."""
         devbox = sdk_client.devbox.create(