Skip to content

Refine SbPlayerGetMediaTimeTest - #4835

Merged
jasonzhangxx merged 1 commit into
youtube:25.lts.1+from
jasonzhangxx:get-meida-time-test
Mar 13, 2025
Merged

jasonzhangxx merged 1 commit into
youtube:25.lts.1+from
jasonzhangxx:get-meida-time-test

Conversation

@jasonzhangxx

Copy link
Copy Markdown
Contributor
  1. Write audio data from seek to time.
  2. Re-enable SbPlayerGetMediaTimeTest for android and win32.

b/347961880

@jasonzhangxx jasonzhangxx added the on_device Trigger on-device tests label Feb 5, 2025
@jasonzhangxx
jasonzhangxx marked this pull request as draft February 18, 2025 19:57
1. Write audio data from seek to time.
2. Re-enable SbPlayerGetMediaTimeTest for android and win32.

b/347961880
@jasonzhangxx
jasonzhangxx force-pushed the get-meida-time-test branch from 89fae11 to 4ce2b5b Compare March 6, 2025 22:25
@jasonzhangxx jasonzhangxx changed the title Fix SbPlayerGetMediaTimeTest with e/ac3 audio Refine SbPlayerGetMediaTimeTest Mar 10, 2025
@jasonzhangxx
jasonzhangxx marked this pull request as ready for review March 10, 2025 18:21
@jasonzhangxx

Copy link
Copy Markdown
Contributor Author

Though arm64_on_device_2 had a failure result, from its log, it seems that all tests were passed. @oxve

@oxve oxve left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These tests failed:

SbSocketAddressTypes/SbSocketGetInterfaceAddressTest.SunnyDayDestination/type_ipv6, where GetParam() = 1
SbSocketAddressTypes/SbSocketGetInterfaceAddressTest.SunnyDaySourceForDestination/type_ipv6, where GetParam() = 1
SbSocketAddressTypes/SbSocketGetInterfaceAddressTest.SunnyDaySourceNotLoopback/type_ipv6, where GetParam() = 1

Looks unrelated to the change

@jasonzhangxx
jasonzhangxx merged commit 9ffd7d7 into youtube:25.lts.1+ Mar 13, 2025
@jasonzhangxx
jasonzhangxx deleted the get-meida-time-test branch August 12, 2025 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

on_device Trigger on-device tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants