google / data-transfer-project

I have a commit to fix this - just wanted a bug to explain it. This came up with my tests in a draft branch started failing (but this bug itself is in master branch).

Here's the buggy line of code (its referencing photos in its ternary instead of videos):

data-transfer-project/portability-types-common/src/main/java/org/datatransferproject/types/common/models/media/MediaContainerResource.java

Line 34 in 94f427a

this.videos = photos == null ? ImmutableList.of() : videos;

here's the stacktrace of how this manifests in my branch's new unit test org.datatransferproject.transfer.microsoft.media.MicrosoftMediaImporterTest #testImportItemAllSuccess:

java.lang.NullPointerException
	at org.datatransferproject.types.common.models.media.MediaContainerResource.transmogrifyTitles(MediaContainerResource.java:90)
	at org.datatransferproject.types.common.models.media.MediaContainerResource.transmogrify(MediaContainerResource.java:70)
	at org.datatransferproject.transfer.microsoft.media.MicrosoftMediaImporter.importItem(MicrosoftMediaImporter.java:103)
	at org.datatransferproject.transfer.microsoft.media.MicrosoftMediaImporterTest.testImportItemAllSuccess(MicrosoftMediaImporterTest.java:229)

MediaContainerResource uses wrong variable in null check