keys: add additional test

[paroga]

@bradh sorry, i somehow forgot to add this test to the last MR

[coderabbitai]

Warning

Review limit reached

Next included review available in 50 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: fe8b91a5-3093-4132-9f99-cd0d758ba1cd

📥 Commits

Reviewing files that changed from the base of the PR and between 8e10e58 and 9c152ca.

📒 Files selected for processing (1)

• src/meta/keys.rs

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: e0a2cc3c-8612-4805-989c-02e07b9be8d5

📥 Commits

Reviewing files that changed from the base of the PR and between bd1f899 and 8e10e58.

📒 Files selected for processing (1)

• src/meta/keys.rs

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

Added a regression test in src/meta/keys.rs. The test creates a keys box with a truncated entry and asserts that Keys::decode returns an error instead of panicking.

Merge Risk: ⚪ Minimal · up to 8e10e

This change adds regression coverage ensuring truncated keys entries return an error rather than panic. No current merge-blocking risk is identified.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description check	✅ Passed	The description explains that the pull request adds a test that was omitted from the previous merge request. It is related to the changeset.
Title check	✅ Passed	The title identifies an additional test for the keys functionality. It accurately summarizes the pull request, although it does not mention the regression case or truncated entry.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 1 files.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}