Sinon: 새 오류 개체를 만드는 거부를 어떻게 재정의 할 수 있습니까?

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

https://github.com/sinonjs/sinon/blob/4310343393b16b92b56526ab4614da4eb2886efa/lib/sinon/default-behaviors.js#L176

문자열을 거부하는 코드를 유지하고 있으며 테스트하는 동안 문제가 발생합니다. 내 블루 버드 캐치가 제대로 작동하지 않습니다.

bluebird.rejects('Something')
    .catch(e => e === 'Something' /* true */, () => console.log(typeof e) /*string*/)
    .catch(() => console.log('should not be here'));

그러나 sinon.stub().usingPromise('bluebird').rejects('Something'); 사용하면 Something이라는 이름의 오류 개체가 생성됩니다. 전혀 예상되지 않습니다.

누군가 우리가 네이티브 약속 만 지원한다고 말하기 전에 그들이 무엇을하는지 조사 해보자!

Promise.reject('Something')
     .catch(e => {
        console.log(e === 'Something'); // true
        console.log(typeof e); // string
    });

흠, 그들은 Error 클래스를 생성하지 않고 던져진 것처럼 주어진 예외를 던집니다.

try { 
    throw 'Something'; 
} catch (e) { 
    console.log(e === 'Something');  // true
    console.log(typeof e); // string
}

Bug Help wanted pinned

출처

JemiloII

가장 유용한 댓글

스택 추적 유용성 및 오류 개체를 제쳐두고 작성해야하는지에 대한 토론, 일반적으로 동의합니다.

const example = sinon.stub().rejects(something);

다음과 동일해야합니다.

const example = () => Promise.reject(something);

현재는 something 가 문자열 인 경우에는 해당되지 않습니다. 현재 동작이 버그라고 말하고 싶습니다.

mantoni 에 2020년 09월 08일

👍4

모든 6 댓글

누군가 우리가 네이티브 약속 만 지원한다고 말하기 전에 그들이 무엇을하는지 조사 해보자!

그것은 짚맨의 오류 입니다. 귀하의 예제에서 사용하는 것처럼 우리가 promise 라이브러리를 지원한다는 것을 충분히 알고 있습니다.

Sinon 관리자는 Promise.reject() 가 전달하는 모든 것을 전달한다는 점을 포함하여 JavaScript promise가 작동하는 방식을 완전히 알고 있습니다.

문자열을 거부하는 코드를 유지하고 있습니다.

제 생각에는,와 약속을 거부 String 대신 Error 정말 우리가 오늘 무엇을 쓰기 디버깅이 미래 개발자에게 몹쓸을하고 있습니다. 문자열은 스택 추적을 캡처하지 않기 때문에 디버깅에 그다지 유용하지 않습니다. 반면에 Error 입니다. 따라서 rejects() 동작이 바람직하다고 생각합니다. 그래야 디버거를 사용하여 무엇이 잘못되었는지 신속하게 파악할 수 있기 때문입니다.

동의하지 않는 것 같지만 당신이 한 유일한 진술은 Sinon이 당신의 기대를 충족시키지 못한다는 것입니다. 기본 동작을 변경해야한다고 생각하는 경우 변경 사항이 많은 사용자에게 영향을 미칠 수 있음을 염두에두고 이유에 대한 인수를 게시하십시오.

기본 동작을 변경하기위한 확실한 주장이 있다면 Sinon의 관리자 중 누구도이를 변경하기위한 pull 요청에 반대 할 것이라고 상상할 수 없습니다.

mroderick 에 2018년 02월 11일

👍1

이 문제는 최근 활동이 없었기 때문에 자동으로 오래된 것으로 표시되었습니다. 더 이상 활동이 발생하지 않으면 폐쇄됩니다. 귀하의 기여에 감사드립니다.

stale[bot] 에 2018년 04월 12일

특정 시나리오에서 스택 추적이 실제로 유용한 지 여부에 대해 다른 의견이있을 수 있습니다. 내 특정 상황에서는 실제로 스택 추적에 신경 쓰지 않고 오류 메시지에만 관심이 있고 다른 곳에 전달합니다. 그래서 저는 문자열과 오류 유형 거부 이유를 모두 처리하는 코드를 작성했습니다. 그러나 sinon은 나를 위해 String-> Error 강제 변환을 수행하므로 문자열을 처리하는 코드 분기를 테스트 할 수 없습니다. 따라서 전체 보험을받을 수 없습니다.

dtom90 에 2020년 09월 08일

스택 추적 유용성 및 오류 개체를 제쳐두고 작성해야하는지에 대한 토론, 일반적으로 동의합니다.

const example = sinon.stub().rejects(something);

다음과 동일해야합니다.

const example = () => Promise.reject(something);

현재는 something 가 문자열 인 경우에는 해당되지 않습니다. 현재 동작이 버그라고 말하고 싶습니다.

mantoni 에 2020년 09월 08일

👍4

나는 동의한다.

JemiloII 에 2020년 09월 17일

stale[bot] 에 2020년 12월 25일

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

0 / 5 - 0 등급

Sinon: 새 오류 개체를 만드는 거부를 어떻게 재정의 할 수 있습니까?

가장 유용한 댓글

모든 6 댓글

관련 문제