| Message ID | 20230509000246.1760327-1-felipe.contreras@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 159f4b9c3b9ccc0beb3a118ae752d61e9fffb681 |
| Headers | show |
| Series | test: rev-parse-upstream: add missing cmp | expand |
Felipe Contreras <felipe.contreras@gmail.com> writes: > It seems pretty clear 5236fce6b4 (t1507: stop losing return codes of git > commands, 2019-12-20) missed a test_cmp. > > Cc: Denton Liu <liu.denton@gmail.com> > Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> > --- > 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.
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' '
It seems pretty clear 5236fce6b4 (t1507: stop losing return codes of git commands, 2019-12-20) missed a test_cmp. Cc: Denton Liu <liu.denton@gmail.com> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> --- t/t1507-rev-parse-upstream.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)