chore: mock import module in favour of bundle require by BioPhoton · Pull Request #1234 · code-pushup/cli

BioPhoton · 2026-02-02T20:00:37Z

This PR includes:

mock importModule instead of bundleRequire to isolate the 3rd party lib in our codebase
duplicate config files to stop relying on centralised test fixtures
reduce tests in cli to minimum.

Problem:
Having a centralised mock folder makes it hard to maintain tests as we can’t easily tell who uses them. Furthermore adding different test fixtures in the same shared project introduces unnecessary dependency to projects.

nx-cloud · 2026-02-02T20:02:55Z

View your CI Pipeline Execution ↗ for commit da60063

Command	Status	Duration	Result
`nx run ci:code-pushup -- merge-diffs --files=/h...`	✅ Succeeded	5s	View ↗
`nx run-many --targets=code-pushup --parallel=fa...`	✅ Succeeded	1m 25s	View ↗
`nx run-many --targets=code-pushup --parallel=fa...`	✅ Succeeded	12m 47s	View ↗
`nx run-many -t unit-test,int-test`	✅ Succeeded	3s	View ↗

☁️ Nx Cloud last updated this comment at 2026-02-08 12:50:57 UTC

pkg-pr-new · 2026-02-03T14:26:45Z

Open in StackBlitz

@code-pushup/ci

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/ci@1234

@code-pushup/cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/cli@1234

@code-pushup/core

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/core@1234

@code-pushup/create-cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/create-cli@1234

@code-pushup/models

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/models@1234

@code-pushup/nx-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/nx-plugin@1234

@code-pushup/axe-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/axe-plugin@1234

@code-pushup/coverage-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/coverage-plugin@1234

@code-pushup/eslint-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/eslint-plugin@1234

@code-pushup/js-packages-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/js-packages-plugin@1234

@code-pushup/jsdocs-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/jsdocs-plugin@1234

@code-pushup/lighthouse-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/lighthouse-plugin@1234

@code-pushup/typescript-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/typescript-plugin@1234

@code-pushup/utils

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/utils@1234

commit: da60063

github-actions · 2026-02-03T14:43:26Z

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit 490bec9 with previous commit 3dbf378.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category	⭐ Previous score	⭐ Current score	🔄 Score change
Updates	🟡 78	🟡 75
Performance	🔴 37	🔴 39
Code coverage	🟢 93	🟢 93
Bug prevention	🟡 75	🟡 75
Axe Accessibility	🟡 88	🟡 88
Code style	🟢 100	🟢 100	–
Security	🔴 47	🔴 47	–
Type Safety	🟡 67	🟡 67	–
Miscellaneous	🟡 67	🟡 67	–
Documentation	🟡 53	🟡 53	–
Accessibility	🟢 92	🟢 92	–
Best Practices	🟢 100	🟢 100	–
SEO	🟢 92	🟢 92	–

👍 2 groups improved, 👎 1 group regressed, 👍 7 audits improved, 👎 1 audit regressed, 21 audits changed without impacting score

🗃️ Groups

🔌 Plugin	🗃️ Group	⭐ Previous score	⭐ Current score
JS packages	npm outdated dependencies	🟡 78	🟡 75
Lighthouse	Performance	🔴 37	🔴 39
Code coverage	Code coverage metrics	🟢 93	🟢 93

