Skip to content

Commit 13647e9

Browse files
rwaldronrwaldron
committed
Fixes assertion counts and dom element fixture issue
1 parent e18cad6 commit 13647e9

File tree

3 files changed

+23
-24
lines changed

3 files changed

+23
-24
lines changed

‎test/index.html

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,7 @@ <h2 id="qunit-userAgent"></h2>
229229
<div id="t6652">
230230
<div></div>
231231
</div>
232+
<div id="no-clone-exception"><object><embed></embed></object></div>
232233
</div>
233234

234235
<div id="tabindex-tests">
@@ -284,8 +285,6 @@ <h2 id="qunit-userAgent"></h2>
284285
</div>
285286

286287
<div id="fx-tests"></div>
287-
288-
<div id="no-clone-exception"><object><embed></embed></object></div>
289288
</div>
290289
</body>
291290
</html>

‎test/unit/attributes.js

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ var functionReturningObj = function(value) { return (function() { return value;
66

77
test("jQuery.attrFix/jQuery.propFix integrity test", function() {
88
expect(2);
9-
9+
1010
// This must be maintained and equal jQuery.attrFix when appropriate
1111
// Ensure that accidental or erroneous property
1212
// overwrites don't occur
@@ -52,7 +52,7 @@ test("attr(String)", function() {
5252
equals( jQuery("<div value='t'></div>").attr("value"), "t", "Check setting custom attr named 'value' on a div" );
5353
equals( jQuery("#form").attr("blah", "blah").attr("blah"), "blah", "Set non-existant attribute on a form" );
5454
equals( jQuery("#foo").attr("height"), undefined, "Non existent height attribute should return undefined" );
55-
55+
5656
// [7472] & [3113] (form contains an input with name="action" or name="id")
5757
var extras = jQuery("<input name='id' name='name' /><input id='target' name='target' />").appendTo("#testForm");
5858
equals( jQuery("#form").attr("action","newformaction").attr("action"), "newformaction", "Check that action attribute was changed" );
@@ -62,7 +62,7 @@ test("attr(String)", function() {
6262
// Bug #3685 (form contains input with name="name")
6363
equals( jQuery("#testForm").attr("name"), undefined, "Retrieving name does not retrieve input with name=name" );
6464
extras.remove();
65-
65+
6666
equals( jQuery("#text1").attr("maxlength"), "30", "Check for maxlength attribute" );
6767
equals( jQuery("#text1").attr("maxLength"), "30", "Check for maxLength attribute" );
6868
equals( jQuery("#area1").attr("maxLength"), "30", "Check for maxLength attribute" );
@@ -255,7 +255,7 @@ test("attr(String, Object)", function() {
255255
commentNode = document.createComment("some comment"),
256256
textNode = document.createTextNode("some text"),
257257
obj = {};
258-
258+
259259
jQuery.each( [commentNode, textNode, attributeNode], function( i, elem ) {
260260
var $elem = jQuery( elem );
261261
$elem.attr( "nonexisting", "foo" );
@@ -295,7 +295,7 @@ test("attr(String, Object)", function() {
295295
j.removeAttr("name");
296296

297297
QUnit.reset();
298-
298+
299299
// Type
300300
var type = jQuery("#check2").attr("type");
301301
var thrown = false;
@@ -457,7 +457,7 @@ test("removeAttr(String)", function() {
457457
equals( jQuery("#foo").attr("style", "position:absolute;").removeAttr("style").attr("style"), undefined, "Check removing style attribute" );
458458
equals( jQuery("#form").attr("style", "position:absolute;").removeAttr("style").attr("style"), undefined, "Check removing style attribute on a form" );
459459
equals( jQuery("#fx-test-group").attr("height", "3px").removeAttr("height").css("height"), "1px", "Removing height attribute has no effect on height set with style attribute" );
460-
460+
461461
jQuery("#check1").removeAttr("checked").prop("checked", true).removeAttr("checked");
462462
equals( document.getElementById("check1").checked, false, "removeAttr sets boolean properties to false" );
463463
jQuery("#text1").prop("readOnly", true).removeAttr("readonly");
@@ -669,11 +669,11 @@ test("val()", function() {
669669
var $button = jQuery("<button value='foobar'>text</button>").insertAfter("#button");
670670
equals( $button.val(), "foobar", "Value retrieval on a button does not return innerHTML" );
671671
equals( $button.val("baz").html(), "text", "Setting the value does not change innerHTML" );
672-
672+
673673
equals( jQuery("<option/>").val("test").attr("value"), "test", "Setting value sets the value attribute" );
674674
});
675675

676-
if ( "value" in document.createElement("meter") &&
676+
if ( "value" in document.createElement("meter") &&
677677
"value" in document.createElement("progress") ) {
678678

679679
test("val() respects numbers without exception (Bug #9319)", function() {
@@ -820,7 +820,7 @@ test("val(select) after form.reset() (Bug #2551)", function() {
820820
same( jQuery("#select3").val(), ["1", "2"], "Call val() on a multiple=\"multiple\" select" );
821821

822822
jQuery("#kk").remove();
823-
});
823+
});
824824

825825
var testAddClass = function(valueObj) {
826826
expect(9);
@@ -852,7 +852,7 @@ var testAddClass = function(valueObj) {
852852
div.attr("class", "foo");
853853
div.addClass( valueObj("bar baz") );
854854
equals( div.attr("class"), "foo bar baz", "Make sure there isn't too much trimming." );
855-
855+
856856
div.removeClass();
857857
div.addClass( valueObj("foo") ).addClass( valueObj("foo") )
858858
equal( div.attr("class"), "foo", "Do not add the same class twice in separate calls." );
@@ -876,11 +876,11 @@ test("addClass(Function)", function() {
876876
});
877877

878878
test("addClass(Function) with incoming value", function() {
879-
expect(45);
879+
expect(48);
880880
var div = jQuery("div"), old = div.map(function(){
881881
return jQuery(this).attr("class") || "";
882882
});
883-
883+
884884
div.addClass(function(i, val) {
885885
if ( this.id !== "_firebugConsole") {
886886
equals( val, old[i], "Make sure the incoming value is correct." );
@@ -948,7 +948,7 @@ test("removeClass(Function) - simple", function() {
948948
});
949949

950950
test("removeClass(Function) with incoming value", function() {
951-
expect(45);
951+
expect(48);
952952

953953
var $divs = jQuery("div").addClass("test"), old = $divs.map(function(){
954954
return jQuery(this).attr("class");
@@ -1120,7 +1120,7 @@ test("addClass, removeClass, hasClass", function() {
11201120
test("contents().hasClass() returns correct values", function() {
11211121
expect(2);
11221122

1123-
var $div = jQuery("<div><span class='foo'></span><!-- comment -->text</div>"),
1123+
var $div = jQuery("<div><span class='foo'></span><!-- comment -->text</div>"),
11241124
$contents = $div.contents();
11251125

11261126
ok( $contents.hasClass("foo"), "Found 'foo' in $contents" );

‎test/unit/traversing.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ test("find(String)", function() {
1515

1616
test("find(node|jQuery object)", function() {
1717
expect( 11 );
18-
18+
1919
var $foo = jQuery("#foo"),
2020
$blog = jQuery(".blogTest"),
2121
$first = jQuery("#first"),
@@ -29,12 +29,12 @@ test("find(node|jQuery object)", function() {
2929
ok( $foo.find( $two ).is(".blogTest"), "Find returns only nodes within #foo" );
3030
ok( $fooTwo.find( $blog ).is(".blogTest"), "Blog is part of the collection, but also within foo" );
3131
ok( $fooTwo.find( $blog[0] ).is(".blogTest"), "Blog is part of the collection, but also within foo(node)" );
32-
32+
3333
equals( $two.find( $foo ).length, 0, "Foo is not in two elements" );
3434
equals( $two.find( $foo[0] ).length, 0, "Foo is not in two elements(node)" );
3535
equals( $two.find( $first ).length, 0, "first is in the collection and not within two" );
3636
equals( $two.find( $first ).length, 0, "first is in the collection and not within two(node)" );
37-
37+
3838
});
3939

4040
test("is(String|undefined)", function() {
@@ -63,7 +63,7 @@ test("is(String|undefined)", function() {
6363
ok( !jQuery("#foo").is(""), "Expected false for an invalid expression - \"\"" );
6464
ok( !jQuery("#foo").is(undefined), "Expected false for an invalid expression - undefined" );
6565
ok( !jQuery("#foo").is({ plain: "object" }), "Check passing invalid object" );
66-
66+
6767
// test is() with comma-seperated expressions
6868
ok( jQuery("#en").is("[lang=\"en\"],[lang=\"de\"]"), "Comma-seperated; Check for lang attribute: Expect en or de" );
6969
ok( jQuery("#en").is("[lang=\"de\"],[lang=\"en\"]"), "Comma-seperated; Check for lang attribute: Expect en or de" );
@@ -88,7 +88,7 @@ test("is(jQuery)", function() {
8888
ok( !jQuery("#radio1").is( jQuery("input:checked") ), "Check for pseudoclass: Expected not checked" );
8989
ok( jQuery("#foo").is( jQuery("div:has(p)") ), "Check for child: Expected a child 'p' element" );
9090
ok( !jQuery("#foo").is( jQuery("div:has(ul)") ), "Check for child: Did not expect 'ul' element" );
91-
91+
9292
// Some raw elements
9393
ok( jQuery("#form").is( jQuery("form")[0] ), "Check for element: A form is a form" );
9494
ok( !jQuery("#form").is( jQuery("div")[0] ), "Check for element: A form is not a div" );
@@ -102,7 +102,7 @@ test("index()", function() {
102102
expect( 2 );
103103

104104
equal( jQuery("#text2").index(), 2, "Returns the index of a child amongst its siblings" );
105-
105+
106106
equal( jQuery("<div/>").index(), -1, "Node without parent returns -1" );
107107
});
108108

@@ -142,7 +142,7 @@ test("filter(Selector|undefined)", function() {
142142
same( jQuery("#form input").filter(":checked").get(), q("radio2", "check1"), "filter(String)" );
143143
same( jQuery("p").filter("#ap, #sndp").get(), q("ap", "sndp"), "filter('String, String')" );
144144
same( jQuery("p").filter("#ap,#sndp").get(), q("ap", "sndp"), "filter('String,String')" );
145-
145+
146146
same( jQuery("p").filter(null).get(), [], "filter(null) should return an empty jQuery object");
147147
same( jQuery("p").filter(undefined).get(), [], "filter(undefined) should return an empty jQuery object");
148148
same( jQuery("p").filter(0).get(), [], "filter(0) should return an empty jQuery object");
@@ -532,7 +532,7 @@ test("add(String|Element|Array|undefined)", function() {
532532

533533
test("add(String, Context)", function() {
534534
expect(6);
535-
535+
536536
deepEqual( jQuery( "#firstp" ).add( "#ap" ).get(), q( "firstp", "ap" ), "Add selector to selector " );
537537
deepEqual( jQuery( document.getElementById("firstp") ).add( "#ap" ).get(), q( "firstp", "ap" ), "Add gEBId to selector" );
538538
deepEqual( jQuery( document.getElementById("firstp") ).add( document.getElementById("ap") ).get(), q( "firstp", "ap" ), "Add gEBId to gEBId" );

0 commit comments

Comments
 (0)