Pip: Bogue potentiel dans "tests.lib.create_basic_wheel_for_package"

Ah, cela a du sens. Le coupable est cette ligne:

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

ce qui donnerait simple-package-1.0-py2.py3-none-any.whl , mais la spécification de roue attend simple_package-1.0-py2.py3-none-any.whl .

Le correctif devrait également être simple:

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

Je pense que c'est la bonne approche, pour canoniser le nom et remplacer - par _ avant de créer le fichier wheel. Laissez-moi créer un correctif pour cela tout de suite et utiliser le même test que j'ai écrit dans le ticket pour vérifier qu'il a été corrigé :)

Je viens de réaliser que PEP 491 inclut en fait une regex pour transformer canoniquement un nom de package pour le nom de fichier d'une roue. Peut-être devrions-nous utiliser cela à la place (avec un lien vers le PEP); ce serait beaucoup plus facile à comprendre pour un futur lecteur.

Je pense que vous voulez dire re.sub("[^\w\d.]+", "_", distribution, re.UNICODE) présent à PEP-491 Escaping et Unicode ?

Nous créerions donc un nom de fichier de roue correct en utilisant cette expression régulière? Souhaitons-nous également faire quelque chose avec le nom de dossier du package pour ce changement?

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

Le test unitaire pour cela sera-t-il également similaire à ce que nous avons discuté dans https://github.com/pypa/pip/pull/8054 ? (Notez que ce qui suit peut ne pas fonctionner en raison de la bizarrerie que nous avons trouvée avec pkg_resources.safe_name jusqu'à ce que nous commencions à utiliser canonicalize_name comme 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

Oui c'est ça. Le même nom peut également être utilisé pour le paquet (je ne pense pas que le paquet soit réellement utilisé par beaucoup de tests de toute façon).

@pradyunsg Cela nécessite-t-il un test?

Oui, je pense que l'ajout d'un test dans test_lib.py est une bonne idée.

Oui, je pense que l'ajout d'un test dans test_lib.py est une bonne idée.

Alors, que fera ce test? Est-il suffisant d'essayer simplement des variantes de noms de packages avec un point, un tiret et un trait de soulignement, et de vérifier que le nom du fichier de roue est correct selon les spécifications pour les 3 cas?