Sinon: हम एक नई त्रुटि वस्तु बनाने को अस्वीकार कैसे कर सकते हैं?

को निर्मित 1 फ़र॰ 2018 · 6टिप्पणियाँ · स्रोत: sinonjs/sinon

https://github.com/sinonjs/sinon/blob/4310343393b16b92b56526ab4614da4eb2886efa/lib/sinon/default-behaviors.js#L176

मैं कुछ कोड बनाए रख रहा हूं जो तारों को अस्वीकार करता है और यह परीक्षण करते समय समस्याएं पैदा कर रहा है। मेरे ब्लूबर्ड कैच सही ढंग से व्यवहार नहीं कर रहे हैं।

bluebird.rejects('Something')
    .catch(e => e === 'Something' /* true */, () => console.log(typeof e) /*string*/)
    .catch(() => console.log('should not be here'));

हालाँकि sinon.stub().usingPromise('bluebird').rejects('Something'); का उपयोग करने से कुछ नाम के साथ एक त्रुटि वस्तु बन जाती है। बिल्कुल अपेक्षित नहीं है।

इससे पहले कि कोई कहे कि हम केवल देशी वादों का समर्थन कर रहे हैं, आइए जाँच करें कि वे क्या करते हैं!

Promise.reject('Something')
     .catch(e => {
        console.log(e === 'Something'); // true
        console.log(typeof e); // string
    });

हम्म, वे एक त्रुटि वर्ग नहीं बनाते हैं, वे केवल उस अपवाद को टॉस करते हैं जो इसे फेंक की तरह दिया गया था।

try { 
    throw 'Something'; 
} catch (e) { 
    console.log(e === 'Something');  // true
    console.log(typeof e); // string
}

Bug Help wanted pinned

स्रोत

JemiloII

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

स्टैक ट्रेस उपयोगिता और चर्चाओं के बारे में कि क्या किसी को त्रुटि वस्तुओं को एक तरफ बनाना चाहिए, मैं सहमत हूं कि आम तौर पर

const example = sinon.stub().rejects(something);

जैसा व्यवहार करना चाहिए

const example = () => Promise.reject(something);

वर्तमान में, यह मामला नहीं है यदि something एक स्ट्रिंग है। मैं कहूंगा कि वर्तमान व्यवहार एक बग है।

mantoni 8 सित॰ 2020

👍4

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

इससे पहले कि कोई कहे कि हम केवल देशी वादों का समर्थन कर रहे हैं, आइए जाँच करें कि वे क्या करते हैं!

यह एक स्ट्रॉमैन झूठ है । आप अच्छी तरह से जानते हैं कि हम वादा पुस्तकालयों का समर्थन करते हैं, जैसा कि आप अपने उदाहरण में उनका उपयोग करते हैं।

सिनॉन अनुरक्षक पूरी तरह से इस बात से अवगत हैं कि जावास्क्रिप्ट कैसे काम करता है, जिसमें यह भी शामिल है कि Promise.reject() आप जो कुछ भी पास करते हैं उसे पास करेंगे।

मैं कुछ कोड बनाए रख रहा हूं जो तारों को अस्वीकार करता है

मेरी राय में, साथ वादों को खारिज String के बजाय Error वास्तव में भविष्य डेवलपर, जो डिबग करने के लिए है कि हम आज क्या लिखने के लिए एक अपकार कर रही है। स्ट्रिंग्स डिबगिंग के लिए बहुत उपयोगी नहीं हैं, क्योंकि वे स्टैक ट्रेस को कैप्चर नहीं करते हैं। दूसरी ओर Error करता है। इसलिए, मुझे लगता है कि rejects() व्यवहार वांछनीय है, क्योंकि इससे डीबगर का उपयोग करना आसान हो जाता है ताकि यह पता चल सके कि क्या गलत हुआ।

ऐसा लगता है कि आप असहमत हैं, लेकिन आपने जो एकमात्र बयान दिया है, वह यह है कि सिनॉन आपकी अपेक्षाओं को पूरा नहीं करता है। यदि आपको लगता है कि डिफ़ॉल्ट व्यवहार को बदला जाना चाहिए, तो कृपया इस बात को ध्यान में रखते हुए कि क्यों कई उपयोगकर्ताओं को प्रभावित करने की क्षमता है, इसके लिए तर्क पोस्ट करें।

यदि डिफ़ॉल्ट व्यवहार को बदलने के लिए ठोस तर्क हैं, तो मैं कल्पना नहीं कर सकता कि सिनॉन का कोई भी अनुरक्षक इसे बदलने के लिए पुल अनुरोध का विरोध करेगा।

mroderick 11 फ़र॰ 2018

👍1

इस समस्या को स्वचालित रूप से पुराने के रूप में चिह्नित कर दिया गया है क्योंकि इसमें हाल की गतिविधि नहीं है। आगे कोई गतिविधि नहीं होने पर इसे बंद कर दिया जाएगा। आपके योगदानों के लिए धन्यवाद।

stale[bot] 12 अप्रैल 2018

मैं कहूंगा कि कुछ परिदृश्यों में स्टैक ट्रेस वास्तव में उपयोगी है या नहीं, इस बारे में अलग-अलग राय हो सकती है। मेरी विशेष स्थिति में, मुझे वास्तव में स्टैक ट्रेस की परवाह नहीं है, मुझे केवल त्रुटि संदेश की परवाह है और फिर इसे कहीं और पास करना है। इसलिए मैंने स्ट्रिंग और त्रुटि प्रकार दोनों को अस्वीकार करने के कारणों को संभालने के लिए कोड लिखा है। हालांकि, चूंकि सिनॉन स्ट्रिंग करता है -> मेरे लिए त्रुटि ज़बरदस्ती, मैं अपने कोड की शाखा का परीक्षण नहीं कर सकता जो स्ट्रिंग्स को संभालती है। ऐसे में मुझे पूरा कवरेज नहीं मिल सकता

dtom90 8 सित॰ 2020

const example = sinon.stub().rejects(something);

जैसा व्यवहार करना चाहिए

const example = () => Promise.reject(something);

mantoni 8 सित॰ 2020

👍4

मैं सहमत हूं।

JemiloII 17 सित॰ 2020

stale[bot] 25 दिस॰ 2020

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

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

Sinon: हम एक नई त्रुटि वस्तु बनाने को अस्वीकार कैसे कर सकते हैं?

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

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

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