CICD

.github/workflows/build.yml

@@ -0,0 +1,51 @@
+name: Build package
+
+on:
+  push:
+    branches: [master]
+  pull_request:
+
+jobs:
+  test:
+    name: Run Build for ${{ matrix.unityVersion }} ${{ matrix.unityVersion }}
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        projectPath:
+          - test-package
+        unityVersion:
+          - 2019.4.40f1
+        targetPlatform:
+          - StandaloneOSX
+          - StandaloneWindows
+          - StandaloneWindows64
+          - StandaloneLinux64
+          - iOS
+          - Android
+          - WebGL
+    steps:
+      - name: Checkout Repo
+        uses: actions/checkout@v4
+        with:
+          lfs: true
+
+      - name: Setup Environment
+        run: |
+          mkdir ${{ matrix.projectPath }}
+          mv Editor Runtime Tests Android iOS Windows package.json ${{ matrix.projectPath }}/
+
+      - if: matrix.targetPlatform == 'Android'
+        uses: jlumbroso/[email protected]
+
+      - uses: game-ci/unity-builder@v4
+        name: Build game for platform ${{ matrix.targetPlatform }}
+        env:
+          UNITY_EMAIL: ${{ secrets.UNITY_EMAIL }}
+          UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
+          UNITY_SERIAL: ${{ secrets.UNITY_SERIAL }}
+        with:
+          unityVersion: ${{ matrix.unityVersion }}
+          targetPlatform: ${{ matrix.targetPlatform }}
+          projectPath: ${{ matrix.projectPath }}/
+          allowDirtyBuild: true

.github/workflows/test.yml

@@ -0,0 +1,46 @@
+name: Test package
+
+on:
+    push:
+        branches: [master]
+    pull_request:
+
+jobs:
+  test:
+    name: Run Tests in ${{ matrix.testMode }} ${{ matrix.unityVersion }}
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        testMode:
+          - playmode
+          # - editmode
+        projectPath:
+          - test-package
+        unityVersion:
+          - 2022.3.19f1
+          - 2019.4.40f1
+          - 2020.3.48f1
+    steps:
+      - name: Checkout Repo
+        uses: actions/checkout@v4
+        with:
+          lfs: true
+
+      - name: Setup Environment
+        run: |
+          mkdir ${{ matrix.projectPath }}
+          mv Editor Runtime Tests Android iOS Windows package.json ${{ matrix.projectPath }}/
+
+      - name: Run Tests
+        uses: game-ci/[email protected]
+        env:
+          UNITY_EMAIL: ${{ secrets.UNITY_EMAIL }}
+          UNITY_PASSWORD: ${{ secrets.UNITY_PASSWORD }}
+          UNITY_SERIAL: ${{ secrets.UNITY_SERIAL }}
+        with:
+          packageMode: true
+          projectPath: ${{ matrix.projectPath }}/
+          testMode: ${{ matrix.testMode }}
+          unityVersion: ${{ matrix.unityVersion }}
+          coverageOptions: "generateAdditionalMetrics;generateHtmlReport;generateBadgeReport;assemblyFilters:+my.assembly.*"

Runtime/BacktraceClient.cs

