Fix flaky test_serde_serialize_recursion_limit in task-sdk#61181
Closed
Eason09053360 wants to merge 5 commits into
Closed
Fix flaky test_serde_serialize_recursion_limit in task-sdk#61181Eason09053360 wants to merge 5 commits into
Eason09053360 wants to merge 5 commits into
Conversation
- Create PluginsPage page object following POM pattern - Add test specs for plugins list display and pagination - Tests work across Chromium, Firefox, and WebKit browsers - All 24 tests passing Fixes apache#60571
… currently fails in CI due to test environment having only 7 plugins, which is insufficient to trigger pagination UI controls. Seeking guidance on whether to keep failing test or remove it.
Eason09053360
requested review from
amoghrajesh,
ashb,
choo121600,
kaxil and
vatsrahul1001
as code owners
boring-cyborg
Bot
added
area:task-sdk
area:UI
Contributor
Author
|
Closing this PR in favor of a cleaner version to avoid mixing unrelated commits. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR fixes a flaky unit test
test_serde_serialize_recursion_limitintask-sdk.The test was previously using
sys.getrecursionlimit() - 1to simulate the recursion depth. However, this system limit (often 1000) does not match the internalMAX_RECURSION_DEPTHconstant (which is 10) used in theserializefunction.Because of this mismatch, the
depthcheck inserializepassed (since 999 != 10), causing the function to attempt to serialize the emptyobject(), resulting in aTypeErrorinstead of the expectedRecursionError.Verification
Ran the test locally with
pytestand it passed:Changes:
MAX_RECURSION_DEPTHfromairflow.sdk.serde.serde.depthparameter to correctly trigger theRecursionError.closes: #51915