FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-131178: Add CLI tests for the profile module by b9788213 · Pull Request #143835 · python/cpython · GitHub

/ cpython Public

gh-131178: Add CLI tests for the profile module - #143835

Open
b9788213 wants to merge 1 commit into
python:mainfrom
b9788213:gh-131178-profile-cli-tests
Open

gh-131178: Add CLI tests for the profile module#143835
b9788213 wants to merge 1 commit into
python:mainfrom
b9788213:gh-131178-profile-cli-tests

Conversation

b9788213 commented Jan 14, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

This PR adds comprehensive command-line interface tests for the profile module to improve test coverage and ensure CLI stability, as part of the effort in gh-131178.

Changes

  • Added test for the help interface (-h, --help).
  • Added test for sorting using the -s option.
  • Added negative test for invalid CLI options

bedevere-app Bot added tests Tests in the Lib/test dir awaiting review labels Jan 14, 2026

aisk commented Jan 14, 2026

Copy link
Copy Markdown
Member

Hi, according to the Dev Guide, test only changes don't require a news entry file, please remove it.

aisk added the skip news label Jan 14, 2026
b9788213 force-pushed the gh-131178-profile-cli-tests branch from 74e9877 to 6360925 Compare January 14, 2026 15:47

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

github-actions Bot added the stale Stale PR or inactive for long period of time. label May 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review skip news stale Stale PR or inactive for long period of time. tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL