Skip to content

test: fix misspelled test identifiers - #4106

Open
wangjingshuiku wants to merge 1 commit into
MetaMask:mainfrom
wangjingshuiku:test-fix-identifier-spelling
Open

test: fix misspelled test identifiers#4106
wangjingshuiku wants to merge 1 commit into
MetaMask:mainfrom
wangjingshuiku:test-fix-identifier-spelling

Conversation

@wangjingshuiku

@wangjingshuiku wangjingshuiku commented Aug 27, 2026

Copy link
Copy Markdown

Fix two spelling errors in test-only identifiers and descriptions:

  • checkumFileschecksumFiles in the checksum test suite description;
  • secondRequstsecondRequest in the sandbox state test.

Assertions, fixtures, snapshots, and production behavior are unchanged. Deprecated compatibility identifiers with historical spelling are intentionally left intact.

Validation:

`npx prettier --check packages/snaps-utils/src/checksum.test.ts packages/snaps-sandbox/src/state.test.ts`

Note

Low Risk
Test-only identifier and describe-block renames with no production or behavioral impact.

Overview
This PR only renames misspelled test-only symbols so names match the real APIs and intent. No assertions, fixtures, or runtime behavior change.

In packages/snaps-sandbox/src/state.test.ts, the history update test renames secondRequst to secondRequest everywhere it is declared, stored in persistedHistoryAtom, and spread into updatedRequest.

In packages/snaps-utils/src/checksum.test.ts, the outer describe label is corrected from checkumFiles to checksumFiles, aligning the suite title with the imported function under test.

Reviewed by Cursor Bugbot for commit ae524a7. Bugbot is set up for automated code reviews on this repo. Configure here.

Signed-off-by: wangjingshuiku <wangjingshuiku@163.com>
@wangjingshuiku
wangjingshuiku requested a review from a team as a code owner August 27, 2026 17:23
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.

1 participant