rjgoyln opened a new pull request, #72701:
URL: https://github.com/apache/airflow/pull/72701

   ## Summary
   
   The operator test module has never run. Two failures stack up:
   
   - `itertools.count` is called without importing `itertools`, so collection 
fails and all 45 tests in the module error.
   - With the import in place, the timeout test's mocked clock jumps 20s per 
read against a 10s timeout, so the poll loop gives up before describing the 
service once and logs no status.
   
   Stepping the mocked clock by the operator's poll interval leaves room for 
exactly one poll — what the loop does between deadline checks on a real clock — 
so the test asserts the last observed status, as its name says. The operator 
needs no change: two real `monotonic()` reads are microseconds apart, so the 
pre-poll deadline check can only trip for a non-positive timeout.
   
   ## Tests
   
   65 passed across the three `snowpark_containers` test modules.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 5)
   
   Generated-by: Claude Code (Opus 5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to