Sinon: 스파이 `returnValue`는 생성기와 함께 작동하지 않습니다.

Sinon은 아직 ES6 기능을 지원하지 않습니다. 이 작업은 아직 시작되지 않았으므로 언제 작동할지 모르겠습니다.

@mantoni , 맞습니다. 이것은 기능 요청에 더 가깝습니다. 예를 들어 로드맵에 추가될 때 쉽게 다시 만들 수 있도록 일종의 태그("es6"?)와 함께 열어 두시겠습니까? -해야 할 일을 평가하는가?

이것은 좋은 기능이 될 것입니다.

@ruimarinho 내가 이것을 찾았습니다 https://github.com/ingameio/SinonES6.JS
require("sinon")을 require("sinon-es6")으로 바꾸면 테스트를 통과하는 것 같습니다.

--
2017년 6월 22일 @fatso83 의 편집:
이것은 사실이 아닌 것 같습니다. 나는 이것을 수동으로 테스트했다. sinon-es6 _만 조롱 을 위한 생성기에 대한 지원을 구현하므로 완전히 의미가 있습니다.

이 작업을 하는 사람이 있습니까?

@emgeee 내가 알고있는 것이 아닙니다.

+1 es6 기능이 있는 @ruimarinho sinon.js가 있으면 정말 좋을 것입니다!

@ingameio 가 변경 사항으로 PR을 하시겠습니까?

@fatso83 제안 감사합니다!
곧 홍보하겠습니다 ;)

패키지 버전에 대해 buster-test를 실행할 때 문제가 있습니다. 디버깅을 좀 해보았지만 요즘 시간이 없어서 문제를 풀지 못했습니다.
누군가가 변경 사항을 내 포크로 푸시할 수 있도록 돕고 싶다면 저에게 말씀해 주십시오.

@gaguirre 반가운 소식입니다. Maximillian은 방금 모든 테스트를 Mocha로 변환하고 테스트 설정을 약간 변경했습니다. 따라서 master 에서 최신 변경 사항을 가져오면 문제가 해결될 수 있습니까?

@gaguirre 누군가 이 스레드를 우연히 보게 될 경우를 대비하여 지나가는 사람이 볼 수 있도록 변경 사항을 포크로 푸시하는 것이 아이디어일 수 있다고 생각합니다.

@fatso83 첫 번째 버전 을 underscope/sinon 으로 푸시했습니다.
주요 문제는 es6(이 경우 phantomjs)을 지원하지 않는 엔진으로 테스트를 실행할 때 발생합니다. 코드를 구문 분석할 때 실패하므로 eval() 를 해결 방법으로 사용하고 있습니다. 잡다. 나는 이것이 받아들여질 수 없다고 생각하지만 그것은 출발점이다.

지금은 생성기 호출을 다른 파일로 래핑하여 동적으로 요구하고 있지만 작동하지 않습니다. browserify가 동적으로 필요한 파일도 번들로 묶고 있기 때문에 작동하지 않는 것 같습니다. npm run test-headless 실행할 때 일부 파일을 제외할 수 있는지 궁금합니다.

어떤 것이 최선의 해결책이 될 수 있다고 생각하십니까?

피드백을 보내주셔서 감사합니다. @gaguirre . 따라서 기본적인 문제는 엔진이 지원하지 않는 경우 구문 분석을 피할 수 있는 방법이 필요하다는 것입니다. WebWorkers 등과 같이 단순한 기능 감지보다 약간 어렵습니다.

실제 검사를 위임하는 것은 if(require('generator-support')){ ... } 만큼 간단하지만 실제 구문 분석 문제는 해결되지 않습니다.

@mantoni 와 @mroderick, 이것을 깔끔하게 처리하는 방법에 대한 아이디어가 있습니까?

추신: 저는 몇 시간 후에 오프라인-휴가로 떨어져 있기 때문에 부활절 이후 얼마 동안 답변을 읽을 수 없을 것입니다.

이 수준의 생성기 스터빙이 실제로 작동하는지(특히 코루틴 지향 시나리오에서) 알 수 없습니다. 비동기 동작의 일부 에뮬레이션이 순서대로 될 수 있는 것 같습니다. 하지만 구문 분석 문제를 해결할 수 있는 방법이 있다고 생각합니다.

제안 :

생성기 코드를 별도의 파일로 추출하고 생성기 함수를 래핑할 때 요청 시에만 해당 파일을 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 전용 환경을 지원하는 소수의 사람들에게 1.x에서 남겨질 것이라고 말할 때일 것입니다.

@evan-king 조건부 요구 사항은 브라우저 빌드에서 중단되므로 솔루션이 아닙니다. 조건부 요구는 종속성 그래프의 정적 분석을 무효화하므로 WebPack, Rollup 및 Browserify와 같은 도구는 이를 처리할 수 없습니다. 완전히 들어가거나 완전히 빠져 있습니다.

