From 81d829b3573e6b3581a7f127040aad84d345a832 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Mon, 18 Jun 2018 18:48:15 +0200 Subject: [PATCH] Test: enable a spec testing CSS whitespace preserving in Edge 17 In Edge 14-16 setting a style property to a whitespace-only value resets it to the default, forcing us to skip a relevant CSS test in Edge. Now that Edge 17 has fixed the issue we can re-enable this test there. Ref gh-3204 Closes gh-4101 --- test/unit/css.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/unit/css.js b/test/unit/css.js index 96a23172..a21a58b9 100644 --- a/test/unit/css.js +++ b/test/unit/css.js @@ -1348,11 +1348,11 @@ QUnit.test( "Keep the last style if the new one isn't recognized by the browser assert.equal( el.css( "position" ), "absolute", "The old style is kept when setting an unrecognized value" ); } ); -// Support: Edge 14 - 15 +// Support: Edge 14 - 16 only // Edge collapses whitespace-only values when setting a style property and // there is no easy way for us to work around it. Just skip the test there // and hope for the better future. -QUnit[ /\bedge\//i.test( navigator.userAgent ) ? "skip" : "test" ]( +QUnit[ /\bedge\/16\./i.test( navigator.userAgent ) ? "skip" : "test" ]( "Keep the last style if the new one is a non-empty whitespace (gh-3204)", function( assert ) { assert.expect( 1 ); -- 2.11.4.GIT