Message ID | 878sgqi52k.fsf@gmail.com |
---|---|
State | Accepted |
Headers | show |
Series | [bug#41845] tests: lint: Add origin patch file name test cases. | expand |
Context | Check | Description |
---|---|---|
cbaines/comparison | success | View comparision |
cbaines/git branch | success | View Git branch |
cbaines/applying patch | fail | View Laminar job |
Hi Chris, Chris Marusich <cmmarusich@gmail.com> skribis: > From 5fa2156f8561afb87d7f9a63de0243334e9da53b Mon Sep 17 00:00:00 2001 > From: Chris Marusich <cmmarusich@gmail.com> > Date: Sat, 13 Jun 2020 22:09:46 -0700 > Subject: [PATCH] tests: lint: Add origin patch file name test cases. > > In particular, "<origin> patches: same file name -> no warnings" would > have caught the issue which was fixed in commit > 21887021b9acf60157b1b0a39c16f2ec6498021b. > > * tests/lint.scm (patches: file names): Rename this test case... > ("file patches: different file name -> warning"): ... to this. > ("file patches: same file name -> no warnings") > ("<origin> patches: different file name -> warning") > ("<origin> patches: same file name -> no warnings"): New test cases. LGTM. Thanks for taking the time to write these tests! Ludo’.
Hi Chris, Ludovic Courtès <ludo@gnu.org> skribis: > Chris Marusich <cmmarusich@gmail.com> skribis: > >> From 5fa2156f8561afb87d7f9a63de0243334e9da53b Mon Sep 17 00:00:00 2001 >> From: Chris Marusich <cmmarusich@gmail.com> >> Date: Sat, 13 Jun 2020 22:09:46 -0700 >> Subject: [PATCH] tests: lint: Add origin patch file name test cases. >> >> In particular, "<origin> patches: same file name -> no warnings" would >> have caught the issue which was fixed in commit >> 21887021b9acf60157b1b0a39c16f2ec6498021b. >> >> * tests/lint.scm (patches: file names): Rename this test case... >> ("file patches: different file name -> warning"): ... to this. >> ("file patches: same file name -> no warnings") >> ("<origin> patches: different file name -> warning") >> ("<origin> patches: same file name -> no warnings"): New test cases. > > LGTM. Thanks for taking the time to write these tests! I went ahead and pushed it as d8ae7852057d5c1818c9c8bb77e8c41407a0d985. Thanks, Ludo’.
From 5fa2156f8561afb87d7f9a63de0243334e9da53b Mon Sep 17 00:00:00 2001 From: Chris Marusich <cmmarusich@gmail.com> Date: Sat, 13 Jun 2020 22:09:46 -0700 Subject: [PATCH] tests: lint: Add origin patch file name test cases. In particular, "<origin> patches: same file name -> no warnings" would have caught the issue which was fixed in commit 21887021b9acf60157b1b0a39c16f2ec6498021b. * tests/lint.scm (patches: file names): Rename this test case... ("file patches: different file name -> warning"): ... to this. ("file patches: same file name -> no warnings") ("<origin> patches: different file name -> warning") ("<origin> patches: same file name -> no warnings"): New test cases. --- tests/lint.scm | 33 ++++++++++++++++++++++++++++++++- 1 file changed, 32 insertions(+), 1 deletion(-) diff --git a/tests/lint.scm b/tests/lint.scm index 4ce45b4a70..53647373e4 100644 --- a/tests/lint.scm +++ b/tests/lint.scm @@ -310,7 +310,7 @@ `(("python-setuptools" ,python-setuptools)))))) (check-inputs-should-not-be-an-input-at-all pkg)))) -(test-equal "patches: file names" +(test-equal "file patches: different file name -> warning" "file names of patches should start with the package name" (single-lint-warning-message (let ((pkg (dummy-package "x" @@ -319,6 +319,37 @@ (patches (list "/path/to/y.patch"))))))) (check-patch-file-names pkg)))) +(test-equal "file patches: same file name -> no warnings" + '() + (let ((pkg (dummy-package "x" + (source + (dummy-origin + (patches (list "/path/to/x.patch"))))))) + (check-patch-file-names pkg))) + +(test-equal "<origin> patches: different file name -> warning" + "file names of patches should start with the package name" + (single-lint-warning-message + (let ((pkg (dummy-package "x" + (source + (dummy-origin + (patches + (list + (dummy-origin + (file-name "y.patch"))))))))) + (check-patch-file-names pkg)))) + +(test-equal "<origin> patches: same file name -> no warnings" + '() + (let ((pkg (dummy-package "x" + (source + (dummy-origin + (patches + (list + (dummy-origin + (file-name "x.patch"))))))))) + (check-patch-file-names pkg))) + (test-equal "patches: file name too long" (string-append "x-" (make-string 100 #\a) -- 2.26.2