emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 2/2] Add tests for reftex.


From: Rüdiger Sonderfeld
Subject: Re: [PATCH 2/2] Add tests for reftex.
Date: Wed, 12 Jun 2013 15:57:47 +0200
User-agent: KMail/4.10.3 (Linux/3.8.0-23-generic; KDE/4.10.3; x86_64; ; )

Hi,

On Wednesday 12 June 2013 14:51:23 Tassilo Horn wrote:
> F reftex-parse-from-file-test
>     Test `reftex-parse-from-file'.
>     (wrong-type-argument stringp nil)

sorry this was a problem caused by an uninitialised variable.  I guess that's 
why I didn't caught it.

Adding (reftex-ensure-compiled-variables) prior to the call to `reftex-parse-
from-file' fixes the issue.

--8<--------------------->8---
diff --git a/test/automated/reftex-tests.el b/test/automated/reftex-tests.el
index 5f5b008..de7f36d 100644
--- a/test/automated/reftex-tests.el
+++ b/test/automated/reftex-tests.el
@@ -115,6 +115,7 @@ (ert-deftest reftex-parse-from-file-test ()
     (with-temp-buffer
       (insert "test\n")
       (write-region (point-min) (point-max) bib-file))
+    (reftex-ensure-compiled-variables)
     (let ((parsed (reftex-parse-from-file tex-file nil temp-dir)))
       (should (equal (car parsed) `(eof ,tex-file)))
       (pop parsed)
--8<--------------------->8---

> I've never used ert.  Is there a way to tell it to also include the line
> number where it failed?

In ert you can get the backtrace by pressing `b'.

Regards
Rüdiger




reply via email to

[Prev in Thread] Current Thread [Next in Thread]