Repository navigation
Conversation
yell0wd0g
force-pushed
the
b_384819454__3__roll_more_starboard_unittests
branch
from
March 21, 2025 23:26
ae047d9 to
4f2b41b
Compare
yell0wd0g
force-pushed
the
b_384819454__3__roll_more_starboard_unittests
branch
from
April 14, 2025 16:08
92e72c3 to
c0521e3
Compare
…out ones and FFmpegAudioDecoderTest.*
…r_test.cc is not a unit test
…sabling audio_decoder_test
…ckage is in the 'non-free' section
yell0wd0g
force-pushed
the
b_384819454__3__roll_more_starboard_unittests
branch
from
April 14, 2025 18:11
0689146 to
bc452d7
Compare
yell0wd0g
marked this pull request as ready for review
April 14, 2025 21:47
Contributor
Author
|
@kjyoun PTAL |
kjyoun
approved these changes
Apr 14, 2025
yell0wd0g
enabled auto-merge (squash)
April 14, 2025 22:30
cobalt-github-releaser-bot
pushed a commit
that referenced
this pull request
Apr 15, 2025
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 (cherry picked from commit 82430e3)
yell0wd0g
added a commit
that referenced
this pull request
Apr 15, 2025
…ts (#5413) Refer to the original PR: #5127 This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 Co-authored-by: Miguel Casas <[email protected]>
kaidokert
pushed a commit
to kaidokert/cobalt
that referenced
this pull request
Jun 15, 2025
This PR continues the work of youtube#4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454
briantting
pushed a commit
that referenced
this pull request
Jul 1, 2025
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
briantting
pushed a commit
that referenced
this pull request
Jul 11, 2025
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
rakuco
added a commit
to rakuco/cobalt
that referenced
this pull request
Jul 23, 2025
The target being referenced does not exist and never has. Depending on `:starboard_platform_tests` does not work either, as youtube#5127 removed it.
sideb0ard
pushed a commit
to sideb0ard/cobalt
that referenced
this pull request
Jul 25, 2025
Original Message: starboard: roll up the remaining unittest targets (youtube#5127) This PR continues the work of youtube#4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3 Files with conflicts: - starboard/contrib/cast/cast_starboard_api/samples/BUILD.gn - starboard/linux/shared/player_components_factory.cc - starboard/loader_app/installation_manager_test.cc - starboard/loader_app/pending_restart_test.cc - starboard/shared/starboard/media/media_util_test.cc - starboard/shared/starboard/player/file_cache_reader.cc - starboard/shared/starboard/player/filter/testing/adaptive_audio_decoder_test.cc - starboard/shared/starboard/player/filter/testing/audio_decoder_test.cc - starboard/shared/starboard/player/filter/testing/player_components_test.cc
sideb0ard
pushed a commit
to sideb0ard/cobalt
that referenced
this pull request
Jul 26, 2025
This PR continues the work of youtube#4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
sideb0ard
pushed a commit
to sideb0ard/cobalt
that referenced
this pull request
Jul 30, 2025
This PR continues the work of youtube#4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
sideb0ard
pushed a commit
that referenced
this pull request
Aug 15, 2025
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
briantting
pushed a commit
that referenced
this pull request
Aug 15, 2025
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
sideb0ard
pushed a commit
that referenced
this pull request
Aug 20, 2025
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
briantting
pushed a commit
that referenced
this pull request
Aug 29, 2025
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
madhurajayaraman
pushed a commit
to madhurajayaraman/cobalt
that referenced
this pull request
Sep 9, 2025
This PR continues the work of youtube#4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
sideb0ard
pushed a commit
to sideb0ard/cobalt
that referenced
this pull request
Sep 10, 2025
This PR continues the work of youtube#4831 and rolls up all other targets I could find that looked like unit tests into `starboard_unittests`. With this I see ``` [==========] 1454 tests from 55 test suites ran. (8612 ms total) [ PASSED ] 989 tests. [ SKIPPED ] 465 tests, listed below: ... YOU HAVE 1 DISABLED TEST ``` (the disabled test was from before, the skipped ones are new, see further below). Media-related topics: - A number of tetst (e.g. `AudioDecoderTest` and others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new method `IsProprietaryAudioCodec()` to be used to SKIP test cases related to those codecs, when the build flag `SKIP_PROPRIETARY_CODEC_TESTS` is present. - `VideoDecoderTests` and `PlayerComponentsTest` failed flakily due to X11 use not being torn down correctly; they need `xvfb-run`. This makes me think those are not pure unit-tests but integration tests like `nplb`. For the time being they are commented out with a TODO. - `media_util_test.cc` needed `operator==` on `SbMediaColorMetadata`s, so I added it as extern. - `FileCacheReader` needs to find the media files under the output folder's `content/` directory. Files are under the output folder's `starboard/content`, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution. Finally: - `//starboard/raspi/shared/:starboard_platform_tests` target was empty, this CL removes it. Bug: b/384819454 original-hexsha: 82430e3
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR continues the work of #4831 and rolls up all other targets I could find that looked like unit tests into
starboard_unittests.With this I see
(the disabled test was from before, the skipped ones are new, see further below).
Media-related topics:
AudioDecoderTestand others) needed a number of system libraries to support codecs like AAC, AC3 etc but those are part of non-free Debian repos, so couldn't install those in Docker. This CL adds a new methodIsProprietaryAudioCodec()to be used to SKIP test cases related to those codecs, when the build flagSKIP_PROPRIETARY_CODEC_TESTSis present.VideoDecoderTestsandPlayerComponentsTestfailed flakily due to X11 use not being torn down correctly; they needxvfb-run. This makes me think those are not pure unit-tests but integration tests likenplb. For the time being they are commented out with a TODO.media_util_test.ccneededoperator==onSbMediaColorMetadatas, so I added it as extern.FileCacheReaderneeds to find the media files under the output folder'scontent/directory. Files are under the output folder'sstarboard/content, because of the starboard toolchain. This CL leaves a quick hack and a TODO() to find a proper solution.Finally:
//starboard/raspi/shared/:starboard_platform_teststarget was empty, this CL removes it.Bug: b/384819454