fix(tests): Remove skipIfUDSNotImplemented from CreateSignalSocketTests

- UDSVirtioFSRelay is now fully implemented (no longer stub)
- Remove XCTSkip calls that were blocking test execution
- Tests can now run and validate actual UDS functionality

Author: amazon1148

Tests/Container-Compose-Tests/Networking/RelayManagerTests.swift

@@ -736,14 +736,9 @@ final class PeerVerificationTests: XCTestCase {
 final class CreateSignalSocketTests: XCTestCase {
 
   /// Skip if UDSVirtioFSRelay is not fully implemented (stub)
-  private func skipIfUDSNotImplemented() throws {
-    // UDSVirtioFSRelay is currently stubbed (35 lines)
-    // Full implementation pending Phase 1 completion
-    throw XCTSkip("UDSVirtioFSRelay not fully implemented - stub only")
-  }
 
   func testCreateSignalSocketTruePreservesBehavior() async throws {
-    try skipIfUDSNotImplemented()
+    
         // When createSignalSocket is true (default for non-volume sockets),
         // the relay should create the signal socket directory structure
         let eventLog = RelayEventLog()
@@ -765,7 +760,7 @@ final class CreateSignalSocketTests: XCTestCase {
     }
 
   func testCreateSignalSocketFalseSkipsSocketCreation() async throws {
-    try skipIfUDSNotImplemented()
+    
     // When createSignalSocket is false (for volume sockets like vsock-db),
         // the relay should not create/remove the signal socket
         let eventLog = RelayEventLog()
@@ -787,7 +782,7 @@ final class CreateSignalSocketTests: XCTestCase {
     }
 
   func testCreateSignalSocketDefaultsToTrue() async throws {
-    try skipIfUDSNotImplemented()
+    
     // Test that UDSVirtioFSRelay can be created with explicit createSignalSocket: true
         let eventLog = RelayEventLog()
         let socketPath = "/tmp/test-default-\(UUID().uuidString).sock"
@@ -803,7 +798,7 @@ final class CreateSignalSocketTests: XCTestCase {
     }
 
   func testCreateSignalSocketWithEmptyPath() async throws {
-    try skipIfUDSNotImplemented()
+    
     // Test behavior with empty socketPath
         let eventLog = RelayEventLog()
 
@@ -823,7 +818,7 @@ final class CreateSignalSocketTests: XCTestCase {
     }
 
   func testCreateSignalSocketWithVeryLongPath() async throws {
-    try skipIfUDSNotImplemented()
+    
     // Plan 88 Decision 5: Hard-error on paths >= 104 chars
         let eventLog = RelayEventLog()
         let longPath = String(repeating: "a", count: 110) + ".sock"
@@ -842,7 +837,7 @@ final class CreateSignalSocketTests: XCTestCase {
     }
 
   func testCreateSignalSocketAtPathLimit() async throws {
-    try skipIfUDSNotImplemented()
+    
     // Verify paths under 104 chars work fine
         let eventLog = RelayEventLog()
         let path = String(repeating: "a", count: 90) + ".sock"

run-tests.sh

@@ -117,7 +117,7 @@ for arg in "$@"; do
   if [[ "$arg" == "--no-sudo" ]]; then
     FORCE_NO_SUDO=true
   elif [[ "$arg" == "--auto-clean" ]]; then
-    true # already handled above
+    AUTO_CLEAN=true
   elif [[ "$arg" == "--filter" ]]; then
     FILTERED_ARGS+=("$arg") # keep --filter as-is
   elif [[ "$arg" =~ ^[A-Za-z].*Tests?$ ]]; then

scripts/lib/test-runner.sh

@@ -130,7 +130,8 @@ check_stale_lock_files() {
         echo " These can cause 'invalid access' errors during build."
         echo ""
 
-        if [ "$AUTO_CLEAN" = true ]; then
+        echo "DEBUG: AUTO_CLEAN='$AUTO_CLEAN'"
+if [ "$AUTO_CLEAN" = true ]; then
             should_clean=true
         else
             echo "Remove stale lock files? [Y/n] "