@@ -839,7 +839,7 @@ private IEnumerator CollectDataAndSend(BacktraceReport report, Action<BacktraceR
 
             if (Database != null && Database.Enabled())
             {
-                yield return new WaitForEndOfFrame();
+                yield return WaitForFrame.Wait();
                 if (EnablePerformanceStatistics)
                 {
                     stopWatch.Restart();
@@ -885,7 +885,7 @@ record = Database.Add(data);
                 stopWatch.Stop();
                 queryAttributes["performance.json"] = stopWatch.GetMicroseconds();
             }
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
             if (string.IsNullOrEmpty(json))
             {
                 yield break;

Runtime/Model/WaitForFrame.cs

@@ -0,0 +1,25 @@
+using Backtrace.Unity.Model.Waiter;
+using UnityEngine;
+
+namespace Backtrace.Unity.Model
+{
+    public class WaitForFrame
+    {
+        private static IWaiter _waiter = CreateWaiterStrategy();
+
+        public static YieldInstruction Wait()
+        {
+            return _waiter.Wait();
+        }
+
+        private static IWaiter CreateWaiterStrategy()
+        {
+            if (Application.isBatchMode)
+            {
+                return new BatchModeWaiter();
+            }
+
+            return new EndOfFrameWaiter();
+        }
+    }
+}

Runtime/Waiter/BatchModeWaiter.cs

@@ -0,0 +1,12 @@
+using UnityEngine;
+
+namespace Backtrace.Unity.Model.Waiter
+{
+    public class BatchModeWaiter : IWaiter
+    {
+        public YieldInstruction Wait()
+        {
+            return null;
+        }
+    }
+}

Runtime/Waiter/EndOfFrameWaiter.cs

@@ -0,0 +1,12 @@
+using UnityEngine;
+
+namespace Backtrace.Unity.Model.Waiter
+{
+    public class EndOfFrameWaiter : IWaiter
+    {
+        public YieldInstruction Wait()
+        {
+            return new WaitForEndOfFrame();
+        }
+    }
+}

Runtime/Waiter/IWaiter.cs

@@ -0,0 +1,10 @@
+using System.Collections;
+using UnityEngine;
+
+namespace Backtrace.Unity.Model.Waiter
+{
+    public interface IWaiter
+    {
+        YieldInstruction Wait();
+    }
+}

Tests/Runtime/BacktraceAttributeTests.cs

@@ -37,7 +37,7 @@ public IEnumerator TesClientAttributeAccessor_BacktraceDataShouldIncludeClientAt
                  return null;
              };
             BacktraceClient.Send(new Exception("foo"));
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
             Assert.IsNotNull(data);
             Assert.AreEqual(data.Attributes.Attributes[key], value);
             yield return null;
@@ -60,9 +60,9 @@ public IEnumerator TesClientAttributes_ReportShouldntExtendClientAttributes_Clie
             foreach (var exceptionMessage in exceptionsMessage)
             {
                 BacktraceClient.Send(new Exception(exceptionMessage));
-                yield return new WaitForEndOfFrame();
-            }
 
+            }
+            yield return WaitForFrame.Wait();
             Assert.AreEqual(data.Attributes.Attributes[key], value);
             Assert.AreEqual(numberOfKeysBeforeSendRequest, BacktraceClient.GetAttributesCount());
             yield return null;
@@ -85,7 +85,7 @@ public IEnumerator TesClientAttributesMethod_BacktraceDataShouldIncludeClientAtt
                 return null;
             };
             BacktraceClient.Send(new Exception("foo"));
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
 
             Assert.IsNotNull(data);
             Assert.AreEqual(data.Attributes.Attributes[key], value);

Tests/Runtime/BacktraceClientTests.cs

@@ -93,7 +93,7 @@ public IEnumerator TestBeforeSendEvent_UpdateReportAttributesForUnhandledExcepti
 
             var unhandledException = new BacktraceUnhandledException("foo", string.Empty);
             BacktraceClient.Send(unhandledException);
-            yield return null;
+            yield return WaitForFrame.Wait();
         }
 
         [UnityTest]
@@ -108,7 +108,7 @@ public IEnumerator TestBeforeSendEvent_ValidConfiguration_EventTrigger()
                 return backtraceData;
             };
             BacktraceClient.Send(new Exception("test exception"));
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
             Assert.IsTrue(trigger);
             yield return null;
         }
@@ -128,7 +128,7 @@ public IEnumerator TestSendingReport_ValidConfiguration_ValidSend()
                 return new BacktraceResult();
             };
             BacktraceClient.Send(new Exception("test exception"));
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
             Assert.IsTrue(trigger);
             yield return null;
         }
@@ -217,9 +217,10 @@ public IEnumerator TestFingerprintBehaviorForNormalizedExceptionMessage_ShouldUs
                 return null;
             };
             BacktraceClient.Send(report);
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
 
             Assert.IsTrue(eventFired);
+            yield return null;
         }
 
         [Test]
@@ -284,15 +285,17 @@ public IEnumerator TestFingerprintBehaviorForNormalizedExceptionMessage_Shouldnt
                 return null;
             };
             BacktraceClient.Send(report);
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
             Assert.IsTrue(eventFired);
+            yield return null;
         }
 
 
         private IEnumerator CallBacktraceClientAndWait(BacktraceReport report)
         {
             BacktraceClient.Send(report);
-            yield return new WaitForEndOfFrame();
+            yield return WaitForFrame.Wait();
+
         }
     }
 }