ci: enable linux test coverage (#856)

* ci: test in Linux

* keep only linux job

* import FoundationNetworking

* fix tests in Linux

* fix(realtime): stabilize channel tests on ci

* test(realtime): disable flaky realtime tests on linux

* revert CI to run all jobs

* fix test

* run tests in parallel and comment out flaky test

* ci: add caches

* disable parallel tests

Author: grdsdev

.github/workflows/ci.yml

@@ -6,33 +6,33 @@ on:
       - main
       - release/*
     paths:
-      - 'Sources/**'
-      - 'Tests/**'
-      - 'Examples/**'
-      - '*.swift'
-      - 'Package.swift'
-      - 'Package.resolved'
-      - '.github/workflows/ci.yml'
-      - 'Makefile'
-      - '*.xcodeproj/**'
-      - '*.xcworkspace/**'
-      - '.swiftpm/**'
+      - "Sources/**"
+      - "Tests/**"
+      - "Examples/**"
+      - "*.swift"
+      - "Package.swift"
+      - "Package.resolved"
+      - ".github/workflows/ci.yml"
+      - "Makefile"
+      - "*.xcodeproj/**"
+      - "*.xcworkspace/**"
+      - ".swiftpm/**"
   pull_request:
     branches:
       - "*"
       - release/*
     paths:
-      - 'Sources/**'
-      - 'Tests/**'
-      - 'Examples/**'
-      - '*.swift'
-      - 'Package.swift'
-      - 'Package.resolved'
-      - '.github/workflows/ci.yml'
-      - 'Makefile'
-      - '*.xcodeproj/**'
-      - '*.xcworkspace/**'
-      - '.swiftpm/**'
+      - "Sources/**"
+      - "Tests/**"
+      - "Examples/**"
+      - "*.swift"
+      - "Package.swift"
+      - "Package.resolved"
+      - ".github/workflows/ci.yml"
+      - "Makefile"
+      - "*.xcodeproj/**"
+      - "*.xcworkspace/**"
+      - ".swiftpm/**"
   workflow_dispatch:
 
 concurrency:
@@ -55,6 +55,14 @@ jobs:
           - { command: test, skip_release: 1 }
     steps:
       - uses: actions/checkout@v5
+      - name: Cache derived data
+        uses: actions/cache@v4
+        with:
+          path: ~/.derivedData
+          key: |
+            deriveddata-xcodebuild-${{ matrix.platform }}-${{ matrix.xcode }}-${{ matrix.command }}-${{ hashFiles('**/Sources/**/*.swift', '**/Tests/**/*.swift') }}
+          restore-keys: |
+            deriveddata-xcodebuild-${{ matrix.platform }}-${{ matrix.xcode }}-${{ matrix.command }}-
       - name: Select Xcode ${{ matrix.xcode }}
         run: sudo xcode-select -s /Applications/Xcode_${{ matrix.xcode }}.app
       - name: List available devices
@@ -119,7 +127,7 @@ jobs:
   spm:
     runs-on: macos-15
     strategy:
-      matrix: 
+      matrix:
         config: [debug, release]
     steps:
       - uses: actions/checkout@v5
@@ -136,11 +144,18 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v5
-      - name: "Remove IntegrationTests"
-        run: rm -r Tests/IntegrationTests/*
+      - name: "Cache Swift Package"
+        uses: actions/cache@v4
+        with:
+          path: .build
+          key: ${{ runner.os }}-spm-${{ hashFiles('**/Package.resolved') }}
+          restore-keys: |
+            ${{ runner.os }}-spm-
       - name: "Build Swift Package"
         run: swift build
-      
+      - name: "Test Swift Package"
+        run: swift test --skip IntegrationTests
+
   # android:
   #   name: Android
   #   runs-on: ubuntu-latest
@@ -164,6 +179,14 @@ jobs:
         xcode: ["16.3"]
     steps:
       - uses: actions/checkout@v5
+      - name: Cache derived data
+        uses: actions/cache@v4
+        with:
+          path: ~/.derivedData
+          key: |
+            deriveddata-library-evolution-${{ matrix.xcode }}-${{ hashFiles('**/Sources/**/*.swift', '**/Tests/**/*.swift') }}
+          restore-keys: |
+            deriveddata-library-evolution-${{ matrix.xcode }}-
       - name: Select Xcode ${{ matrix.xcode }}
         run: sudo xcode-select -s /Applications/Xcode_${{ matrix.xcode }}.app
       - name: Build for library evolution
@@ -200,5 +223,13 @@ jobs:
     runs-on: macos-15
     steps:
       - uses: actions/checkout@v5
+      - name: Cache Swift build
+        uses: actions/cache@v4
+        with:
+          path: .build
+          key: |
+            docs-${{ runner.os }}-${{ hashFiles('**/Package.resolved') }}
+          restore-keys: |
+            docs-${{ runner.os }}-
       - name: Test docs
         run: make test-docs

Sources/Storage/MultipartFormData.swift

@@ -419,8 +419,17 @@ class MultipartFormData {
       var buffer = [UInt8](repeating: 0, count: streamBufferSize)
       let bytesRead = inputStream.read(&buffer, maxLength: streamBufferSize)
 
-      if let error = inputStream.streamError {
-        throw MultipartFormDataError.inputStreamReadFailed(error: error)
+      if bytesRead < 0 {
+        if let error = inputStream.streamError {
+          throw MultipartFormDataError.inputStreamReadFailed(error: error)
+        } else {
+          throw MultipartFormDataError.inputStreamReadFailed(
+            error: MultipartFormDataError.UnexpectedInputStreamLength(
+              bytesExpected: bodyPart.bodyContentLength,
+              bytesRead: UInt64(encoded.count)
+            )
+          )
+        }
       }
 
       if bytesRead > 0 {
@@ -474,9 +483,17 @@ class MultipartFormData {
       let bufferSize = min(streamBufferSize, Int(bytesLeftToRead))
       var buffer = [UInt8](repeating: 0, count: bufferSize)
       let bytesRead = inputStream.read(&buffer, maxLength: bufferSize)
-
-      if let streamError = inputStream.streamError {
-        throw MultipartFormDataError.inputStreamReadFailed(error: streamError)
+      if bytesRead < 0 {
+        if let streamError = inputStream.streamError {
+          throw MultipartFormDataError.inputStreamReadFailed(error: streamError)
+        } else {
+          throw MultipartFormDataError.inputStreamReadFailed(
+            error: MultipartFormDataError.UnexpectedInputStreamLength(
+              bytesExpected: bodyPart.bodyContentLength,
+              bytesRead: bodyPart.bodyContentLength - bytesLeftToRead
+            )
+          )
+        }
       }
 
       if bytesRead > 0 {
@@ -514,8 +531,17 @@ class MultipartFormData {
     while bytesToWrite > 0, outputStream.hasSpaceAvailable {
       let bytesWritten = outputStream.write(buffer, maxLength: bytesToWrite)
 
-      if let error = outputStream.streamError {
-        throw MultipartFormDataError.outputStreamWriteFailed(error: error)
+      if bytesWritten < 0 {
+        if let error = outputStream.streamError {
+          throw MultipartFormDataError.outputStreamWriteFailed(error: error)
+        } else {
+          throw MultipartFormDataError.outputStreamWriteFailed(
+            error: MultipartFormDataError.UnexpectedInputStreamLength(
+              bytesExpected: UInt64(buffer.count),
+              bytesRead: UInt64(buffer.count - bytesToWrite)
+            )
+          )
+        }
       }
 
       bytesToWrite -= bytesWritten
@@ -650,10 +676,10 @@ enum MultipartFormDataError: Error {
 
   var underlyingError: (any Error)? {
     switch self {
-    case let .bodyPartFileNotReachableWithError(_, error),
-      let .bodyPartFileSizeQueryFailedWithError(_, error),
-      let .inputStreamReadFailed(error),
-      let .outputStreamWriteFailed(error):
+    case .bodyPartFileNotReachableWithError(_, let error),
+      .bodyPartFileSizeQueryFailedWithError(_, let error),
+      .inputStreamReadFailed(let error),
+      .outputStreamWriteFailed(let error):
       error
 
     case .bodyPartURLInvalid,
@@ -671,17 +697,17 @@ enum MultipartFormDataError: Error {
 
   var url: URL? {
     switch self {
-    case let .bodyPartURLInvalid(url),
-      let .bodyPartFilenameInvalid(url),
-      let .bodyPartFileNotReachable(url),
-      let .bodyPartFileNotReachableWithError(url, _),
-      let .bodyPartFileIsDirectory(url),
-      let .bodyPartFileSizeNotAvailable(url),
-      let .bodyPartFileSizeQueryFailedWithError(url, _),
-      let .bodyPartInputStreamCreationFailed(url),
-      let .outputStreamFileAlreadyExists(url),
-      let .outputStreamURLInvalid(url),
-      let .outputStreamCreationFailed(url):
+    case .bodyPartURLInvalid(let url),
+      .bodyPartFilenameInvalid(let url),
+      .bodyPartFileNotReachable(let url),
+      .bodyPartFileNotReachableWithError(let url, _),
+      .bodyPartFileIsDirectory(let url),
+      .bodyPartFileSizeNotAvailable(let url),
+      .bodyPartFileSizeQueryFailedWithError(let url, _),
+      .bodyPartInputStreamCreationFailed(let url),
+      .outputStreamFileAlreadyExists(let url),
+      .outputStreamURLInvalid(let url),
+      .outputStreamCreationFailed(let url):
       url
 
     case .inputStreamReadFailed, .outputStreamWriteFailed:

Supabase.xcworkspace/xcshareddata/xcschemes/Supabase.xcscheme

@@ -145,7 +145,8 @@
       </CodeCoverageTargets>
       <Testables>
          <TestableReference
-            skipped = "NO">
+            skipped = "NO"
+            parallelizable = "NO">
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "AuthTests"
@@ -155,7 +156,8 @@
             </BuildableReference>
          </TestableReference>
          <TestableReference
-            skipped = "NO">
+            skipped = "NO"
+            parallelizable = "NO">
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "FunctionsTests"
@@ -165,7 +167,8 @@
             </BuildableReference>
          </TestableReference>
          <TestableReference
-            skipped = "NO">
+            skipped = "NO"
+            parallelizable = "NO">
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "HelpersTests"
@@ -175,7 +178,8 @@
             </BuildableReference>
          </TestableReference>
          <TestableReference
-            skipped = "NO">
+            skipped = "NO"
+            parallelizable = "NO">
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "PostgRESTTests"
@@ -185,7 +189,8 @@
             </BuildableReference>
          </TestableReference>
          <TestableReference
-            skipped = "NO">
+            skipped = "NO"
+            parallelizable = "NO">
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "RealtimeTests"
@@ -195,7 +200,8 @@
             </BuildableReference>
          </TestableReference>
          <TestableReference
-            skipped = "NO">
+            skipped = "NO"
+            parallelizable = "NO">
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "StorageTests"
@@ -205,7 +211,8 @@
             </BuildableReference>
          </TestableReference>
          <TestableReference
-            skipped = "NO">
+            skipped = "NO"
+            parallelizable = "NO">
             <BuildableReference
                BuildableIdentifier = "primary"
                BlueprintIdentifier = "SupabaseTests"