Add testResolveNonExistentServiceWithTargetuserThrows

Author: jdcormie

binder/src/main/java/io/grpc/binder/internal/ServiceBinding.java

@@ -252,10 +252,12 @@ void unbindInternal(Status reason) {
     }
   }
 
+  // Sadly we must call this system API reflectively since it isn't part of the Android SDK.
   private static int getIdentifier(UserHandle userHandle) throws ReflectiveOperationException {
-    return (int) userHandle.getClass().getDeclaredMethod("getIdentifier").invoke(userHandle);
+    return (int) userHandle.getClass().getMethod("getIdentifier").invoke(userHandle);
   }
 
+  // Sadly we must call this system API reflectively since it isn't part of the Android SDK.
   private static ResolveInfo resolveServiceAsUser(
       PackageManager packageManager, Intent intent, int flags, UserHandle targetUserHandle) {
     try {
@@ -279,7 +281,7 @@ public ServiceInfo resolve() throws StatusException {
             : packageManager.resolveService(bindIntent, 0);
     if (resolveInfo == null) {
       throw Status.UNIMPLEMENTED // Same status code as when bindService() returns false.
-          .withDescription("resolveService(" + bindIntent + ") returned null")
+          .withDescription("resolveService(" + bindIntent + " / " + targetUserHandle + ") was null")
           .asException();
     }
     return resolveInfo.serviceInfo;

binder/src/test/java/io/grpc/binder/internal/ServiceBindingTest.java

@@ -312,6 +312,19 @@ public void testResolveNonExistentServiceThrows() throws Exception {
     assertThat(statusException.getStatus().getDescription()).contains("does.not.exist");
   }
 
+  @Test
+  @Config(sdk = 33)
+  public void testResolveNonExistentServiceWithTargetUserThrows() throws Exception {
+    ComponentName doesNotExistService = new ComponentName("does.not.exist", "NoService");
+    binding = newBuilder()
+        .setTargetUserHandle(generateUserHandle(/* userId = */ 12345))
+        .setTargetComponent(doesNotExistService).build();
+    StatusException statusException = assertThrows(StatusException.class, binding::resolve);
+    assertThat(statusException.getStatus().getCode()).isEqualTo(Code.UNIMPLEMENTED);
+    assertThat(statusException.getStatus().getDescription()).contains("does.not.exist");
+    assertThat(statusException.getStatus().getDescription()).contains("12345");
+  }
+
   @Test
   @Config(sdk = 30)
   public void testBindWithDeviceAdmin() throws Exception {