Skip to content

Add tests for the default request timeout#2845

Merged
adamtheturtle merged 1 commit intomainfrom
adamtheturtle/test-default-timeout
Feb 15, 2026
Merged

Add tests for the default request timeout#2845
adamtheturtle merged 1 commit intomainfrom
adamtheturtle/test-default-timeout

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 15, 2026

Summary

Test plan

  • New tests pass locally (16/16)
  • CI passes

🤖 Generated with Claude Code


Note

Low Risk
Test-only changes that don’t modify runtime behavior; low risk aside from potential test flakiness around timing assumptions.

Overview
Adds new parametrized tests asserting the default ~30s HTTP request timeout behavior for both CloudRecoService.query and VWS.add_target.

The tests use MockVWS with an artificial response delay and freezegun to advance time, verifying a 29s delay succeeds while a 31s delay raises requests.exceptions.Timeout.

Written by Cursor Bugbot for commit 2d64771. This will update automatically on new commits. Configure here.

Test that VWS and CloudRecoService requests succeed at 29 seconds
and time out at 31 seconds, verifying the existing default timeout
behavior.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@adamtheturtle adamtheturtle merged commit 69fac31 into main Feb 15, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the adamtheturtle/test-default-timeout branch February 15, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments