Enable lint errors in exoplayer tests

Follow-up to 76db936d68

PiperOrigin-RevId: 692925617

libraries/exoplayer/build.gradle

@@ -34,6 +34,12 @@ android {
         test.assets.srcDir '../test_data/src/test/assets/'
     }
 
+    lintOptions {
+        // TODO: b/353490583 - Move this to common_library_config.gradle once it's enabled for all
+        //  libraries.
+        checkTestSources true
+    }
+
     publishing {
         singleVariant('release') {
             withSourcesJar()

libraries/exoplayer/src/main/java/androidx/media3/exoplayer/audio/AudioSink.java

@@ -160,7 +160,7 @@ public interface AudioSink {
   /** Configuration parameters used for an {@link AudioTrack}. */
   final class AudioTrackConfig {
 
-    /* The {@link C.Encoding} of the audio data. */
+    /** The {@link C.Encoding} of the audio data. */
     public final @C.Encoding int encoding;
 
     /** The sample rate of the audio data. */

libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/spherical/ProjectionDecoder.java

@@ -51,7 +51,7 @@ import java.util.zip.Inflater;
 
   private ProjectionDecoder() {}
 
-  /*
+  /**
    * Decodes the projection data.
    *
    * @param projectionData The projection data.

libraries/exoplayer/src/test/java/androidx/media3/exoplayer/MediaExtractorCompatTest.java

@@ -792,7 +792,7 @@ public class MediaExtractorCompatTest {
     @Override
     public int read(ExtractorInput input, PositionHolder seekPosition) throws IOException {
       if (nextReadActionIndex >= readActions.size()) {
-        return C.RESULT_END_OF_INPUT;
+        return Extractor.RESULT_END_OF_INPUT;
       } else {
         return readActions.get(nextReadActionIndex++).apply(input, seekPosition);
       }

libraries/exoplayer/src/test/java/androidx/media3/exoplayer/audio/DefaultAudioSinkTest.java

@@ -346,6 +346,8 @@ public final class DefaultAudioSinkTest {
         new DefaultAudioSink.Builder()
             .setAudioCapabilities(new AudioCapabilities(new int[] {C.ENCODING_AAC_LC}, 2))
             .build();
+    // TODO: b/376880213 - Resolve this lint error.
+    @SuppressWarnings("WrongConstant")
     Format aacLcFormat =
         STEREO_44_1_FORMAT
             .buildUpon()

libraries/exoplayer/src/test/java/androidx/media3/exoplayer/mediacodec/AsynchronousMediaCodecCallbackTest.java

@@ -207,7 +207,7 @@ public class AsynchronousMediaCodecCallbackTest {
     MediaCodec.BufferInfo bufferInfo1 = new MediaCodec.BufferInfo();
     asynchronousMediaCodecCallback.onOutputBufferAvailable(codec, 0, bufferInfo1);
     MediaCodec.BufferInfo bufferInfo2 = new MediaCodec.BufferInfo();
-    bufferInfo2.set(1, 1, 1, 1);
+    bufferInfo2.set(1, 1, 1, MediaCodec.BUFFER_FLAG_KEY_FRAME);
     asynchronousMediaCodecCallback.onOutputBufferAvailable(codec, 1, bufferInfo2);
 
     MediaCodec.BufferInfo outBufferInfo = new MediaCodec.BufferInfo();

libraries/exoplayer/src/test/java/androidx/media3/exoplayer/source/SilenceMediaSourceTest.java

@@ -74,6 +74,7 @@ public class SilenceMediaSourceTest {
 
   @Test
   public void builderSetDurationUs_nonPositiveValue_throwsIllegalStateException() {
+    @SuppressWarnings("Range") // Deliberately testing an invalid value
     SilenceMediaSource.Factory factory = new SilenceMediaSource.Factory().setDurationUs(-1);
 
     assertThrows(IllegalStateException.class, factory::createMediaSource);

libraries/exoplayer/src/test/java/androidx/media3/exoplayer/trackselection/DefaultTrackSelectorTest.java

@@ -2777,6 +2777,7 @@ public final class DefaultTrackSelectorTest {
                     .build())
             .build());
     // Offload playback with gapless transitions is supported
+    @SuppressWarnings("WrongConstant") // Combining these two values bit-wise is allowed
     RendererCapabilities capabilitiesOffloadSupport =
         new FakeRendererCapabilities(
             C.TRACK_TYPE_AUDIO,

libraries/exoplayer/src/test/java/androidx/media3/exoplayer/upstream/DefaultBandwidthMeterTest.java

@@ -732,7 +732,8 @@ public final class DefaultBandwidthMeterTest {
     setActiveNetworkInfo(networkInfo, TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_NONE);
   }
 
-  @SuppressWarnings("StickyBroadcast")
+  // Adding the permission to the test AndroidManifest.xml doesn't work to appease lint.
+  @SuppressWarnings({"StickyBroadcast", "MissingPermission"})
   private void setActiveNetworkInfo(NetworkInfo networkInfo, int networkTypeOverride) {
     // Set network info in ConnectivityManager and TelephonyDisplayInfo in TelephonyManager.
     Shadows.shadowOf(connectivityManager).setActiveNetworkInfo(networkInfo);

libraries/exoplayer/src/test/java/androidx/media3/exoplayer/upstream/experimental/ExperimentalBandwidthMeterTest.java

@@ -742,7 +742,8 @@ public final class ExperimentalBandwidthMeterTest {
     setActiveNetworkInfo(networkInfo, TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_NONE);
   }
 
-  @SuppressWarnings("StickyBroadcast")
+  // Adding the permission to the test AndroidManifest.xml doesn't work to appease lint.
+  @SuppressWarnings({"StickyBroadcast", "MissingPermission"})
   private void setActiveNetworkInfo(NetworkInfo networkInfo, int networkTypeOverride) {
     // Set network info in ConnectivityManager and TelephonyDisplayInfo in TelephonyManager.
     Shadows.shadowOf(connectivityManager).setActiveNetworkInfo(networkInfo);