Sinon: स्पाई का यह वैल्यू कंस्ट्रक्टर की जासूसी करते समय उदाहरण की ओर इशारा नहीं करता है

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

यह स्निपेट सिनॉन 2.0.0 के साथ ठीक काम करता है लेकिन सिनॉन 4.2.2 के साथ विफल रहता है:

it('returns correct thisValue', function() {
    window.SomeClass = function() {
        this.counter = 0;
        this.inc = function() {
            this.counter++;
        };
        this.getCounter = function() {
            return this.counter;
        };
    };

    var spy = sinon.spy(window, 'SomeClass');
    var some = new window.SomeClass();
    some.inc();

    expect(spy.thisValues[0].counter).toEqual(1);
    expect(spy.thisValues[0].getCounter()).toEqual(1);
    some.inc();
    expect(spy.thisValues[0].counter).toEqual(2);
    expect(spy.thisValues[0].getCounter()).toEqual(2);
});

ऐसा लगता है कि thisValues[0] ऑब्जेक्ट एक प्रॉक्सी है और some के वास्तविक उदाहरण को संदर्भित नहीं करता है।

यह सिनॉन 4.2.2 के साथ त्रुटि है:

Expected undefined to equal 1
TypeError: undefined is not a function (evaluating 'spy.thisValues[0].getCounter()')

Bug Medium Help wanted Regression hacktoberfest pinned

स्रोत

PVince81

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

भले ही यह आमतौर पर उपयोग नहीं किया जाता है, या कम से कम आमतौर पर रिपोर्ट नहीं किया जाता है, मैं इस प्रतिगमन को ठीक करना चाहता हूं।

mroderick 9 अग॰ 2018

👍3

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

यह व्यवहार बदलने के समय को स्थापित करने के लिए git bisect का उपयोग करने के लिए एक अच्छा उम्मीदवार है, जो कुछ सुराग दे सकता है कि परिवर्तन जानबूझकर (एक सुविधा) था या नहीं (एक बग)।

उदाहरण @PVince81 को एक छोटी परीक्षण स्क्रिप्ट में git bisect run लिए किया जा सकता है।

यदि आप git bisect का उपयोग करना नहीं जानते हैं, तो यह स्तर बढ़ाने का एक शानदार अवसर है!

http://www.marclittlemore.com/how-to-find-bugs-using-git-bisect-with-this-easy-guide/

mroderick 11 फ़र॰ 2018

ओह, मुझे द्विभाजित करना पसंद है! मैं इसका ख्याल रखूंगा ..

PVince81 11 फ़र॰ 2018

😄2

यहां हम जाते हैं: 911c498dc14dc4034ba019526bf58f8b24d77da0 जासूस new (#1626) के साथ कॉल करके गुजरता है

PVince81 11 फ़र॰ 2018

👍3

ऐसा करने के लिए धन्यवाद, @PVince81! एक फिक्स # 1265 का भी निरीक्षण करना चाह सकता है, क्योंकि यह स्पर्श करता है कि कंस्ट्रक्टर जासूसी का समर्थन करने के लिए किन परिवर्तनों की आवश्यकता है।

fatso83 12 फ़र॰ 2018

जब कोई वस्तु new के साथ नहीं बनाई जाती है, तो इस विधि को कहा जाता है, जो thisValue की सामग्री को बदल देती है और विस्तार से thisValues । इसके विपरीत, जब नया प्रयोग किया जाता है तो विधि thisValue अपरिवर्तित रहती है।

rgroothuijsen 17 मार्च 2018

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

ऐसा नहीं है कि कई लोगों ने इस मुद्दे के बारे में शिकायत की है, शायद लोग आमतौर पर इस तरह से परीक्षण नहीं करते हैं और एक फिक्स इसके लायक नहीं है (केवल एक अवलोकन, अनुरक्षकों को तय करने के लिए)

PVince81 8 अग॰ 2018

mroderick 9 अग॰ 2018

👍3

एक ही समस्या का सामना करना पड़ा, क्या इसे ठीक करने की कोई योजना है?

ivan-zakharchuk 30 अग॰ 2019

@ ivan-zakharchuk यदि कोई स्वयंसेवा नहीं कर रहा है, तो कोई योजना नहीं है, नहीं। यदि आप इसे ठीक करना चाहते हैं, तो एक स्वाभाविक अगला कदम उठाया जा सकता है ;-)

यह आपत्तिजनक प्रतिबद्धता है: https://github.com/sinonjs/sinon/issues/1683#issuecomment -364794930

fatso83 30 अग॰ 2019

बस कहने के लिए, मैंने इसके लिए एक फिक्स लिखने की कोशिश में काफी समय बिताया है, लेकिन असफल रहा हूं। यह this मूल्यों का एक बुरा सपना है। मूल समस्या यह प्रतीत होती है कि यहां दिए गए फ़ंक्शन में new स्वाभाविक रूप से एक नया this बनाता है, जबकि thisValues[0] (जैसा कि मूल पोस्ट में उपयोग किया गया है) this को संदर्भित करता है new द्वारा बनाया गया (यानी var some = new window.SomeClass(); ) - मुझे पता है कि बहुत भ्रमित है।

अनिवार्य रूप से, इसे ठीक करने का अर्थ है इसे दोनों तरीकों से करने का प्रयास करना - new को कॉल करना बिल्कुल नया this , लेकिन हमें इसे दो बार कॉल करना होगा (यानी ओपी के परीक्षण में से एक और proxy-invoke.js ) ES6 को संतुष्ट करने के लिए। तो आप हमेशा दो this es के साथ समाप्त होने जा रहे हैं। केवल एक चीज जिसके बारे में मैं सोच सकता था, वह थी कार्यों को एक this से दूसरे पर बांधना, लेकिन इसके लिए भी वह बहुत हैक था और कुछ अन्य चीजों को तोड़ दिया।

मुझे आशा है कि उपर्युक्त समझ में आता है, कृपया स्पष्टीकरण को सही/अनुरोध करने के लिए स्वतंत्र महसूस करें। एक पुनश्चर्या के लिए यहां देखें कि new कीवर्ड वास्तव में क्या करता है - TLDR यह एक नई वस्तु बनाता है और इसे this बनाता है, जो भी संदर्भ हो।

किसी भी तरह, एक बात मैंने नोटिस की थी कि 911c498 (प्रतिगमन की शुरुआत करने वाली प्रतिबद्धता) में उस मुद्दे को कवर करने वाला परीक्षण शामिल नहीं है - ES6 कक्षाओं के साथ new का उपयोग करने की आवश्यकता। ऐसा शायद इसलिए है क्योंकि परीक्षणों के लिए लाइनिंग विकल्प class कीवर्ड की अनुमति नहीं देते हैं क्योंकि वे केवल ES5 की पूर्ति करते हैं। मेरे पास एक परीक्षण जोड़ने की प्रतिबद्धता है ताकि वर्तमान मुद्दे के लिए भविष्य में कोई भी फिक्स 911c498 में फिक्स को अन-फिक्स न करे - यदि रुचि हो तो मैं इसे पीआर कर सकता हूं।

gwpmad 12 अप्रैल 2020

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

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

Sinon: स्पाई का यह वैल्यू कंस्ट्रक्टर की जासूसी करते समय उदाहरण की ओर इशारा नहीं करता है

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

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

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