Skip to content

Conversation

@dcalhoun
Copy link
Member

What?

Address JavaScript test failures after merging #286.

 FAIL  src/utils/editor.test.jsx [ src/utils/editor.test.jsx ]
TypeError: Module "file:./GutenbergKit/node_modules/@wordpress/blocks/build-module/api/i18n-block.json" needs an import attribute of type "json"

Why?

Stablize CI checks.

How?

Address an import error by mocking the relevant @wordpress modules. This is likely incompatibility between Vitest configuration and the packages.

Testing Instructions

CI checks should pass.

Accessibility Testing Instructions

N/A, no UX changes.

Screenshots or screencast

N/A, no visual changes.

Address an import error by mocking the relevant `@wordpress` modules.
This is likely incompatibility between Vitest configuration and the
packages.

```
 FAIL  src/utils/editor.test.jsx [ src/utils/editor.test.jsx ]
TypeError: Module "file:./GutenbergKit/node_modules/@wordpress/blocks/build-module/api/i18n-block.json" needs an import attribute of type "json"
```
@dcalhoun dcalhoun added the [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. label Jan 21, 2026
@dcalhoun dcalhoun marked this pull request as ready for review January 21, 2026 22:06
@dcalhoun dcalhoun enabled auto-merge (squash) January 21, 2026 22:06
@dcalhoun dcalhoun requested a review from adalpari January 21, 2026 22:06
Copy link

@adalpari adalpari left a comment

Choose a reason for hiding this comment

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

LGTM!

@dcalhoun dcalhoun merged commit d8e8a82 into trunk Jan 22, 2026
12 checks passed
@dcalhoun dcalhoun deleted the test/fix-javascript-test-failures branch January 22, 2026 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants