Sinon: 샌드 박스에서 '존재하지 않는 속성을 스텁 할 수 없음'오류 발생

# 1512와 관련이 있습니다.

속성이 존재하지 않으면 샌드 박스에 추가 할 필요가 없습니다. 덮어 쓰기 만하면됩니다. 하지만 예, 이전에 효과가 있었고 변경해야한다고 명시 적으로 말하지 않았다면 회귀입니다.

여기서 무엇을해야하는지 잘 모르겠습니다. 존재하지 않는 값을 스터 빙하는 것이 의미가없고 지원되지 않는다고 문서를 업데이트하거나 가능하게 만드시겠습니까?

속성이 존재하지 않으면 샌드 박스에 추가 할 필요가 없습니다. 덮어 쓰기 만하면됩니다.

샌드 박스에 속성을 추가 할 때 좋은 점은 sinon이 sandbox.restore() 를 통해 각 테스트간에 전역 테스트 환경을 깨끗하게 유지하는 데 도움이된다는 것입니다. 특히 API를 제어하지 않는 Google Maps와 같은 타사 라이브러리를 다룰 때 매우 유용한 기능입니다. 3.x 라인에서 작동하도록 만들 수 있다면 좋을 것입니다.

또한 나는 완전한 예를 제공하지 않는 죄를 저질렀다는 것을 깨달았습니다. 내 샌드 박스는 2.4.1 형식으로 생성됩니다.

let sandbox;

before(() => { sandbox = sinon.sandbox.create(); })
afterEach(() => { sandbox.restore(); })

그것이 중요한지 확실하지 않습니다. 더 빨리 제공하지 않은 것에 대해 사과드립니다.

@ZebraFlesh가 설명하는 시나리오와 같은 시나리오에서 텍스트 고정 장치를 더 명시 적으로 사용하는 것이 좋습니다.

// not so explicit, doesn't work with [email protected]
beforeEach(function() {
    const spy = sandbox.spy();
    sandbox.stub(window, 'google').value({
        maps: {
            LatLng: x => x,
            Map: spy
        }
    }); 
});

// more explicit, works with sinon<strong i="9">@2</strong>, sinon<strong i="10">@3</strong>
function setGoogleMapsFixture(sandbox) {
    window.google = {
        maps: {
            LatLng: x => x,
            Map: sandbox.spy()
        }
    };
}

function removeGoogleMapsFixture() {
    delete window.google;
}

beforeEach(function() {
    setGoogleMapsFixture(sandbox)
});

// not using afterEach, as this only needs to happen
// after the last test in this block is run
after(function() {
    removeGoogleMapsFixture();
});

위에서 설명한 것과 같이 조명기를보다 명시 적으로 설정하면 존재하지 않는 자체 속성의 스터 빙을 허용하는 Sinon의 기능이 필요하지 않습니다.

여기서 무엇을해야하는지 잘 모르겠습니다. 존재하지 않는 값을 스터 빙하는 것이 의미가없고 지원되지 않는다고 문서를 업데이트하거나 가능하게 만드시겠습니까?

(@ZebraFlesh에서 설명한 것과 같은) 일부 시나리오에서 편리 할 수 ​​있다는 것을 알고 있지만 존재하지 않는 자체 속성을 스터 빙하는 것은 테스트에서 실수로 이어질 수 있다고 생각합니다. 스텁하려는 기존 자산. 우리는 너무 제한적이지 않으면 서 실수의 가능성을 없애는 것을 목표로해야합니다.

존재하지 않는 자체 속성을 스터 빙하는 것은 지원되지 않아야한다고 생각합니다. 문서를 업데이트해야합니다.

@mroderick 버그를 줄일 수 있다는 점에서 동의하지만 이미 일반 스텁에 대해서는이를 지원합니다. 이 동작 지원을 중단해야하는 경우 일관성을 유지하기 위해 해당 동작도 제거해야합니다. 샌드 박스가 일반적으로 몇 가지 가능성을 _add_하기 때문에 샌드 박스 외부에서만이 기능을 지원하는 것은 이상 할 것입니다. 그리고 지지대를 제거하는 것은 파괴 기능이므로 큰 범프도 필요합니다.

따라서 다음 중 하나입니다.

• 이 차단 기능을 수정하기 위해 샌드 박스에 대한 확인을 즉시 제거하십시오.

또는 / 및 (?)

• 일반 및 샌드 박스 스텁에 대한 기능 제거
  
  
  
  • 업데이트 된 문서로 새로운 주요 버전 출시
  
  

위에서 설명한 것과 같이 조명기를보다 명시 적으로 설정하면 존재하지 않는 자체 속성의 스터 빙을 허용하는 Sinon의 기능이 필요하지 않습니다.

나는 당신의 조명기가 당신의 테스트 사이에 결코 변하지 않는다면 잘 작동한다고 생각합니다. 그러나 내 비품은 그렇습니다. 간단한 예는 성공 및 실패 사례를 모두 다루는 것입니다.

it('handles the success case', () => {
        const spy = sandbox.spy();
        sandbox.stub(window, 'google').value({
            maps: {
                LatLng: x => x,
                Map: spy
            }
        });
        // ... test, including asserting that the spy was called
});

it('handles the failure case', () => {
        const msg = 'test error';
        sandbox.stub(window, 'google').value({
            maps: {
                LatLng: x => x,
                Map: sandbox.stub().throws(new Error(msg))
            }
        });
        // ... test, ignoring spy calls and instead focusing on error handling
});

2.x의 동작은 sandbox.restore() 를 통해 각 테스트 후 모든 것이 올바르게 정리된다는 장점이 있습니다. 위에 설명 된보다 명시적인 조명기 설정 예제를 사용하여 동일한 효과를 얻기 위해 afterEach 후크에서 소유하지 않은 속성을 삭제할 수 있다고 가정합니다.

