Fix variable declarations in test script

Author: tustanivsky

.github/workflows/ci.yml

@@ -205,7 +205,7 @@ jobs:
     secrets: inherit
     strategy:
       fail-fast: false
-      max-parallel: 2
+      max-parallel: 1
       matrix:
         unreal: ['5.4', '5.5', '5.6', '5.7']
     uses: ./.github/workflows/integration-test-android.yml

integration-test/Integration.Android.Tests.ps1

@@ -170,19 +170,19 @@ Describe 'Sentry Unreal Android Integration Tests (<Platform>)' -ForEach $TestTa
         BeforeAll {
             # Crash event is sent during the MESSAGE run (Run 2)
             # But the crash_id comes from the CRASH run (Run 1)
-            $CrashResult = $global:AndroidCrashResult
-            $CrashEvent = $null
+            $script:CrashResult = $global:AndroidCrashResult
+            $script:CrashEvent = $null
 
             # Parse crash event ID from crash run output
-            $eventIds = Get-EventIds -AppOutput $CrashResult.Output -ExpectedCount 1
+            $eventIds = Get-EventIds -AppOutput $script:CrashResult.Output -ExpectedCount 1
 
             if ($eventIds -and $eventIds.Count -gt 0) {
                 Write-Host "Crash ID captured: $($eventIds[0])" -ForegroundColor Cyan
                 $crashId = $eventIds[0]
 
                 # Fetch crash event using the tag (event was sent during message run)
                 try {
-                    $CrashEvent = Get-SentryTestEvent -TagName 'test.crash_id' -TagValue "$crashId"
+                    $script:CrashEvent = Get-SentryTestEvent -TagName 'test.crash_id' -TagValue "$crashId"
                     Write-Host "Crash event fetched from Sentry successfully" -ForegroundColor Green
                 }
                 catch {
@@ -195,60 +195,60 @@ Describe 'Sentry Unreal Android Integration Tests (<Platform>)' -ForEach $TestTa
         }
 
         It "Should output event ID before crash" {
-            $eventIds = Get-EventIds -AppOutput $CrashResult.Output -ExpectedCount 1
+            $eventIds = Get-EventIds -AppOutput $script:CrashResult.Output -ExpectedCount 1
             $eventIds | Should -Not -BeNullOrEmpty
             $eventIds.Count | Should -Be 1
         }
 
         It "Should capture crash event in Sentry (uploaded during next run)" {
-            $CrashEvent | Should -Not -BeNullOrEmpty
+            $script:CrashEvent | Should -Not -BeNullOrEmpty
         }
 
         It "Should have correct event type and platform" {
-            $CrashEvent.type | Should -Be 'error'
-            $CrashEvent.platform | Should -Be 'native'
+            $script:CrashEvent.type | Should -Be 'error'
+            $script:CrashEvent.platform | Should -Be 'native'
         }
 
         It "Should have exception information" {
-            $CrashEvent.exception | Should -Not -BeNullOrEmpty
-            $CrashEvent.exception.values | Should -Not -BeNullOrEmpty
+            $script:CrashEvent.exception | Should -Not -BeNullOrEmpty
+            $script:CrashEvent.exception.values | Should -Not -BeNullOrEmpty
         }
 
         It "Should have stack trace" {
-            $exception = $CrashEvent.exception.values[0]
+            $exception = $script:CrashEvent.exception.values[0]
             $exception.stacktrace | Should -Not -BeNullOrEmpty
             $exception.stacktrace.frames | Should -Not -BeNullOrEmpty
         }
 
         It "Should have user context" {
-            $CrashEvent.user | Should -Not -BeNullOrEmpty
-            $CrashEvent.user.username | Should -Be 'TestUser'
-            $CrashEvent.user.email | Should -Be '[email protected]'
-            $CrashEvent.user.id | Should -Be '12345'
+            $script:CrashEvent.user | Should -Not -BeNullOrEmpty
+            $script:CrashEvent.user.username | Should -Be 'TestUser'
+            $script:CrashEvent.user.email | Should -Be '[email protected]'
+            $script:CrashEvent.user.id | Should -Be '12345'
         }
 
         It "Should have test.crash_id tag for correlation" {
-            $tags = $CrashEvent.tags
+            $tags = $script:CrashEvent.tags
             $crashIdTag = $tags | Where-Object { $_.key -eq 'test.crash_id' }
             $crashIdTag | Should -Not -BeNullOrEmpty
             $crashIdTag.value | Should -Not -BeNullOrEmpty
         }
 
         It "Should have integration test tag" {
-            $tags = $CrashEvent.tags
+            $tags = $script:CrashEvent.tags
             ($tags | Where-Object { $_.key -eq 'test.suite' }).value | Should -Be 'integration'
         }
 
         It "Should have breadcrumbs from before crash" {
-            $CrashEvent.breadcrumbs | Should -Not -BeNullOrEmpty
-            $CrashEvent.breadcrumbs.values | Should -Not -BeNullOrEmpty
+            $script:CrashEvent.breadcrumbs | Should -Not -BeNullOrEmpty
+            $script:CrashEvent.breadcrumbs.values | Should -Not -BeNullOrEmpty
         }
     }
 
     Context "Message Capture Tests" {
         BeforeAll {
-            $MessageResult = $global:AndroidMessageResult
-            $MessageEvent = $null
+            $script:MessageResult = $global:AndroidMessageResult
+            $script:MessageEvent = $null
 
             # Parse event ID from output
             $eventIds = Get-EventIds -AppOutput $MessageResult.Output -ExpectedCount 1
@@ -258,7 +258,7 @@ Describe 'Sentry Unreal Android Integration Tests (<Platform>)' -ForEach $TestTa
 
                 # Fetch event from Sentry (with polling)
                 try {
-                    $MessageEvent = Get-SentryTestEvent -EventId $eventIds[0]
+                    $script:MessageEvent = Get-SentryTestEvent -EventId $eventIds[0]
                     Write-Host "Message event fetched from Sentry successfully" -ForegroundColor Green
                 }
                 catch {
@@ -283,32 +283,32 @@ Describe 'Sentry Unreal Android Integration Tests (<Platform>)' -ForEach $TestTa
         }
 
         It "Should capture message event in Sentry" {
-            $MessageEvent | Should -Not -BeNullOrEmpty
+            $script:MessageEvent | Should -Not -BeNullOrEmpty
         }
 
         It "Should have correct platform" {
             # Android events are captured from Java layer, so platform is 'java' not 'native'
-            $MessageEvent.platform | Should -Be 'java'
+            $script:MessageEvent.platform | Should -Be 'java'
         }
 
         It "Should have message content" {
-            $MessageEvent.message | Should -Not -BeNullOrEmpty
-            $MessageEvent.message.formatted | Should -Match 'Integration test message'
+            $script:MessageEvent.message | Should -Not -BeNullOrEmpty
+            $script:MessageEvent.message.formatted | Should -Match 'Integration test message'
         }
 
         It "Should have user context" {
-            $MessageEvent.user | Should -Not -BeNullOrEmpty
-            $MessageEvent.user.username | Should -Be 'TestUser'
+            $script:MessageEvent.user | Should -Not -BeNullOrEmpty
+            $script:MessageEvent.user.username | Should -Be 'TestUser'
         }
 
         It "Should have integration test tag" {
-            $tags = $MessageEvent.tags
+            $tags = $script:MessageEvent.tags
             ($tags | Where-Object { $_.key -eq 'test.suite' }).value | Should -Be 'integration'
         }
 
         It "Should have breadcrumbs" {
-            $MessageEvent.breadcrumbs | Should -Not -BeNullOrEmpty
-            $MessageEvent.breadcrumbs.values | Should -Not -BeNullOrEmpty
+            $script:MessageEvent.breadcrumbs | Should -Not -BeNullOrEmpty
+            $script:MessageEvent.breadcrumbs.values | Should -Not -BeNullOrEmpty
         }
     }
 }