Subject: Re: bug#26889: Tests for emacs-clojure-mode

From: Alex Kost <[email-masked]>
Date: Sat, 13 May 2017 12:25:10 +0300

Arun Isaac (2017-05-13 10:38 +0530) wrote:

> Here is another patch set with the suggested changes incorporated.

I think Ludovic meant you may commit these patches.  One last comment
from me...

[...]
> From: Arun Isaac <[email-masked]>
> Date: Thu, 11 May 2017 23:28:53 +0530
> Subject: [PATCH 4/4] gnu: emacs-clojure-mode: Enable tests.
>
> * gnu/packages/emacs.scm (emacs-clojure-mode)[arguments]: Add 'check' phase.
> [native-inputs]: Add emacs-dash, emacs-s and ert-runner.
> ---
>  gnu/packages/emacs.scm | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index a9dd3db6c..4f5988b75 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -2876,6 +2876,16 @@ S-expression.")
>                 (base32
>                  "117mvjqh4nm8mvmwmmvy4qmkdg23ldlzk08y91g8b8ac8kxwqg81"))))
>      (build-system emacs-build-system)
> +    (native-inputs
> +     `(("emacs-dash" ,emacs-dash)
> +       ("emacs-s" ,emacs-s)
> +       ("ert-runner" ,ert-runner)))
> +    (arguments
> +     `(#:phases
> +       (modify-phases %standard-phases
> +         (add-after 'install 'check
> +           (lambda* (#:key inputs #:allow-other-keys)

'inputs' are not used, so just (lambda _ ...) here.

> +             (zero? (system* "ert-runner")))))))
>      (home-page "https://github.com/clojure-emacs/clojure-mode")
>      (synopsis "Major mode for Clojure code")
>      (description

-- 
Alex