[go: nahoru, domu]

Fixed model and make values in AutomotiveCarInfo.java.

Fix: 201004136
Test: atest
Change-Id: Ia81fb6bc9e1f647f5318af881e4b4bb42ff587af
diff --git a/car/app/app-automotive/src/main/java/androidx/car/app/hardware/info/AutomotiveCarInfo.java b/car/app/app-automotive/src/main/java/androidx/car/app/hardware/info/AutomotiveCarInfo.java
index 3e2db72..7712336c 100644
--- a/car/app/app-automotive/src/main/java/androidx/car/app/hardware/info/AutomotiveCarInfo.java
+++ b/car/app/app-automotive/src/main/java/androidx/car/app/hardware/info/AutomotiveCarInfo.java
@@ -287,8 +287,8 @@
                         yearValue = getCarValue(value, (Integer) value.getValue());
                     }
                 }
-                Model model = new Model.Builder().setName(makeValue)
-                        .setManufacturer(modelValue)
+                Model model = new Model.Builder().setName(modelValue)
+                        .setManufacturer(makeValue)
                         .setYear(yearValue)
                         .build();
                 listener.onCarDataAvailable(model);
diff --git a/car/app/app-automotive/src/test/java/androidx/car/app/hardware/info/AutomotiveCarInfoTest.java b/car/app/app-automotive/src/test/java/androidx/car/app/hardware/info/AutomotiveCarInfoTest.java
index 9919320..56be908 100644
--- a/car/app/app-automotive/src/test/java/androidx/car/app/hardware/info/AutomotiveCarInfoTest.java
+++ b/car/app/app-automotive/src/test/java/androidx/car/app/hardware/info/AutomotiveCarInfoTest.java
@@ -87,7 +87,7 @@
 public class AutomotiveCarInfoTest {
     private List<CarPropertyResponse<?>> mResponse;
     private CountDownLatch mCountDownLatch;
-    private Executor mExecutor = directExecutor();
+    private final Executor mExecutor = directExecutor();
     private AutomotiveCarInfo mAutomotiveCarInfo;
     @Mock
     private Car mCarMock;
@@ -110,11 +110,9 @@
     public void getModel_verifyResponse() throws InterruptedException {
         // Add "make", "model", "year" values to the response.
         mResponse.add(CarPropertyResponse.create(INFO_MAKE,
-                STATUS_SUCCESS, 1, "Speedy "
-                        + "Model"));
+                STATUS_SUCCESS, 1, "Toy Vehicle"));
         mResponse.add(CarPropertyResponse.create(INFO_MODEL,
-                STATUS_SUCCESS, 2, "Toy "
-                        + "Vehicle"));
+                STATUS_SUCCESS, 2, "Speedy Model"));
         mResponse.add(CarPropertyResponse.create(INFO_MODEL_YEAR,
                 STATUS_SUCCESS, 3, 2020));
         ListenableFuture<List<CarPropertyResponse<?>>> listenableCarPropertyResponse =
@@ -133,8 +131,8 @@
         assertThat(mModel.getName().getValue()).isEqualTo("Speedy Model");
         assertThat(mModel.getManufacturer().getValue()).isEqualTo("Toy Vehicle");
         assertThat(mModel.getYear().getValue()).isEqualTo(2020);
-        assertThat(mModel.getName().getTimestampMillis()).isEqualTo(1);
-        assertThat(mModel.getManufacturer().getTimestampMillis()).isEqualTo(2);
+        assertThat(mModel.getManufacturer().getTimestampMillis()).isEqualTo(1);
+        assertThat(mModel.getName().getTimestampMillis()).isEqualTo(2);
         assertThat(mModel.getYear().getTimestampMillis()).isEqualTo(3);
     }