From ef6cb5d91315d3f5fffca2edf4f80b2e166b8ea1 Mon Sep 17 00:00:00 2001
From: ibaker <ibaker@google.com>
Date: Mon, 12 Aug 2024 01:41:44 -0700
Subject: [PATCH] Use `runCatching` instead of `try/catch` in
 `PlayerExtensionsTest`

PiperOrigin-RevId: 661992546
---
 .../androidx/media3/common/PlayerExtensionsTest.kt  | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/libraries/common_ktx/src/test/java/androidx/media3/common/PlayerExtensionsTest.kt b/libraries/common_ktx/src/test/java/androidx/media3/common/PlayerExtensionsTest.kt
index 63b4da0888..7e6013f618 100644
--- a/libraries/common_ktx/src/test/java/androidx/media3/common/PlayerExtensionsTest.kt
+++ b/libraries/common_ktx/src/test/java/androidx/media3/common/PlayerExtensionsTest.kt
@@ -113,15 +113,14 @@ class PlayerExtensionsTest {
   fun playerListen_onEventsThrowsException_bubblesOutAndUnregistersListener() = runTest {
     val player = PlayerWithListeners(TestExoPlayerBuilder(context).build())
     val exceptionFromListen = async {
-      try {
-        player.listen { events ->
-          if (Player.EVENT_VOLUME_CHANGED in events) {
-            throw IllegalStateException("Volume event!")
+      runCatching {
+          player.listen { events ->
+            if (Player.EVENT_VOLUME_CHANGED in events) {
+              throw IllegalStateException("Volume event!")
+            }
           }
         }
-      } catch (expected: IllegalStateException) {
-        expected
-      }
+        .exceptionOrNull()
     }
     // Wait for the Player.Listener to be registered inside player.listen
     testScheduler.runCurrent()