Opened 7 years ago

Closed 6 years ago

#16118 closed defect (invalid)

IE10 JsonRest test timeout

Reported by: haysmark Owned by: haysmark
Priority: undecided Milestone: 1.8.4
Component: Data Version: 1.8.0
Keywords: dohfail Cc:
Blocked By: Blocking:

Description

     Error: test timeout in testHeaders
     ERROR IN:
 		 function testHeaders(t){
				var d = new doh.Deferred(),
					error,
					expected = 0,
					received = 0;

				// NOTE: Because HTTP headers are case-insensitive they should always be provided as all-lowercase
				// strings to simplify testing.
				function runTest(method, args){
					expected++;

					store[method].apply(store, args).then(function(result){
						received++;

						if(error){
							return;
						}

						var k;

						for(k in requestHeaders){
							if(!result.headers.hasOwnProperty(k) || "" + result.headers[k] !== "" + requestHeaders[k]){
								error = true;
								d.errback(new Error("Header mismatch in " + method + ": " + k));
								return;
							}
						}

						for(k in globalHeaders){
							if(!result.headers.hasOwnProperty(k) || "" + result.headers[k] !== "" + globalHeaders[k]){
								error = true;
								d.errback(new Error("Global header mismatch in " + method + ": " + k));
								return;
							}
						}

						if(expected === received){
							d.callback(true);
						}
					});
				}

				runTest("get", [ "index.php", requestHeaders ]);
				runTest("get", [ "index.php", { headers: requestHeaders } ]);
				runTest("remove", [ "index.php", { headers: requestHeaders } ]);
				runTest("query", [ {}, { headers: requestHeaders, start: 20, count: 42 } ]);
				runTest("put", [ {}, { headers: requestHeaders } ]);
				runTest("add", [ {}, { headers: requestHeaders } ]);

				return d;
			}
 FAILED test: testHeaders 1007 ms

Change History (9)

comment:1 Changed 7 years ago by haysmark

Keywords: dohfail added

comment:2 Changed 7 years ago by bill

#16119 is a duplicate of this ticket.

comment:3 Changed 6 years ago by bill

Milestone: tbd1.8.2

Bulk update of IE10 tickets to 1.8.2, as per meeting. Backports to 1.4 will be handled in a separate ticket. If this ticket just requires a test update, it can be rescheduled for milestone 1.9, and checked into trunk only.

comment:4 Changed 6 years ago by Kenneth G. Franqueiro

I don't see any dojo/store failures running the dojo tests on Win8 IE10 via Browserstack.

comment:5 Changed 6 years ago by Kitson Kelly

I cannot reproduce this on IE 10.0.9200.16384 running util/doh/runner.html?test=dojo/tests/module on trunk (29997).

comment:6 Changed 6 years ago by bill

Owner: changed from Kris Zyp to haysmark
Status: newpending

Doesn't reproduce for me either. Mark, maybe you don't have PHP enabled on your machine? In any case if it's failing for you seems like you are the only one that can debug it.

Last edited 6 years ago by bill (previous) (diff)

comment:7 Changed 6 years ago by Colin Snover

Milestone: 1.8.21.8.3

1.8.2 release candidate is released; moving to next minor release.

comment:8 Changed 6 years ago by Kitson Kelly

Milestone: 1.8.31.8.4

comment:9 Changed 6 years ago by trac-o-bot

Resolution: invalid
Status: pendingclosed

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!

Note: See TracTickets for help on using tickets.