-
Notifications
You must be signed in to change notification settings - Fork 483
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
bug: missing test coverage for newer video logic in MediaContainerResource #1060
Comments
5 tasks
jzacsh
changed the title
add new test coverage for the forked code (the fork's purpose is to introduce video handling; my first step is just porting over the existing original test code, so this isn't fixed by my work yet)
add new test coverage for the video logic in MediaContainerResource
Apr 15, 2022
jzacsh
changed the title
add new test coverage for the video logic in MediaContainerResource
bug: missing test coverage for newer video logic in MediaContainerResource
Apr 15, 2022
jzacsh
added a commit
that referenced
this issue
Apr 18, 2022
backfills missing unit tests of MedaiContainerResourceTest and fixes a bug in the process This change forks test code (`{Photos,Media}ContainerResourceTest`) in line with MediaContainerResource's existing creation, then deletes some transmogrification stuff that we're not doing (per issue #1000) and adds a TODO(#1060) atop the new test to finish the tests more fully for video logic.
is this closed? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
see #1058 for context
task:
The text was updated successfully, but these errors were encountered: