Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bpo-37758: Clean out vestigial script-bits from test_unicodedata.#15126

Merged
benjaminp merged 1 commit into
python:masterpython/cpython:masterfrom
gnprice:pr-unicodedata-spruce-testsgnprice/cpython:pr-unicodedata-spruce-testsCopy head branch name to clipboard
Aug 13, 2019
Merged

bpo-37758: Clean out vestigial script-bits from test_unicodedata.#15126
benjaminp merged 1 commit into
python:masterpython/cpython:masterfrom
gnprice:pr-unicodedata-spruce-testsgnprice/cpython:pr-unicodedata-spruce-testsCopy head branch name to clipboard

Conversation

@gnprice

@gnprice gnprice commented Aug 5, 2019

Copy link
Copy Markdown
Contributor

This file started life as a script, before conversion to a
unittest test file. Clear out some legacies of that conversion
that are a bit confusing about how it works.

Most notably, it's unlikely there's still a good reason to try
to recover from unicodedata failing to import -- as there was
when that logic was first added, when the module was very new.
So take that out entirely. Keep self.db working, though, to
avoid a noisy diff.

https://bugs.python.org/issue37758

This file started life as a script, before conversion to a
`unittest` test file.  Clear out some legacies of that conversion
that are a bit confusing about how it works.

Most notably, it's unlikely there's still a good reason to try
to recover from `unicodedata` failing to import -- as there was
when that logic was first added, when the module was very new.
So take that out entirely.  Keep `self.db` working, though, to
avoid a noisy diff.
@bedevere-bot bedevere-bot added tests Tests in the Lib/test dir awaiting review labels Aug 5, 2019
@benjaminp benjaminp merged commit def97c9 into python:master Aug 13, 2019
@gnprice gnprice deleted the pr-unicodedata-spruce-tests branch August 13, 2019 07:12
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
…thonGH-15126)

This file started life as a script, before conversion to a
`unittest` test file.  Clear out some legacies of that conversion
that are a bit confusing about how it works.

Most notably, it's unlikely there's still a good reason to try
to recover from `unicodedata` failing to import -- as there was
when that logic was first added, when the module was very new.
So take that out entirely.  Keep `self.db` working, though, to
avoid a noisy diff.
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
…thonGH-15126)

This file started life as a script, before conversion to a
`unittest` test file.  Clear out some legacies of that conversion
that are a bit confusing about how it works.

Most notably, it's unlikely there's still a good reason to try
to recover from `unicodedata` failing to import -- as there was
when that logic was first added, when the module was very new.
So take that out entirely.  Keep `self.db` working, though, to
avoid a noisy diff.
websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request Jul 20, 2020
…thonGH-15126)

This file started life as a script, before conversion to a
`unittest` test file.  Clear out some legacies of that conversion
that are a bit confusing about how it works.

Most notably, it's unlikely there's still a good reason to try
to recover from `unicodedata` failing to import -- as there was
when that logic was first added, when the module was very new.
So take that out entirely.  Keep `self.db` working, though, to
avoid a noisy diff.
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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.