Sinon: 가짜 타이머를 사용할 때 네이티브 ES6 약속을 해결해도 콜백이 트리거되지 않습니다.

나는 또한이 문제에 부딪쳤다.

Promise 사양이 setTimeout 를 사용하지 않고 현재 작업 직후에 수행될 새 작업을 예약하기 때문에 이것이 해결될 수 없다고 생각합니다.

테스트에서 약속을 반환하려고 했습니까? 대부분의 테스트 라이브러리는 현재 Promise를 지원하며 it() 함수가 Promise를 반환하면 테스트 실행자는 테스트가 완료되었음을 알리기 전에 Promise가 해결되거나 거부될 때까지 기다립니다.

예, 테스트에서 약속을 반환하면 아마도 작동합니다. 그러나 가짜 타이머를 사용할 때 테스트 기능에서 돌아오기 전에 테스트를 수행할 수 있을 것으로 기대합니다. 그것은 일종의 요점입니다.

Date 객체가 교체되는 것처럼 Promise 객체를 교체하면 이것이 가능하다고 확신합니다. 그렇게 하려면 기본 구현에 의존할 수 없기 때문에 약속 사양의 경쟁 구현이 필요할 수 있습니다.

나는 너희들이 여기서 지적한 문제를 본다. 그러나 약속 사양과 sinon 문서를 기반으로 하여 이것이 "작동해야"하는 것처럼 보였던 특정 상황을 언급하고 싶었습니다. 이 경우 테스트에 약속을 반환하지만 가짜 타이머를 사용하면 약속이 해결되지 않는 것처럼 보입니다.

이 두 가지 테스트가 모두 작동해야 하는 것 같았지만 첫 번째 테스트는 통과하고 두 번째 테스트는 실패했습니다. 나는 mocha와 함께 chai-as-promised 플러그인과 함께 chai를 사용하고 있는데 "timeout of 2000ms가 초과되었습니다. 이 테스트에서 done() 콜백이 호출되고 있는지 확인하십시오."라는 오류가 발생합니다.

describe('chai as promised', function() {

    it('should resolve a promise', function() {
        var promise = new Promise(function( resolve ) {
            setTimeout( resolve, 1000 );
        });
        return expect( promise ).to.have.been.fulfilled;
    });

});

describe('sinon.useFakeTimers()', function() {

    before(function() {
        this.clock = sinon.useFakeTimers();
    });

    after(function() {
        this.clock.restore();
    });

    it('should resolve a promise after ticking', function() {

        var promise = new Promise(function( resolve ) {
            setTimeout( resolve, 1000 );
        });

        this.clock.tick( 1001 );

        return expect( promise ).to.have.been.fulfilled;
    });

});

빠른 테스트 프로젝트를 준비했습니다: https://github.com/JustinLivi/sinon-promises-test

테스트가 완료되기 전에 복원하는 것이 실행 가능한 해결 방법인 것 같습니다.

describe('sinon.useFakeTimers()', function() {

    before(function() {
        this.clock = sinon.useFakeTimers();
    });

    it('should resolve a promise after ticking', function() {

        var promise = new Promise(function( resolve ) {
            setTimeout( resolve, 1000 );
        });

        this.clock.tick( 1001 );
        this.clock.restore();

        return expect( promise ).to.have.been.fulfilled;
    });

});

우리가 사용하는 해결 방법은 테스트를 실행할 때 기본 약속 구현을 setTimeout에 의존하는 간단한 것으로 바꾸는 것입니다.

window.Promise = require('promise-polyfill')

Sinon.JS는 useFakeTimers를 호출할 때 간단히 동일한 작업을 수행할 수 있습니다.

@JustinLivi 가 완료되기 전에 복원했습니다. +1:

그래서, 어떤 해결책이나 해결 방법이 있습니까?
내 테스트에서 그렇게 말했기 때문에 내 앱을 수정하고 ES6 약속 사용을 중지할 수 없습니다. :)

기본 약속이 아니라 es6-promise 폴리필에 문제가 있습니다.
@JustinLivi 의 clock.restore() 솔루션이 문제를 해결합니다.

