Skip to content

gh-151526: Add csv writer tests for quote style edge cases#151527

Open
doma17 wants to merge 1 commit into
python:mainfrom
doma17:gh-csv-quote-style-coverage
Open

gh-151526: Add csv writer tests for quote style edge cases#151527
doma17 wants to merge 1 commit into
python:mainfrom
doma17:gh-csv-quote-style-coverage

Conversation

@doma17

@doma17 doma17 commented Jun 16, 2026

Copy link
Copy Markdown

Adds focused csv.writer coverage for QUOTE_STRINGS and QUOTE_NOTNULL.

The existing test_csv coverage already checks basic output for these quote styles. This adds narrower edge-case coverage for:

  • rejecting quotechar=None when those quote styles enable quoting
  • preserving quote/escape handling with escapechar and doublequote=False

This helps lock down behavior around the newer quoting modes and gives alternative implementations more precise regression coverage.

Testing:

  • ./python.exe -m test test_csv -v
  • ./python.exe -m test test_csv -q

@python-cla-bot

python-cla-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app

bedevere-app Bot commented Jun 16, 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.

@aisk aisk added the skip news label Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants