Skip to content

gh-131178: Add tests for \encodings.rot_13\ command-line interface - #159299

Closed
Bhumika-1432006 wants to merge 3 commits into
python:mainfrom
Bhumika-1432006:gh-131178-add-encodings-rot_13-cli-tests
Closed

Bhumika-1432006 wants to merge 3 commits into
python:mainfrom
Bhumika-1432006:gh-131178-add-encodings-rot_13-cli-tests

Conversation

@Bhumika-1432006

@Bhumika-1432006 Bhumika-1432006 commented Oct 13, 2026 •

Copy link
Copy Markdown

Add Rot13CommandLineTest class to Lib/test/test_codecs.py with
subprocess-based tests for python -m encodings.rot_13.

The tests cover:

  • Basic stdin-to-stdout ROT-13 encoding
  • ROT-13 being its own inverse (applying twice returns original text)
  • Non-alphabetic characters pass through unchanged

Fixes: #131178

@python-cla-bot

python-cla-bot Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app bedevere-app Bot added the type-feature A feature request or enhancement label Oct 13, 2026
@bedevere-app bedevere-app Bot mentioned this pull request Oct 13, 2026
7 of 9 tasks
@bedevere-app bedevere-app Bot added awaiting review tests Tests in the Lib/test dir labels Oct 13, 2026
@bedevere-app

bedevere-app Bot commented Oct 13, 2026

Copy link
Copy Markdown

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@Bhumika-1432006
Bhumika-1432006 force-pushed the gh-131178-add-encodings-rot_13-cli-tests branch from 4491ac9 to ff77d95 Compare October 13, 2026 09:24
@Bhumika-1432006
Bhumika-1432006 force-pushed the gh-131178-add-encodings-rot_13-cli-tests branch from 7e84658 to ae2ca28 Compare October 21, 2026 18:26
@Bhumika-1432006

Copy link
Copy Markdown
Author

All CI checks are passing. Could someone please review? cc @sobolevn @picnixz

@StanFromIreland

Copy link
Copy Markdown
Member

Sorry, but since the CLI is just the bare encoder, I don't think this is worth it.

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

Labels

awaiting review tests Tests in the Lib/test dir type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add tests for command-line interface

2 participants