Skip to content

Commit

Permalink
add grep prefix for grep tests to match other/later searchers (#187)
Browse files Browse the repository at this point in the history
  • Loading branch information
jacktasia authored Dec 15, 2017
1 parent d78d82c commit 3d69721
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion dumb-jump.el
Original file line number Diff line number Diff line change
Expand Up @@ -1247,7 +1247,7 @@ matches when passed input from stdin, which is a crucial feature."
(let ((thefile ".ag.test"))
(dumb-jump-run-test-temp-file test thefile (concat cmd " " thefile))))

(defun dumb-jump-test-rules (&optional run-not-tests)
(defun dumb-jump-test-grep-rules (&optional run-not-tests)
"Test all the grep rules and return count of those that fail.
Optionally pass t for RUN-NOT-TESTS to see a list of all failed rules."
(let ((fail-tmpl "grep FAILURE '%s' %s in response '%s' | CMD: '%s' | rule: '%s'")
Expand Down
12 changes: 6 additions & 6 deletions test/dumb-jump-test.el
Original file line number Diff line number Diff line change
Expand Up @@ -289,8 +289,8 @@
(should (string= (buffer-file-name) js-file))
(should (= (line-number-at-pos) 3)))))

(ert-deftest dumb-jump-test-rules-test ()
(let ((rule-failures (dumb-jump-test-rules)))
(ert-deftest dumb-jump-test-grep-rules-test ()
(let ((rule-failures (dumb-jump-test-grep-rules)))
(dumb-jump-output-rule-test-failures rule-failures)
(should (= (length rule-failures) 0))))

Expand All @@ -312,8 +312,8 @@
(dumb-jump-output-rule-test-failures rule-failures)
(should (= (length rule-failures) 0)))))

(ert-deftest dumb-jump-test-rules-not-test () ;; :not tests
(let ((rule-failures (dumb-jump-test-rules t)))
(ert-deftest dumb-jump-test-grep-rules-not-test () ;; :not tests
(let ((rule-failures (dumb-jump-test-grep-rules t)))
(dumb-jump-output-rule-test-failures rule-failures)
(should (= (length rule-failures) 0))))

Expand All @@ -329,10 +329,10 @@
(dumb-jump-output-rule-test-failures rule-failures)
(should (= (length rule-failures) 0)))))

(ert-deftest dumb-jump-test-rules-fail-test ()
(ert-deftest dumb-jump-test-grep-rules-fail-test ()
(let* ((bad-rule '(:type "variable" :supports ("ag" "grep" "rg" "git-grep") :language "elisp" :regex "\\\(defvarJJJ\\b\\s*" :tests ("(defvar test ")))
(dumb-jump-find-rules (cons bad-rule dumb-jump-find-rules))
(rule-failures (dumb-jump-test-rules)))
(rule-failures (dumb-jump-test-grep-rules)))
;(message "%s" (prin1-to-string rule-failures))
(should (= (length rule-failures) 1))))

Expand Down

0 comments on commit 3d69721

Please sign in to comment.