Fix a some inaccuracies in the original test case for #7912.

- Use fresh div instead of one outside the test-fixture
- make sure the empty string test tests actually that (not 0% 0%)
- actually test for < -10000 (#7193)
- fixed some whitespace issues

diff --git a/test/unit/effects.js b/test/unit/effects.js
index 4bb92fc00120776f98225bfc5e85f7eac5d76527..c0a812f45caabcb49debc2f00380c668b2080225 100644 (file)
--- a/test/unit/effects.js
+++ b/test/unit/effects.js
@@ -558,42 +558,52 @@ jQuery.checkOverflowDisplay = function(){
        start();
 }
 
-test("jQuery.fx.prototype.cur()", function() {
-       expect(5);
-       var nothiddendiv = jQuery('#nothiddendiv').css({
-                       color: '#ABC',
-                       border: '5px solid black',
-                       left: 'auto',
-                       marginBottom: '11000px'
+test( "jQuery.fx.prototype.cur()", 6, function() {
+       var div = jQuery( "<div></div>" ).appendTo( "#main" ).css({
+                       color: "#ABC",
+                       border: "5px solid black",
+                       left: "auto",
+                       marginBottom: "-11000px"
                })[0];
 
        equals(
-               (new jQuery.fx( nothiddendiv, {}, 'color' )).cur(),
-               jQuery.css( nothiddendiv,'color' ),
+               ( new jQuery.fx( div, {}, "color" ) ).cur(),
+               jQuery.css( div, "color" ),
                "Return the same value as jQuery.css for complex properties (bug #7912)"
        );
 
        strictEqual(
-               (new jQuery.fx( nothiddendiv, {}, 'borderLeftWidth' )).cur(),
+               ( new jQuery.fx( div, {}, "borderLeftWidth" ) ).cur(),
                5,
                "Return simple values parsed as Float"
        );
 
+       // backgroundPosition actually returns 0% 0% in most browser
+       // this fakes a "" return
+       jQuery.cssHooks.backgroundPosition = {
+               get: function() {
+                       ok( true, "hook used" );
+                       return "";
+               }
+       };
+
        strictEqual(
-               (new jQuery.fx( nothiddendiv, {}, 'backgroundPosition' )).cur(),
+               ( new jQuery.fx( div, {}, "backgroundPosition" ) ).cur(),
                0,
-               'Return 0 when jQuery.css returns an empty string'
+               "Return 0 when jQuery.css returns an empty string"
        );
 
+       delete jQuery.cssHooks.backgroundPosition;
+
        strictEqual(
-               (new jQuery.fx( nothiddendiv, {}, 'left' )).cur(),
+               ( new jQuery.fx( div, {}, "left" ) ).cur(),
                0,
-               'Return 0 when jQuery.css returns "auto"'
+               "Return 0 when jQuery.css returns 'auto'"
        );
 
        equals(
-               (new jQuery.fx( nothiddendiv, {}, 'marginBottom' )).cur(),
-               11000,
+               ( new jQuery.fx( div, {}, "marginBottom" ) ).cur(),
+               -11000,
                "support negative values < -10000 (bug #7193)"
        );
 });