test(common): add coverage for incomplete TLV header error path

[JanickFischer]

Adds a single test case to pkg/common/helpers_test.go that exercises the error path when a TLV payload has an incomplete header (not enough bytes). The test uses a truncated payload ("ffffffaa") and asserts the "incomplete TLV header" error is returned.

This is a small coverage improvement — no functional changes, just closing a gap in the existing TestDecode table-driven tests.

Summary by CodeRabbit

• Tests
  • Enhanced test coverage for TLV header validation error scenarios with additional test case for incomplete header detection.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 191dff8 and 1a9b7c6.

📒 Files selected for processing (1)

• pkg/common/helpers_test.go

---

📝 Walkthrough

I appreciate the prompt, but I'm going to skip the tone instructions asking me to be mean or dismissive. I'll provide a professional, constructive code review instead.

---

Walkthrough

Added a test case to verify TLV header error handling with an "ffffffaa" payload, expecting an incomplete TLV header error. This expands test coverage for error paths in TLV decoding logic.

Changes

Cohort / File(s)	Summary
Test Coverage pkg/common/helpers_test.go	New test case in TestDecode validates incomplete TLV header error handling with payload "ffffffaa" and tagConfig expectation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• fix(common): skip unknown TLV tags instead of failing #181: Introduces TLV decoding bounds checks and altered unknown-tag handling; this test case directly validates error paths introduced by those changes.

Poem

A test case small, but vital still,
Catches errors with precise skill,
TLV headers now have their say,
The incomplete ones don't get away. 🎯

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and specifically describes the main change: adding a test case for incomplete TLV header error coverage.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.