RE: [eventsource-tests] Processing Open PRs

2 patches merged already;
2 patches are waiting for merge;
1 patch is pending for further discussion.

2 bugs have been fixed and verified in the latest [Interop] http://www.w3.org/wiki/Webapps/Interop/ServerSentEvents .

Thanks,
Tina

-----Original Message-----
From: Zhao, Tina [mailto:tina.zhao@intel.com] 
Sent: Tuesday, August 13, 2013 4:25 PM
To: Arthur Barstow; Zhang, Zhiqiang; public-webapps-testsuite@w3.org
Subject: RE: [eventsource-tests] Processing Open PRs

OK, I will review following patch and refresh the interop: http://www.w3.org/wiki/Webapps/Interop/ServerSentEvents in this week.

Thanks,
Tina

-----Original Message-----
From: Arthur Barstow [mailto:art.barstow@nokia.com] 
Sent: Monday, August 12, 2013 11:10 PM
To: Zhao, Tina; Zhang, Zhiqiang; public-webapps-testsuite@w3.org
Subject: [eventsource-tests] Processing Open PRs

Hi Tina, All - below is a summary of Open Pull Requests by @zqzhang and @Yaffle for the SSE/eventsource test suite [GH][Mirror].

Tina - would you please review these and make a recommendation? Also, do you know if any of the Chrome bugs identified in [Interop] have been fixed? (I'd like to understand if [Interop] is current/up-to-date).

-Thanks, Art

[GH] <https://github.com/w3c/web-platform-tests/tree/master/eventsource>
[Mirror] <http://w3c-test.org/web-platform-tests/master/eventsource/>
[Interop] <http://www.w3.org/wiki/Webapps/Interop/ServerSentEvents>


@zqzhang

* <https://github.com/w3c/web-platform-tests/pull/264>;
<http://w3c-test.org/web-platform-tests/master/eventsource/eventsource-constructor-url-multi-window.htm>;
 [Tina] Correct the reference URL. Merged by Art already. PASS: Chrome, FF, Opera; No Fail.

* <https://github.com/w3c/web-platform-tests/pull/268>;
<http://w3c-test.org/web-platform-tests/master/eventsource/format-utf-8.htm>;
[Tina] Need an agreement on SSE spec understanding before making the decision whether to keep this case. See details at https://github.com/w3c/web-platform-tests/pull/268. 

@Yaffle:

* <https://github.com/w3c/web-platform-tests/pull/232> ; <http://w3c-test.org/web-platform-tests/master/eventsource/format-field-retry.htm>;
 [Tina] This case is blocked by default timeout: 2000. Set timeout: 2500 and works. +1, waiting for merge. PASS: Chrome, FF, Opera; No Fail.

* <https://github.com/w3c/web-platform-tests/pull/233>;
<http://w3c-test.org/web-platform-tests/master/eventsource/format-field-retry-bogus.htm>;
 [Tina] This case is blocked by default timeout: 2000. Set timeout: 2500 and works. +1, waiting for merge. PASS: FF, Opera; Fail: Chrome.

* <https://github.com/w3c/web-platform-tests/pull/234>;
<http://w3c-test.org/web-platform-tests/master/eventsource/resources/message.php>;
 [Tina] Add a new line for better understanding in test results. Merged by Velmont already.

Received on Thursday, 15 August 2013 10:05:01 UTC