Skip to content

gh-37883: Safely skip test_resource file size tests when limits are strict #145579

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
encukou merged 20 commits into from
Apr 8, 2026
Merged

gh-37883: Safely skip test_resource file size tests when limits are strict #145579

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
3cae226
gh-37883: Safely skip test_resource file size tests when limits are s…
Shrey-N Mar 6, 2026
f217fb1
📜🤖 Added by blurb_it.
blurb-it[bot] Mar 6, 2026
ef28dbc
Update News Entry
Shrey-N Mar 6, 2026
f74f58e
Update 2026-03-06-09-43-49.gh-issue-37883.I-3oZq.rst
Shrey-N Mar 8, 2026
d21ceb8
Refactor test_resource.py on the advice
Shrey-N Mar 10, 2026
65b0b88
fix lint error
Shrey-N Mar 10, 2026
1600c31
Update test_resource.py
Shrey-N Mar 10, 2026
7fb6b72
Fix formatting in test_resource.py
Shrey-N Mar 10, 2026
29255c2
Update test_resource.py
Shrey-N Mar 11, 2026
3dc4653
Update test_resource.py
Shrey-N Mar 15, 2026
de3c792
Fix indentation in test_resource.py
Shrey-N Mar 15, 2026
61822dd
Remove unused 'time' import from test_resource.py
Shrey-N Mar 15, 2026
3b02180
Delete Misc/NEWS.d/next/Tests/2026-03-06-09-43-49.gh-issue-37883.I-3o…
Shrey-N Mar 15, 2026
befd67a
Update test_resource.py
Shrey-N Mar 15, 2026
16acaa4
Update test_resource.py
Shrey-N Mar 15, 2026
4f7a349
Refactor resource limit tests for clarity and cleanup
Shrey-N Mar 29, 2026
4564e1f
Merge branch 'main' into fixy-branch
encukou Apr 1, 2026
1808a8a
Update based on suggestion
Shrey-N Apr 7, 2026
6faed32
Update Lib/test/test_resource.py based on suggestion
Shrey-N Apr 7, 2026
58fb908
Update Lib/test/test_resource.py
Shrey-N Apr 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update based on suggestion 
Co-authored-by: Petr Viktorin <encukou@gmail.com>
  • Loading branch information
@Shrey-N @encukou
Shrey-N and encukou authored Apr 7, 2026
commit 1808a8a38f75d4f4aeb662824c24bd7614784872
1 change: 1 addition & 0 deletions Lib/test/test_resource.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ def test_fsize_enforced(self):
# Close will attempt to flush the byte we wrote
# Restore limit first to avoid getting a spurious error
resource.setrlimit(resource.RLIMIT_FSIZE, (cur, max_lim))
Comment thread
Shrey-N marked this conversation as resolved.

@unittest.skipIf(sys.platform == "vxworks",
"setting RLIMIT_FSIZE is not supported on VxWorks")
@unittest.skipUnless(hasattr(resource, 'RLIMIT_FSIZE'), 'requires resource.RLIMIT_FSIZE')
Expand Down
Loading