이 문제가 왜 여기에서 오랫동안 남아 있는지 모르겠지만 이것은 Sinon의 문제가 아닙니다. Promise를 사용하는 것은 본질적으로 비동기 로직을 ​​수행하는 것입니다. 그래도 여기의 모든 테스트는 동기 논리를 사용합니다(OP가 실제로 언급한 대로). 따라서 시간을 속이더라도 여전히 함수가 완료된 후 실행하기 위해 Promise에 의존하고 있으므로 테스트를 약간 변경해야 합니다. 이것은 일반적으로 대부분의 테스트 프레임워크 문서에서 다루지만( 여기에는 Mocha에서 가져온 것입니다), 다가오는 새 사이트에서 테스트 레시피가 포함된 일부 기사를 통해 이 문제를 다룰 것입니다.

따라서 @JustinLivi 의 예제를 약간 변경하면 다음 테스트로 끝납니다.

var sinon = require('sinon');

describe('sinon.useFakeTimers()', function() {

    before(function() {
        this.clock = sinon.useFakeTimers();
    });

    it('should resolve a promise after ticking', function(done) {

        var promise = new Promise(function( resolve ) {
            setTimeout( resolve, 10000 );
        });

        this.clock.tick( 10001 );

        promise
            .then(done) // call done when the promise completes
            .catch(done); // catch any accidental errors
    });

});

실제로 Mocha를 사용하는 경우 Promises를 사용할 때 위와 동일한 코드의 "바로 가기" 버전이 있습니다. Promise를 테스트 프레임워크에 직접 반환하기만 하면 됩니다.

it('should resolve a promise after ticking', function() {
    var promise = new Promise(function( resolve ) {
        setTimeout( resolve, 10000 );
    });

    this.clock.tick( 10001 );

    return promise;
});

이것은

1. Promise 생성자에 매개변수로 전송된 executor 함수를 동기적으로 실행 하여(ES2015 사양 참조) 새 시간 초과를 효과적으로 설정합니다.
2. 생성된 약속을 반환합니다.
3. 시간을 체크하세요.
4. 약속이 해결된 것으로 표시하여 시간 초과 기능을 트리거합니다.
5. process (또는 브라우저)는 새 "틱"을 스택하여 약속을 해결하고 나머지 Thenables 를 호출합니다.

버그가 아닌 것으로 종료합니다.

@fatso83 설명하신 방식으로 테스트를 통과하는 방법을 모르겠습니다. 테스트 저장소에 예제 테스트를 추가했는데 여전히 timeout of 2000ms exceeded. Ensure the done() callback is being called in this test. 가 표시됩니다.

여기를 참조하십시오: https://github.com/JustinLivi/sinon-promises-test/blob/master/test.js#L60
또한 내가 말할 수있는 한이 테스트는 여전히 실패합니다. https://github.com/JustinLivi/Sinon.JS/commit/de106e6db2f5cc076b7e3a78635bd9ae2b6be1c2

편집: @fpirsch 의 의견에 따르면 es6 promise polyfill을 사용할 때인 것 같습니다. 블루버드와 같은 대체 라이브러리를 사용해 본 사람이 있습니까?

@fatso83 이것은 return promise 문제가 아닙니다.
제 경우에는 Phantom(구식, 약속 없음) + es6-promises polyfill + sinon.js 문제입니다. 최신 브라우저에서는 약속이 해결되고 테스트가 잘 실행되지만 약속 폴리필을 사용하면 타이머가 가짜일 때 약속이 해결되지 않습니다.

@JustinLivi 및 @fpirsch : 이 버그 보고서는 _native_ Promise 사용에 관한 것입니다. es6-promise 를 사용하는 @JustinLivi 의 테스트 프로젝트를 보았으므로 보증할 수 없습니다. 다른 폴리필도 마찬가지입니다. 그건 또 다른 문제입니다. 기본 약속을 지원하는 노드 5 및 6으로 이것을 테스트했습니다.

이전 게시물의 예제 코드를 복사하여 붙여넣고 실행(Mocha 및 Sinon 사전 설치):

$ pbpaste > test.js

$ mocha test.js

  sinon.useFakeTimers()
    ✓ should resolve a promise after ticking

  1 passing (12ms)

@fpirsch : return promise 문제라고 말한 적이 없습니다. 나는 그가 테스트를 더 짧은 형식으로 다시 작성할 수 있다고 언급했습니다. 수정이 아니라 팁입니다. 그러나 몇 가지 귀중한 정보를 제공했습니다. 기본 브라우저에서는 작동하지만 약속 폴리필에서는 실패합니다. 그것은 Sinon이 아니라 당신의 promise lib의 잘못입니다. 귀하의 약속 라이브러리는 아마도 setTimeout 및 친구를 캐시하지 않고 기능에 의존하므로 중단됩니다. pinkyswear lib에서 정확히 이 문제를 수정한 적이 있으므로 당연합니다.

