integration-test: add comment for test order constraint of SNP

Author: phip1611

uefi-test-runner/src/proto/network/mod.rs

@@ -4,10 +4,18 @@ pub fn test() {
     info!("Testing Network protocols");
 
     http::test();
-    pxe::test();
-    snp::test();
+    #[cfg(feature = "pxe")]
+    {
+        pxe::test();
+        // Currently, we are in the unfortunate situation that the SNP test
+        // depends on the PXE test, as it assigns an IPv4 address to the
+        // interface.
+        snp::test();
+    }
 }
 
 mod http;
+#[cfg(feature = "pxe")]
 mod pxe;
+#[cfg(feature = "pxe")]
 mod snp;

uefi-test-runner/src/proto/network/pxe.rs

@@ -5,11 +5,6 @@ use uefi::proto::network::pxe::{BaseCode, DhcpV4Packet, IpFilter, IpFilters, Udp
 use uefi::{CStr8, boot};
 
 pub fn test() {
-    // Skip the test if the `pxe` feature is not enabled.
-    if cfg!(not(feature = "pxe")) {
-        return;
-    }
-
     info!("Testing The PXE base code protocol");
 
     let handles = boot::find_handles::<BaseCode>().expect("failed to get PXE base code handles");