diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2021-02-22 12:03:15 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-02-22 12:03:15 -0500 |
| commit | f8ff1727eca1ca726f00f5509fae5d9ca024f516 (patch) | |
| tree | c7c773232dfedb027a360a6e18acd257ac13476e /contrib/cirrus/pr-should-include-tests | |
| parent | cb3af5b05bc6c09c5fb905c8fb99c46e493a8b55 (diff) | |
| parent | d9655b0f014bd6b5ad6ebb2b9c11b409cbcb4a22 (diff) | |
| download | podman-f8ff1727eca1ca726f00f5509fae5d9ca024f516.tar.gz podman-f8ff1727eca1ca726f00f5509fae5d9ca024f516.tar.bz2 podman-f8ff1727eca1ca726f00f5509fae5d9ca024f516.zip | |
Merge pull request #9468 from edsantiago/new_test_checker
pr-should-include-tests: recognized "renamed" tests
Diffstat (limited to 'contrib/cirrus/pr-should-include-tests')
| -rwxr-xr-x | contrib/cirrus/pr-should-include-tests | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/contrib/cirrus/pr-should-include-tests b/contrib/cirrus/pr-should-include-tests index a3b4847a7..e3c5d5da1 100755 --- a/contrib/cirrus/pr-should-include-tests +++ b/contrib/cirrus/pr-should-include-tests @@ -22,7 +22,8 @@ base=$(git merge-base ${DEST_BRANCH:-master} $head) # A foo.c # M bar.c # We look for Added or Modified (not Deleted!) files under 'test'. -if git diff --name-status $base $head | egrep -q '^[AM]\s+(test/|.*_test\.go)'; then +# --no-renames ensures that renamed tests (#9420) show up as 'A'dded. +if git diff --name-status --no-renames $base $head | egrep -q '^[AM]\s+(test/|.*_test\.go)'; then exit 0 fi |