* [GSOC][PATCH v2 0/1] t4121: modernize test style
2023-02-21 20:21 [GSOC][PATCH] " Junio C Hamano
@ 2023-02-21 21:27 ` Vivan Garg
0 siblings, 0 replies; 4+ messages in thread
From: Vivan Garg @ 2023-02-21 21:27 UTC (permalink / raw)
To: git, vdye; +Cc: christian.couder, hariom18599, Vivan Garg
Edited the commit message to add what the commit does and added a period at the end of the sentence.
Added a space after the closing bracket in the second test.
Vivan Garg (1):
t4121: modernize test style
t/t4121-apply-diffs.sh | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
Range-diff against v1:
1: 61f1ed51f4 ! 1: 77fded7759 t4121: modernize test style
@@ Commit message
Test scripts in file t4121-apply-diffs.sh are written in old style,
where the test_expect_success command and test title are written on
- separate lines
+ separate lines, therefore update the tests to adhere to the new
+ style.
Signed-off-by: Vivan Garg <gvivan6@gmail.com>
@@ t/t4121-apply-diffs.sh: test_expect_success 'setup' \
- 'check if contextually independent diffs for the same file apply' \
- '( git diff test~2 test~1 && git diff test~1 test~0 )| git apply'
+test_expect_success 'check if contextually independent diffs for the same file apply' '
-+ ( git diff test~2 test~1 && git diff test~1 test~0 )| git apply
++ ( git diff test~2 test~1 && git diff test~1 test~0 ) | git apply
+'
test_done
--
2.37.0 (Apple Git-136)
^ permalink raw reply [flat|nested] 4+ messages in thread
* [GSOC][PATCH v2 0/1] t4121: modernize test style
[not found] <<20230220235121.34375-1-gvivan6@gmail.com>
@ 2023-02-21 21:35 ` Vivan Garg
2023-02-21 21:35 ` [GSOC][PATCH v2 1/1] " Vivan Garg
0 siblings, 1 reply; 4+ messages in thread
From: Vivan Garg @ 2023-02-21 21:35 UTC (permalink / raw)
To: git, vdye; +Cc: christian.couder, hariom18599, Vivan Garg
Edited the commit message to add what the commit does and added a period at the end of the sentence.
Added a space after the closing bracket in the second test.
Vivan Garg (1):
t4121: modernize test style
t/t4121-apply-diffs.sh | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
Range-diff against v1:
1: 61f1ed51f4 ! 1: 77fded7759 t4121: modernize test style
@@ Commit message
Test scripts in file t4121-apply-diffs.sh are written in old style,
where the test_expect_success command and test title are written on
- separate lines
+ separate lines, therefore update the tests to adhere to the new
+ style.
Signed-off-by: Vivan Garg <gvivan6@gmail.com>
@@ t/t4121-apply-diffs.sh: test_expect_success 'setup' \
- 'check if contextually independent diffs for the same file apply' \
- '( git diff test~2 test~1 && git diff test~1 test~0 )| git apply'
+test_expect_success 'check if contextually independent diffs for the same file apply' '
-+ ( git diff test~2 test~1 && git diff test~1 test~0 )| git apply
++ ( git diff test~2 test~1 && git diff test~1 test~0 ) | git apply
+'
test_done
--
2.37.0 (Apple Git-136)
^ permalink raw reply [flat|nested] 4+ messages in thread
* [GSOC][PATCH v2 1/1] t4121: modernize test style
2023-02-21 21:35 ` [GSOC][PATCH v2 0/1] t4121: modernize test style Vivan Garg
@ 2023-02-21 21:35 ` Vivan Garg
0 siblings, 0 replies; 4+ messages in thread
From: Vivan Garg @ 2023-02-21 21:35 UTC (permalink / raw)
To: git, vdye; +Cc: christian.couder, hariom18599, Vivan Garg
Test scripts in file t4121-apply-diffs.sh are written in old style,
where the test_expect_success command and test title are written on
separate lines, therefore update the tests to adhere to the new
style.
Signed-off-by: Vivan Garg <gvivan6@gmail.com>
---
t/t4121-apply-diffs.sh | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/t/t4121-apply-diffs.sh b/t/t4121-apply-diffs.sh
index a80cec9d11..f1cc42ff71 100755
--- a/t/t4121-apply-diffs.sh
+++ b/t/t4121-apply-diffs.sh
@@ -16,8 +16,8 @@ echo '1
7
8' >file
-test_expect_success 'setup' \
- 'git add file &&
+test_expect_success 'setup' '
+ git add file &&
git commit -q -m 1 &&
git checkout -b test &&
mv file file.tmp &&
@@ -27,10 +27,11 @@ test_expect_success 'setup' \
git commit -a -q -m 2 &&
echo 9 >>file &&
git commit -a -q -m 3 &&
- git checkout main'
+ git checkout main
+'
-test_expect_success \
- 'check if contextually independent diffs for the same file apply' \
- '( git diff test~2 test~1 && git diff test~1 test~0 )| git apply'
+test_expect_success 'check if contextually independent diffs for the same file apply' '
+ ( git diff test~2 test~1 && git diff test~1 test~0 ) | git apply
+'
test_done
--
2.37.0 (Apple Git-136)
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [GSOC][PATCH v2 0/1] t4121: modernize test style
2023-02-20 23:51 [GSOC][PATCH] " Vivan Garg
@ 2023-02-21 21:46 ` Vivan Garg
0 siblings, 0 replies; 4+ messages in thread
From: Vivan Garg @ 2023-02-21 21:46 UTC (permalink / raw)
To: git, vdye; +Cc: christian.couder, hariom18599, Vivan Garg
Edited the commit message to add what the commit does and added a period at the end of the sentence.
Added a space after the closing bracket in the second test.
Vivan Garg (1):
t4121: modernize test style
t/t4121-apply-diffs.sh | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
Range-diff against v1:
1: 61f1ed51f4 ! 1: 77fded7759 t4121: modernize test style
@@ Commit message
Test scripts in file t4121-apply-diffs.sh are written in old style,
where the test_expect_success command and test title are written on
- separate lines
+ separate lines, therefore update the tests to adhere to the new
+ style.
Signed-off-by: Vivan Garg <gvivan6@gmail.com>
@@ t/t4121-apply-diffs.sh: test_expect_success 'setup' \
- 'check if contextually independent diffs for the same file apply' \
- '( git diff test~2 test~1 && git diff test~1 test~0 )| git apply'
+test_expect_success 'check if contextually independent diffs for the same file apply' '
-+ ( git diff test~2 test~1 && git diff test~1 test~0 )| git apply
++ ( git diff test~2 test~1 && git diff test~1 test~0 ) | git apply
+'
test_done
--
2.37.0 (Apple Git-136)
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-02-21 21:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <<20230220235121.34375-1-gvivan6@gmail.com>
2023-02-21 21:35 ` [GSOC][PATCH v2 0/1] t4121: modernize test style Vivan Garg
2023-02-21 21:35 ` [GSOC][PATCH v2 1/1] " Vivan Garg
2023-02-21 20:21 [GSOC][PATCH] " Junio C Hamano
2023-02-21 21:27 ` [GSOC][PATCH v2 0/1] " Vivan Garg
-- strict thread matches above, loose matches on Subject: below --
2023-02-20 23:51 [GSOC][PATCH] " Vivan Garg
2023-02-21 21:46 ` [GSOC][PATCH v2 0/1] " Vivan Garg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).