Skip to content

Test infrastructure upgrade to 3.13.2#5517

Draft
arihant2math wants to merge 20 commits intoRustPython:mainfrom
arihant2math:importlib-313
Draft

Test infrastructure upgrade to 3.13.2#5517
arihant2math wants to merge 20 commits intoRustPython:mainfrom
arihant2math:importlib-313

Conversation

@arihant2math
Copy link
Contributor

@arihant2math arihant2math commented Feb 13, 2025 •
edited
Loading

  • Importlib to 3.13.2
  • test support to 3.13.2
  • libregrtest to 3.13.2
  • added _colorize superseded by Add _colorize at 3.13.2 #5520
  • test_csv to 3.13.2
  • test_cmd_line to 3.13.2

Rustpython issues I found

  • libregrtest has an unsupported match statement
  • what is implemented of match statements is very broken, it managed to hang itself
  • certain classes in libregrtest (like State), would cause a panic when used normally:

panic!("The symbol '{name}' must be present in the symbol table, even when it is undefined in python.");
from compile.rs.

  • Another different panic
>>>  cargo run -r -- -m test -v test_cmd_line
AttributeError: module 'sys' has no attribute 'stderr'
thread 'main' panicked at vm\src\vm\mod.rs:355:30:
essential initialization failed
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

@arihant2math arihant2math changed the title Importlib upgrade to 3.13.2 Test infrastructure upgrade to 3.13.2 Feb 13, 2025
@arihant2math arihant2math marked this pull request as draft February 13, 2025 18:09
@arihant2math arihant2math force-pushed the importlib-313 branch 2 times, most recently from 3112e7a to 512594b Compare February 13, 2025 18:43
@youknowone
Copy link
Member

youknowone commented Feb 14, 2025

#5500

This was referenced Feb 14, 2025
Merged
Closed
@arihant2math
importlib to 3.13.2
0d088cb 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
inspect function shim
9166b80 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
support and libregrtest to 3.13
eda5805 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
trace.py revert
cb6329e 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
fix file that was accidentally edited
4a299c9 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
skip hanging tests
9ad3cd1 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
some-what function multithreading for the test runner
812a32d 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
update test_csv to 3.13.2
1c77678 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
get_c_recursion_limit fix
766a85d 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
removed requires_legacy_unicode_capi
ceddb17 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
update test_cmd_line.py to 3.13.2
697a847 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
fix hang
152bda9
@arihant2math
fix test_http_cookies
d36db76 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
fix test_generators
74beb02 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
fix exceeds_recursion_limit
4577088 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
fix stackoverflow tests
ad6ab5a 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
mark failure on test
3f9198f 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
change recursion tests and disabled it at the root
5339f29 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
test fixes
8a50e44 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
don't skip check_free_after_iterating
d399da9 
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
@arihant2math
Copy link
Contributor Author

arihant2math commented Feb 20, 2025 •
edited
Loading

Current todo list before this can be retried/merged:

  • Fix cargo run -r -- -m test . on windows (it fails right now).
  • Incrementally update test_support to 3.13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arihant2math @youknowone