commit 8b7ac9701eec5aa340cdfcd29459b48a528f3e1f
author Soonil Nagarkar <sooniln@google.com> Thu Jun 04 16:15:06 2020 -0700
committer Soonil Nagarkar <sooniln@google.com> Mon Jun 08 19:56:12 2020 +0000
tree d79d1bcb9eb67c4d4f6eeff6c6a05a7b44f33006
parent 34a961ff4cee7d67b589434ab4c0a167253bb6cc

Disable GNSS tests on non-GNSS devices

Bug: 158136732
Bug: 158136616
Test: presubmits
Change-Id: I3ec3b325558a5ad6c958bd3db4ea01c9412841c3

tests/location/location_fine/src/android/location/cts/fine/LocationManagerFineTest.java

diff --git a/tests/location/location_fine/src/android/location/cts/fine/LocationManagerFineTest.java b/tests/location/location_fine/src/android/location/cts/fine/LocationManagerFineTest.java
index 9e0ed0b..e6e69df 100644
--- a/tests/location/location_fine/src/android/location/cts/fine/LocationManagerFineTest.java
+++ b/tests/location/location_fine/src/android/location/cts/fine/LocationManagerFineTest.java
@@ -32,6 +32,7 @@
 import static com.google.common.truth.Truth.assertThat;
 
 import static org.junit.Assert.fail;
+import static org.junit.Assume.assumeTrue;
 
 import android.app.PendingIntent;
 import android.content.BroadcastReceiver;
@@ -555,6 +556,8 @@
     @Test
     @AppModeFull(reason = "Instant apps can't hold ACCESS_LOCATION_EXTRA_COMMANDS permission")
     public void testRequestGpsUpdates_B9758659() throws Exception {
+        assumeTrue(hasGpsFeature());
+
         // test for b/9758659, where the gps provider may reuse network provider positions creating
         // an unnatural feedback loop
         assertThat(mManager.isProviderEnabled(GPS_PROVIDER)).isTrue();
@@ -1101,11 +1104,14 @@
 
     @Test
     public void testGetGnssYearOfHardware() {
+        assumeTrue(hasGpsFeature());
         mManager.getGnssYearOfHardware();
     }
 
     @Test
     public void testGetGnssHardwareModelName() {
+        assumeTrue(hasGpsFeature());
+
         // model name should be longer than 4 characters
         String gnssHardwareModelName = mManager.getGnssHardwareModelName();