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

[Rüdiger Sonderfeld]

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