confluentinc / confluentinc/vscode

Add unit tests over `fetchResults` in `FlinkStatementResultsManager`

Open
#1,763 0 comments 0 reactions 0 assignees View on GitHub
testing
Dominant language
TypeScript
Stars
34
Forks
17
Avg merge
1d 22h
Merged PRs (30d)
8

Description

The existing tests over `FlinkStatementResultsManager` test the "system" as a whole. While that may be sufficient, it would be also be nice to have unit tests over the core logic of `FlinkStatementResultsManager` in the `fetchResults`.

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.