|
From: | Ben Woodcroft |
Subject: | Re: [PATCH] import: Move beautify description from cran to utils. (was Re: [PATCH 6/6] gnu: Add emacs-slime.) |
Date: | Thu, 14 Jan 2016 22:15:33 +1000 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 |
On 14/01/16 21:30, Ricardo Wurmus wrote:
I just tested these with "make check" and also by ./pre-inst-env guile tests/cran.scm, and they seem to pass. Am I doing something incorrectly?Ben Woodcroft <address@hidden> writes:On 14/01/16 20:31, Ricardo Wurmus wrote:This breaks two of the cran importer tests: (test-equal "beautify-description: use double spacing" "This is a package. It is great. Trust me Mr. Hendrix." ((@@ (guix import cran) beautify-description) "This is a package. It is great. Trust me Mr. Hendrix.")) (test-equal "beautify-description: transform fragment into sentence" "This package provides a function to establish world peace" ((@@ (guix import cran) beautify-description) "A function to establish world peace"))
However, ironically tests/gem fails.
Happy to provide a patch fixing the gem test, and moving the (failing?) cran tests to this new file since even if they pass that is a more appropriate place for them. It'll be the weekend before I have time for this though, apologies for leaving master broken.How about moving these tests elsewhere? (Or should they be removed?)Ah, sorry. Should they be moved to a new file tests/import-utils? Why do you suggest that they should be removed?I don’t have any preference for a name. “tests/import-utils” sounds good to me. If we can keep (and later enhance) the tests it would obviously be better. (I just meant that I wouldn’t be too sad if they were removed for whatever reasons.) Would you like to submit a patch to add tests/import-utils (including adding it to SCM_TESTS in Makefile.am) and move these two test cases there?
ben
[Prev in Thread] | Current Thread | [Next in Thread] |