Mopidy: परीक्षण के बाहर प्रयुक्त कथन का दावा करें

मैंने इसे एक अच्छे पहले मुद्दे के रूप में चिह्नित किया है। तत्काल समाधान से बदलना है:

assert condition, "Error message"

प्रति:

if not condition:
    raise AssertionError("error message")

यह कोड को ठीक उसी तरह व्यवहार करता है जैसा वह आज करता है, लेकिन -O के साथ पायथन चलाते समय इसे अनुकूलित नहीं किया जाएगा।

@jodal , क्या मुझे इसे हल करना चाहिए या अन्य पहली बार योगदानकर्ताओं को हल करने देना चाहिए?

इसके अलावा, यदि आप चाहते हैं कि फ्रेशर्स इसे हल करें तो मुझे आपसे एक सुझाव की आवश्यकता है, अर्थात, यदि हम इस मुद्दे को कई अन्य मुद्दों में तोड़ दें तो क्या होगा? मेरा मतलब फाइलों के हिसाब से मामले को तोड़ना है।

यदि आप रुचि रखते हैं तो इससे निपटने के लिए स्वतंत्र महसूस करें!

यदि यह समस्या के बहुत से उदाहरण थे, तो कोडबेस के क्षेत्र द्वारा इसे कई पुल अनुरोधों में विभाजित करने का अर्थ हो सकता है। हालांकि, ऐसा लगता है कि ऐसे कई उदाहरण नहीं हैं, इसलिए मेरा सुझाव है कि सभी मामलों को ठीक करने के लिए केवल एक ही पीआर बनाएं।

परीक्षण को छोड़कर, प्रति फ़ाइल assert की संख्या:

$ rg -c assert mopidy
mopidy/ext.py:3
mopidy/core/actor.py:1
mopidy/core/tracklist.py:8
mopidy/audio/utils.py:1
mopidy/core/playback.py:1

अरे @pnijhara @jodal मैं इस मुद्दे पर काम करना चाहता हूं। क्या आप कृपया मुझे यह निर्दिष्ट कर सकते हैं क्योंकि मैं पहली बार योगदानकर्ता हूं।

https://docs.mopidy.com/en/latest/contributing/ और https://docs.mopidy.com/en/latest/devenv/ पर भी जानकारी है कि कैसे एक विकास वातावरण स्थापित करें और परीक्षण चलाएं और लिंटर आदि