The GitHub Actions job "Tests" on airflow.git/fix-triggerer-comms-race has 
succeeded.
Run started by GitHub user ashb (triggered by ashb).

Head commit for run:
8b11ca0661978cabb3064c1809f670af5a42d9d9 / Ash Berlin-Taylor <[email protected]>
Remove defunct tests from Task SDK

Because the mock forced `send()` to `raise RuntimeError("You cannot use
AsyncToSync...")` directly — simulating what async_to_sync used to raise.
After our change, that exception falls through to except Exception: return
None.

But that RuntimeError can no longer actually occur in production. With the new
`send()`, when called from the event-loop thread with a portal it goes through
`greenback.await_()z successfully — it never raises that error in the first
place. The mock was simulating a scenario that no longer exists, so the test
was faithfully testing dead code.

Report URL: https://github.com/apache/airflow/actions/runs/25390242944

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to