그리고 예, ES5 호환성은 중요합니다. ES6 생성기 지원 은 기껏해야 조악하고 사람들이 프로젝트에 Sinon을 사용하기 위해 추가 도구를 사용하도록 강요하면 테스트 수행의 기준이 높아집니다. 그리고 많은 사람들에게 그 문턱은 있는 그대로 충분히 높습니다. 스크립트 태그를 포함하여 작동하도록 하는 것은 IMHO의 중요한 기능입니다. 우리는 언젠가 ES5 호환성을 깨뜨릴 수 있지만 로드맵에 없고 Sinon 3에 대해 논의조차 하지 않았습니다. Sinon 2는 사실상 나온 지 꽤 되었습니다. 우리가 공식 출시를 하지 못하게 막는 약간의 성가신 일이 있습니다.

기존 ES5 런타임 호환성을 깨뜨리지 않고 ES6 호환성을 얻는 방법은 기본적으로 두 가지가 있으며, 마지막 방법은 확실하지 않습니다(테스트하지 않음).

모듈의 조건부 평가

이것은 해킹이지만 상당히 간단합니다. 이것이 의미하는 바는 우리가 필요한 모듈을 평가할지 여부를 결정하기 위해 정적 자산 인라이닝 및 ES6 기능 테스트에 의존할 수 있다는 것입니다. 이것은 ES5 호환성을 보장합니다(Sinon은 런타임이 구문을 지원하는 경우에만 패치됩니다). 라이브러리 제작자 측이나 클라이언트 사용자 모두에서 Babel 형태의 추가 도구가 필요하지 않으며 ES6을 테스트하면 ES5 브라우저가 중단됩니다. 어떤 경우에도 실패했을 것입니다.

brfs 와 같은 것이 있다고 가정하면 코드는 다음과 같을 것입니다.

_런타임 기능.js_

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

_es6-generator-wrapper.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"); }
}

바벨리파잉 시논

이건 제가 실제로 해보지 않아서 잘 모르겠습니다. Babel을 통해 빌드를 ES5로 변환하면 ES6 코드를 여기저기에 작성할 수 있고 위에서 했던 것처럼 점프를 피하고 생성기에 대해 동일한 종류의 검사를 계속 사용할 수 있습니다. 그것들은 ES5 구조를 사용하여 구현될 것입니다. 물론 ES5 브라우저에서 ES6을 테스트하는 것은 여전히 ​​실패합니다. 이것은 클라이언트 측의 이전 것과 동일한 장점이 있지만 yield , async , function*() 와 같은 항목에 대한 ES2015 지식이 도달하지 않았기 때문에 기여를 방해할 수 있습니다. 넓은 청중.

+1

@rpavlovs , 스레드에 +1을 추가하는 것은 실제로 의미가 없습니다. GitHub UI에는 감정을 표현해야 하는 경우 각 댓글 상단에 "반응 추가" 버튼이 있습니다. +1은 아무 것도 하지 않습니다. 반면에 위의 제안 중 하나(또는 더 영리한 것)를 구현하는 풀 리퀘스트는 이 문제를 해결할 가능성이 훨씬 더 높습니다 😄

제너레이터에 대한 API 지원이 어떻게 보일지 에 대한 의견을 듣고 싶습니다. 이에 대해 몇 시간을 사용한 후에도 사람들이 무엇을 보고 싶어하는지 아직 확실하지 않습니다.

이를 구체화하기 위해 @ingameio 의 모의 API 수정 사항을 유지하면서 동시에 ES5 호환성을 깨뜨리지 않는 새 분기 를 만들었습니다(위에서 언급한 해킹 사용).

나를 조금 짜증나게 하는 것은 테스트 예제가 작동하지 않기 때문에 Ingameio에서 원래 변경 사항을 테스트하는 방법을 정말로 말할 수 없다는 것입니다. 포크의 예제도 완전하지 않고 사전/포스트를 깨는 항목을 얻을 수 없습니다. 변경.

발전기는 단순한 것입니다. 과거를 기억하는 부드러운 동기적 존재입니다. 따라서 co 및 관련되지 않은 다른 것으로 예제를 수수께끼로 만들지 마십시오. 원하는/잘못된 것을 보기가 더 어렵기 때문입니다. 예를 들어 위의 예는 매우 복잡하고 "yield expression"의 반환 값이 "yield value"와 같을 것으로 예상하기 때문에 yield 가 하는 일을 잘못 이해하는 것 같습니다. 수익률의 result 는 생성기의 next() ( MDN )에 전달된 값입니다.

co 를 사용하는 예제는 아마도 Mocha 테스트에서 yield 를 직접 사용할 수 있을 때만 사용할 수 있지만 IIFE로 예제를 래핑하거나 동일한 결과를 얻을 수 있는 다른 방법으로 더 명확하게 하기 위해.

이것은 생성기가 지원하는 방법에 대한 간단한 테스트입니다(오늘날의 Sinon에서 작동함).

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');
?

cc @ruimarinho

원래 테스트에 오류가 있었고 Sinon에서 발전기를 처리하는 데 문제가 없었기 때문에 이 문제를 닫습니다.

생성기(및 관련 반복기)를 처리하기 위한 API가 문제 #1467에서 어떻게 보이는지에 대한 토론에 참여하세요.