tree 926b54910d6c67f9b87fb791f608a738453a344d
parent 716de601faaf7f6f015a2df3ca345abcfab8bb86
author Youssef Elghareeb <ghareeb@google.com> 1634023367 +0200
committer Youssef Elghareeb <ghareeb@google.com> 1634042117 +0200

ListOfFilesUnchangedPredicate: Ignore due to rebase file modifications

This predicate checks that the user modified the same set of files
between two patchsets by inspecting the {base vs. prior PS} diff and
checking if it has the exact same file paths and change types as the
{base vs. current PS} diff. Sometimes these two diffs are not exact and
the difference is in some file path that was modified due to rebase
(i.e. a modification between the two parent commits of both patchsets).
We are fixing this issue by checking that these difference files exist
in the {prior PS vs. current PS} diff. For example if {base vs. current}
contains a file that's not in {base vs. prior}, but {prior vs. current}
does not have this file, then this file was modified due to rebase and
the diff cache skipped this file in the {prior vs. current diff}, so we
can also ignore this file.

The added test fails before this fix and passes with it.

Bug: Google b/202528151
Change-Id: I50f3509febe774fb29ac44a04da5c73cf9d303fa