Skip to content

Conversation

Tony133
Copy link
Contributor

@Tony133 Tony133 commented Jun 7, 2025

Proposals:

  • migrated errors.test.js from tap to node:test 🔥
  • added prefix .test 🔥
  • added @matteo.collina/snap package for snapshot testing 🔥

@Tony133
Copy link
Contributor Author

Tony133 commented Aug 14, 2025

Perhaps something needs to be configured on the pipeline for snapshots? 🤔 Since we now use @matteo.collina/snap package? 🤔

screenshot-pipeline-ci screenshot-test-ok

@Tony133 Tony133 changed the title test: migrated test/errors.test.js from tap to node:test test: migrated errors.test.js from tap to node:test Aug 18, 2025
Copy link
Collaborator

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina merged commit 94c4356 into mercurius-js:master Aug 18, 2025
10 checks passed
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