Sinon: स्टबिंग सेट डिफ़ॉल्ट रूप से तत्काल अवांछित व्यवहार की ओर जाता है

को निर्मित 7 जन॰ 2016 · 4टिप्पणियाँ · स्रोत: sinonjs/sinon

मैंने यह पता लगाने की कोशिश करते हुए देखा कि सिस्टम के समय को कम करने से सीएसवी लेखन विफल क्यों हो गया। यह पता चला है कि पुस्तकालय में एक setImmediate कॉल था जो अवरुद्ध कर रहा था: https://github.com/C2FO/fast-csv/blob/master/lib/extended.js#L18

ऐसा लगता है कि पुस्तकालयों का एक समूह है जो अहानिकर चीजों के लिए setImmediate पर निर्भर हैं, और यह कि setImmediate का ढोंग करने से उन सभी में अप्रत्याशित व्यवहार होता है:

अनुरोध: https://github.com/sinonjs/sinon/issues/484
async: https://github.com/caolan/async/issues/609
एक्सप्रेस.जेएस: https://github.com/strongloop/express/issues/2447
ब्लूबर्ड: http://stackoverflow.com/q/31403818/329700
सह: https://github.com/sinonjs/sinon/issues/527

शायद यह अपेक्षा करना अनुचित है कि सिस्टम समय को कम करने से सीएसवी लेखन विफल नहीं होता है? इस मामले में इसे ट्रैक करना विशेष रूप से कठिन है, क्योंकि जावास्क्रिप्ट में आप केवल ctrl+c हिट नहीं कर सकते हैं और एक स्टैक ट्रेस प्राप्त कर सकते हैं जो आपको दिखा रहा है कि क्या गलत हुआ। कोड का कौन सा हिस्सा फ्रीज हो रहा था, यह पहचानने में और फिर यह समझने में लगभग 90 मिनट लग गए कि सिनॉन जिम्मेदार क्यों था।

मुझे यकीन नहीं है कि फिक्स क्या होना चाहिए, लेकिन ऐसा लगता है कि यह बहुत सारे पुस्तकालयों को प्रभावित कर रहा है, और उनमें से कम से कम एक ने setImmediate का अपना संस्करण बनाकर प्रतिक्रिया दी है जिसे उलझाया नहीं जा सकता है।

अगर जवाब है कि मुझे useFakeTimers(0, "Date") पर कॉल करना चाहिए, तो शायद यह डिफ़ॉल्ट होना चाहिए? हो सकता है कि यह मेरी/लोगों की अपेक्षाओं के साथ बेहतर ढंग से फिट बैठता हो कि दुनिया कैसे व्यवहार करती है?

स्रोत

kevinburkeshyp

👍1

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

भविष्य के संदर्भ के लिए, फ़ंक्शन का यह रूप अब आपको यह निर्दिष्ट करने की अनुमति देता है कि कौन से फ़ंक्शन नकली हैं:

var clock = sinon.useFakeTimers([now, ]prop1, prop2, ...);

Sets the clock start timestamp and names functions to fake.

Possible functions are setTimeout, clearTimeout, setInterval, clearInterval, setImmediate, clearImmediate and Date. Can also be called without the timestamp.

तो एक्सप्रेस के साथ काम करते समय इसने मेरे लिए समस्या हल कर दी:

clock = sinon.useFakeTimers(1495020000, 'Date');

neocolmartin 19 मई 2017

👍3 ❤1

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

मुझे यकीन नहीं है कि फिक्स क्या होना चाहिए, लेकिन ऐसा लगता है कि यह बहुत सारे पुस्तकालयों को प्रभावित कर रहा है, और उनमें से कम से कम एक ने सेटइमीडिएट का अपना संस्करण बनाकर प्रतिक्रिया दी है जिसे उलझाया नहीं जा सकता है।

जब आप सेटटाइमआउट का उपयोग कर रहे हों, तो यह अनुशंसित समाधान है, एक नया कॉल स्टैक प्राप्त करने के प्रभाव के लिए तत्काल सेट करें (और समय को नियंत्रित करने के लिए नहीं)।

https://github.com/taylorhakes/promise-polyfill/pull/15

mroderick 7 जन॰ 2016

हां, मैंने व्यावसायिक कोडबेस में इसका निदान और समाधान करने में अभी 90 मिनट बिताए हैं।

कम से कम, दस्तावेज़ों को उल्लेख करना चाहिए कि सेट तत्काल नकली हो जाएगा।

NealEhardt 24 फ़र॰ 2017

var clock = sinon.useFakeTimers([now, ]prop1, prop2, ...);

Sets the clock start timestamp and names functions to fake.

Possible functions are setTimeout, clearTimeout, setInterval, clearInterval, setImmediate, clearImmediate and Date. Can also be called without the timestamp.

तो एक्सप्रेस के साथ काम करते समय इसने मेरे लिए समस्या हल कर दी:

clock = sinon.useFakeTimers(1495020000, 'Date');

neocolmartin 19 मई 2017

👍3 ❤1

दस्तावेज़ीकरण अब इसकी व्याख्या करता है: http://sinonjs.org/releases/v2.3.2/fake-timers/#var -clock--sinonusefaketimersnow-prop1-prop2-

mroderick 27 मई 2017

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

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

Sinon: स्टबिंग सेट डिफ़ॉल्ट रूप से तत्काल अवांछित व्यवहार की ओर जाता है

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

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

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