confluentinc / confluentinc/vscode

Boost unit test coverage for `consume.ts`

Open
#3,253 0 comments 0 reactions 1 assignee Claimed by @noeldevelops View on GitHub
message viewer
Dominant language
TypeScript
Stars
34
Forks
17
Avg merge
1d 22h
Merged PRs (30d)
8

Description

Add unit test coverage for `consume.ts` that at minimum tests simple functions and sets up required state (think of this ticket as a kind of prototype for what would be required to cover more deeply, and a small % win).

Some options for small refactoring wins: Move command registration out of this file? Add a Message Viewer Provider class for state? See also tickets under https://github.com/confluentinc/vscode/issues/2049 which we may want to link this under.

Since it might take some large refactoring to boost coverage to our goal of 80% per file, E2E tests are a path forward to give confidence for feature additions and maintenance. [Boosting E2E coverage](https://github.com/confluentinc/vscode/issues/3252) can be done in parallel

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.