MetaMask / MetaMask/core

Improve test coverage for `src/gas/gas-util.ts`

Open
#734 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

team-wallet-framework wf-test-quality-improvement
Dominant language
TypeScript
Stars
413
Forks
308
Avg merge
1d 4h
Merged PRs (30d)
253

Description

Test coverage for this file is so low that changes are liable to go past the threshold we've set. At the moment we've disabled coverage checks for this file so that they don't break CI, but we should re-enable this and add the missing tests.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading src/gas/gas-util.ts and locating the coverage configuration or exclusions for this file. Add tests for the uncovered behavior, re-enable coverage checks for the file, and run the relevant test and CI coverage commands to confirm the threshold passes.

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
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.