flake8
passed.
JSHint
passed.
Review Request #10613 — Created July 1, 2019 and submitted
SVN diffs can contain entries for binary files that also introduce
property changes. We weren't handling these properly, skipping any
parsed files that contained property changes (with the assumption being
that they were standalone entries), but we don't want to skip them for
binary files.This simply updates our logic to only skip for non-binary (standalone
proeprty) changes.
Unit tests pass. They failed without the fix.
Manually tested uploading sample diffs that were missing entries due to
the property change. Verified that all files were present and the diff was
correct after upload.