Jest: वैश्विक `जेस्ट-क्ली` चलाना त्रुटि के बजाय चेतावनी देना चाहिए

को निर्मित 14 जुल॰ 2016 · 3टिप्पणियाँ · स्रोत: facebook/jest

जेस्ट के वैश्विक रूप से स्थापित संस्करण को चलाने पर process.exit(1) कॉल ट्रिगर हो जाता है जिसके परिणामस्वरूप विषम परीक्षा परिणाम मिलते हैं - यहां तक कि जब सभी परीक्षण पास हो जाते हैं, तो प्रक्रिया एक गैर-शून्य निकास कोड लौटाती है, और यह तुरंत स्पष्ट नहीं है कि क्यों।

जब jest को npm test (जो एक बहुत ही सामान्य परंपरा है) को सौंपा जाता है, तो आपको मिलता है:

Please run `npm install` to use the version of Jest intended for this project.
...
163 tests passed (163 total in 30 test suites, run time 3.292s)
npm ERR! Test failed.  See above for more details.

कोई स्पष्ट विवरण नहीं है ("कृपया चलाएं ..." लाइन एक त्रुटि संदेश की तरह नहीं दिखती है") और कोई असफल परीक्षण नहीं है, इसलिए डिबगिंग अजीब है।

सिफारिशों

मैं पूरी तरह से getJest से जेस्ट के पैकेज-स्थानीय संस्करण के लिए परीक्षण को हटाना पसंद करूंगा। अगर लोग npm install चलाए बिना जेस्ट के गैर-स्थानीय संस्करण का उपयोग करना चाहते हैं, तो मुझे यकीन नहीं है कि पैकेज को क्यों ध्यान रखना चाहिए?
यदि परीक्षण बने रहने के लिए पर्याप्त उपयोगी है, तो इसे निकास कोड को प्रभावित किए बिना console.warn उपयोग करना चाहिए क्योंकि परीक्षण अभी भी बिना किसी परेशानी के चल सकते हैं, और चेतावनी पर्याप्त संकेत है कि आउटपुट ठीक उसी तरह प्रतिबिंबित नहीं हो सकता है जो उपयोगकर्ता चाहता था।
यदि एक गैर-शून्य निकास कोड लौटाना पसंद किया जाता है, तो परीक्षण को वास्तविक Error फेंकना चाहिए ताकि डिबगिंग को आसान बनाया जा सके - नोड की अंतर्निहित त्रुटि प्रबंधन console.error तुलना में कहीं अधिक स्पष्ट है और उसके बाद process.exit(1) ।
यदि आप वास्तव में कोई त्रुटि नहीं फेंकना चाहते हैं, तो सभी परीक्षणों के चलने की प्रतीक्षा करने के बजाय तुरंत process.exit(1) कॉल करें - यदि स्थानीय रूप से परिभाषित निर्भरता गायब होने पर जेस्ट को विश्व स्तर पर चलाना ठीक नहीं है, तो इसे करना चाहिए बिल्कुल नहीं भागो।

स्रोत

mmcgahan

👍3 ❤1

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

@mmcgahan @cpojer यार्न वर्कस्पेस मोनोरेपो के भीतर चाइल्ड पैकेज के अंदर जेस्ट चलाते समय मुझे यह गैर-घातक त्रुटि दिखाई दे रही है। ऐसा लगता है कि शायद इसे गलती से दिखाया जा रहा है क्योंकि जिस तरह से आप jest की तलाश कर रहे हैं, वह मोनोरेपो रूट तक फहराए जा रहे पैकेजों से निपट नहीं रहा है?

jedrichards 13 दिस॰ 2017

👍7

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

इसे लाने और मुद्दे के विस्तृत विवरण के लिए धन्यवाद। सच कहूं तो, मैं खुद इस सामान को लेकर परेशान रहा हूं और यह निश्चित रूप से जेस्ट के टेस्ट सूट को निपटने के लिए थोड़ा कठिन बना देता है।

मैं इस बात से भी सहमत हूं कि जिस तरह से यह काम कर रहा है उसका कोई मतलब नहीं है अगर जेस्ट अभी भी चल रहे परीक्षणों को समाप्त करता है। मुझे खेद है कि आपने इस पर समय बर्बाद किया।

क्या आप पुल अनुरोध भेजने के इच्छुक होंगे?

मेरा सुझाव इस प्रकार है:

process.exit(1) निकालें।
process.on('exit', () => console.log(chalk.red('The error message '))) ताकि टेस्ट रन के अंत में हमें एक अच्छा, लाल संदेश मिले जो आपको बताए कि कुछ वैसा नहीं है जैसा उसे होना चाहिए।

यह लोगों को जेस्ट के गलत संस्करण का उपयोग करने की अनुमति देगा (जिसकी अनुशंसा नहीं की जाती है, क्योंकि ब्रेकिंग परिवर्तन हो सकते हैं) लेकिन यह अभी भी परीक्षण को ठीक से पूरा करेगा और नीचे एक संदेश प्रिंट करेगा। तुम क्या सोचते हो?

cpojer 14 जुल॰ 2016

@cpojer जो मेरे लिए काम करता है, त्वरित प्रतिक्रिया के लिए धन्यवाद! शीघ्र ही पीआर प्रस्तुत करेंगे

mmcgahan 14 जुल॰ 2016

jedrichards 13 दिस॰ 2017

👍7

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

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

Jest: वैश्विक `जेस्ट-क्ली` चलाना त्रुटि के बजाय चेतावनी देना चाहिए

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

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

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