From: J.Son Date: Thu, 29 Feb 2024 23:40:41 +0000 (+0800) Subject: Tests: Use allowlist instead of whitelist X-Git-Tag: 4.0.0-beta.2~41 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2b97b6bbcfc67c234b86d41451aac7cdd778e855;p=jquery.git Tests: Use allowlist instead of whitelist Closes gh-5420 --- diff --git a/test/unit/css.js b/test/unit/css.js index 5567b7437..ea3055f77 100644 --- a/test/unit/css.js +++ b/test/unit/css.js @@ -1278,10 +1278,10 @@ QUnit.test( "Do not append px to most properties not accepting integer values", assert.equal( $div.css( "letter-spacing" ), "2px", "Do not append px to 'letter-spacing'" ); } ); -QUnit.test( "Append px to whitelisted properties", function( assert ) { +QUnit.test( "Append px to allowlisted properties", function( assert ) { var prop, $div = jQuery( "
" ).appendTo( "#qunit-fixture" ), - whitelist = { + allowlist = { margin: "marginTop", marginTop: undefined, marginRight: undefined, @@ -1314,10 +1314,10 @@ QUnit.test( "Append px to whitelisted properties", function( assert ) { borderLeftWidth: undefined }; - assert.expect( ( Object.keys( whitelist ).length ) * 2 ); + assert.expect( ( Object.keys( allowlist ).length ) * 2 ); - for ( prop in whitelist ) { - var propToCheck = whitelist[ prop ] || prop, + for ( prop in allowlist ) { + var propToCheck = allowlist[ prop ] || prop, kebabProp = prop.replace( /[A-Z]/g, function( match ) { return "-" + match.toLowerCase(); } ),