luk707 / luk707/imprest

Add test coverage across the project

Open
#8 0 comments 0 reactions 1 assignee View on GitHub

@luk707 is already working on this.

Since Aug 1, 2019.

enhancement
Dominant language
TypeScript
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

**Is your feature request related to a problem? Please describe.**
There are a few methods in the project I’m not entirely confident in their implementation, specifically for edge cases.

**Describe the solution you'd like**
Adding test coverage, with Jest or Ava would eliminate these worries, and also help improve the overall quality of the codebase.

**Additional context**
Solution must be TypeScript compatible, and ideally support a common reporting interface so that coverage badges can be added to the README. This may also be an ideal opportunity to take a look at Drone CI.

Contributor guide

Open the contributing guide

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.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.