fix: adjust ad break exclusion logic

[Mansi-mParticle]

Instructions

1. PR target branch should be against main
2. PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
3. PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary

• pauseContentDuringAdBreaks renamed to excludeAdBreaksFromContentTime to accurately describe behavior.
  Updated boolean flag usage for clarity.
  Test cases modified to remove flaky timing assertions and ensure correctness.

Testing Plan

• Tested with sample app

Reference Issue

• Closes https://go.mparticle.com/work/REPLACEME

[thomson-t]

Can we update the changelog and Readme as well?

[rmi22186]

I think the following test makes sense to ensure user pausing and ad break ending is correctly ensuring contentTime is calculated and excluding ad breaks. The scenario is let's say you are watching a youtube video and mid ad, a user pauses, then the ad break ends (perhaps the "skip" button is shown and the user clicks it, creating an ad break end).

@Test
fun testUserPausesDuringAdBreak_DoesNotAutoResume() {
    val mparticle = MockMParticle()
    val mediaSession = MediaSession.builder(mparticle) {
        title = "hello"
        mediaContentId = "123"
        duration = 1000
        excludeAdBreaksFromContentTime = true
    }

    mediaSession.logPlay()
    Thread.sleep(500)
    
    mediaSession.logAdBreakStart { id = "break-1" }
    // User pauses during the ad
    mediaSession.logPause()
    Thread.sleep(500)
    mediaSession.logAdBreakEnd()
    
    val contentTimeAfterAdBreak = mediaSession.mediaContentTimeSpent
    
    // Wait to see if time is incorrectly accumulating
    Thread.sleep(500)
    
    val contentTimeLater = mediaSession.mediaContentTimeSpent
    
    // Content time should NOT have resumed after ad break
    // because user paused during the ad
    assertEquals(contentTimeAfterAdBreak, contentTimeLater, 0.2)
}

[rmi22186]

2 minor comments

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[mparticle-automation]

🎉 This PR is included in version 1.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