Sinon: fakeServer는 1.17.4에서 .respond (500)로 손상되었습니다.

나는 이것이 1.17.4에서 발생하지만 1.17.3에서는 발생하지 않음을 확인할 수 있습니다. 나는 karma-sinon과 비슷한 설정을 가지고 있습니다.

내 생각 엔 문제는이 커밋 https://github.com/sinonjs/sinon/commit/2cfbacd5cea5b63c014076d3a65b6642b2200793

1.17.4 태그가 npm 레지스트리로 푸시되었지만이 저장소에서이 태그의 흔적을 찾지 못했습니다. 어떻게 된 거예요?

내 생각 엔 태그가 아직 생성되지 않은 것 같습니다. 3 시간 전에 만 출시되었습니다.

@mbarlock 예, 아마도 — 그럼에도 불구하고 GitHub의 태그가 먼저 릴리스되면 더 나을 것이라고 생각합니다. 적어도 우리는 PR이나 수정해야 할 것을 살펴보고 도움을 줄 것입니다.

내 잘못이야. git push --tags 잊어 버렸습니다. 버그에 대한 정보에 감사드립니다.

2cfbacd 커밋을 확인했는데 mozilla / loop # 400에서 테스트가 실패하게되었습니다.

# 1031의 패치를 로컬에 적용했고 테스트를 수정했습니다.

1.17.4가 기존 동작을 변경한다면 새로운 주요 릴리스의 일부가 아니어야합니까? 현재는 1.17.3과 호환되는 것으로 간주되므로 sinon 종속성을 "^ 1.17.3"으로 지정하는 package.json은 1.17.4를 얻고 작동하던 테스트에 실패 할 수 있습니다.

1.17.3이 작동하는 방식은 회귀입니다. 나를 바로 잡으십시오. 그럴 경우에는 고쳐야하며 고장난 상태로 유지해서는 안됩니다.

업데이트 : 이것은 실제 버그처럼 보입니다.

오, https://github.com/sinonjs/sinon/pull/861 @fearphage 에서 원래 토론을

그 점을 감안할 때, 내 쪽에서해야 할 올바른 일은 xhr.onerror 대신 xhr.onabort에 의존하도록 테스트를 변경하는 것 같습니다. 로컬에서 실행되는 단위 테스트가 node_modules 디렉터리에있는 경우 모든 종속성을 자동으로 다시 다운로드하지 않기 때문에이 변경 사항이 잠시 동안 혼란을 야기 할 것이라고 생각합니다. package.json (Travis CI가 테스트를 위해 처음부터 npm을 설치하기 때문에 빠르게 깨달았습니다).

