Skip to content

Commit

Permalink
Tests: Really fix tests in IE 8 this time
Browse files Browse the repository at this point in the history
IE 8 doesn't have indexOf on arrays. Also, one toLowerCase() was missing. Oops.

This commit is not necessary on master but has been brought here to keep
tests similar in both branches.

(cherry-picked from 1b48eef)
  • Loading branch information
mgol committed Sep 8, 2015
1 parent 5914b10 commit 1b566d3
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion test/data/testinit.js
Expand Up @@ -271,7 +271,7 @@ QUnit.config.autostart = false;
this.loadTests = function() {
var loadSwarm,
url = window.location.search,
basicTests = url.substring( 1 ).split( "&" ).indexOf( "module=basic" ) > -1;
basicTests = jQuery.inArray( "module=basic", url.substring( 1 ).split( "&" ) ) > -1;

url = decodeURIComponent( url.slice( url.indexOf( "swarmURL=" ) + "swarmURL=".length ) );
loadSwarm = url && url.indexOf( "http" ) === 0;
Expand Down
2 changes: 1 addition & 1 deletion test/unit/basic.js
Expand Up @@ -200,7 +200,7 @@ QUnit.test( "manipulation", function( assert ) {

// Support: IE 8 only
// IE 8 prints tag names in upper case.
elem1.html(),
elem1.html().toLowerCase(),
"<div></div><b></b><span></span><a></a>",
".after/.before"
);
Expand Down

0 comments on commit 1b566d3

Please sign in to comment.