Fix tests

Author: gjcairo

Sources/GRPCNIOTransportCore/TLSConfig.swift

@@ -30,8 +30,8 @@ public enum TLSConfig: Sendable {
 
   /// A description of where a certificate is coming from: either a byte array or a file.
   /// The serialization format is specified by ``TLSConfig/SerializationFormat``.
-  public struct CertificateSource: Sendable {
-    package enum Wrapped {
+  public struct CertificateSource: Sendable, Equatable {
+    package enum Wrapped: Equatable {
       case file(path: String, format: SerializationFormat)
       case bytes(bytes: [UInt8], format: SerializationFormat)
     }
@@ -90,8 +90,8 @@ public enum TLSConfig: Sendable {
   }
 
   /// A description of where the trust roots are coming from: either a custom certificate chain, or the system default trust store.
-  public struct TrustRootsSource: Sendable {
-    package enum Wrapped {
+  public struct TrustRootsSource: Sendable, Equatable {
+    package enum Wrapped: Equatable {
       case certificates([CertificateSource])
       case systemDefault
     }
@@ -113,8 +113,8 @@ public enum TLSConfig: Sendable {
   }
 
   /// How to verify certificates.
-  public struct CertificateVerification: Sendable {
-    package enum Wrapped {
+  public struct CertificateVerification: Sendable, Equatable {
+    package enum Wrapped: Equatable {
       case doNotVerify
       case fullVerification
       case noHostnameVerification

Tests/GRPCNIOTransportHTTP2Tests/HTTP2TransportNIOTransportServicesTests.swift

@@ -199,11 +199,8 @@ final class HTTP2TransportNIOTransportServicesTests: XCTestCase {
   }
 
   func testServerConfig_Defaults() throws {
-    try XCTSkipIf(true)
-
-    let identityProvider = Self.loadIdentity
     let grpcTLSConfig = HTTP2ServerTransport.TransportServices.Config.TLS.defaults(
-      identityProvider: identityProvider
+      identityProvider: Self.loadIdentity
     )
     let grpcConfig = HTTP2ServerTransport.TransportServices.Config.defaults(
       transportSecurity: .tls(grpcTLSConfig)
@@ -213,17 +210,15 @@ final class HTTP2TransportNIOTransportServicesTests: XCTestCase {
     XCTAssertEqual(grpcConfig.connection, HTTP2ServerTransport.Config.Connection.defaults)
     XCTAssertEqual(grpcConfig.http2, HTTP2ServerTransport.Config.HTTP2.defaults)
     XCTAssertEqual(grpcConfig.rpc, HTTP2ServerTransport.Config.RPC.defaults)
-    XCTAssertEqual(try grpcTLSConfig.identityProvider(), try identityProvider())
+
+    XCTAssertNotNil(grpcTLSConfig.identityProvider)
+    XCTAssertEqual(grpcTLSConfig.trustRoots, .systemDefault)
+    XCTAssertEqual(grpcTLSConfig.clientCertificateVerification, .noVerification)
     XCTAssertEqual(grpcTLSConfig.requireALPN, false)
   }
 
   func testClientConfig_Defaults() throws {
-    try XCTSkipIf(true)
-
-    let identityProvider = Self.loadIdentity
-    let grpcTLSConfig = HTTP2ClientTransport.TransportServices.Config.TLS(
-      identityProvider: identityProvider
-    )
+    let grpcTLSConfig = HTTP2ClientTransport.TransportServices.Config.TLS.defaults()
     let grpcConfig = HTTP2ClientTransport.TransportServices.Config.defaults(
       transportSecurity: .tls(grpcTLSConfig)
     )
@@ -232,7 +227,11 @@ final class HTTP2TransportNIOTransportServicesTests: XCTestCase {
     XCTAssertEqual(grpcConfig.connection, HTTP2ClientTransport.Config.Connection.defaults)
     XCTAssertEqual(grpcConfig.http2, HTTP2ClientTransport.Config.HTTP2.defaults)
     XCTAssertEqual(grpcConfig.backoff, HTTP2ClientTransport.Config.Backoff.defaults)
-    XCTAssertEqual(try grpcTLSConfig.identityProvider(), try identityProvider())
+
+    XCTAssertNil(grpcTLSConfig.identityProvider)
+    XCTAssertNil(grpcTLSConfig.serverHostname)
+    XCTAssertEqual(grpcTLSConfig.serverCertificateVerification, .fullVerification)
+    XCTAssertEqual(grpcTLSConfig.trustRoots, .systemDefault)
   }
 }
 #endif