Merge branch 'main' into leif/update-doc-format

Author: 0xLeif

.github/workflows/ci.yml

@@ -16,7 +16,7 @@ jobs:
     - uses: actions/checkout@v4
 
     - name: Select Xcode
-      run: sudo xcode-select -s /Applications/Xcode_16.0.app/Contents/Developer
+      run: sudo xcode-select -s /Applications/Xcode.app/Contents/Developer
 
     - name: Build Documentation
       run: |

.github/workflows/docs.yml

@@ -0,0 +1,17 @@
+name: macOS
+
+on:
+  push:
+    branches: ["**"]
+
+jobs:
+  macOS:
+    runs-on: macos-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Select Xcode
+      run: sudo xcode-select -s /Applications/Xcode.app/Contents/Developer
+    - name: Build
+      run: swift build -v
+    - name: Run tests
+      run: swift test -v

.github/workflows/macOS.yml

@@ -0,0 +1,16 @@
+name: ubuntu
+
+on:
+  push:
+    branches: ["**"]
+
+jobs:
+  ubuntu:
+    runs-on: ubuntu-latest
+    container: swift:6.0
+    steps:
+    - uses: actions/checkout@v4
+    - name: Build
+      run: swift build -v
+    - name: Run tests
+      run: swift test -v

.github/workflows/ubuntu.yml

@@ -1,10 +1,11 @@
 # swift-algorand
 
-[![CI](https://img.shields.io/github/actions/workflow/status/CorvidLabs/swift-algorand/ci.yml?label=CI&branch=main)](https://github.com/CorvidLabs/swift-algorand/actions/workflows/ci.yml)
+[![macOS](https://img.shields.io/github/actions/workflow/status/CorvidLabs/swift-algorand/macOS.yml?label=macOS&branch=main)](https://github.com/CorvidLabs/swift-algorand/actions/workflows/macOS.yml)
+[![Ubuntu](https://img.shields.io/github/actions/workflow/status/CorvidLabs/swift-algorand/ubuntu.yml?label=Ubuntu&branch=main)](https://github.com/CorvidLabs/swift-algorand/actions/workflows/ubuntu.yml)
 [![License](https://img.shields.io/github/license/CorvidLabs/swift-algorand)](https://github.com/CorvidLabs/swift-algorand/blob/main/LICENSE)
 [![Version](https://img.shields.io/github/v/release/CorvidLabs/swift-algorand)](https://github.com/CorvidLabs/swift-algorand/releases)
 
-> **Pre-1.0 Notice**: This SDK is under active development. The API may change between minor versions until 1.0. Not yet audited by a third-party security firm.
+> **Pre-1.0 Notice**: This SDK is under active development. The API may change between minor versions until 1.0.
 
 A modern Swift SDK for the Algorand blockchain. Built with Swift 6 and async/await.
 

README.md

@@ -134,13 +134,13 @@ public struct AssetCreateTransaction: Transaction {
             apar["df"] = .bool(true)
         }
 
-        if let unitName = assetParams.unitName {
+        if let unitName = assetParams.unitName, !unitName.isEmpty {
             apar["un"] = .string(unitName)
         }
-        if let assetName = assetParams.assetName {
+        if let assetName = assetParams.assetName, !assetName.isEmpty {
             apar["an"] = .string(assetName)
         }
-        if let url = assetParams.url {
+        if let url = assetParams.url, !url.isEmpty {
             apar["au"] = .string(url)
         }
         if let metadataHash = assetParams.metadataHash {

Sources/Algorand/AssetTransaction.swift

@@ -11,6 +11,11 @@ final class ComprehensiveIntegrationTest: XCTestCase {
     override func setUp() async throws {
         try await super.setUp()
 
+        // Skip integration tests on CI - they require Docker and local Algorand node
+        guard ProcessInfo.processInfo.environment["CI"] == nil else {
+            throw XCTSkip("Integration tests require local Algorand node - skipping on CI")
+        }
+
         let network = ProcessInfo.processInfo.environment["ALGORAND_NETWORK"] ?? "localnet"
         guard network == "localnet" else {
             throw XCTSkip("Comprehensive test only runs on LocalNet")

Tests/AlgorandTests/ComprehensiveIntegrationTest.swift

@@ -14,6 +14,11 @@ final class IntegrationTests: XCTestCase {
     override func setUp() async throws {
         try await super.setUp()
 
+        // Skip integration tests on CI - they require Docker and local Algorand node
+        guard ProcessInfo.processInfo.environment["CI"] == nil else {
+            throw XCTSkip("Integration tests require local Algorand node - skipping on CI")
+        }
+
         let network = ProcessInfo.processInfo.environment["ALGORAND_NETWORK"] ?? "localnet"
         isLocalNet = network == "localnet"