Skip to content

Add tool for easier test updates #6089

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
youknowone merged 34 commits into from
Sep 7, 2025
Merged

Add tool for easier test updates #6089

Changes from 1 commit
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
7c0a8fe
Base version of automated test marker
ShaharNaveh Aug 11, 2025
69ed76b
Remove debug code
ShaharNaveh Aug 11, 2025
dc3a6de
Add newline
ShaharNaveh Aug 11, 2025
5c09ef0
Apply RustPython patch
ShaharNaveh Aug 11, 2025
ad061b2
Convert to tool with args
ShaharNaveh Aug 20, 2025
a6f2324
Apply patch
ShaharNaveh Aug 22, 2025
302b3d1
Remove old script
ShaharNaveh Aug 22, 2025
db74d2f
Add textwrap.py
ShaharNaveh Aug 22, 2025
11694e3
ruff fmt
ShaharNaveh Aug 22, 2025
d8b4e26
Add more modules
ShaharNaveh Aug 22, 2025
2308a5a
Merge remote-tracking branch 'upstream/main' into auto-updater
ShaharNaveh Aug 25, 2025
2fb6842
Add `gen` subcommand
ShaharNaveh Aug 29, 2025
db5eb4b
Use `_generate_next_value_`
ShaharNaveh Aug 29, 2025
af1c28d
Gen & patch
ShaharNaveh Aug 30, 2025
42365d2
Remove old tool
ShaharNaveh Aug 30, 2025
fdce40b
Merge remote-tracking branch 'upstream/main' into auto-updater
ShaharNaveh Aug 30, 2025
32baa80
Revert changes under `Lib/`
ShaharNaveh Aug 30, 2025
fb7324d
Don't crash if cls renamed/moved
ShaharNaveh Aug 30, 2025
f4056ac
Update `Lib/test/test_os.py` with tool
ShaharNaveh Aug 30, 2025
4296b59
apply patch
ShaharNaveh Aug 30, 2025
e2aa220
Fix double assignment
ShaharNaveh Aug 30, 2025
74b47a0
Better args
ShaharNaveh Aug 30, 2025
51c6ad9
Update `test_list.py` as well
ShaharNaveh Aug 30, 2025
01a90ef
Less complex print
ShaharNaveh Aug 30, 2025
a0e56ae
Improve exoectedFailure match
ShaharNaveh Aug 30, 2025
43a63a8
fix list slice
ShaharNaveh Aug 30, 2025
3e2c1f1
Add __doc__ and to help
ShaharNaveh Sep 2, 2025
789cf6e
Merge remote-tracking branch 'upstream/main' into auto-updater
ShaharNaveh Sep 2, 2025
3e9872c
Update scripts/lib_updater.py
ShaharNaveh Sep 4, 2025
d0763e4
Clearer output arg
ShaharNaveh Sep 5, 2025
e00bb28
Don't crash on missing id
ShaharNaveh Sep 5, 2025
8a5875e
Merge remote-tracking branch 'upstream/main' into auto-updater
ShaharNaveh Sep 5, 2025
6c615cc
Merge remote-tracking branch 'origin/auto-updater' into auto-updater
ShaharNaveh Sep 5, 2025
c0e90cc
Fix comment regex
ShaharNaveh Sep 6, 2025
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
Add newline
  • Loading branch information
@ShaharNaveh
ShaharNaveh committed Aug 22, 2025
commit dc3a6de1f7319212aceaf60a561a0aa51d4b1f4d
1 change: 1 addition & 0 deletions tools/test_marker/main.py
View file
Open in desktop
Copy link

@reactive-firewall reactive-firewall Aug 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ShaharNaveh, Overall LGTM! With the caveat, that as a POC, it has lots of potential for improvements.

Most of my inline comments are off-the-cuff, I recognize this is a proof-of-concept and I like the concept, so feel-free to dismiss any comments that are off-topic without response.

Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,7 @@ def iter_patches(tree: ast.Module, conf: Conf) -> "Iterator[tuple[int, str]]":
if isinstance(dec_node, (ast.Attribute, ast.Call))
):
continue

lineno = min(
(dec_node.lineno for dec_node in fn_node.decorator_list),
default=fn_node.lineno,
Expand Down