es6-promise 의 소스를 확인했고 setTimeout 에 대한 참조를 캐시하지 않으므로 sinon이 하는 모든 것의 영향을 받습니다. 그러나 나는 그 스케줄러가 여기에서 어떻게 관련이 있는지 알지 못합니다 ... 또한 폴리필이 타임아웃 스케줄러로 돌아가기 전에 시도할 다른 옵션의 거대한 목록이 있습니다 . 그 대체에 착륙하는 모든 브라우저는 고대여야 합니다. :)

그러나 어쨌든 @fpirsch 와 @JustinLivi 가 언급한 것은 Sinon과 폴리필 간의 상호 운용성에 관한 문제입니다. 그리고 그것은 또 다른 문제입니다. Sinon이 ATM에 대해 많은 것을 할 수 있는 방법을 알지 못하지만(모든 PR은 es6-polyfill 로 끝날 가능성이 더 높음), 이것이 Sinon 문제인 경우 자유롭게 새 문제를 여십시오.

추신: @ropez 의 팁은 promise-polyfill 를 사용하는 것이었으며 이는 실제로 최소 폴리필에 대한 권장 사항이기도 하지만 제공된 것과 반대의 이유입니다. @mroderick 은 setTimeout 에 대한 참조를 캐시하기 위해 1월에 해당 프로젝트를 패치했습니다( @ropez 주석 후 몇 개월). 그러면 약속이 _sinon이 무엇을 하고 있든_ 글로벌 setTimeout 참조로 해결됩니다. 자세한 내용은 taylorhakes/promise-polyfill#15를 참조하세요.