올바른 행동 방침이 무엇인지 모르겠습니다. "onerror"에 의존하는 일부 테스트가 "onabort"를 사용해야한다고 지정하는 1.17.4의 변경 로그에 메모를 추가 할 수 있습니까? (btw,이 주석에서 http://sinonjs.org/Changelog.txt에는 아직 1.17.4가 포함되어 있지 않습니다).

나는 그것을 되 찾는다. 내 테스트 수정을 조사한 결과 https://github.com/sinonjs/sinon/commit/2cfbacd5cea5b63c014076d3a65b6642b2200793에 새로운 버그가 도입되었음을 깨달았습니다. onReadyStateChange 함수는 이제 정의 된 경우 onerror를 직접 호출하기 위해 중단 함수에 의존하는 대신 "오류"ProgressEvent를 트리거합니다. 문제는 FakeXMLHttpRequest에 현재 "오류"이벤트에 대한 리스너가 없다는 것입니다.

누락 된 오류 eventListener 키를 추가하기 위해 PR https://github.com/sinonjs/sinon/pull/1042 를 만들었습니다. 이 변경 사항이 없으면 합법적 인 서버 오류 응답 (예 : 500)에서 호출되는 onerror 핸들러 함수를 확인하는 모든 단위 테스트가 실패합니다. 당신의 생각을 알려주세요, @fearphage @ fatso83

이 스레드를 처음 읽었을 때 https://github.com/sinonjs/sinon/pull/1041을 보지 못했습니다. 누락 된 eventListener 키와 이벤트 순서를 주장하는 추가 코드를 추가합니다. 대신 내 PR을 무시해도됩니다.

이제 # 1041 (# 1042와 동일)이 병합되었습니다.

@overcaffeinated 업데이트를 방해 한 문서의 빌드 스크립트 실패로 인한 누락 된 변경 로그와 관련하여 (https://github.com/sinonjs/sinon/issues/991#issuecomment-216651347 참조). 미안합니다. 다시 작동하게하고 변경 사항에 대한 메모를 추가 할 수 있기를 바랍니다.

이와 같은 것들로 인해 2.0을 출시하여 1.x 브랜치에 너무 많은 에너지를 투자하지 않도록합니다.

1.7.4로 인해 여러 테스트가 실패했습니다. 1.7.5가이 문제를 해결할 것으로 예상합니까?

불행히도 # 1031은 아직 수정되지 않았기 때문에 모든 사람을 위해 수정되지는 않습니다. 오늘 저녁에 도움을 드리겠습니다.

정보 주셔서 감사합니다, @ Standard8 !

XHR 사양에 대한 해석을 기반으로이 문제를 수정했습니다 (고장?). 이번에는 옳은지 확인하기 위해 이것을 비교할 브라우저 구현이 있다면 좋을 것입니다. 이벤트가 올바른 순서로 실행되고 올바른 이벤트가 실행되는지 확인하기 위해 sinon의 가짜 xhr 구현을 실제와 비교하는 테스트 베드가 필요합니다.

수취인이 있습니까?

@fearphage 그런 테스트 베드가 어떻게

나는 그것이 browserscope.org와 같거나 적어도 결과를 저장하는 백엔드로 사용할 수 있다고 생각합니다.

XHR에 대해 http://www.acidtests.org/ 를 상상해보십시오

다음은 도움이되는 몇 가지 요청 기반 유틸리티입니다.

https://httpbin.org/
http://requestb.in/

훌륭한 자원 인 것 같습니다! Browserscope가 다운되었습니다.

@fearphage www.browserscope.org에 무언가를 넣는 방법을 알아낼 시간이 없었기 때문에 대신 웹 페이지를 작성했습니다.

http://people.mozilla.org/~mbanner2/sinonXHRBrowserTest.html

최신 버전의 Firefox, Chrome, IE 및 Safari에서로드됩니다.

@ Standard8 감사합니다. 나는 이것을 사용하여 sinon의 가짜 서버 구현을 비교했습니다. 감사합니다.

@fearphage @ fatso83 ,이 문제의 현재 상태에 대한 개요를 제공해 주시겠습니까?

빠른 훑어보기에서 우리는 v1.17.3 기능을 되돌리고 v1.17.5 릴리스를 제공하는 것을 고려해야하는 것처럼 들립니다. 이전 기능이 손상 되었더라도 이는 API의 주요 변경 사항이므로 2.0 릴리스로 롤링해야합니까?

@fearphage 가 이것에 대한 세부 사항에 더 많이 있지만 잘 요약했다고 생각합니다. 나는 v1.17 브랜치에있는 것과 master 에있는 것을 내 머릿속에 붙잡 으려고 계속해서 압도 당한다. 나는 대부분의 문제가 master 에서 수정되었다고 생각하지만 v1.17 브랜치 (# 1031 및 # 1041 AFAIK와 같은 수정 사항이 없음)에는 적용되지 않는다고 생각합니다. 나는 (아마도) 착각 할 수 있습니다.

가장 실용적인 해결책은 당신이 말한대로하는 것입니다.

• 1.17 브랜치에 대한 # 1017 (및 관련?)을 되돌려 노이즈를 줄이고 네트워크 API를 동일하게 유지하고 1.17.5 릴리스를 제공하고 구현이 버전 2보다 덜 정확하다는 사실을 인정하십시오.
• master 의 변경 사항을 유지하고 마이그레이션 가이드에 변경된 사항을 문서화하십시오 (# 1090 참조).

나는 여기서 무슨 일이 일어나고 있는지 따라 잡고 있습니다. 되돌리기보다 수정하는 것이 더 낫습니까?

가장 큰 변화는 error / onerror 입니다. 그렇죠?

@fearphage 차임에 감사드립니다! 속도가 느리기 때문에 모든 수정 사항이 v1.17 브랜치에 적용되었을 때 기존 최종 사용자의 관점에서 어떤 API 변경 사항이 분명해 질지 5 줄로 요약 해 주시겠습니까?

새로운 메이저 버전에서는 큰 변경이 가능하지만, 1.x에서 많은 테스트를 중단하기 시작하는 모든 것은 아마도 보류 상태 여야하지만 master 적용된 수정 사항이 있는지 확실하지 않았습니다

내가 이해하는 한, 200 개가 아닌 요청은 여전히 error 이벤트를 트리거해야한다는 점만 깨졌습니다. 이것에 더 있습니까? 나는 그것이 필요한 모든 것이 마스터의 수정이라고 믿습니다.

Github와 NPM에서도 v1.7.4 를 가져 오는 것도 좋은 생각 인 것 같습니다. 대부분의 사람들은 되돌 리거나 업그레이드 할 수 없습니다.

그게 다 바뀌었다면 master 포함 된 수정 사항을 포함하고 최대한 빨리 1.17.5 릴리스를 출시하는 것이 좋습니다. 할 수 있겠 어? 휴가를 위해 떠납니다 ( Date.now() ).

NPM 버전을 제거하는 것은 "좋은"것으로 간주되지 않으므로 1.17.4에 대해 npm deprecate 명령을 실행했습니다. 일부 사람들이 태그를 사용하는 경우 태그를 제거하는 것도 확실하지 않습니다. 그러나 그것은 또 다른 논의이며, 수정 사항을 제공하는 데 집중할 것을 제안합니다.

계획 같은 소리. 이번 주말에 갈 수있을 것 같아요.

누군가 # 1102에 무게를두고 싶다면이 문제에 대한 해결책을 제안했습니다.

누군가 그것을 필요로한다면, 나는 @ Standard8 의 테스트 파일을 약간 수정했고 이것이 내가 가짜 xhr을 브라우저 동작에 더 가깝게 만들기 위해 사용했던 것입니다.

https://dl.dropboxusercontent.com/u/2400/tc/sinon/xhr-browser-test.htm

@ Standard8 다시 감사합니다! :박수:

사양에 따르면 onerror 이벤트는 DNS 시간 초과 또는 응답하지 않는 서버와 같은 네트워크 수준 이벤트가 발생할 때만 트리거됩니다. 500 또는 404는 정상적인 HTTP 응답이며 오류가 발생했는지 여부를 결정하는 것은 애플리케이션에 달려 있습니다. https://www.w3.org/TR/XMLHttpRequest/#event -xhr-error
사양이 평소처럼 너무 간결합니다. 2xx가 아닌 응답은 jQuery에 의해 오류로 간주되므로 많은 사람들이 XMLHttpRequest의 동작에 혼란스러워합니다.

@ nyk0r : Phred의 수정이하는 일입니다. :)

@gil :이 문제는 # 1102, # 1108, # 1109에서 @fearphage 의 훌륭한 작업으로 수정되어야합니다. 테스트 케이스가 불완전하므로 (확인 / 확인 없음) 현재 v1.17 브랜치의 코드에 의해 실제로 수정되었는지 확인 하시겠습니까? 그렇게하시면 최대한 빨리 새 패치 릴리스를 배송 할 수 있습니다.

또는 @wlepinski 또는 @mbarlock 이 v1.17 브랜치에서 수정되었는지 확인할 수 있습니까? 단지에서 sinon 의존성 변경 package.json 읽어 : sinon#v1.17 GitHub의에서 직접 권리 분기를 사용 할 수 있습니다.

좋습니다. 다음 테스트를 실행하여 수정되었는지 확인했습니다.

"call load handler on non-2xx statuses" : function(){
  var stub = sinon.stub();
  this.xhr.addEventListener("load", stub);
  this.xhr.open("GET", "/");
  this.xhr.send();

  this.xhr.respond(500, { 'Content-Type' : 'application/json' }, JSON.stringify({}));
  assert(stub.called);
}

이것은 1.17.4에서는 작동하지 않았지만 최신 수정에서는 작동합니다.

@fearphage는 이미 bf709a7f (1797 행)에서이 테스트를 다루었지만 헤이 ...

수정 된대로 닫힙니다.

안타깝게도 더 이상 테스트 할 프로젝트에 액세스 할 수 없습니다. 지금은 새로운 회사에서 일하고 있습니다. 하지만 라이브러리를 업데이트하려는 경우 알려 드리겠습니다. 고쳐 주셔서 감사합니다 !!