fix(utils): handle rounding of small trend values

code-pushup · Sep 12, 2024 · c23e38e · c23e38e
1 parent 3dde651
commit c23e38e
Show file tree

Hide file tree

Showing 9 changed files with 116 additions and 53 deletions.
diff --git a/packages/utils/src/lib/reports/__snapshots__/report-diff-added.md b/packages/utils/src/lib/reports/__snapshots__/report-diff-added.md
@@ -4,11 +4,11 @@
 
 ## 🏷️ Categories
 
-| 🏷️ Category   | ⭐ Previous score | ⭐ Current score |                         🔄 Score change                          |
-| :------------- | :--------------: | :-------------: | :--------------------------------------------------------------: |
-| Bug prevention |      🟡 68       |    🟡 **63**    | ![↓ −5](https://img.shields.io/badge/%E2%86%93%20%E2%88%925-red) |
-| Performance    |    _n/a (\*)_    |    🟢 **94**    |                            _n/a (\*)_                            |
-| Code style     |      🟡 54       |    🟡 **54**    |                                –                                 |
+| 🏷️ Category   | ⭐ Previous score | ⭐ Current score |                           🔄 Score change                            |
+| :------------- | :--------------: | :-------------: | :------------------------------------------------------------------: |
+| Bug prevention |      🟡 68       |    🟡 **68**    | ![↓ −0.1](https://img.shields.io/badge/%E2%86%93%20%E2%88%920.1-red) |
+| Performance    |    _n/a (\*)_    |    🟢 **94**    |                              _n/a (\*)_                              |
+| Code style     |      🟡 54       |    🟡 **54**    |                                  –                                   |
 
 _(\*) New category._
 

diff --git a/packages/utils/src/lib/reports/__snapshots__/report-diff-improved.md b/packages/utils/src/lib/reports/__snapshots__/report-diff-improved.md
@@ -33,9 +33,9 @@
 | [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Enforce a maximum number of lines of code in a function](https://eslint.org/docs/latest/rules/max-lines-per-function)                               |   🟥 1 warning    |  🟩 **passed**   | ![↓ −100 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%92100%E2%80%89%25-green) |
 | [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Require `const` declarations for variables that are never reassigned after declared](https://eslint.org/docs/latest/rules/prefer-const)             |   🟥 1 warning    |  🟩 **passed**   | ![↓ −100 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%92100%E2%80%89%25-green) |
 | [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Disallow missing `key` props in iterators/collection literals](https://github.com/jsx-eslint/eslint-plugin-react/tree/master/docs/rules/jsx-key.md) |   🟥 1 warning    |  🟩 **passed**   | ![↓ −100 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%92100%E2%80%89%25-green) |
-| Lighthouse                                                         | [Largest Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/largest-contentful-paint/)                                       |     🟨 1.5 s      |   🟨 **1.4 s**   |   ![↓ −8 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%928%E2%80%89%25-green)   |
-| Lighthouse                                                         | [First Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/first-contentful-paint/)                                           |     🟨 1.2 s      |   🟨 **1.1 s**   |   ![↓ −4 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%924%E2%80%89%25-green)   |
-| Lighthouse                                                         | [Speed Index](https://developer.chrome.com/docs/lighthouse/performance/speed-index/)                                                                 |     🟩 1.2 s      |   🟩 **1.1 s**   |   ![↓ −4 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%924%E2%80%89%25-green)   |
+| Lighthouse                                                         | [Largest Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/largest-contentful-paint/)                                       |     🟨 1.5 s      |   🟨 **1.4 s**   | ![↓ −8.2 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%928.2%E2%80%89%25-green) |
+| Lighthouse                                                         | [First Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/first-contentful-paint/)                                           |     🟨 1.2 s      |   🟨 **1.1 s**   | ![↓ −3.6 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%923.6%E2%80%89%25-green) |
+| Lighthouse                                                         | [Speed Index](https://developer.chrome.com/docs/lighthouse/performance/speed-index/)                                                                 |     🟩 1.2 s      |   🟩 **1.1 s**   | ![↓ −3.6 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%923.6%E2%80%89%25-green) |
 
 41 other audits are unchanged.
 

diff --git a/packages/utils/src/lib/reports/__snapshots__/report-diff-mixed.md b/packages/utils/src/lib/reports/__snapshots__/report-diff-mixed.md
@@ -4,11 +4,11 @@
 
 ## 🏷️ Categories
 
-| 🏷️ Category   | ⭐ Previous score | ⭐ Current score |                         🔄 Score change                          |
-| :------------- | :--------------: | :-------------: | :--------------------------------------------------------------: |
-| Bug prevention |      🟡 68       |    🟡 **63**    | ![↓ −5](https://img.shields.io/badge/%E2%86%93%20%E2%88%925-red) |
-| Performance    |      🟢 92       |    🟢 **94**    |   ![↑ +2](https://img.shields.io/badge/%E2%86%91%20%2B2-green)   |
-| Code style     |      🟡 54       |    🟡 **54**    |                                –                                 |
+| 🏷️ Category   | ⭐ Previous score | ⭐ Current score |                           🔄 Score change                            |
+| :------------- | :--------------: | :-------------: | :------------------------------------------------------------------: |
+| Performance    |      🟢 92       |    🟢 **94**    |     ![↑ +2](https://img.shields.io/badge/%E2%86%91%20%2B2-green)     |
+| Bug prevention |      🟡 68       |    🟡 **68**    | ![↓ −0.1](https://img.shields.io/badge/%E2%86%93%20%E2%88%920.1-red) |
+| Code style     |      🟡 54       |    🟡 **54**    |                                  –                                   |
 
 <details>
 <summary>👍 <strong>1</strong> group improved, 👍 <strong>3</strong> audits improved, 👎 <strong>1</strong> audit regressed</summary>
@@ -23,12 +23,12 @@
 
 ## 🛡️ Audits
 
-| 🔌 Plugin                                                          | 🛡️ Audit                                                                                                      | 📏 Previous value | 📏 Current value |                                 🔄 Value change                                  |
-| :----------------------------------------------------------------- | :------------------------------------------------------------------------------------------------------------- | :---------------: | :--------------: | :------------------------------------------------------------------------------: |
-| [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Disallow unused variables](https://eslint.org/docs/latest/rules/no-unused-vars)                               |     🟩 passed     |  🟥 **1 error**  | ![↑ +∞ %](https://img.shields.io/badge/%E2%86%91%20%2B%E2%88%9E%E2%80%89%25-red) |
-| Lighthouse                                                         | [Largest Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/largest-contentful-paint/) |     🟨 1.5 s      |   🟨 **1.4 s**   | ![↓ −8 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%928%E2%80%89%25-green) |
-| Lighthouse                                                         | [First Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/first-contentful-paint/)     |     🟨 1.2 s      |   🟨 **1.1 s**   | ![↓ −4 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%924%E2%80%89%25-green) |
-| Lighthouse                                                         | [Speed Index](https://developer.chrome.com/docs/lighthouse/performance/speed-index/)                           |     🟩 1.2 s      |   🟩 **1.1 s**   | ![↓ −4 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%924%E2%80%89%25-green) |
+| 🔌 Plugin                                                          | 🛡️ Audit                                                                                                      | 📏 Previous value | 📏 Current value |                                   🔄 Value change                                    |
+| :----------------------------------------------------------------- | :------------------------------------------------------------------------------------------------------------- | :---------------: | :--------------: | :----------------------------------------------------------------------------------: |
+| [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Disallow unused variables](https://eslint.org/docs/latest/rules/no-unused-vars)                               |     🟩 passed     |  🟥 **1 error**  |   ![↑ +∞ %](https://img.shields.io/badge/%E2%86%91%20%2B%E2%88%9E%E2%80%89%25-red)   |
+| Lighthouse                                                         | [Largest Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/largest-contentful-paint/) |     🟨 1.5 s      |   🟨 **1.4 s**   | ![↓ −8.2 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%928.2%E2%80%89%25-green) |
+| Lighthouse                                                         | [First Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/first-contentful-paint/)     |     🟨 1.2 s      |   🟨 **1.1 s**   | ![↓ −3.6 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%923.6%E2%80%89%25-green) |
+| Lighthouse                                                         | [Speed Index](https://developer.chrome.com/docs/lighthouse/performance/speed-index/)                           |     🟩 1.2 s      |   🟩 **1.1 s**   | ![↓ −3.6 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%923.6%E2%80%89%25-green) |
 
 49 other audits are unchanged.
 

diff --git a/packages/utils/src/lib/reports/__snapshots__/report-diff-monorepo-with-portal.md b/packages/utils/src/lib/reports/__snapshots__/report-diff-monorepo-with-portal.md
@@ -37,9 +37,9 @@
 | [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Enforce a maximum number of lines of code in a function](https://eslint.org/docs/latest/rules/max-lines-per-function)                               |   🟥 1 warning    |  🟩 **passed**   | ![↓ −100 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%92100%E2%80%89%25-green) |
 | [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Require `const` declarations for variables that are never reassigned after declared](https://eslint.org/docs/latest/rules/prefer-const)             |   🟥 1 warning    |  🟩 **passed**   | ![↓ −100 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%92100%E2%80%89%25-green) |
 | [ESLint](https://www.npmjs.com/package/@code-pushup/eslint-plugin) | [Disallow missing `key` props in iterators/collection literals](https://github.com/jsx-eslint/eslint-plugin-react/tree/master/docs/rules/jsx-key.md) |   🟥 1 warning    |  🟩 **passed**   | ![↓ −100 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%92100%E2%80%89%25-green) |
-| Lighthouse                                                         | [Largest Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/largest-contentful-paint/)                                       |     🟨 1.5 s      |   🟨 **1.4 s**   |   ![↓ −8 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%928%E2%80%89%25-green)   |
-| Lighthouse                                                         | [First Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/first-contentful-paint/)                                           |     🟨 1.2 s      |   🟨 **1.1 s**   |   ![↓ −4 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%924%E2%80%89%25-green)   |
-| Lighthouse                                                         | [Speed Index](https://developer.chrome.com/docs/lighthouse/performance/speed-index/)                                                                 |     🟩 1.2 s      |   🟩 **1.1 s**   |   ![↓ −4 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%924%E2%80%89%25-green)   |
+| Lighthouse                                                         | [Largest Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/largest-contentful-paint/)                                       |     🟨 1.5 s      |   🟨 **1.4 s**   | ![↓ −8.2 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%928.2%E2%80%89%25-green) |
+| Lighthouse                                                         | [First Contentful Paint](https://developer.chrome.com/docs/lighthouse/performance/first-contentful-paint/)                                           |     🟨 1.2 s      |   🟨 **1.1 s**   | ![↓ −3.6 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%923.6%E2%80%89%25-green) |
+| Lighthouse                                                         | [Speed Index](https://developer.chrome.com/docs/lighthouse/performance/speed-index/)                                                                 |     🟩 1.2 s      |   🟩 **1.1 s**   | ![↓ −3.6 %](https://img.shields.io/badge/%E2%86%93%20%E2%88%923.6%E2%80%89%25-green) |
 
 41 other audits are unchanged.