Pip: "टेस्ट.लिब.क्रिएट_बासिक_व्हील_फॉरपैक" में संभावित बग

को निर्मित 16 अप्रैल 2020 · 7टिप्पणियाँ · स्रोत: pypa/pip

वातावरण

पाइप संस्करण: 20.1.dev0
पायथन संस्करण: 3.8.2
OS: OSX 10.15.4

विवरण

जब हेल्पर tests.lib.create_basic_wheel_for_package का उपयोग इकाई परीक्षण में किया जाता है, जिसमें - , जैसे simple-package वाले नाम का एक पहिया बनाया जाता है, और फिर पहिया स्थापित किया जाता है, तो पैकेज का नाम लिया जाता है simple और संस्करण package रूप में लिया जाता है

अपेक्षित् व्यवहार

PEP-491 फ़ाइल नाम सम्मेलन के अनुसार लगता है, पैकेज नाम में - नहीं हो सकते हैं, इसलिए परीक्षण उद्देश्यों के लिए पहियों को बनाते समय ऐसे पैकेज नामों की अनुमति नहीं दी जानी चाहिए। (पीईपी की मेरी समझ यहां भी गलत हो सकती है)

कैसे पुन: पेश करें

निम्नलिखित इकाई परीक्षण निष्पादित करें

def test_create_wheel_bug(script):

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

उत्पादन

परीक्षण एक अभिकथन त्रुटि के साथ विफल वजह से उत्पादन pip list --format=json सूची के रूप में पैकेज का नाम simple और के रूप में संस्करण package

 assert 'simple-package' in [{'name': 'pip', 'version': '20.1.dev0'}, 
{'name': 'setuptools', 'version': '46.1.3'}, {'name': 'simple', 'version': 'package'}]

tests auto-locked bug

स्रोत

deveshks

सबसे उपयोगी टिप्पणी

आह, यह समझ में आता है। अपराधी यह पंक्ति है:

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

uranusjr 16 अप्रैल 2020

👍3

सभी 7 टिप्पणियाँ

आह, यह समझ में आता है। अपराधी यह पंक्ति है:

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

uranusjr 16 अप्रैल 2020

👍3

मुझे लगता है कि सही दृष्टिकोण है, नाम canonicalize और बदलने के लिए - के साथ _ पहिया फ़ाइल करने से पहले। मुझे तुरंत इसके लिए एक फ़िक्स बनाना है और उसी टेस्ट का उपयोग करना है जो मैंने टिकट में लिखा था कि यह सत्यापित करें कि यह तय हो गया है :)

deveshks 16 अप्रैल 2020

मुझे बस एहसास हुआ कि पीईपी 491 में वास्तव में पहिया के फ़ाइल नाम के लिए पैकेज नाम को कैनोनिक रूप से बदलने के लिए एक रेगेक्स शामिल है। हो सकता है कि हम इसके बजाय (पीईपी के लिंक के साथ) का उपयोग करें; भविष्य के पाठक के लिए इसे समझना बहुत आसान होगा।

uranusjr 16 अप्रैल 2020

👀1 👍1

मुझे लगता है कि आप PEP-491 एस्केपिंग और यूनिकोड में re.sub("[^\w\d.]+", "_", distribution, re.UNICODE) मौजूद हैं?

तो हम उस regex का उपयोग करके एक सही व्हील फ़ाइल नाम बनाएंगे? क्या हम इस बदलाव के लिए पैकेज के फ़ोल्डर नाम के साथ भी कुछ करना चाहते हैं?

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#discussuss_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

deveshks 16 अप्रैल 2020

हाँ बस यही। उसी नाम का उपयोग पैकेज के लिए भी किया जा सकता है (मुझे नहीं लगता कि पैकेज वास्तव में वैसे भी बहुत सारे परीक्षणों द्वारा उपयोग किया जाता है)।

@pradyunsg क्या इसे एक परीक्षण की आवश्यकता है?

uranusjr 16 अप्रैल 2020

हाँ, मुझे लगता है कि test_lib.py में एक परीक्षण जोड़ना एक अच्छा विचार है।

pradyunsg 17 अप्रैल 2020

हाँ, मुझे लगता है कि test_lib.py में एक परीक्षण जोड़ना एक अच्छा विचार है।

तो वह परीक्षण क्या करेगा? क्या यह केवल डॉट, डैश और अंडरस्कोर के साथ पैकेज नामों के वेरिएंट को आज़माने के लिए पर्याप्त है, और सत्यापित करें कि सभी 3 मामलों के लिए व्हील फ़ाइल नाम सही है?

deveshks 17 अप्रैल 2020

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Pip: "टेस्ट.लिब.क्रिएट_बासिक_व्हील_फॉरपैक" में संभावित बग

सबसे उपयोगी टिप्पणी

सभी 7 टिप्पणियाँ

संबंधित मुद्दों