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,
)

Я думаю, что это правильный подход - канонизировать имя и заменить - на _ перед созданием файла wheel. Позвольте мне сразу исправить это и использовать тот же тест, который я написал в заявке, чтобы убедиться, что он исправлен :)

Я только что понял, что PEP 491 на самом деле включает регулярное выражение для канонического преобразования имени пакета в имя файла колеса. Возможно, нам следует использовать это вместо этого (со ссылкой на PEP); это будет намного легче понять будущему читателю.

Я думаю, вы имеете в виду re.sub("[^\w\d.]+", "_", distribution, re.UNICODE) присутствующие в PEP-491 Escaping и Unicode ?

Итак, мы могли бы создать правильное имя файла колеса, используя это регулярное выражение? Также хотим ли мы что-нибудь сделать с именем папки пакета для этого изменения?

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

Также будет ли модульный тест для этого аналогичен тому, что мы обсуждали в https://github.com/pypa/pip/pull/8054 ? (Обратите внимание, что приведенное ниже может не работать из-за причуды, которую мы обнаружили с pkg_resources.safe_name пока мы не начнем использовать canonicalize_name как https://github.com/pypa/pip/pull/8054#discussion_r409654545)

@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 случаев?