From 432b1e996139d4e53d6787ed12f15753add0bb10 Mon Sep 17 00:00:00 2001
From: PastaClaw <thepastaclaw@users.noreply.github.com>
Date: Sat, 21 Feb 2026 15:27:54 -0600
Subject: [PATCH 1/2] test(swift-sdk): remove scaffold tests and fix
 tautological assertions

- Remove scaffold test files: CrashDebugTests, DebugTests, MinimalAsyncTest
- Remove testSimpleAsync from StateTransitionTests (scaffold)
- Remove testTransactionBuilderAddOutput and testTransactionBuilderChangeAddress
  (properties not accessible through public API, tests were tautological)
- Replace XCTAssertTrue(true) with proper assertions throughout
- Fix SDKMethodTests to XCTFail on unexpected success
- Fix SDKTests version check: use semver regex instead of hardcoded '2.0.0'
- Add missing swift_dash_string_free calls in SDKTests
---
 .../CrashDebugTests.swift                     | 121 -----------------
 .../SwiftExampleAppTests/DebugTests.swift     | 122 ------------------
 .../MinimalAsyncTest.swift                    |  36 ------
 .../SwiftExampleAppTests/SDKMethodTests.swift |   8 +-
 .../StateTransitionTests.swift                |   8 --
 .../WalletTests/TransactionTests.swift        |  13 --
 .../SwiftDashSDKTests/DataContractTests.swift |   3 -
 .../SwiftDashSDKTests/IdentityTests.swift     |   6 +-
 .../MemoryManagementTests.swift               |  18 ---
 .../Tests/SwiftDashSDKTests/SDKTests.swift    |  17 ++-
 10 files changed, 15 insertions(+), 337 deletions(-)
 delete mode 100644 packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/CrashDebugTests.swift
 delete mode 100644 packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/DebugTests.swift
 delete mode 100644 packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/MinimalAsyncTest.swift

