Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Fix Tizen test failures #6540

Merged
merged 1 commit into from
May 7, 2024

Conversation

joeyparrish
Copy link
Member

Fixes these failures on Tizen:

Player
rebufferGoal
✗ state orchestration and buffer length [Safari 3.0 (Tizen 3.0)]
Player Src Equals
✗ can control trick play rate [Safari 3.0 (Tizen 3.0)]

This was not in any releases.

Reverts "fix: reBufferingGoal is not respected (#6433)"

This reverts commit 99ed5db.

Reopens #6355

Fixes these failures on Tizen:

Player
    rebufferGoal
      ✗ state orchestration and buffer length [Safari 3.0 (Tizen 3.0)]
  Player Src Equals
    ✗ can control trick play rate [Safari 3.0 (Tizen 3.0)]

This was not in any releases.

Reverts "fix: reBufferingGoal is not respected (shaka-project#6433)"

This reverts commit 99ed5db.

Reopens shaka-project#6355
@joeyparrish joeyparrish changed the title tests: Fix Tizen test failures test: Fix Tizen test failures May 7, 2024
@shaka-bot
Copy link
Collaborator

Incremental code coverage: 100.00%

Copy link

@alekzz alekzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit; IMHO it's worth to put the info about reverting another CL in the title.

@joeyparrish
Copy link
Member Author

nit; IMHO it's worth to put the info about reverting another CL in the title.

I hear you, but the titles are used to automate the generation of change logs. A revert in the title is hard to follow in the change log. Since the main consequence of this was fixing test failures, I put that in the title and moved the revert details to the commit message.

@joeyparrish joeyparrish merged commit 475e604 into shaka-project:main May 7, 2024
15 of 21 checks passed
@joeyparrish joeyparrish deleted the fix-tizen-tests branch May 7, 2024 02:08
joeyparrish added a commit that referenced this pull request May 7, 2024
Fixes these failures on Tizen:

Player
    rebufferGoal
      ✗ state orchestration and buffer length [Safari 3.0 (Tizen 3.0)]
  Player Src Equals
    ✗ can control trick play rate [Safari 3.0 (Tizen 3.0)]

This was not in any releases.

Reverts "fix: reBufferingGoal is not respected (#6433)"

This reverts commit 99ed5db.

Reopens #6355
joeyparrish added a commit that referenced this pull request May 7, 2024
Fixes these failures on Tizen:

Player
    rebufferGoal
      ✗ state orchestration and buffer length [Safari 3.0 (Tizen 3.0)]
  Player Src Equals
    ✗ can control trick play rate [Safari 3.0 (Tizen 3.0)]

This was not in any releases.

Reverts "fix: reBufferingGoal is not respected (#6433)"

This reverts commit 99ed5db.

Reopens #6355
joeyparrish added a commit that referenced this pull request May 7, 2024
Fixes these failures on Tizen:

Player
    rebufferGoal
      ✗ state orchestration and buffer length [Safari 3.0 (Tizen 3.0)]
  Player Src Equals
    ✗ can control trick play rate [Safari 3.0 (Tizen 3.0)]

This was not in any releases.

Reverts "fix: reBufferingGoal is not respected (#6433)"

This reverts commit 99ed5db.

Reopens #6355
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Jul 6, 2024
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Jul 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants