[qmonnet]

We have a warning when building the shuttle tests for nat:

$ cargo check --features shuttle --all-targets -p dataplane-nat
[...]
warning: function `vpcd1` is never used
  --> nat/src/masquerade/apalloc/test_alloc.rs:43:12
   |
43 |     pub fn vpcd1() -> VpcDiscriminant {
   |            ^^^^^
   |
   = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default

It's not blocking, but let's suppress it anyway.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 4abafa33-149e-4dd1-929c-928aaa867057

📥 Commits

Reviewing files that changed from the base of the PR and between 557fa3c and cd11e47.

📒 Files selected for processing (1)

• nat/src/masquerade/apalloc/test_alloc.rs

✅ Files skipped from review due to trivial changes (1)

• nat/src/masquerade/apalloc/test_alloc.rs

---

📝 Walkthrough

Walkthrough

test_alloc.rs updates warning suppression in the context test-helper module from unused to dead_code. No helper logic, control flow, or public API changes are made.

Changes

Test helper lint suppression

Layer / File(s)	Summary
Update helper warning attributes nat/src/masquerade/apalloc/test_alloc.rs	Replaces #[allow(unused)] with #[allow(dead_code)] on helper items in context, and adds a matching #[allow(dead_code)] before the next helper.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the warning-suppression change for NAT shuttle tests.
Description check	✅ Passed	The description matches the code change by explaining the dead_code warning and intent to suppress it.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

---

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

[Copilot]

Pull request overview

Suppresses a dead_code warning when compiling NAT shuttle tests by annotating an unused test helper.

Changes:

• Add a lint suppression attribute to context::vpcd1() to avoid dead_code warnings under shuttle builds.