influxdata / influxdata/influxdb

Add TimeMachine Reducer Test

Open
#16,294 0 comments 0 reactions 0 assignees View on GitHub
area/tests testing
Dominant language
Rust
Stars
31.7k
Forks
3.7k
Avg merge
13h 37m
Merged PRs (30d)
8

Description

The logic to remove tag selector values in the reducer is untested. https://github.com/influxdata/influxdb/blob/master/ui/src/timeMachine/reducers/index.ts#L832.

Add some tests to it, yo!

Originally ideated at: https://github.com/influxdata/influxdb/pull/16226/files#diff-afb81e9ccfb919d0134e48bd7176aa6bR839

Contributor guide

Open the contributing guide

Research direction

Start at ui/src/timeMachine/reducers/index.ts around line 832 and read the reducer logic that removes tag selector values. Review the linked pull request diff for context, then add focused reducer tests covering that behavior and run the relevant TimeMachine reducer test command to confirm the cases pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend, testing
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.