From e64272fa229dca709c8400f760a850cbf7739e24 Mon Sep 17 00:00:00 2001 From: Ludovico Fischer <43557+ludofischer@users.noreply.github.com> Date: Wed, 28 Apr 2021 14:20:17 +0200 Subject: [PATCH] Add tests for fixed edge cases (#1059) * test(postcss-merge-longhand): check invalid border property handling * test(postcss-convert-values): add test case for #958 Co-authored-by: Shipov Mikhail --- .../src/__tests__/index.js | 20 +++++++++++++++++++ .../src/__tests__/borders.js | 7 +++++++ 2 files changed, 27 insertions(+) diff --git a/packages/postcss-convert-values/src/__tests__/index.js b/packages/postcss-convert-values/src/__tests__/index.js index febabdb90..1ec0e2ed4 100644 --- a/packages/postcss-convert-values/src/__tests__/index.js +++ b/packages/postcss-convert-values/src/__tests__/index.js @@ -382,6 +382,26 @@ test( passthroughCSS('h1{top:0\\9\\0;left:0lightyear}') ); +test( + 'should not try to convert keyframe names in animation', + passthroughCSS( + 'h1{ -webkit-animation: e836684w2 } h2{ animation: e836684w2 }' + ) +); + +test( + 'should not try to convert keyframe names in animation (case 2)', + passthroughCSS( + ` +.e4yw0Q { + animation: e4yw0Q; +} + +@keyframes e4yw0Q {} + ` + ) +); + ['stroke-dasharray', 'stroke-dashoffset', 'stroke-width'].forEach( (property) => { test( diff --git a/packages/postcss-merge-longhand/src/__tests__/borders.js b/packages/postcss-merge-longhand/src/__tests__/borders.js index 39e99bc11..515436855 100644 --- a/packages/postcss-merge-longhand/src/__tests__/borders.js +++ b/packages/postcss-merge-longhand/src/__tests__/borders.js @@ -1203,6 +1203,13 @@ test( ) ); +test( + 'should not break border rules mixing custorm and regular properties', + passthroughCSS( + 'h1{border:var(--v1) solid var(--v2, #abc123);border-right-color:blue}' + ) +); + test( 'should overwrite some border-width props and save fallbacks and preserve case #648 2', processCSS(