From 756fd0bccf82d6c5b100ff094c98def49968ea62 Mon Sep 17 00:00:00 2001
From: Michael Housh <mhoush@houshhomeenergy.com>
Date: Mon, 18 Nov 2024 17:16:44 -0500
Subject: [PATCH] feat: cli client test updates

---
 Sources/CliClient/CliClient.swift         |  3 +--
 Tests/CliClientTests/CliClientTests.swift | 19 +++++++++++++++++++
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/Sources/CliClient/CliClient.swift b/Sources/CliClient/CliClient.swift
index 50f1b03..fd9f4cf 100644
--- a/Sources/CliClient/CliClient.swift
+++ b/Sources/CliClient/CliClient.swift
@@ -192,8 +192,7 @@ extension MQTTClient.Version {
   }
 }
 
-@_spi(Internal)
-public extension Logger.Level {
+extension Logger.Level {
 
   /// Parse a `Logger.Level` from the loaded `EnvVars`.
   static func from(environment envVars: EnvVars) -> Self {
diff --git a/Tests/CliClientTests/CliClientTests.swift b/Tests/CliClientTests/CliClientTests.swift
index 93cf3cf..0af6b31 100644
--- a/Tests/CliClientTests/CliClientTests.swift
+++ b/Tests/CliClientTests/CliClientTests.swift
@@ -115,6 +115,25 @@ final class CliClientTests: XCTestCase {
     let environment = EnvironmentDependency.liveValue
     XCTAssertEqual(environment.processInfo(), ProcessInfo.processInfo.environment)
   }
+
+  func testMakeClient() throws {
+    @Dependency(\.cliClient) var cliClient
+    let envVars = EnvVars.test
+    let client = try cliClient.makeClient(.init(
+      environment: envVars,
+      eventLoopGroup: .init(numberOfThreads: 1),
+      logger: nil
+    ))
+
+    XCTAssertEqual(client.host, envVars.host)
+    XCTAssertEqual(client.port, Int(envVars.port!))
+    XCTAssertEqual(client.identifier, envVars.identifier)
+    XCTAssertEqual(client.configuration.version, .v5_0)
+    XCTAssertEqual(client.configuration.userName, envVars.userName)
+    XCTAssertEqual(client.configuration.password, envVars.password)
+
+    try client.syncShutdownGracefully()
+  }
 }
 
 // - MARK: Helper