Merge branch 'main' into bug-copy

Author: Ronitsabhaya75

Sources/ContainerResource/Network/NetworkMode.swift

@@ -14,10 +14,8 @@
 // limitations under the License.
 //===----------------------------------------------------------------------===//
 
-import ArgumentParser
-
 /// Networking mode that applies to client containers.
-public enum NetworkMode: String, Codable, Sendable, ExpressibleByArgument {
+public enum NetworkMode: String, Codable, Sendable {
     /// NAT networking mode.
     /// Containers do not have routable IPs, and the host performs network
     /// address translation to allow containers to reach external services.

Sources/Helpers/NetworkVmnet/NetworkVmnetHelper+Start.swift

@@ -29,6 +29,8 @@ enum Variant: String, ExpressibleByArgument {
     case allocationOnly
 }
 
+extension NetworkMode: ExpressibleByArgument {}
+
 extension NetworkVmnetHelper {
     struct Start: AsyncParsableCommand {
         static let configuration = CommandConfiguration(

Tests/CLITests/Subcommands/Containers/TestCLIExec.swift

@@ -122,9 +122,15 @@ class TestCLIExecCommand: CLITest {
             do {
                 _ = try doExec(name: name, cmd: ["sleep", "infinity"])
             } catch CLIError.executionFailed(let message) {
-                #expect(message.contains("is not running"))
+                // There's no nice way to check fail reason here
+                #expect(message.contains("is not running"), "expected container is not running if exec failed")
+            }
+
+            // Give time for the exec (or start) error handling settles down
+            sleep(1)
+            #expect(throws: Never.self, "expected the container remains") {
+                try getContainerStatus(name)
             }
-            #expect(try getContainerStatus(name) == "stopped")
         }
     }
 }