# [PATCH] test: rev-parse-upstream: add missing cmp 2 messages from 2023-05-09 to 2023-05-10. Participants: Felipe Contreras, Junio C Hamano. Thread: https://gitlist.dev/t/59716 ## Felipe Contreras, 2023-05-09 00:02 Subject: [PATCH] test: rev-parse-upstream: add missing cmp Message-ID: <20230509000246.1760327-1-felipe.contreras@gmail.com> URL: https://gitlist.dev/e/20230509000246.1760327-1-felipe.contreras%40gmail.com ``` It seems pretty clear 5236fce6b4 (t1507: stop losing return codes of git commands, 2019-12-20) missed a test_cmp. Cc: Denton Liu Signed-off-by: Felipe Contreras --- t/t1507-rev-parse-upstream.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/t/t1507-rev-parse-upstream.sh b/t/t1507-rev-parse-upstream.sh index d94c72c672..cb9ef7e329 100755 --- a/t/t1507-rev-parse-upstream.sh +++ b/t/t1507-rev-parse-upstream.sh @@ -97,7 +97,8 @@ test_expect_success 'my-side@{u} resolves to correct commit' ' commit_subject my-side >actual && test_cmp expect actual && echo 5 >expect && - commit_subject my-side@{u} >actual + commit_subject my-side@{u} >actual && + test_cmp expect actual ' test_expect_success 'not-tracking@{u} fails' ' -- 2.40.0+fc1 ``` ## Junio C Hamano, 2023-05-10 03:56 Subject: Re: [PATCH] test: rev-parse-upstream: add missing cmp Message-ID: URL: https://gitlist.dev/e/xmqqpm78zvo0.fsf%40gitster.g In-Reply-To: <20230509000246.1760327-1-felipe.contreras@gmail.com> ``` Felipe Contreras writes: > It seems pretty clear 5236fce6b4 (t1507: stop losing return codes of git > commands, 2019-12-20) missed a test_cmp. > > Cc: Denton Liu > Signed-off-by: Felipe Contreras > --- > t/t1507-rev-parse-upstream.sh | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/t/t1507-rev-parse-upstream.sh b/t/t1507-rev-parse-upstream.sh > index d94c72c672..cb9ef7e329 100755 > --- a/t/t1507-rev-parse-upstream.sh > +++ b/t/t1507-rev-parse-upstream.sh > @@ -97,7 +97,8 @@ test_expect_success 'my-side@{u} resolves to correct commit' ' > commit_subject my-side >actual && > test_cmp expect actual && > echo 5 >expect && > - commit_subject my-side@{u} >actual > + commit_subject my-side@{u} >actual && > + test_cmp expect actual > ' > > test_expect_success 'not-tracking@{u} fails' ' Good eyes. a93cbe8d (t1507: assert output of rev-parse, 2023-03-18) and wanted to do the same to this script, and the series that contains the commit wanted to do the same to many other test scripts, but it seems to have missed this one. Will queue (on top of a93cbe8d, naturally). Thanks. ```