[SeaMonkey, WINNT 5.2, Debug] mochitests-1: intermittent "test_ws_basic_tests.html | Test timed out"

RESOLVED FIXED in mozilla2.0b9

Status

()

Core
Networking: WebSockets
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Blocks: 2 bugs)

Trunk
mozilla2.0b9
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [SeaMonkey "orange" issue moved to bug 736325], URL)

Attachments

(3 attachments, 2 obsolete attachments)

(Assignee)

Description

7 years ago
Bug 574070 comment 16:
{{
Serge Gautherie (:sgautherie)      2010-11-18 21:28:27 PST

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1290134463.1290139655.3881.gz
WINNT 5.2 comm-central-trunk debug test mochitests-1/5 on 2010/11/18 18:41:03
{
...
31011 INFO TEST-PASS | /tests/content/base/test/test_ws_basic_tests.html |
Wrong message - "test message100" should equal "test message100"
31012 ERROR TEST-UNEXPECTED-FAIL |
/tests/content/base/test/test_ws_basic_tests.html | Test timed out.
31013 INFO TEST-END | /tests/content/base/test/test_ws_basic_tests.html |
finished in 354923ms
}

354.923 s +=- 360 s : this test can simply be to slow as is.
}}

***

Test has completed the expected 100 iterations of testWebSocket3 onmessage(),
but testWebSocket3 onclose() doesn't run :-(
(Assignee)

Updated

7 years ago
Summary: [SeaMonkey, WINNT 5.2, Debug] → [SeaMonkey, WINNT 5.2, Debug] mochitests-1: intermittent "test_ws_basic_tests.html | Test timed out"
(Assignee)

Comment 1

7 years ago
Created attachment 499709 [details] [diff] [review]
(Av1) s/ok/is/ typo fix from bug 562681
[Checked in: Comment 9]
Attachment #499709 - Flags: review?(Olli.Pettay)
(Assignee)

Comment 2

7 years ago
Created attachment 499710 [details] [diff] [review]
(Bv1) Add/Re-enable '.wasClean' checks from (bug 573227 ->) bug 572975
[Checked in: Comment 13]
Attachment #499710 - Flags: review?(Olli.Pettay)
(Assignee)

Updated

7 years ago
Blocks: 562681, 572975
(Assignee)

Comment 3

7 years ago
Created attachment 499715 [details] [diff] [review]
(Cv1) Document/Improve code, somewhat
Attachment #499715 - Flags: review?(Olli.Pettay)
(Assignee)

Updated

7 years ago
Blocks: 621390
(Assignee)

Comment 4

7 years ago
Created attachment 499724 [details] [diff] [review]
(Dv1) testWebSocket4(): stop generating a 508 KiB log line from bug 562681
[Checked in: See comment 10]

I wanted to check a SeaMonkey green run log, I found bug 621390 instead :-/

Let's fix this, then we'll see...
Attachment #499724 - Flags: review?(Olli.Pettay)
(Assignee)

Updated

7 years ago
Attachment #499724 - Attachment description: (Dv1) testWebSocket4(): stop generating a 508 KiB log line → (Dv1) testWebSocket4(): stop generating a 508 KiB log line from bug 562681

Updated

7 years ago
Attachment #499709 - Flags: review?(Olli.Pettay) → review+

Comment 5

7 years ago
Um, somewhat unrelated patches in one bug :/

Updated

7 years ago
Attachment #499724 - Flags: review?(Olli.Pettay) → review+

Comment 6

7 years ago
Comment on attachment 499710 [details] [diff] [review]
(Bv1) Add/Re-enable '.wasClean' checks from (bug 573227 ->) bug 572975
[Checked in: Comment 13]

Why you change the comments in the ok() checks?

Comment 7

7 years ago
Comment on attachment 499715 [details] [diff] [review]
(Cv1) Document/Improve code, somewhat

Kind of hard to see what is changed and why.
Will review a bit later.
(Assignee)

Comment 8

