futurile/guix-play
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tests: lint: Add origin patch file name test cases.

Browse Source 
In particular, "<origin> patches: same file name -> no warnings" would
have caught the issue which was fixed in commit
21887021b9.

* 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.

Signed-off-by: Ludovic Courtès <ludo@gnu.org>
This commit is contained in:
Chris Marusich 2020-06-13 22:09:46 -07:00 committed by Ludovic Courtès
parent aecd2a13cb
commit d8ae785205
No known key found for this signature in database
GPG Key ID: 090B11993D9AEBB5
1 changed files with 32 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
tests/lint.scm
View File

@ -315,7 +315,7 @@
`(("python-setuptools" ,python-setuptools)))))) `(("python-setuptools" ,python-setuptools))))))
(check-inputs-should-not-be-an-input-at-all pkg)))) (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" "file names of patches should start with the package name"
(single-lint-warning-message (single-lint-warning-message
(let ((pkg (dummy-package "x" (let ((pkg (dummy-package "x"
@ -324,6 +324,37 @@
(patches (list "/path/to/y.patch"))))))) (patches (list "/path/to/y.patch")))))))
(check-patch-file-names pkg)))) (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" (test-equal "patches: file name too long"
(string-append "x-" (string-append "x-"
(make-string 100 #\a) (make-string 100 #\a)