MetaMask / MetaMask/snaps

Unit test `getSnapChecksum` more closely

Open
#1,958 1 comment 0 reactions 0 assignees View on GitHub
snaps-deferred snaps-test
Dominant language
TypeScript
Stars
853
Forks
662
Avg merge
2d 8h
Merged PRs (30d)
8

Description

We should write unit tests specifically for `getSnapChecksum` that test edge-cases and require new tests to be added when new functionality is added that should be taken into account for the checksum.

Contributor guide

Open the contributing guide

Research direction

Search the repository for getSnapChecksum and inspect its implementation alongside any existing tests. Add focused unit coverage for edge cases and ensure future checksum-relevant functionality has corresponding tests; done means the relevant test suite passes with checksum behavior covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.