test: Remove empty disabled test stubs from DeviceDataCollectorTest

The disabled tests were placeholders that couldn't be implemented as
unit tests due to Android framework static fields (Build.*, etc.).
Individual collectors have their own tests, and full integration
testing should use instrumented tests.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: oschwald

device-sdk/src/test/java/com/maxmind/device/collector/DeviceDataCollectorTest.kt

@@ -1,196 +1,22 @@
 package com.maxmind.device.collector
 
-import android.app.ActivityManager
 import android.content.Context
-import android.content.pm.PackageInfo
-import android.content.pm.PackageManager
-import android.os.Environment
-import android.os.StatFs
-import android.provider.Settings
-import android.util.DisplayMetrics
-import android.view.Display
-import android.view.WindowManager
-import io.mockk.every
 import io.mockk.mockk
-import io.mockk.mockkStatic
-import io.mockk.slot
-import io.mockk.unmockkStatic
-import org.junit.jupiter.api.AfterEach
 import org.junit.jupiter.api.Assertions.assertNotNull
-import org.junit.jupiter.api.Assertions.assertTrue
-import org.junit.jupiter.api.BeforeEach
-import org.junit.jupiter.api.Disabled
 import org.junit.jupiter.api.Test
-import java.io.File
 
 /**
  * Tests for DeviceDataCollector.
  *
- * Note: Most tests require Android framework classes (Build.*, Typeface, etc.)
- * that cannot be mocked in pure unit tests. These tests are disabled and should
- * be run in instrumented tests on real devices.
- *
- * Uses MockK to mock Android framework classes and services.
+ * Full integration testing of collect() requires instrumented tests
+ * due to Android framework static fields (Build.*, etc.).
  */
 internal class DeviceDataCollectorTest {
-    private lateinit var mockContext: Context
-    private lateinit var mockWindowManager: WindowManager
-    private lateinit var mockActivityManager: ActivityManager
-    private lateinit var mockPackageManager: PackageManager
-    private lateinit var mockDisplay: Display
-
-    @BeforeEach
-    internal fun setUp() {
-        mockContext = mockk(relaxed = true)
-        mockWindowManager = mockk(relaxed = true)
-        mockActivityManager = mockk(relaxed = true)
-        mockPackageManager = mockk(relaxed = true)
-        mockDisplay = mockk(relaxed = true)
-
-        every { mockContext.getSystemService(Context.WINDOW_SERVICE) } returns mockWindowManager
-        every { mockContext.getSystemService(Context.ACTIVITY_SERVICE) } returns mockActivityManager
-        every { mockContext.packageManager } returns mockPackageManager
-        every { mockContext.packageName } returns "com.test.app"
-
-        // Mock display metrics
-        val metricsSlot = slot<DisplayMetrics>()
-        @Suppress("DEPRECATION")
-        every { mockWindowManager.defaultDisplay } returns mockDisplay
-        @Suppress("DEPRECATION")
-        every { mockDisplay.getMetrics(capture(metricsSlot)) } answers {
-            metricsSlot.captured.widthPixels = 1080
-            metricsSlot.captured.heightPixels = 1920
-            metricsSlot.captured.densityDpi = 480
-            metricsSlot.captured.density = 3.0f
-        }
-        every { mockDisplay.refreshRate } returns 60.0f
-
-        // Mock memory info
-        every { mockActivityManager.getMemoryInfo(any()) } answers {
-            val info = firstArg<ActivityManager.MemoryInfo>()
-            info.totalMem = 4_000_000_000L
-        }
-
-        // Mock package info
-        val packageInfo = PackageInfo()
-        packageInfo.firstInstallTime = 1700000000000L
-        packageInfo.lastUpdateTime = 1700100000000L
-        packageInfo.versionName = "1.0.0"
-        @Suppress("DEPRECATION")
-        packageInfo.versionCode = 1
-        every { mockPackageManager.getPackageInfo("com.test.app", 0) } returns packageInfo
-
-        // Mock content resolver for DeviceIDsCollector
-        val mockContentResolver = mockk<android.content.ContentResolver>(relaxed = true)
-        every { mockContext.contentResolver } returns mockContentResolver
-
-        mockkStatic(Settings.Secure::class)
-        every {
-            Settings.Secure.getString(mockContentResolver, Settings.Secure.ANDROID_ID)
-        } returns "test-android-id"
-
-        // Mock Environment and StatFs for storage
-        mockkStatic(Environment::class)
-        every { Environment.getDataDirectory() } returns File("/data")
-
-        mockkStatic(StatFs::class)
-    }
-
-    @AfterEach
-    internal fun tearDown() {
-        unmockkStatic(Settings.Secure::class)
-        unmockkStatic(Environment::class)
-        unmockkStatic(StatFs::class)
-    }
-
-    // Note: The following tests are disabled because DeviceDataCollector.collect()
-    // accesses Android framework static fields (Build.*, Typeface, etc.) that
-    // cannot be mocked in pure unit tests without Robolectric.
-    //
-    // These should be tested in instrumented tests (androidTest).
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with build info`() {
-        // This test requires Build.FINGERPRINT, etc. which are null in unit tests
-    }
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with display info`() {
-        // This test requires WindowManager which is mocked but inner calls fail
-    }
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with hardware info`() {
-        // Requires Android framework
-    }
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with installation info`() {
-        // Requires Android framework
-    }
 
     @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with locale info`() {
-        // Requires Android framework
-    }
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with timezone offset`() {
-        // Requires Android framework
-    }
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with device time`() {
-        // Requires Android framework
-    }
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `collect returns DeviceData with device IDs`() {
-        // Requires Android framework
-    }
-
-    @Test
-    @Disabled("Requires Android framework - use instrumented tests")
-    internal fun `build info supportedAbis is not empty`() {
-        // Requires Build.SUPPORTED_ABIS which is null in unit tests
-    }
-
-    // Tests that can run without full Android framework
-
-    @Test
-    internal fun `collector can be instantiated with mock context`() {
-        // Just verify the collector can be created
+    internal fun `collector can be instantiated with context`() {
+        val mockContext = mockk<Context>(relaxed = true)
         val collector = DeviceDataCollector(mockContext)
         assertNotNull(collector)
     }
-
-    @Test
-    internal fun `locale info uses system defaults`() {
-        // Verify Locale.getDefault() works in unit tests
-        val locale = java.util.Locale.getDefault()
-        assertNotNull(locale.language)
-    }
-
-    @Test
-    internal fun `timezone offset is within valid range`() {
-        // Verify TimeZone.getDefault() works in unit tests
-        val offset = java.util.TimeZone.getDefault().rawOffset / 60000
-        assertTrue(offset >= -720)
-        assertTrue(offset <= 840)
-    }
-
-    @Test
-    internal fun `device time is current`() {
-        // Verify System.currentTimeMillis() works in unit tests
-        val time = System.currentTimeMillis()
-        assertTrue(time > 0)
-    }
 }