Skip to content
Permalink
Browse files

Remove duplicate expect. Add QUnit.config.requireExpects to testrunne…

…r.js QUnit configurations.
  • Loading branch information...
rwaldron committed Oct 15, 2012
1 parent 0c44743 commit 611d7660cd23747bcc7948ad2ee2a663e00ff105
Showing with 5 additions and 2 deletions.
  1. +3 −0 test/data/testrunner.js
  2. +2 −2 test/unit/css.js
@@ -156,6 +156,9 @@ function testSubproject( label, url, risTests ) {
// and start()'s the next test.
QUnit.config.testTimeout = 20 * 1000; // 20 seconds

// Enforce an "expect" argument or expect() call in all test bodies.
QUnit.config.requireExpects = true;

/**
* Load the TestSwarm listener if swarmURL is in the address.
*/
@@ -782,7 +782,7 @@ if ( jQuery.fn.offset ) {
});
}

test("Do not append px to 'fill-opacity' #9548", 1, function() {
test("Do not append px to 'fill-opacity' #9548", function() {
expect( 1 );

var $div = jQuery("<div>").appendTo("#qunit-fixture").css("fill-opacity", 1);
@@ -791,7 +791,7 @@ test("Do not append px to 'fill-opacity' #9548", 1, function() {

test("css('width') and css('height') should respect box-sizing, see #11004", function() {
expect( 4 );

var el_dis = jQuery("<div style='width:300px;height:300px;margin:2px;padding:2px;-moz-box-sizing:border-box;-webkit-box-sizing:border-box;box-sizing:border-box;'>test</div>"),
el = el_dis.clone().appendTo("#qunit-fixture");

1 comment on commit 611d766

@Krinkle

This comment has been minimized.

Copy link
Member

commented on 611d766 Oct 15, 2012

This commit introduced errors IE6-8 (some were already fixed since then, but the following is still happening):

support:
Expected 1 assertions, but 30 were run 
Please sign in to comment.
You can’t perform that action at this time.