Test skip jobs

.github/workflows/android.yaml

@@ -25,10 +25,11 @@ jobs:
         id: check_changes
         run: |
           ./ci/check_bazel.sh //examples/android:android_app || ./ci/files_changed.sh .github/workflows/android.yaml || ./ci/files_changed.sh "^platform/jvm/gradle-test-app/.*\.(gradle|kts|kt|xml)$" && ./ci/run_tests.sh
-          true
   build_apk:
     runs-on: ubuntu-latest
-    if: needs.pre_check.outputs.should_run == 'true'
+    # TODO Remove hardcoding after testing
+    if: false
+#    if: needs.pre_check.outputs.should_run == 'true'
     needs: pre_check
     steps:
       # Checkout repo to Github Actions runner
@@ -49,7 +50,9 @@ jobs:
     runs-on: ubuntu-latest-8-cores
     env:
       SKIP_PROTO_GEN: 1
-    if: needs.pre_check.outputs.should_run == 'true'
+    # TODO Remove hardcoding after testing
+    if: false
+#    if: needs.pre_check.outputs.should_run == 'true'
     needs: pre_check
     steps:
     - name: Checkout project sources
@@ -115,6 +118,8 @@ jobs:
         script: cd platform/jvm && adb uninstall io.bitdrift.gradletestapp.test; adb uninstall io.bitdrift.gradletestapp; cd ../.. && ./tools/android_sdk_wrapper.sh platform/jvm/gradlew -p platform/jvm gradle-test-app:check gradle-test-app:connectedCheck --stacktrace
 
   verify_android_hello_world_per_version:
+    # TODO Remove hardcoding after testing
+    if: false
     needs: build_apk
     runs-on: ubuntu-latest-8-cores
     strategy:
@@ -178,7 +183,7 @@ jobs:
   # job completing, we are able to gate it on all the previous jobs without explicitly enumerating them.
   verify_android:
     runs-on: ubuntu-latest
-    needs: ["build_apk", "verify_android_hello_world_per_version", "gradle_tests"]
+    needs: ["pre_check", "build_apk", "verify_android_hello_world_per_version", "gradle_tests"]
     if: always()
     steps:
     # Checkout repo to Github Actions runner
@@ -187,4 +192,4 @@ jobs:
       with:
         fetch-depth: 1
     - name: check result
-      run: ./ci/check_result.sh ${{ needs.build_apk.result }} && ./ci/check_result.sh ${{ needs.verify_android_hello_world_per_version.result }} && ./ci/check_result.sh ${{ needs.gradle_tests.result }}
+      run: ./ci/check_result.sh ${{ needs.pre_check.result }} && ./ci/check_result.sh ${{ needs.build_apk.result }} && ./ci/check_result.sh ${{ needs.verify_android_hello_world_per_version.result }} && ./ci/check_result.sh ${{ needs.gradle_tests.result }}

WORKSPACE