diff --git a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/CrashDebugTests.swift b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/CrashDebugTests.swift
deleted file mode 100644
index 7961dacbf73..00000000000
--- a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/CrashDebugTests.swift
+++ /dev/null
@@ -1,121 +0,0 @@
-import DashSDKFFI
-import SwiftDashSDK
-import XCTest
-
-@testable import SwiftExampleApp
-
-final class CrashDebugTests: XCTestCase {
-
-  func testCatchCrash() async throws {
-    print("=== Starting crash debug test ===")
-
-    // Install exception handler (without capturing context)
-    let handler = NSGetUncaughtExceptionHandler()
-    NSSetUncaughtExceptionHandler { exception in
-      print("!!! Caught exception: \(exception)")
-      print("!!! Reason: \(exception.reason ?? "unknown")")
-      print("!!! User info: \(exception.userInfo ?? [:])")
-      print("!!! Call stack: \(exception.callStackSymbols)")
-    }
-
-    defer {
-      NSSetUncaughtExceptionHandler(handler)
-    }
-
-    // Try the problematic code
-    do {
-      print("Initializing SDK...")
-      SDK.initialize()
-
-      print("Creating SDK instance...")
-      let sdk = try SDK(network: DashSDKNetwork(rawValue: 1))
-
-      print("SDK created, checking methods...")
-
-      // Try to call the method with minimal setup
-      _ = "test"  // fromId
-      let toId = "test2"
-      let amount: UInt64 = 1
-      let key = Data(repeating: 0, count: 32)
-
-      print("Creating identity and signer...")
-
-      // Create a dummy identity
-      let identity = DPPIdentity(
-        id: Data(repeating: 0, count: 32),
-        publicKeys: [:],
-        balance: 0,
-        revision: 0
-      )
-
-      // Create signer from private key
-      let signerResult = key.withUnsafeBytes { keyBytes in
-        dash_sdk_signer_create_from_private_key(
-          keyBytes.bindMemory(to: UInt8.self).baseAddress!,
-          UInt(key.count)
-        )
-      }
-
-      guard signerResult.error == nil,
-        let signer = signerResult.data
-      else {
-        print("Failed to create signer")
-        return
-      }
-
-      defer {
-        dash_sdk_signer_destroy(signer.assumingMemoryBound(to: SignerHandle.self))
-      }
-
-      print("Calling transferCredits...")
-      _ = try await sdk.transferCredits(
-        from: identity,
-        toIdentityId: toId,
-        amount: amount,
-        signer: OpaquePointer(signer)
-      )
-
-      print("Method call completed")
-    } catch {
-      print("Caught error: \(error)")
-      print("Error type: \(type(of: error))")
-      print("Error localized: \(error.localizedDescription)")
-
-      let nsError = error as NSError
-      print("NSError domain: \(nsError.domain)")
-      print("NSError code: \(nsError.code)")
-      print("NSError userInfo: \(nsError.userInfo)")
-    }
-
-    print("=== Crash debug test completed ===")
-  }
-
-  func testMethodExistence() {
-    print("=== Testing method existence ===")
-
-    // Check if the SDK has the method we're trying to call
-    let sdkClass: AnyClass? = NSClassFromString("SwiftDashSDK.SDK")
-    print("SDK class: \(String(describing: sdkClass))")
-
-    if let cls = sdkClass {
-      // List all methods
-      var methodCount: UInt32 = 0
-      let methods = class_copyMethodList(cls, &methodCount)
-
-      print("Found \(methodCount) methods in SDK class:")
-      if let methods = methods {
-        for i in 0..<Int(methodCount) {
-          let method = methods[i]
-          let selector = method_getName(method)
-          let name = NSStringFromSelector(selector)
-          if name.contains("identity") || name.contains("transfer") || name.contains("credit") {
-            print("  - \(name)")
-          }
-        }
-        free(methods)
-      }
-    }
-
-    print("=== Method existence test completed ===")
-  }
-}
diff --git a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/DebugTests.swift b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/DebugTests.swift
deleted file mode 100644
index 4525d813f9c..00000000000
--- a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/DebugTests.swift
+++ /dev/null
@@ -1,122 +0,0 @@
-import SwiftDashSDK
-import XCTest
-
-@testable import SwiftExampleApp
-
-final class DebugTests: XCTestCase {
-
-  // Test 1: Simple synchronous test
-  func testSimpleSync() throws {
-    print("=== testSimpleSync ===")
-    XCTAssertTrue(true)
-    print("✅ Simple sync test passed")
-  }
-
-  // Test 2: Simple async test
-  func testSimpleAsync() async throws {
-    print("=== testSimpleAsync ===")
-    try await Task.sleep(nanoseconds: 100_000_000)
-    XCTAssertTrue(true)
-    print("✅ Simple async test passed")
-  }
-
-  // Test 3: Test with "Transfer" in name (non-async)
-  func testTransferSync() throws {
-    print("=== testTransferSync ===")
-    XCTAssertTrue(true)
-    print("✅ Transfer sync test passed")
-  }
-
-  // Test 4: Test with "Transfer" in name (async)
-  func testTransferAsync() async throws {
-    print("=== testTransferAsync ===")
-    try await Task.sleep(nanoseconds: 100_000_000)
-    XCTAssertTrue(true)
-    print("✅ Transfer async test passed")
-  }
-
-  // Test 5: Test with "CreditTransfer" in name
-  func testCreditTransferDebug() async throws {
-    print("=== testCreditTransferDebug ===")
-    try await Task.sleep(nanoseconds: 100_000_000)
-    XCTAssertTrue(true)
-    print("✅ Credit transfer debug test passed")
-  }
-
-  // Test 6: Test SDK loading
-  func testSDKLoading() throws {
-    print("=== testSDKLoading ===")
-
-    // Initialize SDK library
-    SDK.initialize()
-    print("✅ SDK library initialized")
-
-    // Try to create SDK instance
-    do {
-      let sdk = try SDK(network: DashSDKNetwork(rawValue: 1))
-      print("✅ SDK instance created")
-      XCTAssertNotNil(sdk.handle)
-    } catch {
-      XCTFail("Failed to create SDK: \(error)")
-    }
-  }
-
-  // Test 7: Test environment loading (EnvLoader is @MainActor)
-  @MainActor
-  func testEnvLoading() throws {
-    print("=== testEnvLoading ===")
-
-    // Load env file
-    EnvLoader.loadEnvFile()
-
-    // Try to get test identity ID
-    if let identityId = EnvLoader.get("TEST_IDENTITY_ID") {
-      print("✅ Found TEST_IDENTITY_ID: \(identityId)")
-      XCTAssertFalse(identityId.isEmpty)
-    } else {
-      print("⚠️ TEST_IDENTITY_ID not found in environment")
-    }
-  }
-
-  // Test 8: Test with exact method signature
-  func testIdentityCreditTransferDebug() async throws {
-    print("=== testIdentityCreditTransferDebug ===")
-
-    do {
-      print("Starting test...")
-      try await Task.sleep(nanoseconds: 100_000_000)
-      print("✅ Test completed successfully")
-      XCTAssertTrue(true)
-    } catch {
-      print("❌ Test failed: \(error)")
-      throw error
-    }
-  }
-
-  // Test 9: Test method discovery
-  func testMethodDiscovery() throws {
-    print("=== testMethodDiscovery ===")
-
-    let testClass = StateTransitionTests.self
-    print("Test class: \(testClass)")
-
-    // List all test methods
-    var methodCount: UInt32 = 0
-    let methods = class_copyMethodList(testClass as? AnyClass, &methodCount)
-
-    if let methods = methods {
-      print("Found \(methodCount) methods:")
-      for i in 0..<Int(methodCount) {
-        let method = methods[i]
-        let selector = method_getName(method)
-        let name = NSStringFromSelector(selector)
-        if name.hasPrefix("test") {
-          print("  - \(name)")
-        }
-      }
-      free(methods)
-    }
-
-    XCTAssertTrue(true)
-  }
-}
diff --git a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/MinimalAsyncTest.swift b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/MinimalAsyncTest.swift
deleted file mode 100644
index dbd7c375781..00000000000
--- a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/MinimalAsyncTest.swift
+++ /dev/null
@@ -1,36 +0,0 @@
-import XCTest
-
-final class MinimalAsyncTest: XCTestCase {
-    
-    // Test 1: Simple async test
-    func testSimpleAsync() async throws {
-        print("Starting simple async test")
-        try await Task.sleep(nanoseconds: 100_000_000)
-        print("Simple async test completed")
-        XCTAssertTrue(true)
-    }
-    
-    // Test 2: Async test with "Transfer" in name
-    func testTransferAsync() async throws {
-        print("Starting transfer async test")
-        try await Task.sleep(nanoseconds: 100_000_000)
-        print("Transfer async test completed")
-        XCTAssertTrue(true)
-    }
-    
-    // Test 3: Async test with "CreditTransfer" in name
-    func testCreditTransferAsync() async throws {
-        print("Starting credit transfer async test")
-        try await Task.sleep(nanoseconds: 100_000_000)
-        print("Credit transfer async test completed")
-        XCTAssertTrue(true)
-    }
-    
-    // Test 4: Async test with exact failing name
-    func testIdentityCreditTransfer() async throws {
-        print("Starting identity credit transfer test")
-        try await Task.sleep(nanoseconds: 100_000_000)
-        print("Identity credit transfer test completed")
-        XCTAssertTrue(true)
-    }
-}
\ No newline at end of file
diff --git a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/SDKMethodTests.swift b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/SDKMethodTests.swift
index e67681746d8..aa9f0e45438 100644
--- a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/SDKMethodTests.swift
+++ b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/SDKMethodTests.swift
@@ -81,14 +81,10 @@ final class SDKMethodTests: XCTestCase {
         amount: amount,
         signer: OpaquePointer(signer)
       )
-      print("✅ Method call succeeded (unexpected)")
+      XCTFail("transferCredits should fail with dummy data")
     } catch {
-      print("Method call failed with error: \(error)")
-      print("Error type: \(type(of: error))")
-      // This is expected since we're using dummy data
+      // Expected: dummy data causes an error
     }
-
-    XCTAssertTrue(true)
   }
 
   func testSimpleIdentityFetch() async throws {
diff --git a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/StateTransitionTests.swift b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/StateTransitionTests.swift
index 256395a977d..fec70c25a2f 100644
--- a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/StateTransitionTests.swift
+++ b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/StateTransitionTests.swift
@@ -72,13 +72,6 @@ final class StateTransitionTests: XCTestCase {
     print("✅ SDK initialized successfully")
   }
 
-  func testSimpleAsync() async throws {
-    // Test that async tests work at all
-    print("Starting simple async test")
-    try await Task.sleep(nanoseconds: 100_000_000)  // 0.1 second
-    print("Simple async test completed")
-    XCTAssertTrue(true)
-  }
 
   func testIdentityCreditTransferDebug() async throws {
     print("Test started")
@@ -180,7 +173,6 @@ final class StateTransitionTests: XCTestCase {
 
     // This test just verifies setup is correct
     // The actual async transfer would be executed in testIdentityCreditTransferAsync
-    XCTAssertTrue(true)
   }
 
   func testBasicSetup() throws {
diff --git a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/WalletTests/TransactionTests.swift b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/WalletTests/TransactionTests.swift
index b34147e6f94..f9d88622451 100644
--- a/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/WalletTests/TransactionTests.swift
+++ b/packages/swift-sdk/SwiftExampleApp/SwiftExampleAppTests/WalletTests/TransactionTests.swift
@@ -21,20 +21,7 @@ final class TransactionTests: XCTestCase {
     // SDKTransactionBuilder.addInput takes Input(txid:vout:scriptPubKey:privateKey), not HDUTXO
   }
 
-  func testTransactionBuilderAddOutput() throws {
-    let builder = SDKTransactionBuilder(feePerKB: 1000)
-    let address = "yTsGq4wV8WySdQTYgGqmiUKMxb8RBr6wc6"
-    let amount: UInt64 = 50_000_000
-    try builder.addOutput(SDKTransactionBuilder.Output(address: address, amount: amount))
-    XCTAssertTrue(true)
-  }
 
-  func testTransactionBuilderChangeAddress() throws {
-    let builder = SDKTransactionBuilder(feePerKB: 1000)
-    let changeAddress = "yXdUfGBfX6rQmNq5speeNGD5HfL2qkYBNe"
-    try builder.setChangeAddress(changeAddress)
-    XCTAssertTrue(true)
-  }
 
   func testTransactionBuilderInsufficientBalance() throws {
     let builder = SDKTransactionBuilder(feePerKB: 1000)
diff --git a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/DataContractTests.swift b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/DataContractTests.swift
index 903c0b58b21..e494906c67f 100644
--- a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/DataContractTests.swift
+++ b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/DataContractTests.swift
@@ -118,9 +118,6 @@ class DataContractTests: XCTestCase {
             
             // Clean up
             swift_dash_string_free(jsonString)
-        } else {
-            // No history is also valid for test vectors
-            XCTAssertTrue(true, "Contract history may return nil if no history exists")
         }
     }
     
diff --git a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/IdentityTests.swift b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/IdentityTests.swift
index fe5f565bb65..442909af303 100644
--- a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/IdentityTests.swift
+++ b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/IdentityTests.swift
@@ -135,6 +135,7 @@ class IdentityTests: XCTestCase {
         }
         
         let result = swift_dash_identity_resolve_name(sdk, "dash")
+        XCTAssertNotNil(result, "Should resolve existing alias")
         
         if let jsonString = result {
             let jsonStr = String(cString: jsonString)
@@ -157,9 +158,6 @@ class IdentityTests: XCTestCase {
             
             // Clean up
             swift_dash_string_free(jsonString)
-        } else {
-            // Name not found is also valid for test vectors
-            XCTAssertTrue(true, "Name resolution may return nil if not found in test vectors")
         }
     }
     
@@ -312,4 +310,4 @@ class IdentityTests: XCTestCase {
             swift_dash_error_free(error)
         }
     }
-}
\ No newline at end of file
+}
diff --git a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/MemoryManagementTests.swift b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/MemoryManagementTests.swift
index 9691d02546c..788d7355220 100644
--- a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/MemoryManagementTests.swift
+++ b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/MemoryManagementTests.swift
@@ -26,7 +26,6 @@ class MemoryManagementTests: XCTestCase {
     func testStringFreeWithNullPointer() {
         // Should not crash
         swift_dash_string_free(nil)
-        XCTAssertTrue(true, "String free with null pointer should not crash")
     }
     
     func testStringFreeWithValidPointer() {
@@ -44,7 +43,6 @@ class MemoryManagementTests: XCTestCase {
             swift_dash_string_free(version)
         }
         
-        XCTAssertTrue(true, "String free with valid pointer should not crash")
     }
     
     // MARK: - Error Memory Management Tests
@@ -52,7 +50,6 @@ class MemoryManagementTests: XCTestCase {
     func testErrorFreeWithNullPointer() {
         // Should not crash
         swift_dash_error_free(nil)
-        XCTAssertTrue(true, "Error free with null pointer should not crash")
     }
     
     func testErrorFreeWithValidPointer() {
@@ -71,7 +68,6 @@ class MemoryManagementTests: XCTestCase {
             swift_dash_error_free(error)
         }
         
-        XCTAssertTrue(true, "Error free with valid pointer should not crash")
     }
     
     // MARK: - Binary Data Memory Management Tests
@@ -79,7 +75,6 @@ class MemoryManagementTests: XCTestCase {
     func testBinaryDataFreeWithNullPointer() {
         // Should not crash
         swift_dash_binary_data_free(nil)
-        XCTAssertTrue(true, "Binary data free with null pointer should not crash")
     }
     
     // MARK: - Info Structure Memory Management Tests
@@ -87,31 +82,26 @@ class MemoryManagementTests: XCTestCase {
     func testIdentityInfoFreeWithNullPointer() {
         // Should not crash
         swift_dash_identity_info_free(nil)
-        XCTAssertTrue(true, "Identity info free with null pointer should not crash")
     }
     
     func testDataContractInfoFreeWithNullPointer() {
         // Should not crash
         swift_dash_data_contract_info_free(nil)
-        XCTAssertTrue(true, "Data contract info free with null pointer should not crash")
     }
     
     func testDocumentInfoFreeWithNullPointer() {
         // Should not crash
         swift_dash_document_info_free(nil)
-        XCTAssertTrue(true, "Document info free with null pointer should not crash")
     }
     
     func testTransferCreditsResultFreeWithNullPointer() {
         // Should not crash
         swift_dash_transfer_credits_result_free(nil)
-        XCTAssertTrue(true, "Transfer credits result free with null pointer should not crash")
     }
     
     func testTokenInfoFreeWithNullPointer() {
         // Should not crash
         swift_dash_token_info_free(nil)
-        XCTAssertTrue(true, "Token info free with null pointer should not crash")
     }
     
     // MARK: - Signer Memory Management Tests
@@ -119,7 +109,6 @@ class MemoryManagementTests: XCTestCase {
     func testSignerFreeWithNullPointer() {
         // Should not crash
         swift_dash_signer_free(nil)
-        XCTAssertTrue(true, "Signer free with null pointer should not crash")
     }
     
     func testSignerCreateAndFree() {
@@ -142,7 +131,6 @@ class MemoryManagementTests: XCTestCase {
             swift_dash_signer_free(signer)
         }
         
-        XCTAssertTrue(true, "Signer create and free should not crash")
     }
     
     // MARK: - Bytes Memory Management Tests
@@ -150,7 +138,6 @@ class MemoryManagementTests: XCTestCase {
     func testBytesFreeWithNullPointer() {
         // Should not crash
         swift_dash_bytes_free(nil, 0)
-        XCTAssertTrue(true, "Bytes free with null pointer should not crash")
     }
     
     func testBytesFreeWithValidPointer() {
@@ -169,7 +156,6 @@ class MemoryManagementTests: XCTestCase {
             swift_dash_bytes_free(bytes, size)
         }
         
-        XCTAssertTrue(true, "Bytes free with valid pointer should not crash")
     }
     
     // MARK: - SDK Handle Memory Management Tests
@@ -177,7 +163,6 @@ class MemoryManagementTests: XCTestCase {
     func testSDKDestroyWithNullHandle() {
         // Should not crash
         swift_dash_sdk_destroy(nil)
-        XCTAssertTrue(true, "SDK destroy with null handle should not crash")
     }
     
     func testMultipleSDKCreateAndDestroy() {
@@ -199,7 +184,6 @@ class MemoryManagementTests: XCTestCase {
             swift_dash_sdk_destroy(sdk)
         }
         
-        XCTAssertTrue(true, "Multiple SDK create and destroy should not crash")
     }
     
     // MARK: - Memory Leak Prevention Tests
@@ -236,7 +220,6 @@ class MemoryManagementTests: XCTestCase {
             }
         }
         
-        XCTAssertTrue(true, "Memory leak prevention tests completed")
     }
     
     // MARK: - Double Free Protection Tests
@@ -252,6 +235,5 @@ class MemoryManagementTests: XCTestCase {
             swift_dash_string_free(version)
         }
         
-        XCTAssertTrue(true, "Double free protection test completed")
     }
 }
\ No newline at end of file
diff --git a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/SDKTests.swift b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/SDKTests.swift
index def7e05e9b9..edafd0b458e 100644
--- a/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/SDKTests.swift
+++ b/packages/swift-sdk/SwiftTests/Tests/SwiftDashSDKTests/SDKTests.swift
@@ -12,9 +12,11 @@ class SDKTests: XCTestCase {
     // MARK: - Initialization Tests
     
     func testSDKInitialization() {
-        // SDK should be initialized in setUp
-        // If we get here without crashing, initialization worked
-        XCTAssertTrue(true, "SDK initialized successfully")
+        let version = swift_dash_sdk_get_version()
+        XCTAssertNotNil(version, "SDK should provide a version after initialization")
+        if let version = version {
+            swift_dash_string_free(version)
+        }
     }
     
     func testSDKVersion() {
@@ -24,7 +26,11 @@ class SDKTests: XCTestCase {
         if let version = version {
             let versionString = String(cString: version)
             XCTAssertFalse(versionString.isEmpty)
-            XCTAssertTrue(versionString.contains("2.0.0"))
+            XCTAssertTrue(
+                versionString.range(of: #"^\d+\.\d+\.\d+(?:[-+][A-Za-z0-9.-]+)?$"#, options: .regularExpression) != nil,
+                "Version should match semantic version format"
+            )
+            swift_dash_string_free(version)
         }
     }
     
@@ -95,7 +101,6 @@ class SDKTests: XCTestCase {
     func testSDKDestroyNullHandle() {
         // Should not crash
         swift_dash_sdk_destroy(nil)
-        XCTAssertTrue(true, "Destroying null handle should not crash")
     }
     
     func testGetNetworkWithNullHandle() {
@@ -121,4 +126,4 @@ class SDKTests: XCTestCase {
         XCTAssertTrue(settings.ban_failed_address)
         XCTAssertEqual(settings.user_fee_increase, 10)
     }
-}
\ No newline at end of file
+}

From 6a26099250ccf2ae66a2365ae9cf7e56e9bf84e7 Mon Sep 17 00:00:00 2001
From: PastaClaw <thepastaclaw@users.noreply.github.com>
Date: Mon, 23 Feb 2026 20:56:13 -0600
Subject: [PATCH 2/2] docs(swift-sdk): clarify SwiftDashSDKMock.c usage in
 tests

---
 .../SwiftTests/Sources/SwiftDashSDKMock/SwiftDashSDKMock.c    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/swift-sdk/SwiftTests/Sources/SwiftDashSDKMock/SwiftDashSDKMock.c b/packages/swift-sdk/SwiftTests/Sources/SwiftDashSDKMock/SwiftDashSDKMock.c
index 7594688a537..aac1efa5efc 100644
--- a/packages/swift-sdk/SwiftTests/Sources/SwiftDashSDKMock/SwiftDashSDKMock.c
+++ b/packages/swift-sdk/SwiftTests/Sources/SwiftDashSDKMock/SwiftDashSDKMock.c
@@ -1,5 +1,5 @@
-// Mock implementation of Swift Dash SDK for testing
-// This provides mock implementations of all the C functions
+// Test-only C mock implementation of SwiftDashSDK FFI, linked by SwiftTests/Package.swift target `SwiftDashSDKMock`.
+// Used by SwiftTests unit tests to exercise Swift wrapper behavior without depending on a real Rust backend.
 
 #include "SwiftDashSDK.h"
 #include <stdlib.h>