31 other groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value
Lighthouse	Defer offscreen images	🟨 Potential savings of 16 KiB	🟩 0
Lighthouse	Minify CSS	🟨 Potential savings of 10 KiB	🟩 0
Lighthouse	Speed Index	🟥 7.3 s	🟥 6.1 s
Lighthouse	First Contentful Paint	🟥 3.5 s	🟥 3.3 s
JS packages	Outdated npm prod dependencies.	🟨 20 outdated package versions (5 major, 11 minor, 4 patch)	🟨 20 outdated package versions (6 major, 10 minor, 4 patch)
Lighthouse	Total Blocking Time	🟥 1,310 ms	🟥 1,230 ms
Lighthouse	Time to Interactive	🟥 13.4 s	🟥 13.2 s
Code coverage	Branch coverage	🟨 89.2 %	🟨 89.2 %
Lighthouse	Avoids enormous network payloads	🟩 Total size was 2,119 KiB	🟩 Total size was 2,063 KiB
Lighthouse	Uses efficient cache policy on static assets	🟨 31 resources found	🟨 18 resources found
Lighthouse	Minimizes main-thread work	🟥 10.2 s	🟥 8.4 s
Lighthouse	Largest Contentful Paint	🟥 12.3 s	🟥 11.9 s
Lighthouse	Server Backend Latencies	🟩 1,220 ms	🟩 880 ms
Lighthouse	JavaScript execution time	🟥 3.2 s	🟥 2.9 s
Lighthouse	Metrics	🟩 100%	🟩 100%
Lighthouse	Reduce unused CSS	🟥 Potential savings of 146 KiB	🟥 Potential savings of 111 KiB
Lighthouse	Max Potential First Input Delay	🟥 1,080 ms	🟥 950 ms
Lighthouse	Initial server response time was short	🟩 Root document took 560 ms	🟩 Root document took 480 ms
Lighthouse	Avoids an excessive DOM size	🟥 2,366 elements	🟥 2,327 elements
Axe	`<li>` elements must be contained in a `<ul>` or `<ol>`	🟩 65 elements	🟩 52 elements
Axe	Interactive controls must not be nested	🟩 60 elements	🟩 47 elements
Axe	Elements must only use supported ARIA attributes	🟩 100 elements	🟩 89 elements
Axe	ARIA attributes must be used as specified for the element's role	🟩 100 elements	🟩 89 elements
Axe	Elements must only use permitted ARIA attributes	🟩 100 elements	🟩 89 elements
Axe	ARIA attributes must conform to valid names	🟩 100 elements	🟩 89 elements
Lighthouse	Network Round Trip Times	🟩 70 ms	🟩 70 ms
JS packages	Vulnerabilities for npm dev dependencies.	🟥 47 vulnerabilities (3 critical, 9 high, 32 moderate, 3 low)	🟥 48 vulnerabilities (3 critical, 9 high, 32 moderate, 4 low)
TypeScript	Semantic errors	🟥 43 errors	🟥 44 errors
Code coverage	Line coverage	🟩 92.8 %	🟩 92.8 %

650 other audits are unchanged.

github-actions · 2026-02-03T14:47:59Z

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit 490bec9 with previous commit 3dbf378.

💼 Project `cli`

🥳 Code PushUp report has improved.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category	⭐ Previous score	⭐ Current score	🔄 Score change
Code coverage	🟢 94	🟢 94

5 other categories are unchanged.

👍 1 group improved, 👍 1 audit improved

🗃️ Groups

🔌 Plugin	🗃️ Group	⭐ Previous score	⭐ Current score	🔄 Score change
Code coverage	Code coverage metrics	🟢 94	🟢 94

14 other groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value	🔄 Value change
Code coverage	Branch coverage	🟨 86.9 %	🟨 87.3 %

443 other audits are unchanged.

💼 Project `utils`

🥳 Code PushUp report has improved.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category	⭐ Previous score	⭐ Current score	🔄 Score change
Code coverage	🟢 95	🟢 95
Documentation	🟡 61	🟡 61

4 other categories are unchanged.

👍 2 groups improved, 👍 1 audit improved, 1 audit changed without impacting score

🗃️ Groups

🔌 Plugin	🗃️ Group	⭐ Previous score	⭐ Current score	🔄 Score change
Code coverage	Code coverage metrics	🟢 95	🟢 95
JSDocs coverage	Documentation coverage	🟡 61	🟡 61

13 other groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value	🔄 Value change
Code coverage	Branch coverage	🟩 91.9 %	🟩 91.9 %
Code coverage	Line coverage	🟩 97.8 %	🟩 97.8 %

442 other audits are unchanged.

💼 Project `plugin-coverage`

😟 Code PushUp report has regressed.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category	⭐ Previous score	⭐ Current score	🔄 Score change
Code coverage	🟢 92	🟢 92

5 other categories are unchanged.

👎 1 group regressed, 👎 1 audit regressed

🗃️ Groups

🔌 Plugin	🗃️ Group	⭐ Previous score	⭐ Current score	🔄 Score change
Code coverage	Code coverage metrics	🟢 92	🟢 92

14 other groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value	🔄 Value change
Code coverage	Line coverage	🟨 89.1 %	🟨 89.1 %

443 other audits are unchanged.

💼 Project `core`

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

All of 6 categories are unchanged.

1 audit changed without impacting score

🗃️ Groups

All of 15 groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value	🔄 Value change
Code coverage	Line coverage	🟩 94.8 %	🟩 94.8 %

443 other audits are unchanged.

💼 Project `plugin-lighthouse`

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

All of 6 categories are unchanged.

1 audit changed without impacting score

🗃️ Groups

All of 15 groups are unchanged.

🛡️ Audits

🔌 Plugin	🛡️ Audit	📏 Previous value	📏 Current value	🔄 Value change
Code coverage	Line coverage	🟩 99.8 %	🟩 99.8 %

443 other audits are unchanged.

