Skip to content

Test/texttrack cue test - #899

Open
wvanhaevre wants to merge 2 commits into
developfrom
test/texttrack-cue-test
Open

Test/texttrack cue test#899
wvanhaevre wants to merge 2 commits into
developfrom
test/texttrack-cue-test

Conversation

@wvanhaevre

@wvanhaevre wvanhaevre commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread e2e/src/tests/index.ts
import Version from './Version.spec';

export default [Version, Basic, Ads, PresentationMode, ContentMatching, Connector];
export default [Version, Basic, Ads, PresentationMode, Connector, ContentMatching, TextTracks];

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔍 Existing spec order changed

Beyond appending TextTracks, ContentMatching and Connector are swapped in the export array. If specs share device/player state, this reorder can change outcomes.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

That is not the case. No issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant