Sinon: जासूस `रिटर्नवैल्यू` जनरेटर के साथ काम नहीं करता है

सिनॉन अभी तक ES6 सुविधाओं का समर्थन नहीं करता है। इस पर अभी काम शुरू नहीं हुआ है इसलिए पता नहीं यह कब काम करेगा।

@mantoni , ठीक है, यह एक फीचर अनुरोध से अधिक था :) क्या आप इसे खुला छोड़ना चाहते हैं, उदाहरण के लिए, किसी प्रकार के टैग ("es6"?) - मूल्यांकन करें कि क्या करने की आवश्यकता है?

यह एक बड़ी विशेषता होगी।

@ruimarinho मुझे यह मिला https://github.com/ingameio/SinonES6.JS
आवश्यकता ("sinon") को आवश्यकता ("sinon-es6") के साथ बदलने से आपके परीक्षण पास हो जाते हैं

--
22 जून 2017 को @ fatso83 द्वारा संपादित करें:
यह सच नहीं लगता। मैंने मैन्युअल रूप से इसका परीक्षण किया। यह सभी समान रूप से विफल रहता है, जो कुल समझ में आता है क्योंकि sinon-es6 _ केवल _ का मजाक उड़ाने के लिए जनरेटर के लिए अपना समर्थन लागू करता है।

क्या कोई इस पर काम कर रहा है?

@emgeee ऐसा नहीं है कि मुझे पता है।

+1 @ruimarinho sinon.js es6 सुविधाओं के साथ होना बहुत अच्छा होगा!

@ingameio अपने परिवर्तनों के साथ पीआर बनाने की परवाह करता है?

@ fatso83 सुझाव के लिए धन्यवाद!
मैं जल्द ही पीआर करूँगा;)

पैकेज्ड संस्करण के विरुद्ध बस्टर-टेस्ट चलाते समय मुझे समस्या आ रही है। मैंने कुछ डिबगिंग की लेकिन मेरे पास इस दिन समय की कमी है इसलिए मैं इस मुद्दे को हल नहीं कर सका।
अगर कोई मदद करना चाहता है तो मैं अपने कांटे में बदलावों को आगे बढ़ा सकता हूं, बस मुझे बताएं और मैं इसे करूंगा।

@gaguirre यह अच्छी खबर है। मैक्सिमिलियन ने अभी-अभी सभी परीक्षणों को मोचा में परिवर्तित कर दिया है, और इस दौरान परीक्षण सेटअप में कुछ बदलाव किए हैं, इसलिए यदि आप master से नवीनतम परिवर्तनों को शामिल करते हैं तो हो सकता है कि चीजें काम करें?

@gaguirre अगर किसी को इस धागे पर ठोकर खानी चाहिए, तो मुझे लगता है कि किसी भी तरह से आपके कांटे में बदलाव को धक्का देना एक विचार हो सकता है ताकि एक राहगीर इसे देख सके।

@ fatso83 मैंने पहले संस्करण को अंडरस्कोप/सिनॉन पर धकेल दिया।
मुख्य समस्या तब होती है जब एक इंजन के साथ परीक्षण चलाते हैं जो es6 (इस मामले में प्रेत) का समर्थन नहीं करता है: यह कोड को पार्स करते समय विफल हो जाता है इसलिए मैं eval() का उपयोग वर्कअराउंड के रूप में कर रहा हूं क्योंकि एक कोशिश से घिरा हो सकता है/ पकड़। मुझे लगता है कि यह स्वीकार्य नहीं है लेकिन यह एक शुरुआती बिंदु है।

अभी मैं जेनरेटर कॉल को किसी अन्य फ़ाइल में लपेटने की कोशिश कर रहा हूं और इसे गतिशील रूप से आवश्यकता है, लेकिन यह काम नहीं कर रहा है, मुझे लगता है क्योंकि ब्राउज़र को गतिशील रूप से आवश्यक फाइलों को भी बंडल कर रहा है। मैं सोच रहा हूं कि npm run test-headless चलाते समय कुछ फाइलों को बाहर रखा जा सकता है।

आपको क्या लगता है कि सबसे अच्छा समाधान क्या हो सकता है?

प्रतिक्रिया के लिए धन्यवाद, @gaguirre । तो मूल मुद्दा यह है कि अगर इंजन इसका समर्थन नहीं करता है तो हमें पार्सिंग से बचने का कोई तरीका चाहिए। यह साधारण फीचर डिटेक्शन की तुलना में थोड़ा कठिन है, जैसे हम वेबवर्कर्स आदि के साथ करते हैं।

केवल वास्तविक जांच को प्रत्यायोजित करना उतना ही सरल है जितना if(require('generator-support')){ ... } , लेकिन यह वास्तविक पार्सिंग समस्या को ठीक नहीं करता है।

@mantoni और @mrodrick से कैसे सफाई से निपटना है, इस पर कोई विचार?

पुनश्च मैं कुछ घंटों में ऑफ़लाइन-अवकाश पर हूं, इसलिए ईस्टर के कुछ समय बाद तक उत्तर पढ़ने में सक्षम नहीं होगा।

मैं यह नहीं बता सकता कि जेनरेटर स्टबिंग का यह स्तर वास्तव में काम करता है (विशेष रूप से कोरआउटिन-उन्मुख परिदृश्यों में) - ऐसा लगता है कि एसिंक व्यवहार का कुछ अनुकरण क्रम में हो सकता है। लेकिन मुझे लगता है कि पार्सिंग मुद्दे के आसपास एक रास्ता है।

सुझाव :

जेनरेटर कोड को एक अलग फ़ाइल में निकालें और require() उस फ़ाइल को केवल मांग पर जेनरेटर फ़ंक्शन को लपेटते समय निकालें, यानी:

?/es6-support.js:

"use strict";

exports.getGeneratorWrapper = function(method, ctx, args) {
    return function* () {
        return mockObject.invokeMethod(method, ctx, args);
    }
}

lib/sinon/mock.js->उम्मीद...:

var wrapper = function () {
    return mockObject.invokeMethod(method, this, arguments);
});

if (/^function\s*\*/.test(method.toString())) {
    wrapper = require('es6-support').getGeneratorWrapper(method, this, arguments);
}

wrapMethod(this.object, method, wrapper);

(इकाई परीक्षणों के साथ भी ऐसा ही करें, और सुनिश्चित करें कि कोड कवरेज की ट्रैकिंग 'es6-support' फ़ाइल को स्वचालित रूप से शामिल करने का कारण नहीं बनती है।)

वैकल्पिक सुझाव :

जब तक sinon 2.0 रिलीज के लिए तैयार नहीं हो जाता, क्या ES5 संगतता अभी भी एक सार्थक उद्देश्य होने जा रही है? शायद अब कुछ लोगों को यह बताने का समय है कि वे अभी भी ट्रांसपिलर के उपयोगकर्ता के बिना विरासत ES5-only वातावरण का समर्थन कर रहे हैं कि वे 1.x पर पीछे रह जाएंगे।

@ इवान-किंग सशर्त आवश्यकता कोई समाधान नहीं है, क्योंकि यह हमारे ब्राउज़र के निर्माण में टूट जाएगा। चूंकि सशर्त आवश्यकता निर्भरता ग्राफ के स्थिर विश्लेषण को हरा देती है, इसलिए वेबपैक, रोलअप और ब्राउज़रिफ़ जैसे उपकरण इससे निपटने में असमर्थ होंगे। यह या तो पूरी तरह से अंदर है या पूरी तरह से बाहर है।

और हाँ, ES5 संगतता एक चीज है। ES6 जनरेटर समर्थन सबसे अच्छा है, और लोगों को अपनी परियोजनाओं के लिए सिनॉन का उपयोग करने के लिए अतिरिक्त टूलिंग का उपयोग करने के लिए मजबूर करना परीक्षण करने के लिए बार बढ़ाएगा। और कई लोगों के लिए, वह दहलीज काफी अधिक है। सरल करने में सक्षम होने के कारण इसे काम करने के लिए एक स्क्रिप्ट टैग शामिल करना IMHO की एक महत्वपूर्ण विशेषता है। हम कुछ समय में ES5 संगतता तोड़ सकते हैं, लेकिन यह हमारे रोडमैप पर नहीं है और सिनॉन 3 के लिए भी चर्चा नहीं की गई है। सिनॉन 2 काफी समय से बाहर है; बस कुछ छोटी-छोटी परेशानियाँ हैं जो हमें आधिकारिक रिलीज़ करने से रोक रही हैं।

मौजूदा ES5 रनटाइम संगतता को तोड़े बिना ES6 संगतता प्राप्त करने के मूल रूप से दो तरीके हैं जो मैं ith पर आ सकता हूं, और मैं वास्तव में पिछले एक के बारे में निश्चित नहीं हूं (इसे परीक्षण नहीं किया है):

मॉड्यूल का सशर्त मूल्यांकन

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

यह मानते हुए कि brfs जैसी कोई चीज़ मौजूद है, कोड कुछ इस तरह दिखाई देगा:

_रनटाइम-फीचर्स.जेएस_

var features = {};
try { 
    new Function("var foo = function* foo(){ }") ;
    features.generatorSupport = true; 
} 
catch(err){ features.generatorSupport = false; }
module.exports = features;

_es6-जनरेटर-आवरण.js_

return function* () {
    return mockObject.invokeMethod(method, ctx, args);
}

_es6.js_

var features = require('./runtime-features');

if( features.generatorSupport ) {
    var code = fs.readFileSync('./es6-generator-wrapper.js');
    module.exports.getGeneratorWrapper = new Function("mockObject", "method", "ctx", "args", code);
} else {
    module.exports.getGeneratorWrapper = function() { throw new TypeError("You tried using a generator function in a runtime only capable of running ES5 code"); }
}

बेबेलिफाइंग सिनोन

