8000 gh-109162: Refactor libregrtest WorkerJob by vstinner · Pull Request #109171 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-109162: Refactor libreg 8000 rtest WorkerJob #109171

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
Merged
merged 1 commit into from
Sep 9, 2023
Merged

gh-109162: Refactor libregrtest WorkerJob #109171

merged 1 commit into from
Sep 9, 2023

Conversation

vstinner
Copy link
Member
@vstinner vstinner commented Sep 9, 2023 •
edited by bedevere-bot
Loading

  • Rename --worker-args command line option to --worker-json.

  • Rename _parse_worker_args() to _parse_worker_json().

  • WorkerJob:

    • Add runtests attribute
    • Remove test_name and rerun attribute

  • Rename run_test_in_subprocess() to create_worker_process().

  • Rename run_tests_worker() to worker_process().

  • create_worker_process() uses json.dump(): write directly JSON to stdout.

  • Convert MultiprocessResult to a frozen dataclass.

  • Rename RunTests.match_tests to RunTests.match_tests_dict.

@vstinner
pythongh-109162: Refactor libregrtest WorkerJob
ed3bc63 
* Rename --worker-args command line option to --worker-json.
* Rename _parse_worker_args() to _parse_worker_json().
* WorkerJob:

  * Add runtests attribute
  * Remove test_name and rerun attribute

* Rename run_test_in_subprocess() to create_worker_process().
* Rename run_tests_worker() to worker_process().
* create_worker_process() uses json.dump(): write directly JSON to
  stdout.
* Convert MultiprocessResult to a frozen dataclass.
* Rename RunTests.match_tests to RunTests.match_tests_dict.
@bedevere-bot bedevere-bot mentioned this pull request Sep 9, 2023
Closed
@vstinner vstinner enabled auto-merge (squash) September 9, 2023 00:35
@vstinner vstinner merged commit a56c928 into python:main Sep 9, 2023
@vstinner vstinner deleted the regrtest_refactor4 branch September 9, 2023 01:03
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
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vstinner @bedevere-bot
0