Skip to content

Commit f7cd45c

Browse files
harp-intelclaude
andauthored
Add comprehensive unit tests for MergeUncoreGroups function (#566)
* Add comprehensive unit tests for MergeUncoreGroups function This commit adds unit tests for the MergeUncoreGroups function that was recently modified to eliminate duplicate uncore events across groups (commit cd5bcf2). The duplicate event elimination prevents perf from misreporting event values when the same event appears in multiple groups. Test coverage includes: - Duplicate event elimination across 2 and 3 groups - Verification that first occurrence is preserved - Group merging behavior after deduplication - Preservation of metric names during merge - Edge cases: empty input, single group, all empty events - Validation that empty events don't count as duplicates - Incompatible unit handling All tests follow the project's established patterns using table-driven tests and the testify assertion library. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]> Signed-off-by: Harper, Jason M <[email protected]> * Add .claude directory to .gitignore The .claude directory contains Claude Code session state and should not be tracked in version control. Signed-off-by: Jason Harper <[email protected]> Signed-off-by: Harper, Jason M <[email protected]> --------- Signed-off-by: Harper, Jason M <[email protected]> Signed-off-by: Jason Harper <[email protected]> Co-authored-by: Claude <[email protected]>
1 parent e9572e3 commit f7cd45c

File tree

2 files changed

+409
-0
lines changed

2 files changed

+409
-0
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,3 +10,4 @@
1010
/internal/script/resources/aarch64
1111
/test
1212
/__debug_bin*.log
13+
/.claude

0 commit comments

Comments
 (0)