Pip: 「tests.lib.create_basic_wheel_for_package」の潜在的なバグ

ああ、これは理にかなっています。 犯人はこの行です：

archive_name = "{}-{}-py2.py3-none-any.whl".format(name, version)

これはsimple-package-1.0-py2.py3-none-any.whlになりますが、ホイールの仕様ではsimple_package-1.0-py2.py3-none-any.whlです。

修正も簡単なはずです。

archive_name = "{}-{}-py2.py3-none-any.whl".format(
    canonicalize_name(name).replace('-', '_'),
    version,
)

ホイールファイルを作成する前に、名前を正規化し、 -を_に置き換えるのが正しいアプローチだと思います。 すぐに修正を作成し、チケットに書き込んだのと同じテストを使用して、修正されたことを確認します:)

PEP 491には​​、実際には、ホイールのファイル名のパッケージ名を正規に変換するための正規表現が含まれていることに気付きました。 代わりにそれを使用する必要があるかもしれません（PEPへのリンク付き）。 将来の読者にとっては、はるかに理解しやすいでしょう。

PEP-491 EscapingとUnicodeにre.sub("[^\w\d.]+", "_", distribution, re.UNICODE)存在するという意味だと思いますか？

では、その正規表現を使用して正しいホイールファイル名を作成しますか？ また、この変更のために、パッケージのフォルダー名でも何かをしたいですか？

https://github.com/pypa/pip/blob/2f3a1be1185e3434c0c8d9cc58d4271beff1d122/tests/lib/__init__.py#L1000

また、これの単体テストは、 https：//github.com/pypa/pip/pull/8054で説明したものと同様ですか？ （https://github.com/pypa/pip/pull/8054#discussion_r409654545としてcanonicalize_nameを使い始めるまで、 pkg_resources.safe_name見つかった癖のため、以下が機能しない可能性があることに注意してください）

@pytest.mark.parametrize(
    'package_name',
    ['simple-package', 'simple_package', 'simple.package'],
)
def test_create_wheel_bug(script):

    package = create_basic_wheel_for_package(script, package_name, '1.0')
    script.pip("install", "--no-cache-dir", "--no-index", package)
    result = script.pip('list', '--format=freeze')
    assert package_name in result.stdout

はい、それだけです。 パッケージにも同じ名前を使用できます（とにかく、パッケージが実際に多くのテストで使用されているとは思いません）。

@pradyunsgこれにはテストが必要ですか？

ええ、test_lib.pyにテストを追加するのは良い考えだと思います。

ええ、test_lib.pyにテストを追加するのは良い考えだと思います。

では、そのテストは何をするのでしょうか？ ドット、ダッシュ、アンダースコアを使用してパッケージ名のバリエーションを試して、ホイールファイル名が3つのケースすべての仕様に従って正しいことを確認するだけで十分ですか？