Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

Add working decryption failure test

Open
  • Is something wrong?
#75 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
typescript
Domain
cryptography, testing

Research direction

Read PR #74 and its linked comment first; the issue provides no file or test path. Locate the decryption failure test being removed, then ensure the replacement exercises failure to decrypt an existing payload rather than failure caused by a missing variable. Done means the test fails for the intended decryption reason and passes with the corrected behavior.

Written by the indexing model from the issue text.

Description

enhancement

Our current decryption failure test is incorrect, and is being removed:

It tests if an absent variable can decrypt a payload, and so it fails for the wrong reason: An error is thrown because alice does not exist, not because she cannot decrypt the variable.

Highlighted here:
https://github.com/MetaMask/eth-sig-util/pull/74#issuecomment-572330875

Dominant language
TypeScript
Stars
584
Forks
230
No merged PRs in 30d

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

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.

More from MetaMask/eth-sig-util

All issues in MetaMask/eth-sig-util

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.