Sinon: 생성자를 감시 할 때 스파이의 thisValue가 인스턴스를 가리 키지 않습니다.

에 만든 2018년 02월 09일 · 10코멘트 · 출처: sinonjs/sinon

이 스 니펫은 Sinon 2.0.0에서는 잘 작동하지만 Sinon 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 의 실제 인스턴스를 참조하지 않는 것 같습니다.

이것은 Sinon 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 에 2018년 08월 09일

👍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 에 2018년 02월 11일

오, 나는 이등분하는 것을 좋아합니다! 내가 알아서 할게 ..

PVince81 에 2018년 02월 11일

😄2

911c498dc14dc4034ba019526bf58f8b24d77da0 스파이는 new (# 1626)로 통화를 통과합니다.

PVince81 에 2018년 02월 11일

👍3

해주셔서 감사합니다, @ PVince81! 수정 사항은 생성자 감시를 지원하기 위해 필요한 변경 사항을 다루기 때문에 # 1265를 검사 할 수도 있습니다.

fatso83 에 2018년 02월 12일

new 로 객체가 생성되지 않으면 이 메서드 가 호출되어 thisValue 의 내용을 변경하고 thisValues 확장합니다. 반대로 new가 사용될 때 호출되는 메서드는 thisValue 그대로 둡니다.

rgroothuijsen 에 2018년 03월 17일

우리 코드가 이것의 단일 용도만을 가지고 있다는 점을 고려하면, 나는 sinon의 내부를 둘러 쌀 시간이없고 자격이 없다고 느끼기 때문에 빠른 경로로 가서 이것을 유발 한 우리의 단일 테스트를 변경하고 싶을 것입니다. 잠재적으로 획기적인 변경

많은 사람들이이 문제에 대해 불평하지 않는 것 같기 때문에, 아마도 사람들은 일반적으로 이런 방식으로 테스트하지 않고 수정할 가치가 없을 것입니다 (단지 관찰, 결정은 유지 관리자까지).

PVince81 에 2018년 08월 08일

일반적으로 사용되지 않거나 적어도 일반적으로보고되지 않는 경우에도이 회귀를 수정하고 싶습니다.

mroderick 에 2018년 08월 09일

👍3

같은 문제가 발생했습니다.이 문제를 해결할 계획이 있습니까?

ivan-zakharchuk 에 2019년 08월 30일

@ ivan-zakharchuk 아무도 자원하지 않으면 계획이 없습니다. 당신이 그것을 고치고 싶다면, 취할 수있는 자연스러운 다음 단계가 있습니다 ;-)

이것은 문제가되는 커밋입니다 : https://github.com/sinonjs/sinon/issues/1683#issuecomment -364794930

fatso83 에 2019년 08월 30일

간단히 말해서, 나는 이것에 대한 수정을 위해 오랜 시간을 보냈지 만 성공하지 못했습니다. this 가치의 악몽입니다. 핵심 문제는 여기에 제공된 함수를 new 하면 본질적으로 새로운 this 생성하는 반면 thisValues[0] (원래 게시물에서 사용됨)는 this 참조한다는 것입니다 new 에 의해 생성 된 this (예 : var some = new window.SomeClass(); )-매우 혼란 스럽습니다.

본질적으로, 이것을 고치는 것은 두 가지 방법을 시도하는 것을 의미합니다- new 호출 this 를 의미하지만 우리는 두 번 호출해야합니다 (즉, OP의 테스트에서 하나, proxy-invoke.js 에서 하나) this es로 끝날 것입니다. 내가 생각할 수있는 유일한 것은 하나의 this 에서 다른 기능으로 함수를 바인딩하는 것이었지만, 이것에도 너무 엉망이었고 다른 몇 가지를 망쳤습니다.

위의 내용이 이해되기를 바랍니다. 언제든지 수정 / 설명을 요청하세요. new 키워드가 실제로 수행하는 작업에 대한 복습을 보려면 여기 를 참조 this 만듭니다.

어쨌든 내가 알아 차린 한 가지는 911c498 (회귀를 도입 한 커밋)에 ES6 클래스에서 new 를 사용해야하는 문제를 다루는 테스트가 포함되어 있지 않다는 것입니다. 이는 아마도 테스트의 linting 옵션이 ES5만을 지원하기 때문에 class 키워드를 허용하지 않기 때문일 것입니다. 현재 문제에 대한 향후 수정 사항이 911c498의 수정 사항을 해제하지 않도록 테스트를 추가했습니다. 관심이 있으면 홍보 할 수 있습니다.

gwpmad 에 2020년 04월 12일

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Sinon: 생성자를 감시 할 때 스파이의 thisValue가 인스턴스를 가리 키지 않습니다.

가장 유용한 댓글

모든 10 댓글

관련 문제