실수로 기존 속성의 이름을 입력하여 잠재적 인 실수를 유발하는 문제를 해결하기 위해 sinon은 공용 API를 수정할 수 있습니다.

• stub.ownValue() : 자신의 속성 만 스텁, 소유하지 않은 속성에 대해 발생
• stub.value() : 소유하지 않은 속성 만 스텁하고 자신의 속성에 대해 throw합니다.

API가 더욱 명확 해지고 소비자는 당면한 작업에 적합한 도구를 선택해야합니다.

@lucasfcosta는 반대 견해를 가지고 시간 (정상 스텁 다루고 있지만) 이것은 매우 # 1508에서 토론 관련이있다 - 우리가하지 않는 throw_ _should이 undefined 속성. 우리가 무엇에 착수하든, 나는 일반 스텁과 샌드 박스를위한 스터 빙 API에서 _ 일관 적이어야한다고 믿습니다. 우리는 한 경우에만 지원해서는 안되며 다른 경우에는 지원하지 않아야합니다.

현재 상황은 다음과 같습니다.

• 1.x를 던지는 데 사용되는 일반 스텁이 2.0에서 변경되었으며 더 이상 던지지 않습니다.
• 샌드 박스는 던지는 데 사용되지 않았지만 3.1 (?)에서 던지기 시작했습니다.

그래서 잠시 동안 기능 패리티가 있었지만 다시 잃어 버렸습니다.이 지그재그가 사용자에게별로 유익하지 않다고 생각하므로이 논의를 시작해야합니다. 좀 더 구체적인 테스트를 할 수 있다는 점에서 Morgan과 동의하지만 두 가지 주요 릴리스에 대한 동작을 삭제 한 다음 다시 추가하는 것을 좋아하지 않습니다. 이 회귀를 되돌리려면 최소한의 소음 (고객에 대한 수정,이 추적기의 질문 / 문제)을 만들 것이라고 생각합니다.

불편 함을 이해하지만 최소한의 코드 변경으로 쉬운 해결 방법이있는 것 같습니다.

before(function() {
  window.google = 'This is a placeholder for sinon to overwrite.';
});

after(function() {
  delete window.google;
});

이를 통해 sinon 코드가 변경되지 않은 상태로 유지됩니다.

회귀 vs 불량 문서

테스트가 있기 때문에 예상되는 동작 인 것 같습니다. 제 생각에 그것을 반영하기 위해 문서를 업데이트해야합니다. 주요 변경 사항이 허용됩니다.

@fearphage 현상 유지는 존재하지 않는 필드를

해결 방법은 # 1557에서 구현되었습니다.

다양한 스레드를 읽었으며 왜 이런 일이 발생했는지 알 수 있지만 클래스 프로토 타입에서 구현되는 함수를 자주 사용하는 Typescript에서는 정말 고통 스럽습니다.이 경우 모든 것이 괜찮아 보이지만 sinon이 더미를 뱉어냅니다. 현명합니다 ( keyof YourType 는 프로토 타입 체인 아래에 정의 된 모든 공용 함수를 기꺼이 허용하므로).

나는 Typescript가 아마도 당신들에게 우선 순위가 아니라는 것을 알지만 JS에서도 myObject.callMe() 는 완벽하게 행복하게 실행되는 반면 sinon.stub(myObject, "callMe") 는 그렇지 않습니다. 나는 그것을 스터브하는 방법을 알기 위해 특정 객체가 어떻게 결합되었는지 조사하지 않고 싶지 않습니다.

클래스가 JS에서 더 많은 기본 지원을 받고 있다는 점을 고려할 때 이것이 행복한 경로를 만드는 중요한 사용 사례라고 생각합니다.

메서드가 객체에 정의되어 있지 않다는 오류가 발생하면 오류가 프로토 타입에있을 가능성이 있음을 알 수 있습니다. 그런 다음 myObject.callMe = sinon.stub(); 사용하여 개체를 직접 수정하는 것은 그렇게 번거롭지 않은 것 같습니다. IMHO ... 또한 프로토 타입이 변경되지 않았기 때문에 정리 / 해체 기능을 만들지 않아도됩니다.

네, 해결하기가 그리 어렵지 않은 것 같아요. 사물이 어떻게 구현되는지 알기 위해 나에게 더 많은인지 부하를 가하고 있습니다.

또한 예상치 못한 것처럼 보였기 때문에 동일한 객체에 대해 연속 된 두 줄의 스터 빙 코드가 다른 이유와 분해에서 스터브 중 하나를 수동으로 삭제하는 이유를 설명하기 위해 테스트에 주석을 추가해야한다고 느꼈습니다. 하지만 다른 하나는 샌드 박스에서 처리했습니다.

~이 동작을 복원 해 주셔서 감사합니다. ~

존재하지 않는 속성 스터 빙을 지원하는 사용 사례를 추가하고 싶었습니다.

내 사용 사례에서는 구성 개체의 속성을 스터 빙하고 있습니다. config 개체에는 다양한 선택적 키가 있으며 개발자 컴퓨터에서 파일을로드하여 초기화됩니다. 특정 테스트를 실행할 때 알려진 값으로 설정된 키 중 하나가 필요하며 개발자의 개체를있는 그대로 복원하고 싶습니다.

sandbox.stub(serverSecrets, 'the_key_i_need_set').value(fakeValue) 는이를 전달하는 매우 명확한 방법입니다. 런타임에 키가 설정되었는지 여부를 알지 못하더라도 동일한 동작을 얻는 것이 좋습니다 _.