only grant permission for unit test if we are on a device that needs it

smartdevicelink · Feb 13, 2024 · 0b0f350 · 0b0f350
1 parent 1eaf6aa
commit 0b0f350
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/...sdl_android/src/androidTest/java/com/smartdevicelink/transport/TransportManagerTests.java b/...sdl_android/src/androidTest/java/com/smartdevicelink/transport/TransportManagerTests.java
@@ -2,6 +2,7 @@
 
 import android.Manifest;
 import android.content.ComponentName;
+import android.os.Build;
 import android.os.Looper;
 
 import androidx.test.ext.junit.runners.AndroidJUnit4;
@@ -33,7 +34,7 @@
 public class TransportManagerTests {
 
     @Rule
-    public GrantPermissionRule btRuntimePermissionRule = GrantPermissionRule.grant(Manifest.permission.BLUETOOTH_CONNECT);
+    public GrantPermissionRule btRuntimePermissionRule;
     MultiplexTransportConfig config;
     final TransportRecord defaultBtRecord = new TransportRecord(TransportType.BLUETOOTH, "12:34:56:78:90");
     final ComponentName routerServiceComponentName = new ComponentName("com.smartdevicelink.test", "com.smartdevicelink.test.SdlRouterService");
@@ -67,6 +68,10 @@ public boolean onLegacyModeEnabled(String info) {
 
     @Before
     public void setUp() throws Exception {
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+            btRuntimePermissionRule =
+                    GrantPermissionRule.grant(Manifest.permission.BLUETOOTH_CONNECT);
+        }
         config = new MultiplexTransportConfig(getInstrumentation().getContext(), SdlUnitTestContants.TEST_APP_ID);
         config.setService(routerServiceComponentName);
         if (Looper.myLooper() == null) {

diff --git a/.../sdl_android/src/androidTest/java/com/smartdevicelink/util/MediaStreamingStatusTests.java b/.../sdl_android/src/androidTest/java/com/smartdevicelink/util/MediaStreamingStatusTests.java
@@ -37,7 +37,7 @@
 @RunWith(AndroidJUnit4.class)
 public class MediaStreamingStatusTests {
     @Rule
-    public GrantPermissionRule btRuntimePermissionRule = GrantPermissionRule.grant(Manifest.permission.BLUETOOTH_CONNECT);
+    public GrantPermissionRule btRuntimePermissionRule;
 
     @Mock
     private AudioManager audioManager = mock(AudioManager.class);
@@ -63,6 +63,9 @@ public Object answer(InvocationOnMock invocation) {
 
     @Before
     public void setUp() throws Exception {
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
+            btRuntimePermissionRule = GrantPermissionRule.grant(Manifest.permission.BLUETOOTH_CONNECT);
+        }
         mockedContext = mock(Context.class);
         doAnswer(onGetSystemService).when(mockedContext).getSystemService(Context.AUDIO_SERVICE);
         defaultMediaStreamingStatus = new MediaStreamingStatus(mockedContext, mock(MediaStreamingStatus.Callback.class));