* [PATCH 2/2] Add tests for reftex.
[not found] <752231878d20023a1d624e71ea4edc15bd10cf78.1370979894.git.ruediger@c-plusplus.de>
@ 2013-06-11 19:46 ` Rüdiger Sonderfeld
2013-06-12 12:51 ` Tassilo Horn
0 siblings, 1 reply; 4+ messages in thread
From: Rüdiger Sonderfeld @ 2013-06-11 19:46 UTC (permalink / raw)
To: auctex-devel; +Cc: emacs-devel
* test/automated/reftex-tests.el: New file.
Signed-off-by: Rüdiger Sonderfeld <ruediger@c-plusplus.de>
---
test/automated/reftex-tests.el | 200 +++++++++++++++++++++++++++++++++++++++++
1 file changed, 200 insertions(+)
create mode 100644 test/automated/reftex-tests.el
diff --git a/test/automated/reftex-tests.el b/test/automated/reftex-tests.el
new file mode 100644
index 0000000..5f5b008
--- /dev/null
+++ b/test/automated/reftex-tests.el
@@ -0,0 +1,200 @@
+;;; reftex-tests.el --- Test suite for reftex. -*- lexical-binding: t -*-
+
+;; Copyright (C) 2013 Free Software Foundation, Inc.
+
+;; Author: Rüdiger Sonderfeld <ruediger@c-plusplus.de>
+;; Keywords: internal
+;; Human-Keywords: internal
+
+;; This file is part of GNU Emacs.
+
+;; GNU Emacs is free software: you can redistribute it and/or modify
+;; it under the terms of the GNU General Public License as published by
+;; the Free Software Foundation, either version 3 of the License, or
+;; (at your option) any later version.
+
+;; GNU Emacs is distributed in the hope that it will be useful,
+;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+;; GNU General Public License for more details.
+
+;; You should have received a copy of the GNU General Public License
+;; along with GNU Emacs. If not, see <http://www.gnu.org/licenses/>.
+
+;;; Code:
+
+(require 'ert)
+
+;;; reftex
+(require 'reftex)
+
+;;; reftex-parse
+(require 'reftex-parse)
+
+(ert-deftest reftex-locate-bibliography-files ()
+ "Test `reftex-locate-bibliography-files'."
+ (let ((temp-dir (make-temp-file "reftex-bib" 'dir))
+ (files '("ref1.bib" "ref2.bib"))
+ (test '(("\\addbibresource{ref1.bib}\n" . ("ref1.bib"))
+ ("\\\\addbibresource[label=x]{ref2.bib}\\n" . ("ref2.bib"))
+ ("\\begin{document}\n\\bibliographystyle{plain}\n
+\\bibliography{ref1,ref2}\n\\end{document}" . ("ref1.bib" "ref2.bib"))))
+ (reftex-bibliography-commands
+ ;; Default value: See reftex-vars.el `reftex-bibliography-commands'
+ '("bibliography" "nobibliography" "setupbibtex\\[.*?database="
+ "addbibresource")))
+ (with-temp-buffer
+ (insert "test\n")
+ (mapc
+ (lambda (file)
+ (write-region (point-min) (point-max) (expand-file-name file
+ temp-dir)))
+ files))
+ (mapc
+ (lambda (data)
+ (with-temp-buffer
+ (insert (car data))
+ (let ((res (mapcar #'file-name-nondirectory
+ (reftex-locate-bibliography-files temp-dir))))
+ (should (equal res (cdr data))))))
+ test)
+ (delete-directory temp-dir 'recursive)))
+
+(ert-deftest reftex-what-environment-test ()
+ "Test `reftex-what-environment'."
+ (with-temp-buffer
+ (insert "\\begin{equation}\n x=y^2\n")
+ (let ((pt (point))
+ pt2)
+ (insert "\\end{equation}\n")
+ (goto-char pt)
+
+ (should (equal (reftex-what-environment 1) '("equation" . 1)))
+ (should (equal (reftex-what-environment t) '(("equation" . 1))))
+
+ (insert "\\begin{something}\nxxx")
+ (setq pt2 (point))
+ (insert "\\end{something}")
+ (goto-char pt2)
+ (should (equal (reftex-what-environment 1) `("something" . ,pt)))
+ (should (equal (reftex-what-environment t) `(("something" . ,pt)
+ ("equation" . 1))))
+ (should (equal (reftex-what-environment t pt) `(("something" . ,pt))))
+ (should (equal (reftex-what-environment '("equation"))
+ '("equation" . 1))))))
+
+(ert-deftest reftex-roman-number-test ()
+ "Test `reftex-roman-number'."
+ (let ((hindu-arabic '(1 2 4 9 14 1050))
+ (roman '("I" "II" "IV" "IX" "XIV" "ML")))
+ (while (and hindu-arabic roman)
+ (should (string= (reftex-roman-number (car hindu-arabic))
+ (car roman)))
+ (pop roman)
+ (pop hindu-arabic))))
+
+(ert-deftest reftex-parse-from-file-test ()
+ "Test `reftex-parse-from-file'."
+ (let* ((temp-dir (make-temp-file "reftex-parse" 'dir))
+ (tex-file (expand-file-name "test.tex" temp-dir))
+ (bib-file (expand-file-name "ref.bib" temp-dir)))
+ (with-temp-buffer
+ (insert
+"\\begin{document}
+\\section{test}\\label{sec:test}
+\\subsection{subtest}
+
+\\begin{align*}\\label{eq:foo}
+ x &= y^2
+\\end{align*}
+
+\\bibliographystyle{plain}
+\\bibliography{ref}
+\\end{document}")
+ (write-region (point-min) (point-max) tex-file))
+ (with-temp-buffer
+ (insert "test\n")
+ (write-region (point-min) (point-max) bib-file))
+ (let ((parsed (reftex-parse-from-file tex-file nil temp-dir)))
+ (should (equal (car parsed) `(eof ,tex-file)))
+ (pop parsed)
+ (while parsed
+ (let ((entry (pop parsed)))
+ (cond
+ ((eq (car entry) 'bib)
+ (should (string= (cadr entry) bib-file)))
+ ((eq (car entry) 'toc)) ;; ...
+ ((string= (car entry) "eq:foo"))
+ ((string= (car entry) "sec:test"))
+ ((eq (car entry) 'bof)
+ (should (string= (cadr entry) tex-file))
+ (should (null parsed)))
+ (t (should-not t)))))
+ (delete-directory temp-dir 'recursive))))
+
+;;; reftex-cite
+(require 'reftex-cite)
+
+(ert-deftest reftex-parse-bibtex-entry-test ()
+ "Test `reftex-parse-bibtex-entry'."
+ (let ((entry "@Book{Stallman12,
+ author = {Richard Stallman\net al.},
+ title = {The Emacs Editor},
+ publisher = {GNU Press},
+ year = 2012,
+ edition = {17th},
+ note = {Updated for Emacs Version 24.2}
+}")
+ (check (function
+ (lambda (parsed)
+ (should (string= (reftex-get-bib-field "&key" parsed)
+ "Stallman12"))
+ (should (string= (reftex-get-bib-field "&type" parsed)
+ "book"))
+ (should (string= (reftex-get-bib-field "author" parsed)
+ "Richard Stallman et al."))
+ (should (string= (reftex-get-bib-field "title" parsed)
+ "The Emacs Editor"))
+ (should (string= (reftex-get-bib-field "publisher" parsed)
+ "GNU Press"))
+ (should (string= (reftex-get-bib-field "year" parsed)
+ "2012"))
+ (should (string= (reftex-get-bib-field "edition" parsed)
+ "17th"))
+ (should (string= (reftex-get-bib-field "note" parsed)
+ "Updated for Emacs Version 24.2"))))))
+ (funcall check (reftex-parse-bibtex-entry entry))
+ (with-temp-buffer
+ (insert entry)
+ (funcall check (reftex-parse-bibtex-entry nil (point-min)
+ (point-max))))))
+
+(ert-deftest reftex-get-bib-names-test ()
+ "Test `reftex-get-bib-names'."
+ (let ((entry (reftex-parse-bibtex-entry "@article{Foo123,
+ author = {Jane Roe and\tJohn Doe and W. Public},
+}")))
+ (should (equal (reftex-get-bib-names "author" entry)
+ '("Jane Roe" "John Doe" "Public"))))
+ (let ((entry (reftex-parse-bibtex-entry "@article{Foo123,
+ editor = {Jane Roe and\tJohn Doe and W. Public},
+}")))
+ (should (equal (reftex-get-bib-names "author" entry)
+ '("Jane Roe" "John Doe" "Public")))))
+
+(ert-deftest reftex-format-citation-test ()
+ "Test `reftex-format-citation'."
+ (let ((entry (reftex-parse-bibtex-entry
+"@article{Foo13,
+ author = {Jane Roe and John Doe and Jane Q. Taxpayer},
+ title = {Some Article},
+ journal = {Some Journal},
+ year = 2013,
+ pages = {1--333}
+}")))
+ (should (string= (reftex-format-citation entry nil) "\\cite{Foo13}"))
+ (should (string= (reftex-format-citation entry "%l:%A:%y:%t %j %P %a")
+ "Foo13:Jane Roe:2013:Some Article Some Journal 1 Jane Roe, John Doe \\& Jane Taxpayer"))))
+
+(provide 'reftex-tests)
+;;; reftex-tests.el ends here.
--
1.8.3
_______________________________________________
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] Add tests for reftex.
2013-06-11 19:46 ` [PATCH 2/2] Add tests for reftex Rüdiger Sonderfeld
@ 2013-06-12 12:51 ` Tassilo Horn
2013-06-12 13:57 ` Rüdiger Sonderfeld
0 siblings, 1 reply; 4+ messages in thread
From: Tassilo Horn @ 2013-06-12 12:51 UTC (permalink / raw)
To: Rüdiger Sonderfeld; +Cc: auctex-devel, emacs-devel
Rüdiger Sonderfeld <ruediger@c-plusplus.de> writes:
> * test/automated/reftex-tests.el: New file.
Nice, applied, thanks.
But one test fails:
--8<---------------cut here---------------start------------->8---
Selector: t
Passed: 6
Failed: 1 (1 unexpected)
Total: 7/7
Started at: 2013-06-12 14:47:42+0200
Finished.
Finished at: 2013-06-12 14:47:43+0200
....F..
F reftex-parse-from-file-test
Test `reftex-parse-from-file'.
(wrong-type-argument stringp nil)
--8<---------------cut here---------------end--------------->8---
I've never used ert. Is there a way to tell it to also include the line
number where it failed?
Bye,
Tassilo
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] Add tests for reftex.
2013-06-12 12:51 ` Tassilo Horn
@ 2013-06-12 13:57 ` Rüdiger Sonderfeld
2013-06-12 14:47 ` Tassilo Horn
0 siblings, 1 reply; 4+ messages in thread
From: Rüdiger Sonderfeld @ 2013-06-12 13:57 UTC (permalink / raw)
To: Tassilo Horn; +Cc: auctex-devel, emacs-devel
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
_______________________________________________
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 2/2] Add tests for reftex.
2013-06-12 13:57 ` Rüdiger Sonderfeld
@ 2013-06-12 14:47 ` Tassilo Horn
0 siblings, 0 replies; 4+ messages in thread
From: Tassilo Horn @ 2013-06-12 14:47 UTC (permalink / raw)
To: Rüdiger Sonderfeld; +Cc: auctex-devel, emacs-devel
Rüdiger Sonderfeld <ruediger@c-plusplus.de> writes:
> 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.
Indeed, committed, thanks!
>> 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'.
Ah, good to know.
Bye,
Tassilo
_______________________________________________
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-06-12 14:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <752231878d20023a1d624e71ea4edc15bd10cf78.1370979894.git.ruediger@c-plusplus.de>
2013-06-11 19:46 ` [PATCH 2/2] Add tests for reftex Rüdiger Sonderfeld
2013-06-12 12:51 ` Tassilo Horn
2013-06-12 13:57 ` Rüdiger Sonderfeld
2013-06-12 14:47 ` Tassilo Horn
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).