@fatso83 정보 감사합니다. 불행히도 @mroderick 이 promise-polyfill 에 대해 했던 것처럼 setTimeout 에 대한 참조를 저장하는 것은 제 경우에는 작동하지 않습니다. es6-promise 를 promise-polyfill 로 바꾸지 않습니다. :-(
편집: 단순한 팬텀+모카 스택에서는 수행되지만 카르마가 있는 전체 스택에서는 수행되지 않습니다. 너무 피곤합니다.

@fatso83 여기에 예를 들겠습니다: faketimers
결국 Karma에 문제가 있을 수 있습니다. 이 문제에 대해 더 깊이 파고들도록 노력하겠습니다.

@fpirsch : 뭔가 하고 있는 것 같아요. 당신과 @JustinLivi 모두 카르마를 테스트 러너로 사용하고 있습니다. 그들은 매우 비슷해 보입니다. 추신: Justin이 문제를 보여주는 테스트 사례를 이미 제공했기 때문에 완전히 불필요하다고 생각하여 내 의견을 삭제했지만 귀하의 예제 프로젝트는 이것이 Karma 일이라는 가설을 강화합니다. 감사합니다!

이것은 관련이 있을 수 있지만 확실하지 않습니다. 아래의 세 번째 테스트가 실패하는 이유는 무엇입니까? 특정 실패는 일반적인 테스트 시간 초과입니다.

     Error: timeout of 2000ms exceeded. Ensure the done() callback is being called in this test.

describe.only('working', () => {
  let clock;
  beforeEach(() => { clock = Sinon.useFakeTimers(); })
  afterEach(() => { clock.restore() })

  const delay = (ms) => (
    new Promise((resolve/* , reject */) => {
      setTimeout(resolve, ms)
    })
  );

  it('1 OK', () => {
    const promise = new Promise((resolve) => {
      setTimeout(resolve, 100)
    })
    clock.tick(200)
    return promise
  })

  it('2 OK', () => {
    const promise = delay(100)
    clock.tick(200)
    return promise
  })

  it('3 FAIL', () => {
    const promise = delay(50).then(() => delay(50))
    clock.tick(200)
    return promise
  })
})

@jasonkuhrt 세 번째 테스트는 clock.tick 이 실행기를 동기적으로 처리하므로 비동기 특성 으로 인해 .then(...) 가 실행되지 않기 때문에 실패합니다.

onFulfilled 또는 onRejected는 실행 컨텍스트 스택에 플랫폼 코드만 포함될 때까지 호출되지 않아야 합니다.

@fatso83 타이머 를 추가하는 체인 프라미스에 대한 롤렉스 개발자의 입장을 모르겠습니다. 현재 제한 사항은 최소한 문서에 자세히 나와 있어야 합니다.

이것이 지원되어야 하는지 여부에 따라 이 문제는 다시 열릴 수 있고(또는 새 문제가 열릴 수 있음) lolex 의 새 문제에 종속될 수 있습니다.
또한 이 사용 사례(타이머를 추가하는 연결 약속)를 고려해야 하는 경우, 이로 인해 lolex API 수정이 중단될 수 있습니다(AFAIK clock.tick 은 비동기식이어야 함).

@gautaz 아 이제 이해가 됩니다. 설명해주셔서 감사합니다!

lolex 는 동기화 라이브러리이고 @jasonkuhrt 의 문제를 어떻게 해결해야 할지 완전히 모르겠습니다. 자유롭게 PR을 제공할 수 있지만 clock.tick 를 수정하는 것은 그다지 매력적이지 않습니다. 차라리 추가 비동기 방법을보고 싶습니다.

나는 약속과 시계 똑딱과 관련하여 많은 문제가 없었지만 아마도 그것이 동기식이고 그 사이에 추가 시계 틱을 추가한 것을 보았기 때문일 것입니다.

@fatso83 맞습니다 . 비동기 tick 를 추가하면 덜 방해가 됩니다(사실 전혀 방해가 되지 않음).
따라서 asyncTick 와 같은 것이 적합할 수 있습니다. 시간이 되면 조사하겠습니다.

@gautaz lolex 에 패치를 제공할 기회가 있었는지 궁금합니다.

@fatso83 안녕하세요, 저는 작년에 제 옆에서 지점을 시작했지만 지금은 그 일을 끝낼 시간이 없습니다.
나에게도 같은 문제로 걸려 넘어진 동료가 있으므로 문제가 나에게 추가 시간을 제공할 만큼 충분히 커지기를 바랄 뿐입니다.

그 동안 이 특정 지점에서 lolex API와 관련하여 변경 사항이 있습니까?

그것은 없어야합니다. 지난 반년 동안 코드베이스에 거의 변화가 없었습니다. 꽤 안정적입니다.

여기에 간단한 해결 방법이 있습니까? 같은 문제가 있습니다. 두 가지 약속이 있으며 setTimeout으로 해결됩니다. 해결 전, 두 번째 해결 전, 첫 번째 해결 후, 모든 해결 후 항목을 확인해야 합니다.

당신이 필요로 하는 유일한 것은 약속 마이크로태스크가 실행될 때까지 기다리는 것입니다.
따라서 다음 접근 방식이 완벽하게 작동합니다.

const tick = async (ms) => {
  clock.tick(ms);
};

it("imitates promise fulfill when called once", async () => {
    new Promise(resolve => setTimeout(resolve, 400)).then(callback);
    expect(callback.callCount).to.eql(0);

    await tick(200);
    expect(callback.callCount).to.eql(0);

    await tick(200);
    expect(callback.callCount).to.eql(1);
  });

@jakwuh 의 트릭은 then 콜백이 틱에 의해 연기되었다는 점을 제외하고 저에게 효과적이었습니다(노드 8, 트랜스파일 없음, 기본 약속).

업데이트된 의견 : 내 원래 의견(아래)의 솔루션에 문제가 있습니다. 때로는 실제로 모든 것을 플러시하기 위해 여러 개의 연속적인 await Promise.resolve() 호출이 필요했습니다. 다음은 조금 더 잘 작동하는 것 같습니다.

beforeEach(function() {
    const originalSetImmediate = setImmediate;
    this.clock = sinon.useFakeTimers();
    this.tickAsync = async ms => {
        this.clock.tick(ms);
        await new Promise(resolve => originalSetImmediate(resolve));
    }
});

afterEach(function() {
    this.clock.restore();
});

원래 주석 [경고: 위의 코드처럼 작동하지 않습니다.]

tick 도우미에 비동기 간격을 추가하면 문제가 해결되었습니다.

```js
const 틱 = 비동기(ms) => {
clock.tick(ms);
Promise.resolve()를 기다립니다.
};

이 영역에서 Sinon을 개선하는 작업(실제로는 형제 프로젝트 lolex )에 관심이 있는 사람들은 여기에서 토론을 확인하십시오.

• https://github.com/sinonjs/lolex/issues/114
• https://github.com/sinonjs/lolex/pull/105