Skip to content

test_exceptions.ExceptionTests.testMemoryErrorBigSource fails on certain platforms #110800

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

Jump to bottom
Closed
FFY00 opened this issue Oct 13, 2023 · 1 comment
Closed

test_exceptions.ExceptionTests.testMemoryErrorBigSource fails on certain platforms #110800

FFY00 opened this issue Oct 13, 2023 · 1 comment
Labels
tests Tests in the Lib/test dir type-bug An unexpected behavior, bug, or error

Comments

@FFY00
Copy link
Member

FFY00 commented Oct 13, 2023 •
edited by bedevere-app bot
Loading

Bug report

Bug description:

Example: https://buildbot.python.org/all/#/builders/1056/builds/3329/steps/10/logs/stdio

======================================================================
FAIL: testMemoryErrorBigSource (test.test_exceptions.ExceptionTests.testMemoryErrorBigSource)
----------------------------------------------------------------------
OverflowError: cannot fit 'int' into an index-sized integer
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/opt/buildbot/bcannon-wasm/3.x.bcannon-wasm.emscripten-node-dl/build/Lib/test/test_exceptions.py", line 322, in testMemoryErrorBigSource
    with self.assertRaisesRegex(OverflowError, "column offset overflow"):
AssertionError: "column offset overflow" does not match "cannot fit 'int' into an index-sized integer"
----------------------------------------------------------------------
Ran 95 tests in 1.969s

This seems to be due to the PySequence_Repeat taking a Py_ssize_t, which is platform dependent.

CPython versions tested on:

CPython main branch

Operating systems tested on:

No response

Linked PRs
@FFY00 FFY00 added type-bug An unexpected behavior, bug, or error tests Tests in the Lib/test dir labels Oct 13, 2023
FFY00 added a commit to FFY00/cpython that referenced this issue Oct 13, 2023
@FFY00
pythonGH-110800: fix unexpected OverflowError in testMemoryErrorBigSo…
6918768 
…urce

Signed-off-by: Filipe Laíns <[email protected]>
@bedevere-app bedevere-app bot mentioned this issue Oct 13, 2023
Closed
@FFY00 FFY00 mentioned this issue Oct 13, 2023
Merged
@AlexWaygood
Copy link
Member

Duplicate of #110767

@AlexWaygood AlexWaygood marked this as a duplicate of #110767 Oct 13, 2023
@AlexWaygood AlexWaygood closed this as not planned Won't fix, can't repro, duplicate, stale Oct 13, 2023
FFY00 added a commit to FFY00/cpython that referenced this issue Oct 13, 2023
@FFY00
Merge remote-tracking branch 'origin/main' into pythongh-110800
2faae5e 
Signed-off-by: Filipe Laíns <[email protected]>
FFY00 added a commit to FFY00/cpython that referenced this issue Oct 13, 2023
@FFY00
Merge remote-tracking branch 'origin/main' into pythongh-110800
45c3392 
Signed-off-by: Filipe Laíns <[email protected]>
lysnikolaou added a commit to FFY00/cpython that referenced this issue Oct 13, 2023
@lysnikolaou
Merge branch 'main' into pythongh-110800
6626787
FFY00 added a commit to FFY00/cpython that referenced this issue Oct 13, 2023
@FFY00
pythonGH-110800: fix unexpected OverflowError in testMemoryErrorBigSo…
f932c96 
…urce

Signed-off-by: Filipe Laíns <[email protected]>

add the suggested test markers

Signed-off-by: Filipe Laíns <[email protected]>

fix incorrect logic

Signed-off-by: Filipe Laíns <[email protected]>

add _size arg that got lost in the merge

Signed-off-by: Filipe Laíns <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tests Tests in the Lib/test dir type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@FFY00 @AlexWaygood