aRustyDev / aRustyDev/helm-charts

test: Tier 3 footer parsing edge cases fail

Open
#114 0 comments 0 reactions 1 assignee Claimed by @aRustyDev View on GitHub
bug
Dominant language
Go Template
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

## Description

Tests 60, 64, 65 in `test-tier3-footer.bats` fail for edge cases in commit footer parsing.

## Failing Tests

### Test 60: tier3:footer when file matches related pattern
- Function returns empty instead of `tier3:footer`

### Test 64: handles multiple related footers
- Output has leading spaces that don't match expected lines

### Test 65: parses valid UUID from requires footer
- `parse_commit_footer_requires_uuid` returns empty for valid UUID

## Impact

Low - Basic footer parsing works correctly. These are edge cases.

## Files

- `.github/actions/atomize/lib/relationships.sh` - Footer parsing functions
- `.github/tests/relationships/test-tier3-footer.bats` - Tests 60, 64, 65

## Suggested Fixes

1. Test 60: Debug `check_tier3_relationship` glob matching logic
2. Test 64: Trim whitespace from parsed footer values
3. Test 65: Check regex in `parse_commit_footer_requires_uuid` function

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.