ReshuffleTest: reuse ContainsKVs in GroupByKeyTest
Open
improvement
P3
tests
- Dominant language
- Java
- Stars
- 8.7k
- Forks
- 4.7k
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 205
Description
ContainsKVs in GroupByKeyTest could be reused to replace AssertThatHasExpectedContents in ReshuffleTest.
Imported from Jira [BEAM-2663](https://issues.apache.org/jira/browse/BEAM-2663). Original Jira may contain additional context.
Reported by: peihe0@gmail.com.
Contributor guide
Research direction
Start by locating GroupByKeyTest and ReshuffleTest, then compare ContainsKVs with AssertThatHasExpectedContents and the assertions that use them. Replace the ReshuffleTest usage with the reusable helper and run the relevant tests to confirm they still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100