Skip to content

Commit

Permalink
Revert "Chrome fails in ajax.js tests because TestSwarm/Ngnix won't g…
Browse files Browse the repository at this point in the history
…ive HEAD requests any respect. Skip them rather than hear Swarmy call FAILURE on each run."

This reverts commit 9c5089a.

These were not the droids we were looking for. Actually caused by a BrowserStack issue.
  • Loading branch information
dmethvin committed Oct 1, 2012
1 parent 631fbc7 commit ebf4d43
Showing 1 changed file with 1 addition and 12 deletions.
13 changes: 1 addition & 12 deletions test/unit/ajax.js
Expand Up @@ -814,16 +814,6 @@ test("jQuery.ajax - xml: non-namespace elements inside namespaced elements (over
}); });
}); });


// Skip HEAD tests in TestSwarm/Ngnix with Chrome because they consistently hang
if ( location.search.indexOf("swarmURL=") >= 0 && navigator.userAgent.indexOf("Chrome/") >= 0 ) {

test("jQuery.ajax - HEAD requests (SKIPPED)", function() {
expect(1);
ok( true, "Skipping HEAD request tests for Chrome in TestSwarm" );
});

} else {

test("jQuery.ajax - HEAD requests", function() { test("jQuery.ajax - HEAD requests", function() {
expect(2); expect(2);


Expand All @@ -847,9 +837,8 @@ test("jQuery.ajax - HEAD requests", function() {
}); });
} }
}); });
});


} });


test("jQuery.ajax - beforeSend", function() { test("jQuery.ajax - beforeSend", function() {
expect(1); expect(1);
Expand Down

2 comments on commit ebf4d43

@jaubourg
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

\o/

@RedWolves
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can go about your business!

Please sign in to comment.