[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Emacs-diffs] master 82a487d 03/10: Fix reading of regex-resources in re
From: |
Michal Nazarewicz |
Subject: |
[Emacs-diffs] master 82a487d 03/10: Fix reading of regex-resources in regex-tests |
Date: |
Tue, 2 Aug 2016 16:05:34 +0000 (UTC) |
branch: master
commit 82a487d522106c1c5afb00f2fa6c9055e37ba5ad
Author: Michal Nazarewicz <address@hidden>
Commit: Michal Nazarewicz <address@hidden>
Fix reading of regex-resources in regex-tests
* test/src/regex-tests.el (regex-tests-generic-line): Referring to
‘buffer-file-name’ does not work when running the test from command
line, i.e. via make, which results in (wrong-type-argument stringp nil)
failures. Replace it with hard-coded path.
(regex-tests-BOOST, regex-tests-PCRE, regex-tests-PTESTS-whitelist,
regex-tests-TESTS-whitelist): ‘regex-tests-generic-line’ now includes
the ‘regex-resources’ path component so the tests don’t need to specify
it explicitly.
---
test/src/regex-tests.el | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/test/src/regex-tests.el b/test/src/regex-tests.el
index 13a9f86..1407441 100644
--- a/test/src/regex-tests.el
+++ b/test/src/regex-tests.el
@@ -99,8 +99,7 @@ are known failures, and are skipped."
`(with-temp-buffer
(modify-syntax-entry ?_ "w;; ") ; tests expect _ to be a word
- (insert-file-contents ,(concat (file-name-directory (buffer-file-name))
test-file))
-
+ (insert-file-contents ,(concat "src/regex-resources/" test-file))
(let ((case-fold-search nil)
(line-number 1)
(whitelist-idx 0))
@@ -419,7 +418,7 @@ differences in behavior.")
(let (failures
basic icase newline notbol noteol)
(regex-tests-generic-line
- ?; "regex-resources/BOOST.tests" regex-tests-BOOST-whitelist
+ ?; "BOOST.tests" regex-tests-BOOST-whitelist
(if (save-excursion (re-search-forward "^-" nil t))
(setq basic (save-excursion (re-search-forward "REG_BASIC" nil t))
icase (save-excursion (re-search-forward "REG_ICASE" nil t))
@@ -496,7 +495,7 @@ differences in behavior.")
(let (failures
pattern icase string what-failed matches-observed)
(regex-tests-generic-line
- ?# "regex-resources/PCRE.tests" regex-tests-PCRE-whitelist
+ ?# "PCRE.tests" regex-tests-PCRE-whitelist
(cond
@@ -570,7 +569,7 @@ differences in behavior.")
(defun regex-tests-PTESTS ()
(let (failures)
(regex-tests-generic-line
- ?# "regex-resources/PTESTS" regex-tests-PTESTS-whitelist
+ ?# "PTESTS" regex-tests-PTESTS-whitelist
(let* ((fields (split-string (buffer-string) "¦"))
;; string has 1-based index of first char in the
@@ -632,7 +631,7 @@ differences in behavior.")
(defun regex-tests-TESTS ()
(let (failures)
(regex-tests-generic-line
- ?# "regex-resources/TESTS" regex-tests-TESTS-whitelist
+ ?# "TESTS" regex-tests-TESTS-whitelist
(if (save-excursion (re-search-forward
"^\\([^:]+\\):\\(.*\\):\\([^:]*\\)$" nil t))
(let* ((what-failed
(let ((raw (string-to-number (match-string 1))))
- [Emacs-diffs] master updated (5391761 -> 54a3c0c), Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master 82a487d 03/10: Fix reading of regex-resources in regex-tests,
Michal Nazarewicz <=
- [Emacs-diffs] master 9a418e0 07/10: Remove dead opcodes in regex bytecode, Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master da9c55d 09/10: Get rid of re_set_whitespace_regexp, Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master 7496844 04/10: Don’t (require 'cl), Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master 4538a5e 06/10: Refactor regex character class parsing in [:name:], Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master 91d5309 02/10: Added driver for the regex tests, Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master 04d96ec 08/10: Get rid of re_set_syntax, Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master e725706 05/10: Split regex glibc test cases into separet tests, Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master 54a3c0c 10/10: Hardcode regex syntax to remove dead code handling different syntax, Michal Nazarewicz, 2016/08/02
- [Emacs-diffs] master 6db7277 01/10: New regex tests imported from glibc 2.21, Michal Nazarewicz, 2016/08/02