test(nat): Remove warning for shuttle tests

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.

On Copilot review's suggestion, suppress "dead_code" warnings rather
than "unused", that might be broader (such as unused variables or
imports inside of body functions).

Signed-off-by: Quentin Monnet <qmo@qmon.net>

Author: qmonnet

nat/src/masquerade/apalloc/test_alloc.rs

@@ -21,15 +21,15 @@ mod context {
     use std::net::{IpAddr, Ipv4Addr};
     use std::str::FromStr;
 
-    #[allow(unused)]
+    #[allow(dead_code)]
     pub fn addr_v4(ip: &str) -> Ipv4Addr {
         Ipv4Addr::from_str(ip).unwrap()
     }
-    #[allow(unused)]
+    #[allow(dead_code)]
     pub fn addr_v4_bits(ip: &str) -> u32 {
         addr_v4(ip).to_bits()
     }
-    #[allow(unused)]
+    #[allow(dead_code)]
     pub fn ipaddr(ip: &str) -> IpAddr {
         IpAddr::from_str(ip).unwrap()
     }
@@ -40,6 +40,7 @@ mod context {
     pub fn vni2() -> Vni {
         Vni::new_checked(200).unwrap()
     }
+    #[allow(dead_code)]
     pub fn vpcd1() -> VpcDiscriminant {
         VpcDiscriminant::from_vni(vni1())
     }