9 other projects are unchanged.

packages/cli/mocks/configs/code-pushup.needs-tsconfig.config.ts

packages/plugin-lighthouse/src/lib/runner/utils.unit.test.ts

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

hanna-skryl

What was the problem with the centralized fixtures in testing/test-fixtures? The PR description says "stop relying on centralised test fixtures," but doesn't explain what issue that caused. Now the same config exists in both cli/mocks and core/mocks, so if the schema changes, both need to be updated separately.

BioPhoton · 2026-02-03T20:44:25Z

@hanna-skryl i updated the pr description. I will also go ahead and reduce the tests in CLI as mentioned above.

packages/cli/src/lib/implementation/core-config.middleware.int.test.ts

BioPhoton added 3 commits February 2, 2026 20:58

fix: avoid bundle require mocking in tests

7f473ec

fix: avoid depending on central mock files

1b205be

fix: delete centralised mock files

18a6c40

BioPhoton added 🔬 testing writing tests 🧹 maintenance labels Feb 2, 2026

BioPhoton mentioned this pull request Feb 3, 2026

feat: use jiti for plugin loading #1228

Draft

refactor: resolve lint issues

27f92ed

github-actions bot added 🧩 cli 🧩 core 🧩 utils 🧩 lighthouse-plugin 🧩 coverage-plugin 🛠️ tooling labels Feb 3, 2026

BioPhoton requested a review from hanna-skryl February 3, 2026 14:32

BioPhoton marked this pull request as ready for review February 3, 2026 14:32

BioPhoton requested a review from matejchalk as a code owner February 3, 2026 14:32

refactor: add defaults for backwards compat

b06c96b

hanna-skryl reviewed Feb 3, 2026

View reviewed changes

packages/cli/mocks/configs/code-pushup.needs-tsconfig.config.ts Show resolved Hide resolved

packages/plugin-lighthouse/src/lib/runner/utils.unit.test.ts Outdated Show resolved Hide resolved

BioPhoton and others added 3 commits February 3, 2026 19:02

Update packages/plugin-lighthouse/src/lib/runner/utils.unit.test.ts

30802a5

Co-authored-by: Hanna Skryl <80118140+hanna-skryl@users.noreply.github.com>

refactor: wip

e8c4526

refactor: wip

e2685b3

BioPhoton enabled auto-merge (squash) February 3, 2026 18:49

BioPhoton requested a review from hanna-skryl February 3, 2026 18:52

BioPhoton self-assigned this Feb 3, 2026

hanna-skryl reviewed Feb 3, 2026

View reviewed changes

refactor: remove redundant tests

48e900f

BioPhoton requested a review from hanna-skryl February 3, 2026 21:14

hanna-skryl reviewed Feb 4, 2026

View reviewed changes

packages/cli/src/lib/implementation/core-config.middleware.int.test.ts Outdated Show resolved Hide resolved

BioPhoton added 2 commits February 8, 2026 12:34

refactor: fix test description

6f99f0f

Merge branch 'main' into chore/mock-import-module

da60063

BioPhoton requested a review from hanna-skryl February 8, 2026 12:34

hanna-skryl approved these changes Feb 8, 2026

View reviewed changes

BioPhoton merged commit 3137af7 into main Feb 8, 2026
24 checks passed

BioPhoton deleted the chore/mock-import-module branch February 8, 2026 17:01

Conversation

BioPhoton commented Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nx-cloud bot commented Feb 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pkg-pr-new bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code PushUp

🏷️ Categories

🗃️ Groups

🛡️ Audits

Uh oh!

github-actions bot commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code PushUp

💼 Project cli

🗃️ Groups

🛡️ Audits

💼 Project utils

🗃️ Groups

🛡️ Audits

💼 Project plugin-coverage

🗃️ Groups

🛡️ Audits

💼 Project core

🗃️ Groups

🛡️ Audits

💼 Project plugin-lighthouse

🗃️ Groups

🛡️ Audits

Uh oh!

Uh oh!

Uh oh!

hanna-skryl left a comment

Choose a reason for hiding this comment

Uh oh!

BioPhoton commented Feb 3, 2026

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

BioPhoton commented Feb 2, 2026 •

edited

Loading

nx-cloud bot commented Feb 2, 2026 •

edited

Loading

pkg-pr-new bot commented Feb 3, 2026 •

edited

Loading

github-actions bot commented Feb 3, 2026 •

edited

Loading

github-actions bot commented Feb 3, 2026 •

edited

Loading

💼 Project `cli`

💼 Project `utils`

💼 Project `plugin-coverage`

💼 Project `core`

💼 Project `plugin-lighthouse`