MetaMask / MetaMask/detect-provider
Test harness is outdated
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 238
- Forks
- 63
- PR merge metrics
- No merged PRs in 30d
Description
To achieve consistency with our other libraries, we should
- Convert the tests in this repo from Tape to Jest (also using TypeScript instead of JavaScript)
- Add `@types/jest`, `jest`, `jest-it-up`, and `ts-jest` as dependencies (use same versions as in module template)
- Update the `test` package script to run `jest && jest-it-up` instead of `tape` (copy from the module template)
- Add a `test:watch` package script (copy from the module template)
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the repository's existing Tape-based tests and package scripts, then compare them with the module template referenced in the issue. Convert the test suite and dependencies to the template's Jest and TypeScript setup, update the test scripts, and confirm the regular and watch workflows are defined as requested.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, typescript
- Domain
- testing, tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100