airbnb / airbnb/streamalert

Add Unit Test Coverage for StreamAlert CLI Component

Open
#995 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
2.9k
Forks
324
PR merge metrics
No merged PRs in 30d

Description

## Background

As of #992, StreamAlert CLI is now modular with commands that are each modularized. @ryandeivert has mentioned that we intentionally omitted the [StreamAlert CLI package](https://github.com/airbnb/streamalert/blob/master/setup.cfg#L32) from unit test coverage in the past because it wasn't considered necessary for core stability.

## Desired Change

We should figure out how to add Unit Test Coverage to StreamAlert CLI.

Contributor guide

Open the contributing guide

Research direction

Start with the StreamAlert CLI package referenced in setup.cfg and review the modular command changes from #992. Identify the CLI commands and current test structure, then define coverage for the package's behavior; done means the CLI has meaningful unit tests without reducing existing coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
cli, testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.