7 years ago
(In reply to comment #6)
> Why you change the comments in the ok() checks?

"Should have" should be implied by ok(), I think.
(Yet I'll use whatever comment you want.)
(Assignee)

Comment 9

7 years ago
Comment on attachment 499709 [details] [diff] [review]
(Av1) s/ok/is/ typo fix from bug 562681
[Checked in: Comment 9]

http://hg.mozilla.org/mozilla-central/rev/d14e3a26d5a0
Attachment #499709 - Attachment description: (Av1) s/ok/is/ typo fix from bug 562681 → (Av1) s/ok/is/ typo fix from bug 562681 [Checked in: Comment 9]
(Assignee)

Comment 10

7 years ago
Comment on attachment 499724 [details] [diff] [review]
(Dv1) testWebSocket4(): stop generating a 508 KiB log line from bug 562681
[Checked in: See comment 10]

http://hg.mozilla.org/mozilla-central/rev/ec3e4786877d
Dv1, but applied before patches B and C.
Attachment #499724 - Attachment description: (Dv1) testWebSocket4(): stop generating a 508 KiB log line from bug 562681 → (Dv1) testWebSocket4(): stop generating a 508 KiB log line from bug 562681 [Checked in: See comment 10]
(Assignee)

Comment 11

7 years ago
Created attachment 500318 [details] [diff] [review]
(Cv1a) Document/Improve code, somewhat
[Moved to bug 744663]

Cv1, unbitrotted.
Attachment #499715 - Attachment is obsolete: true
Attachment #500318 - Flags: review?(Olli.Pettay)
Attachment #499715 - Flags: review?(Olli.Pettay)
Comment on attachment 499710 [details] [diff] [review]
(Bv1) Add/Re-enable '.wasClean' checks from (bug 573227 ->) bug 572975
[Checked in: Comment 13]

If this causes new random orange, please back out and
file bugs for those problems.
Attachment #499710 - Flags: review?(Olli.Pettay) → review+
(Assignee)

Comment 13

7 years ago
Comment on attachment 499710 [details] [diff] [review]
(Bv1) Add/Re-enable '.wasClean' checks from (bug 573227 ->) bug 572975
[Checked in: Comment 13]

http://hg.mozilla.org/mozilla-central/rev/13adcde8f131
Attachment #499710 - Attachment description: (Bv1) Add/Re-enable '.wasClean' checks from (bug 573227 ->) bug 572975 → (Bv1) Add/Re-enable '.wasClean' checks from (bug 573227 ->) bug 572975 [Checked in: Comment 13]
Comment on attachment 500318 [details] [diff] [review]
(Cv1a) Document/Improve code, somewhat
[Moved to bug 744663]

Is this still needed?
Attachment #500318 - Flags: review?(Olli.Pettay)
(Assignee)

Updated

5 years ago
Blocks: 736325
(Assignee)

Updated

5 years ago
No longer blocks: 452942
(Assignee)

Updated

5 years ago
No longer blocks: 438871
(Assignee)

Updated

5 years ago
Assignee: nobody → sgautherie.bz
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Component: DOM → Networking: WebSockets
Flags: in-testsuite+
OS: Windows Server 2003 → All
QA Contact: general → networking.websockets
Hardware: x86 → All
Resolution: --- → FIXED
Whiteboard: [orange] → [SeaMonkey issue moved to bug 736325] [orange]
Target Milestone: --- → mozilla2.0b9
(Assignee)

Updated

5 years ago
Blocks: 744663
(Assignee)

Comment 15

5 years ago
Comment on attachment 500318 [details] [diff] [review]
(Cv1a) Document/Improve code, somewhat
[Moved to bug 744663]

(In reply to Olli Pettay [:smaug] from comment #14)
> Is this still needed?

Yes: "Bug 744663 Submitted".
Attachment #500318 - Attachment description: (Cv1a) Document/Improve code, somewhat → (Cv1a) Document/Improve code, somewhat [Moved to bug 744663]
Attachment #500318 - Attachment is obsolete: true
Blocks: 438871
(Assignee)

Updated

5 years ago
No longer blocks: 438871
Whiteboard: [SeaMonkey issue moved to bug 736325] [orange] → [SeaMonkey "orange" issue moved to bug 736325]
You need to log in before you can comment on or make changes to this bug.