Skip to content

test(e2e): test CLI in addition to jest integration#880

Merged
oetr merged 2 commits into
mainfrom
fix_dependabot
Apr 20, 2026
Merged

test(e2e): test CLI in addition to jest integration#880
oetr merged 2 commits into
mainfrom
fix_dependabot

Conversation

@oetr
Copy link
Copy Markdown
Contributor

@oetr oetr commented Apr 20, 2026

  • previously, only Jest test was run in our e2e tests
  • dependabot upgraded yargs to 18 (ESM-first) and nothing failed, even though node version14 and 16 should fail with this change

- yargs 18+ is ESM-first and does not work on node 14 and 16
@oetr oetr marked this pull request as ready for review April 20, 2026 10:11
Copilot AI review requested due to automatic review settings April 20, 2026 10:11
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR strengthens end-to-end CI by adding a direct Jazzer.js CLI smoke test across multiple Node versions, ensuring incompatibilities (e.g., ESM-first yargs upgrades) are caught even when the Jest integration still passes.

Changes:

  • Add an end-to-end npm script to run a minimal CLI fuzz invocation (fuzz:cli).
  • Introduce a minimal CLI fuzz target (cli-smoke.cjs) used by the smoke test.
  • Update the PR workflow to run the CLI smoke test for each Node runtime (14–24), and configure Dependabot to ignore yargs >=18.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
end-to-end/package.json Adds fuzz:cli script to exercise the Jazzer.js CLI.
end-to-end/cli-smoke.cjs Adds a minimal fuzz target module used by the CLI smoke test.
.github/workflows/run-all-tests-pr.yaml Runs the new CLI smoke test for each Node version in the e2e job.
.github/dependabot.yml Prevents Dependabot from proposing yargs >=18 updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread end-to-end/cli-smoke.cjs Outdated
- before this, only jest was tested
@oetr oetr enabled auto-merge (rebase) April 20, 2026 10:46
@oetr oetr disabled auto-merge April 20, 2026 10:46
@oetr oetr merged commit 7db8c66 into main Apr 20, 2026
14 checks passed
@oetr oetr deleted the fix_dependabot branch April 20, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants