Skip to content

Conversation

@github-actions
Copy link

The tests was overusing mocking so the error was hidden!

I found that while migrating ti.run() to Task SDK execution path.
(cherry picked from commit e52e8ac)

Co-authored-by: Kaxil Naik kaxilnaik@gmail.com

@potiuk potiuk marked this pull request as ready for review May 21, 2025 15:59
@potiuk potiuk requested review from amoghrajesh, ashb and kaxil as code owners May 21, 2025 15:59
…0898)

The tests was overusing mocking so the error was hidden!

I found that while migrating `ti.run()` to Task SDK execution path.
(cherry picked from commit e52e8ac)

Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
@kaxil kaxil force-pushed the backport-e52e8ac-v3-0-test branch from 24655ff to 6b03645 Compare May 21, 2025 16:04
@potiuk potiuk merged commit ef45aff into v3-0-test May 22, 2025
51 checks passed
@potiuk potiuk deleted the backport-e52e8ac-v3-0-test branch May 22, 2025 11:42
kaxil added a commit that referenced this pull request Jun 3, 2025
…0898) (#50903)

The tests was overusing mocking so the error was hidden!

I found that while migrating `ti.run()` to Task SDK execution path.
(cherry picked from commit e52e8ac)

Co-authored-by: Kaxil Naik <kaxilnaik@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants