react-native-queue-player 1.1.2 → 2.0.0

package/ios/Tests/StallRecoveryTests.swift

@@ -0,0 +1,97 @@
+import AVFoundation
+import XCTest
+@testable import QueuePlayer
+
+/// Playback runs with `automaticallyWaitsToMinimizeStalling = false`, so
+/// AVPlayer does not resume itself after a buffer underrun. A stall arms a
+/// recovery that re-issues play when the buffer refills, and transport intent
+/// decides whether it arms at all.
+///
+/// The resume itself needs a real item whose buffer actually refills, so that
+/// half is covered by the streaming demo suites; what is covered here is the
+/// arming and clearing, which is where the intent bugs would be.
+final class StallRecoveryTests: XCTestCase {
+
+  func testAStallArmsRecoveryWhilePlaybackIsIntended() {
+    let player = TrackPlayer()
+    let active = seedActiveItem(player)
+    player.wantsToPlay = true
+
+    player.handlePlaybackStalled(
+      Notification(name: AVPlayerItem.playbackStalledNotification, object: active))
+    drainPlayerQueue(player)
+
+    XCTAssertTrue(player.isRecoveringFromStall)
+  }
+
+  /// The observer sees every item in the process; a stall on an item that
+  /// is not the one being heard is not this player's stall.
+  func testAStallOnAnotherItemDoesNotArmRecovery() {
+    let player = TrackPlayer()
+    _ = seedActiveItem(player)
+    player.wantsToPlay = true
+    let other = AVPlayerItem(url: URL(fileURLWithPath: "/dev/null/other.mp3"))
+
+    player.handlePlaybackStalled(
+      Notification(name: AVPlayerItem.playbackStalledNotification, object: other))
+    drainPlayerQueue(player)
+
+    XCTAssertFalse(player.isRecoveringFromStall)
+  }
+
+  /// A notification with no item names nothing this player is playing — also
+  /// while the player itself has no active item, where "no item" and "no
+  /// active item" must not be read as a match.
+  func testAStallWithNoItemDoesNotArmRecovery() {
+    let player = TrackPlayer()
+    player.wantsToPlay = true
+
+    player.handlePlaybackStalled(Notification(name: AVPlayerItem.playbackStalledNotification))
+    drainPlayerQueue(player)
+
+    XCTAssertFalse(player.isRecoveringFromStall)
+  }
+
+  func testAStallWhilePausedDoesNotArmRecovery() {
+    let player = TrackPlayer()
+    let active = seedActiveItem(player)
+    player.wantsToPlay = false
+
+    player.handlePlaybackStalled(
+      Notification(name: AVPlayerItem.playbackStalledNotification, object: active))
+    drainPlayerQueue(player)
+
+    XCTAssertFalse(
+      player.isRecoveringFromStall,
+      "a stall arriving while paused must not arm a resume that fires later")
+  }
+
+  func testAnExplicitPauseDisarmsAPendingRecovery() async throws {
+    let player = TrackPlayer()
+    player.wantsToPlay = true
+    player.isRecoveringFromStall = true
+
+    _ = try await player.pause().await()
+
+    XCTAssertFalse(
+      player.isRecoveringFromStall,
+      "an explicit pause outranks a pending recovery, or the next refill "
+        + "restarts audio the user just stopped")
+  }
+
+  /// Installs an engine holding one item and returns that item.
+  private func seedActiveItem(_ player: TrackPlayer) -> AVPlayerItem {
+    let engine = ActiveItemEngineStub()
+    let item = AVPlayerItem(url: URL(fileURLWithPath: "/dev/null/active.mp3"))
+    engine.item = item
+    player.engine = engine
+    return item
+  }
+
+  /// The handler hops to the player queue, so the assertion has to follow it.
+  private func drainPlayerQueue(_ player: TrackPlayer) {
+    let drained = expectation(description: "player queue drained")
+    player.playerQueue.async { drained.fulfill() }
+    wait(for: [drained], timeout: 2.0)
+  }
+}

package/ios/Tests/StreamingBitrateProbeTests.swift

@@ -1,29 +1,26 @@
 import XCTest
 @testable import QueuePlayer
 
-/// Pins `StreamingBitrateProbe`'s network-isolation seam: a remote probe
-/// must fetch via the injected `URLSession`, never the global
-/// `URLSession.shared`. The bitrate-parsing math is exercised end-to-end
-/// through the demo format/replaygain suites.
+/// The probe derives a bitrate from a local file's first 256 KB and reads
+/// nothing that is not a `file://` URL.
 final class StreamingBitrateProbeTests: XCTestCase {
 
-  func testRemoteProbeUsesInjectedSession() async {
-    let config = URLSessionConfiguration.ephemeral
-    config.protocolClasses = [StubURLProtocol.self]
-    let session = URLSession(configuration: config)
-    StubURLProtocol.reset()
-    defer { StubURLProtocol.reset() }
-    // Unique URL per run so the probe's process-wide bitrate cache can't
-    // satisfy the call without a fetch. `.test` is a reserved
-    // non-resolvable TLD, so a real-network leak would fail with -1003
-    // instead of reaching the stub.
-    let url = URL(string: "https://example.test/probe-\(UUID().uuidString).m4a")!
-    StubURLProtocol.stub(url: url, data: Data(count: 4096), statusCode: 200)
+  func testALocalCbrMp3ReportsItsEncodedBitrate() async throws {
+    let url = try PlayerFixtures.fixtureURL(name: "silence-cbr-128", ext: "mp3", for: self)
+    let bitrate = await StreamingBitrateProbe.bitrate(forUrl: url)
+    XCTAssertEqual(try XCTUnwrap(bitrate), 128_000, accuracy: 128_000 * 0.05)
+  }
 
-    _ = await StreamingBitrateProbe.bitrate(forUrl: url, urlSession: session)
+  func testARemoteUrlIsNotProbed() async throws {
+    let url = try XCTUnwrap(URL(string: "http://127.0.0.1:1/\(UUID().uuidString).mp3"))
+    let bitrate = await StreamingBitrateProbe.bitrate(forUrl: url)
+    XCTAssertNil(bitrate)
+  }
 
-    XCTAssertEqual(
-      StubURLProtocol.requestCount(for: url), 1,
-      "remote probe must fetch through the injected session, never URLSession.shared")
+  func testAMissingFileReportsNoBitrate() async {
+    let url = FileManager.default.temporaryDirectory
+      .appendingPathComponent("absent-\(UUID().uuidString).mp3")
+    let bitrate = await StreamingBitrateProbe.bitrate(forUrl: url)
+    XCTAssertNil(bitrate)
   }
 }

package/ios/Tests/TopUpWindowGateTests.swift

@@ -0,0 +1,394 @@
+import AVFoundation
+import XCTest
+@testable import QueuePlayer
+@_spi(QueuePlayerTests) import QueuePlayer
+
+/// Extending the installed run waits for a crossfade rather than cutting it
+/// short, and reads the run's start from what is actually being heard.
+final class TopUpWindowGateTests: XCTestCase {
+
+  private var player: TrackPlayer!
+
+  override func setUp() async throws {
+    try await super.setUp()
+    player = TrackPlayer()
+  }
+
+  override func tearDown() async throws {
+    player = nil
+    try await super.tearDown()
+  }
+
+  /// The same run, the same owed top-up: only the pending fade differs.
+  func testAPendingFadeIsWhatHoldsTheTopUpBack() async throws {
+    let engine = WindowEngineStub(count: 3)
+    engine.fadePending = true
+    try await seed(engine: engine, trackCount: 20)
+
+    await onQueue { $0.topUpWindow() }
+    XCTAssertTrue(
+      engine.inserted.isEmpty,
+      "a pending fade must not be cut short to extend the run")
+
+    engine.fadePending = false
+    await onQueue { $0.topUpWindow() }
+    XCTAssertFalse(
+      engine.inserted.isEmpty,
+      "the same run extends once there is no fade to protect")
+  }
+
+  /// The run starts at the item being heard. An engine that retains played
+  /// items reports those first, and measuring from them would call the run
+  /// satisfied while the playhead sat near its end.
+  func testTheRunIsMeasuredFromTheItemBeingHeardNotTheFirstInstalled() async throws {
+    // Ten installed positions, but the playhead is at the last of them — so
+    // the run ahead is exhausted and a top-up is owed.
+    let engine = WindowEngineStub(count: 10)
+    engine.leadingIndex = 9
+    try await seed(engine: engine, trackCount: 30)
+
+    await onQueue { $0.topUpWindow() }
+
+    XCTAssertFalse(
+      engine.inserted.isEmpty,
+      "measured from the playhead the run is exhausted, so it must extend")
+  }
+
+  /// A skip is applied inside a mutation, and the transition it raises is not
+  /// allowed to top up while one is in flight — so the skip extends the run
+  /// itself when it lands on the last installed position.
+  func testASkipOntoTheLastInstalledPositionExtendsTheRun() async throws {
+    let engine = WindowEngineStub(count: 4)
+    try await seed(engine: engine, trackCount: 20)
+
+    await onQueue { $0.applyNewCurrentIndex(newIndex: 3, reason: .userSkipToIndex) }
+
+    XCTAssertFalse(
+      engine.inserted.isEmpty,
+      "the run ahead of the skip target is empty and must be extended")
+  }
+
+  /// The crossfade steady state: the whole queue installed, the playhead part
+  /// way into it. This is where the claim that the machinery is inert has to
+  /// hold, and a shorter-than-floor queue does not test it.
+  func testAWhollyInstalledQueueNeedsNoTopUpFromMidRun() async throws {
+    let engine = WindowEngineStub(count: 30)
+    engine.leadingIndex = 5
+    try await seed(engine: engine, trackCount: 30)
+
+    await onQueue { $0.topUpWindow() }
+
+    XCTAssertTrue(
+      engine.inserted.isEmpty,
+      "nothing is owed when the queue is already installed to its end")
+  }
+
+  /// The engine refusing the append falls back to installing the slice.
+  func testAnEngineThatRefusesTheAppendFallsBackToAFullInstall() async throws {
+    let engine = WindowEngineStub(count: 3)
+    engine.refuseInserts = true
+    try await seed(engine: engine, trackCount: 20)
+
+    await onQueue { $0.topUpWindow() }
+
+    XCTAssertTrue(engine.inserted.isEmpty)
+    XCTAssertTrue(
+      engine.didSetItems, "a refused append must be recovered by a full install")
+  }
+
+  func testAFullTailNeedsNoTopUp() async throws {
+    let engine = WindowEngineStub(count: 4)
+    try await seed(engine: engine, trackCount: 4)
+
+    await onQueue { $0.topUpWindow() }
+
+    XCTAssertTrue(
+      engine.inserted.isEmpty,
+      "the queue is wholly installed, so there is nothing to append")
+  }
+
+  /// A queue that has ended is not extended by a top-up, even while the
+  /// engine still holds a short run and the model has tracks past it: the
+  /// finished track would otherwise get a run appended behind it, and the
+  /// next play() would fade into it.
+  func testTopUpDoesNotExtendAnEndedQueue() async throws {
+    let engine = WindowEngineStub(count: 3)
+    engine.leadingIndex = 2
+    try await seed(engine: engine, trackCount: 20)
+    await onQueue { $0.reachedQueueEnd = true }
+
+    await onQueue { $0.topUpWindow() }
+
+    XCTAssertTrue(engine.inserted.isEmpty, "nothing is appended behind the finished track")
+    XCTAssertFalse(engine.didSetItems, "an ended queue must not be seated again by a top-up")
+  }
+
+  /// The same holds for a drained engine that has not ended: seating it is the
+  /// caller's job, not the top-up's.
+  func testTopUpDoesNotSeatADrainedEngine() async throws {
+    let engine = WindowEngineStub(count: 0)
+    try await seed(engine: engine, trackCount: 20)
+
+    await onQueue { $0.topUpWindow() }
+
+    XCTAssertFalse(engine.didSetItems)
+  }
+
+  /// An add that fits inside the window is inserted surgically, so the items
+  /// already buffering ahead of it are kept.
+  func testAnAddInsideTheWindowIsInsertedWithoutReinstalling() async throws {
+    let engine = WindowEngineStub(count: QueueWindowArithmetic.floor)
+    try await seed(engine: engine, trackCount: QueueWindowArithmetic.floor)
+
+    _ = try await player.addToQueue(
+      tracks: [
+        PlayerFixtures.track(id: "n0", url: "https://example.com/n0.m4a"),
+        PlayerFixtures.track(id: "n1", url: "https://example.com/n1.m4a"),
+      ],
+      insertBefore: 3
+    ).await()
+
+    XCTAssertEqual(engine.inserted.count, 2, "both added items are inserted")
+    XCTAssertFalse(engine.didSetItems, "nothing is reinstalled")
+    let ids = try await onQueueValue { $0.queueItemIds }
+    XCTAssertEqual(
+      engine.allMediaItems.compactMap(\.queueItemId), Array(ids.prefix(QueueWindowArithmetic.floor + 2)),
+      "the run is the queue's positions in order with the two added items in place")
+  }
+
+  /// An add that runs past the window is not built item by item: the engine
+  /// is reconciled to the slice, and `topUpWindow` materialises the rest as
+  /// the playhead advances.
+  func testAnAddPastTheWindowBuildsOnlyTheSlice() async throws {
+    let engine = WindowEngineStub(count: QueueWindowArithmetic.floor)
+    try await seed(engine: engine, trackCount: QueueWindowArithmetic.floor)
+    let album = (0 ..< 30).map {
+      PlayerFixtures.track(id: "n\($0)", url: "https://example.com/n\($0).m4a")
+    }
+
+    _ = try await player.addToQueue(tracks: album, insertBefore: 5).await()
+
+    XCTAssertEqual(
+      engine.inserted.count, QueueWindowArithmetic.floor - 5,
+      "only the positions inside the window are built")
+    XCTAssertFalse(engine.didSetItems, "the run is trimmed and extended, not reinstalled")
+    let ids = try await onQueueValue { $0.queueItemIds }
+    XCTAssertEqual(
+      engine.allMediaItems.compactMap(\.queueItemId), Array(ids.prefix(QueueWindowArithmetic.floor)),
+      "the engine holds exactly the window: the five kept positions and the first five added")
+  }
+
+  /// The same after the end of the queue on an engine that keeps its items:
+  /// the finished item is a materialised anchor, and an item inserted behind
+  /// it would be what the next play() fades into.
+  func testAnAppendAfterTheEndOfTheQueueLeavesARetainingEngineAlone() async throws {
+    let engine = WindowEngineStub(count: 3)
+    engine.leadingIndex = 2
+    try await seed(engine: engine, trackCount: 3)
+    await onQueue { $0.reachedQueueEnd = true }
+
+    _ = try await player.addToQueue(
+      tracks: [PlayerFixtures.track(id: "t3", url: "https://example.com/t3.m4a")],
+      insertBefore: nil
+    ).await()
+
+    XCTAssertTrue(engine.inserted.isEmpty, "nothing is inserted behind the finished item")
+    XCTAssertFalse(engine.didSetItems)
+    let state = try await onQueueValue { ($0.currentTrackIndex, $0.reachedQueueEnd) }
+    XCTAssertEqual(state.0, 2)
+    XCTAssertTrue(state.1)
+  }
+
+  /// A drain reported while the queue is already over does not re-decide it:
+  /// with tracks added behind the finished one, the verdict would otherwise be
+  /// `.windowDrained` and seat the added track.
+  func testARepeatedEndReportWhileEndedSeatsNothing() async throws {
+    let engine = WindowEngineStub(count: 0)
+    engine.leadingIndex = 2
+    try await seed(engine: engine, trackCount: 3)
+    await onQueue {
+      $0.reachedQueueEnd = true
+      _ = $0.queueState.insert(
+        [PlayerFixtures.track(id: "t3", url: "https://example.com/t3.m4a")], at: 3)
+    }
+
+    await onQueue { $0.applyEndVerdict() }
+
+    XCTAssertFalse(engine.didSetItems, "the ended queue is not re-seated")
+    let state = try await onQueueValue { ($0.currentTrackIndex, $0.reachedQueueEnd) }
+    XCTAssertEqual(state.0, 2, "the index stays on the track that finished")
+    XCTAssertTrue(state.1)
+  }
+
+  /// An append after the end of the queue extends the model, not the engine:
+  /// inserting into a drained engine would seat playback on the new track
+  /// while `currentTrackIndex` still names the one that finished, so the
+  /// next `play()` would start a track the consumer was never told about.
+  func testAnAppendAfterTheEndOfTheQueueLeavesTheDrainedEngineAlone() async throws {
+    let engine = WindowEngineStub(count: 0)
+    engine.leadingIndex = 2
+    try await seed(engine: engine, trackCount: 3)
+    await onQueue { $0.reachedQueueEnd = true }
+
+    _ = try await player.addToQueue(
+      tracks: [PlayerFixtures.track(id: "t3", url: "https://example.com/t3.m4a")],
+      insertBefore: nil
+    ).await()
+
+    XCTAssertTrue(engine.inserted.isEmpty, "nothing is inserted into a drained engine")
+    XCTAssertFalse(engine.didSetItems)
+    let state = try await onQueueValue {
+      ($0.tracks.count, $0.currentTrackIndex, $0.reachedQueueEnd)
+    }
+    XCTAssertEqual(state.0, 4, "the queue model carries the append")
+    XCTAssertEqual(state.1, 2, "the current index stays on the track that finished")
+    XCTAssertTrue(state.2, "the queue is still over until a skip or seek says otherwise")
+  }
+
+  /// Reinstalling the run the engine is already rendering from restarts the
+  /// track being heard unless its playhead is carried across.
+  func testAForcedReinstallKeepsThePlayheadOfTheTrackBeingHeard() async throws {
+    let engine = WindowEngineStub(count: 12)
+    engine.leadingIndex = 4
+    engine.position = 37.5
+    try await seed(engine: engine, trackCount: 20)
+
+    await onQueue { p in
+      p.performingMutation { p.fullRebuildPlayerQueue(forcingReinstall: true) }
+    }
+
+    XCTAssertTrue(engine.didSetItems, "the run is installed whole")
+    XCTAssertEqual(
+      engine.seatedAtPosition, 37.5,
+      "the track being heard resumes where it was, not from its start")
+  }
+
+  /// A reinstall that seats on a different track is a track change, and a
+  /// track change starts at the beginning.
+  func testAReinstallOntoADifferentTrackStartsItFromZero() async throws {
+    let engine = WindowEngineStub(count: 12)
+    engine.leadingIndex = 4
+    engine.position = 37.5
+    try await seed(engine: engine, trackCount: 20)
+    await onQueue { $0.currentTrackIndex = 9 }
+
+    await onQueue { p in
+      p.performingMutation { p.fullRebuildPlayerQueue(forcingReinstall: true) }
+    }
+
+    XCTAssertEqual(engine.seatedAtPosition, 0)
+  }
+
+  // MARK: - Helpers
+
+  private func seed(engine: WindowEngineStub, trackCount: Int) async throws {
+    let player = try XCTUnwrap(self.player)
+    await player.onPlayerQueue {
+      let ids = player.queueState.replaceAll(
+        (0 ..< trackCount).map {
+          PlayerFixtures.track(id: "t\($0)", url: "https://example.com/\($0).m4a")
+        })
+      engine.stampIds(Array(ids.prefix(engine.installedCount)))
+      player.currentTrackIndex = engine.leadingIndex
+      player.engine = engine
+    }
+  }
+
+  private func onQueue(_ body: @escaping (TrackPlayer) -> Void) async {
+    guard let player else { return }
+    await player.onPlayerQueue { body(player) }
+  }
+
+  private func onQueueValue<T>(_ body: @escaping (TrackPlayer) -> T) async throws -> T {
+    let player = try XCTUnwrap(self.player)
+    return await player.onPlayerQueueValue { body(player) }
+  }
+}
+
+/// Reports an installed run of `count` items and records what is appended to
+/// it. Every other member is inert — the window logic reads the run, the
+/// leading identity and the fade flag, nothing else.
+private final class WindowEngineStub: PlaybackEngine {
+  let installedCount: Int
+  var leadingIndex: Int = 0
+  var fadePending = false
+  /// `CrossfadeEngine.insertItems` refuses an anchor it does not hold; the
+  /// caller's recovery from that is a contract worth exercising.
+  var refuseInserts = false
+  /// The engine's own playhead, which a reinstall of the run it is already
+  /// rendering has to carry across.
+  var position: TimeInterval = 0
+  private(set) var inserted: [AVPlayerItem] = []
+  private(set) var didSetItems = false
+  private(set) var seatedAtPosition: TimeInterval?
+  private var items: [AVPlayerItem] = []
+
+  init(count: Int) {
+    self.installedCount = count
+  }
+
+  func stampIds(_ ids: [String]) {
+    items = ids.map { id in
+      let item = AVPlayerItem(url: URL(string: "https://example.com/\(id).m4a")!)
+      item.queueItemId = id
+      return item
+    }
+  }
+
+  var allMediaItems: [AVPlayerItem] { items }
+  var leadingQueueItemId: String? { items[safe: leadingIndex]?.queueItemId }
+  var isFadePendingOrActive: Bool { fadePending }
+  /// The whole installed run stands in for what is attached; this stub's
+  /// contract is the window, not the legs.
+  var attachedQueueItemIds: [String] { items.compactMap { $0.queueItemId } }
+
+  /// Inserts after the anchor, as the engines do, so the run's shape is
+  /// observable and not only the count of what was appended.
+  @discardableResult
+  func insertItems(_ newItems: [AVPlayerItem], after: AVPlayerItem?) -> Bool {
+    guard !refuseInserts else { return false }
+    let position: Int
+    if let after {
+      guard let index = items.firstIndex(where: { $0 === after }) else { return false }
+      position = index + 1
+    } else {
+      position = items.count
+    }
+    inserted.append(contentsOf: newItems)
+    items.insert(contentsOf: newItems, at: position)
+    return true
+  }
+
+  func setAudioMixProvider(_ provider: AudioMixProvider?) {}
+  func refreshAllItemMixes() {}
+  func release() {}
+  func setItems(_ items: [AVPlayerItem], startIndex: Int, startPositionSeconds: TimeInterval) {
+    didSetItems = true
+    seatedAtPosition = startPositionSeconds
+    self.items = items
+  }
+  func remove(_ item: AVPlayerItem) { items.removeAll { $0 === item } }
+  func removeAllItems() { items.removeAll() }
+  func canInsert(_ item: AVPlayerItem, after: AVPlayerItem?) -> Bool { true }
+  func play() {}
+  func pause() {}
+  func stop() {}
+  /// A seek moves the playhead: the run is measured from the new leading item.
+  func seek(toIndex index: Int, position: TimeInterval) { leadingIndex = index }
+  func skipToNext() {}
+  func skipToPrevious() {}
+  func setVolume(_ volume: Float) {}
+  func setPlaybackSpeed(_ speed: Float) {}
+  func seedPlaybackSpeed(_ speed: Float) {}
+  func setRepeatMode(_ mode: RepeatMode) {}
+  func setPitchCorrectionMode(_ mode: PitchCorrectionMode) {}
+
+  var currentMediaItemIndex: Int { leadingIndex }
+  var currentPositionSeconds: TimeInterval { position }
+  var currentDurationSeconds: TimeInterval { 0 }
+  var bufferedPositionSeconds: TimeInterval { 0 }
+  var isPlaying: Bool { false }
+  var timeControlStatus: AVPlayer.TimeControlStatus { .paused }
+  var currentMediaItem: AVPlayerItem? { items[safe: leadingIndex] }
+  var delegate: PlaybackEngineDelegate?
+}

package/ios/Tests/TrackPlayer+TestHops.swift

@@ -0,0 +1,14 @@
+@testable import QueuePlayer
+
+extension TrackPlayer {
+  /// Runs `body` on `playerQueue` and hands its value back — the read side of
+  /// the tests' player-state assertions. The library has no async getter that
+  /// needs this; its own hops are `onPlayerQueue` and `onPlayerQueueSync`.
+  func onPlayerQueueValue<T>(_ body: @escaping () -> T) async -> T {
+    await withCheckedContinuation { (cont: CheckedContinuation<T, Never>) in
+      playerQueue.async {
+        cont.resume(returning: body())
+      }
+    }
+  }
+}

package/ios/Tests/TrackPlayerCallOrderTests.swift

@@ -0,0 +1,93 @@
+import XCTest
+@testable import QueuePlayer
+
+/// Queue mutations reach the player in the order JS called them.
+///
+/// Each method is issued and a probe is enqueued immediately afterwards, the
+/// way a transport call issued on the next line would be. The probe runs on
+/// `playerQueue` behind whatever the method enqueued, so what it observes is
+/// what that next call would act on.
+final class TrackPlayerCallOrderTests: XCTestCase {
+
+  private var player: TrackPlayer!
+
+  override func setUp() async throws {
+    try await super.setUp()
+    player = TrackPlayer()
+    _ = try await player.configure(config: PlayerFixtures.config()).await()
+  }
+
+  override func tearDown() async throws {
+    if let player {
+      _ = try await player.destroy().await()
+    }
+    player = nil
+    try await super.tearDown()
+  }
+
+  func testSetQueueIsVisibleToWorkEnqueuedAfterIt() async throws {
+    let tracks = try PlayerFixtures.localTracks(count: 3, for: self)
+
+    _ = try player.setQueue(tracks: tracks, startAtIndex: 1)
+    let seen = try await probe { (count: $0.tracks.count, index: $0.currentTrackIndex) }
+
+    XCTAssertEqual(seen.count, 3)
+    XCTAssertEqual(seen.index, 1)
+  }
+
+  func testAddToQueueIsVisibleToWorkEnqueuedAfterIt() async throws {
+    _ = try await player.setQueue(tracks: PlayerFixtures.localTracks(count: 3, for: self), startAtIndex: 0).await()
+
+    _ = try player.addToQueue(tracks: PlayerFixtures.localTracks(count: 2, for: self), insertBefore: nil)
+    let count = try await probe { $0.tracks.count }
+
+    XCTAssertEqual(count, 5)
+  }
+
+  /// The immediate-next slot is the insert `AVQueuePlayer` prerolls, so it is
+  /// the one whose commit is most at risk of being deferred.
+  func testAnImmediateNextInsertIsVisibleToWorkEnqueuedAfterIt() async throws {
+    _ = try await player.setQueue(tracks: PlayerFixtures.localTracks(count: 3, for: self), startAtIndex: 0).await()
+
+    _ = try player.addToQueue(tracks: PlayerFixtures.localTracks(count: 1, for: self), insertBefore: 1)
+    let count = try await probe { $0.tracks.count }
+
+    XCTAssertEqual(count, 4)
+  }
+
+  func testRemoveFromQueueIsVisibleToWorkEnqueuedAfterIt() async throws {
+    _ = try await player.setQueue(tracks: PlayerFixtures.localTracks(count: 4, for: self), startAtIndex: 0).await()
+
+    _ = try player.removeFromQueue(indices: [2, 3])
+    let count = try await probe { $0.tracks.count }
+
+    XCTAssertEqual(count, 2)
+  }
+
+  func testMoveInQueueIsVisibleToWorkEnqueuedAfterIt() async throws {
+    _ = try await player.setQueue(tracks: PlayerFixtures.localTracks(count: 3, for: self), startAtIndex: 0).await()
+
+    _ = try player.moveInQueue(fromIndex: 2, toIndex: 1)
+    let idAtOne = try await probe { $0.tracks[1].id }
+
+    XCTAssertEqual(idAtOne, "t2")
+  }
+
+  /// A start index of NaN reaches `Int(_:)` unless it is guarded in `Double`
+  /// space, where the conversion is undefined behaviour.
+  func testANonFiniteStartIndexResolvesToTheQueueHead() async throws {
+    _ = try player.setQueue(tracks: PlayerFixtures.localTracks(count: 3, for: self), startAtIndex: Double.nan)
+    let index = try await probe { $0.currentTrackIndex }
+
+    XCTAssertEqual(index, 0)
+  }
+
+  // MARK: - Helpers
+
+  /// Enqueue a read behind whatever the call under test just enqueued.
+  private func probe<T>(_ body: @escaping (TrackPlayer) -> T) async throws -> T {
+    let player = try XCTUnwrap(self.player)
+    return await player.onPlayerQueueValue { body(player) }
+  }
+
+}