यह वह है जिसके बारे में मुझे यकीन नहीं है क्योंकि मैंने वास्तव में कोशिश नहीं की है। यदि हम बिल्ड को बैबेल के माध्यम से ES5 में ट्रांसपाइल करते हैं, तो हम सभी जगह ES6 कोड लिख सकते हैं, हुप्स के माध्यम से कूदने से बच सकते हैं जैसे मैंने ऊपर किया था, और हम अभी भी जनरेटर के लिए उसी प्रकार के चेक का उपयोग कर सकते हैं। उन्हें सिर्फ ES5 निर्माणों का उपयोग करके लागू किया जाएगा। बेशक, ES5 ब्राउज़र में ES6 का परीक्षण अभी भी विफल होगा। यह क्लाइंट साइड पर पिछले वाले के समान ही है, लेकिन हम योगदान में बाधा डाल सकते हैं क्योंकि yield , async , function*() जैसे सामान पर ES2015 ज्ञान पहुंच से बहुत दूर है। एक व्यापक दर्शक।

+1

@rpavlovs , थ्रेड में +1 जोड़ने का वास्तव में कोई मतलब नहीं है। यदि आपको अपनी भावनाओं को व्यक्त करने की आवश्यकता है, तो GitHub UI में प्रत्येक टिप्पणी के शीर्ष पर "प्रतिक्रिया जोड़ें" बटन है। एक +1 कुछ नहीं करेगा। एक पुल अनुरोध जो ऊपर दिए गए सुझावों में से एक को लागू करता है (या कुछ और चालाक), दूसरी ओर, इस मुद्दे को ठीक करने का एक बेहतर मौका है

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

इसे शुरू करने के लिए, मैंने एक नई शाखा बनाई है जिसमें नकली एपीआई में @ingameio के संशोधन हैं, जबकि साथ ही ES5 संगतता (ऊपर उल्लिखित हैक का उपयोग करके) को तोड़ना नहीं है।

जो बात मुझे थोड़ी परेशान करती है, वह यह है कि मैं वास्तव में यह नहीं बता सकता कि इनगैमियो द्वारा मूल परिवर्तनों का परीक्षण कैसे किया जाए, क्योंकि कोई भी परीक्षण उदाहरण काम नहीं करता है - यहां तक ​​कि कांटे में उदाहरण भी पूरा नहीं हुआ है, और मुझे पूर्व/पोस्ट को तोड़ने के लिए सामान नहीं मिल सकता है परिवर्तन।

जेनरेटर सरल चीजें हैं: कोमल तुल्यकालिक प्राणी जो अपने अतीत को याद करते हैं। तो कृपया किसी भी उदाहरण को co और अन्य चीजों के साथ पहेली न करें जो असंबंधित हैं, क्योंकि इससे यह देखना कठिन हो जाता है कि क्या वांछित है/काम नहीं करता है। उदाहरण के लिए शीर्ष उदाहरण काफी जटिल है, और यह भी गलती लगता है कि yield क्या करता है, क्योंकि यह "उपज अभिव्यक्ति" के वापसी मूल्य को "उपज मूल्य" के समान होने की अपेक्षा करता है। उपज का result जनरेटर के next() ( MDN ) में दिया गया मान है

मुझे co का उपयोग करने वाले उदाहरणों का एहसास है शायद इसका उपयोग केवल मोचा परीक्षण में yield का उपयोग करने में सक्षम होने के लिए करता है, लेकिन अपने उदाहरण को आईआईएफई या इसे प्राप्त करने के किसी अन्य तरीके में लपेटें अधिक स्पष्टता के लिए।

यह एक सरल परीक्षण है कि कैसे जनरेटर समर्थन करते हैं (आज के सिनॉन में काम करता है):

require("should");

var sinon = require("../sinon");

var foo = {
    bar: function () {
        return "bar";
    },
    biz: function *() {
        return "biz";
    }
};

describe("generator support", function () {
    it('should work with generator functions',  function(){
        var spy = sinon.spy(foo, 'biz');

        var iterator = foo.biz();
        var result = iterator.next();

        result.value.should.equal('biz');
        result.done.should.equal(true);
        spy.firstCall.returnValue.should.be.an.Object();
        spy.firstCall.returnValue.next.should.be.a.Function();
    });
});

अब, हम कौन से API एक्सटेंशन चाहते हैं?
मूल परीक्षण से, मुझे लगता है कि हम कुछ इस तरह देखना चाहेंगे

foo.biz.firstGeneratedValue.should.equal('biz');
या
foo.biz.generatedValue[0].should.equal('biz');
?

सीसी @ruimarinho

मैं इस मुद्दे को बंद कर रहा हूं, क्योंकि मूल परीक्षण में एक त्रुटि थी, और मैं सिनॉन में जनरेटर से निपटने के साथ कोई समस्या नहीं देख सकता।

कृपया इस चर्चा में शामिल हों कि जेनरेटर (और इससे जुड़े इटरेटर्स) से निपटने के लिए एक एपीआई #1467 अंक में कैसे दिखेगा