Skip to content

Conversation

JanEricNitschke
Copy link
Contributor

@JanEricNitschke JanEricNitschke commented Sep 8, 2025

Previously:

Name             Stmts   Miss Branch BrPart  Cover   Missing
------------------------------------------------------------
Lib/difflib.py     677     19    328     21    96%   523, 697, 699, 865, 867, 871, 885-886, 974, 978, 1061, 1150->1157, 1170->1162, 1239->1246, 1252->1258, 1261->1238, 1263->1262, 1447, 1514-1516, 1526->1532, 1814-1815, 1981->1984, 2090-2091
------------------------------------------------------------
TOTAL              677     19    328     21    96%



Now:

Name             Stmts   Miss Branch BrPart  Cover   Missing
------------------------------------------------------------
Lib/difflib.py     677     12    328     16    97%   871, 885-886, 974, 978, 1061, 1150->1157, 1170->1162, 1239->1246, 1252->1258, 1261->1238, 1263->1262, 1447, 1514-1516, 1526->1532, 1814-1815, 1981->1984
------------------------------------------------------------
TOTAL              677     12    328     16    97%


I thought about also testing the "pat" argument for IS_LINE_JUNK, but decided against it because it is completely undocumented.

Additionally i considered adding the raising tests to the doctests, but decided against that to keep them concise.

@bedevere-app bedevere-app bot added awaiting review tests Tests in the Lib/test dir labels Sep 8, 2025
@bedevere-app

This comment was marked as resolved.

Copy link
Member

@encukou encukou left a comment

Choose a reason for hiding this comment

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

I'd add a comment to avoid implying that users can rely on opcodes being the same object.

Otherwise, this looks great! Thank you!

@encukou encukou merged commit 4499161 into python:main Sep 12, 2025
43 checks passed
@encukou encukou added the needs backport to 3.14 bugs and security fixes label Sep 12, 2025
@miss-islington-app
Copy link

Thanks @JanEricNitschke for the PR, and @encukou for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

@encukou encukou added the needs backport to 3.13 bugs and security fixes label Sep 12, 2025
@miss-islington-app
Copy link

Thanks @JanEricNitschke for the PR, and @encukou for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 12, 2025
(cherry picked from commit 4499161)

Co-authored-by: Jan-Eric Nitschke <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 12, 2025
(cherry picked from commit 4499161)

Co-authored-by: Jan-Eric Nitschke <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Sep 12, 2025

GH-138817 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Sep 12, 2025
@bedevere-app
Copy link

bedevere-app bot commented Sep 12, 2025

GH-138818 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Sep 12, 2025
gpshead pushed a commit that referenced this pull request Sep 14, 2025
…8818)

gh-138669: Increase test coverage for difflib (GH-138670)
(cherry picked from commit 4499161)

Co-authored-by: Jan-Eric Nitschke <[email protected]>
hugovk pushed a commit that referenced this pull request Sep 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants