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 and 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 에서 논의한 것과 유사합니까? ( canonicalize_name 를 https://github.com/pypa/pip/pull/8054#discussion_r409654545로 사용하기 시작할 때까지 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에 테스트를 추가하는 것이 좋은 생각이라고 생각합니다.

그렇다면 그 테스트는 무엇을할까요? 점, 대시 및 밑줄이있는 패키지 이름의 변형을 시험해보고 휠 파일 이름이 세 가지 경우 모두의 사양에 따라 올바른지 확인하는 것으로 충분합니까?