Permalink
Browse files

remove ie8 hack from tests. for some reason it is no longer needed.

  • Loading branch information...
1 parent 2f42f21 commit 4e922fd709e2f8586ba8a6c811b4d90c4bb4f9d3 @n1mmy n1mmy committed Oct 9, 2013
Showing with 8 additions and 3 deletions.
  1. +8 −3 packages/http/httpcall_tests.js
@@ -249,11 +249,16 @@ testAsyncMulti("httpcall - methods", [
test.equal(result.statusCode, 200);
var data = result.data;
test.equal(data.url, "/foo");
+
// IE <= 8 turns seems to turn POSTs with no body into
// GETs, inexplicably.
- if (Meteor.isClient && $.browser.msie && $.browser.version <= 8
- && meth === "POST")
- meth = "GET";
+ //
+ // XXX Except now it doesn't!? Not sure what changed, but
+ // these lines now break the test...
+ // if (Meteor.isClient && $.browser.msie && $.browser.version <= 8
+ // && meth === "POST")
+ // meth = "GET";
+
test.equal(data.method, meth);
}));
};

0 comments on commit 4e922fd

Please sign in to comment.