@@ -275,9 +275,9 @@ http_archive(
 
 http_archive(
     name = "robolectric",
-    sha256 = "ba1269064f5509531b024cdea70349e97756f0f639e53b7cbb0938127218d6f8",
-    strip_prefix = "robolectric-bazel-4.11",
-    urls = ["https://github.com/robolectric/robolectric-bazel/archive/4.11.tar.gz"],
+    sha256 = "a270fd6fd83f9f024623e787696e6b73c44664b7c95f3d937ed35bf0a94a67ae",
+    strip_prefix = "robolectric-bazel-4.13",
+    urls = ["https://github.com/robolectric/robolectric-bazel/releases/download/4.13/robolectric-bazel-4.13.tar.gz"],
 )
 
 load("@robolectric//bazel:robolectric.bzl", "robolectric_repositories")

bazel/capture_dependencies.bzl

@@ -18,7 +18,7 @@ def jvm_dependencies():
             "org.mockito:mockito-inline:4.11.0",
             "com.nhaarman.mockitokotlin2:mockito-kotlin:2.2.0",
             "androidx.test:core:1.6.0",
-            "org.robolectric:robolectric:4.11",
+            "org.robolectric:robolectric:4.13",
             "org.assertj:assertj-core:3.22.0",
             "com.squareup.okhttp3:mockwebserver:{}".format(okhttp_version),
             "junit:junit:4.13.2",

ci/files_changed.sh

@@ -1,10 +1,22 @@
 #!/bin/bash
 
-# Checks whether the files in provided regex via the command line has changed when comparing the HEAD ref and
-# $GITHUB_BASE_REF, i.e. the target branch (usually main). Returns true if the current branch is main.
-#
-# Usage: ./ci/files_changed.sh <regex>
-
 set -e
 
-git rev-parse --abbrev-ref HEAD | grep -q ^main$ || git diff --name-only "origin/$GITHUB_BASE_REF" | grep -E "$1"
+# Get the current branch name
+current_branch=$(git rev-parse --abbrev-ref HEAD)
+
+# Find the remote associated with the base branch (GITHUB_BASE_REF)
+remote_name=$(git remote -v | grep fetch | awk '{print $1}' | head -n 1)
+
+# If the current branch is "main", exit successfully (no need to check file changes)
+if [[ "$current_branch" == "main" ]]; then
+  exit 0
+fi
+
+# Fetch the base branch if it doesn't exist locally
+if ! git show-ref --quiet refs/remotes/$remote_name/$GITHUB_BASE_REF; then
+  git fetch $remote_name $GITHUB_BASE_REF
+fi
+
+# Check if any files matching the regex have changed between the current HEAD and the base branch
+git diff --name-only "$remote_name/$GITHUB_BASE_REF" | grep -E "$1"

platform/jvm/capture/src/test/kotlin/io/bitdrift/capture/ClientAttributesTest.kt

@@ -0,0 +1,187 @@
+// capture-sdk - bitdrift's client SDK
+// Copyright Bitdrift, Inc. All rights reserved.
+//
+// Use of this source code is governed by a source available license that can be found in the
+// LICENSE file or at:
+// https://polyformproject.org/wp-content/uploads/2020/06/PolyForm-Shield-1.0.0.txt
+
+package io.bitdrift.capture
+
+import android.content.Context
+import android.content.pm.PackageInfo
+import android.content.pm.PackageManager
+import androidx.lifecycle.Lifecycle
+import androidx.lifecycle.LifecycleOwner
+import androidx.test.core.app.ApplicationProvider
+import io.bitdrift.capture.attributes.ClientAttributes
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.mockito.ArgumentMatchers.any
+import org.mockito.ArgumentMatchers.eq
+import org.mockito.Mockito.anyString
+import org.mockito.Mockito.doReturn
+import org.mockito.Mockito.mock
+import org.mockito.Mockito.spy
+import org.mockito.Mockito.verify
+import org.robolectric.RobolectricTestRunner
+import org.robolectric.annotation.Config
+
+@Suppress("DEPRECATION")
+@RunWith(RobolectricTestRunner::class)
+@Config(sdk = [21])
+class ClientAttributesTest {
+
+    @Test
+    fun foreground() {
+        val context = ApplicationProvider.getApplicationContext<Context>()
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("foreground", "1")
+    }
+
+    @Test
+    fun not_foreground() {
+        val context = ApplicationProvider.getApplicationContext<Context>()
+        val mockedLifecycleOwnerLifecycleStateCreated = obtainMockedLifecycleOwnerWith(Lifecycle.State.CREATED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateCreated).invoke()
+
+        assertThat(clientAttributes).containsEntry("foreground", "0")
+    }
+
+    @Test
+    fun app_id() {
+        val packageName = "my.bitdrift.test"
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        doReturn(packageName).`when`(context).packageName
+        val mockedLifecycleOwnerLifecycleStateStarted =
+            obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("app_id", packageName)
+    }
+
+    @Test
+    fun app_id_unknown() {
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        doReturn(null).`when`(context).packageName
+        val mockedLifecycleOwnerLifecycleStateStarted =
+            obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("app_id", "unknown")
+    }
+
+    @Test
+    fun app_version() {
+        val versionName = "1.2.3.4"
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        val mockedPackageInfo = obtainMockedPackageInfo(context)
+        mockedPackageInfo.versionName = versionName
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("app_version", versionName)
+    }
+
+    @Test
+    fun app_version_unknown() {
+        val context = ApplicationProvider.getApplicationContext<Context>()
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("app_version", "?.?.?")
+    }
+
+    @Test
+    fun app_version_code() {
+        val versionCode = 66
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        val mockedPackageInfo = obtainMockedPackageInfo(context)
+        mockedPackageInfo.versionCode = versionCode
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("_app_version_code", versionCode.toString())
+    }
+
+    @Test
+    fun app_version_code_unknown() {
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        val packageManager = obtainMockedPackageManager(context)
+        doReturn(null).`when`(packageManager).getPackageInfo(anyString(), eq(0))
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("_app_version_code", "-1")
+    }
+
+    @Test
+    @Config(sdk = [28])
+    fun app_version_code_android_pie() {
+        val versionCode = 66L
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        val mockedPackageInfo = obtainMockedPackageInfo(context)
+        doReturn(versionCode).`when`(mockedPackageInfo).longVersionCode
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("_app_version_code", versionCode.toString())
+    }
+
+    @Test
+    @Config(sdk = [28])
+    fun app_version_code_unknown_android_pie() {
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        val packageManager = obtainMockedPackageManager(context)
+        doReturn(null).`when`(packageManager).getPackageInfo(anyString(), eq(0))
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        val clientAttributes = ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        assertThat(clientAttributes).containsEntry("_app_version_code", "-1")
+    }
+
+    @Test
+    @Config(sdk = [33])
+    fun package_info_android_tiramisu() {
+        val context = spy(ApplicationProvider.getApplicationContext<Context>())
+        val packageManager = obtainMockedPackageManager(context)
+        val mockedLifecycleOwnerLifecycleStateStarted = obtainMockedLifecycleOwnerWith(Lifecycle.State.STARTED)
+
+        ClientAttributes(context, mockedLifecycleOwnerLifecycleStateStarted).invoke()
+
+        verify(packageManager).getPackageInfo(anyString(), any(PackageManager.PackageInfoFlags::class.java))
+    }
+
+    private fun obtainMockedLifecycleOwnerWith(state: Lifecycle.State): LifecycleOwner {
+        val mockedLifecycle = mock(Lifecycle::class.java)
+        doReturn(state).`when`(mockedLifecycle).currentState
+        val mockedLifecycleOwner = mock(LifecycleOwner::class.java)
+        doReturn(mockedLifecycle).`when`(mockedLifecycleOwner).lifecycle
+        return mockedLifecycleOwner
+    }
+
+    private fun obtainMockedPackageInfo(context: Context): PackageInfo {
+        val mockedPackageManager = obtainMockedPackageManager(context)
+        val mockedPackageInfo: PackageInfo = mock(PackageInfo::class.java)
+        doReturn(mockedPackageInfo).`when`(mockedPackageManager).getPackageInfo(anyString(), eq(0))
+        return mockedPackageInfo
+    }
+
+    private fun obtainMockedPackageManager(context: Context): PackageManager {
+        val mockedPackageManager: PackageManager = mock(PackageManager::class.java)
+        doReturn(mockedPackageManager).`when`(context).packageManager
+        return mockedPackageManager
+    }
+}

platform/jvm/capture/src/test/kotlin/io/bitdrift/capture/DeviceAttributesTest.kt

@@ -0,0 +1,35 @@
+// capture-sdk - bitdrift's client SDK
+// Copyright Bitdrift, Inc. All rights reserved.
+//
+// Use of this source code is governed by a source available license that can be found in the
+// LICENSE file or at:
+// https://polyformproject.org/wp-content/uploads/2020/06/PolyForm-Shield-1.0.0.txt
+
+package io.bitdrift.capture
+
+import androidx.test.core.app.ApplicationProvider
+import io.bitdrift.capture.attributes.DeviceAttributes
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.robolectric.RobolectricTestRunner
+import org.robolectric.annotation.Config
+
+@RunWith(RobolectricTestRunner::class)
+@Config(sdk = [21])
+class DeviceAttributesTest {
+
+    @Test
+    fun model() {
+        val deviceAttributes = DeviceAttributes(ApplicationProvider.getApplicationContext()).invoke()
+
+        assertThat(deviceAttributes).containsEntry("model", "robolectric")
+    }
+
+    @Test
+    fun locale() {
+        val deviceAttributes = DeviceAttributes(ApplicationProvider.getApplicationContext()).invoke()
+
+        assertThat(deviceAttributes).containsEntry("_locale", "en_US")
+    }
+}