master 986bf7a: Remove duplicated tests in checkdoc-tests.el

[Mauro Aranda]

branch: master
commit 986bf7ac0dbfee0522cbe1e89872309d3a4f182c
Author: Mauro Aranda <maurooaranda@gmail.com>
Commit: Mauro Aranda <maurooaranda@gmail.com>

    Remove duplicated tests in checkdoc-tests.el
    
    * test/lisp/emacs-lisp/checkdoc-tests.el (checkdoc-cl-defmethod-ok)
    (checkdoc-cl-defmethod-with-types-ok, checkdoc-cl-defun-with-key-ok)
    (checkdoc-cl-defun-with-allow-other-keys-ok)
    (checkdoc-cl-defun-with-default-optional-value-ok)
    (checkdoc-cl-defun-with-destructuring-ok): This tests were duplicated,
    so keep one copy of them.  Checked by diffing two files with the
    suspected tests, and supported by the fact that running occur with the
    regexp "^(ert-deftest" reported 14 matches, while the tests being run
    were 8.
---
 test/lisp/emacs-lisp/checkdoc-tests.el | 45 ----------------------------------
 1 file changed, 45 deletions(-)

diff --git a/test/lisp/emacs-lisp/checkdoc-tests.el 
b/test/lisp/emacs-lisp/checkdoc-tests.el
index cf7baf4..93015fb 100644
--- a/test/lisp/emacs-lisp/checkdoc-tests.el
+++ b/test/lisp/emacs-lisp/checkdoc-tests.el
@@ -82,51 +82,6 @@
     (insert "(cl-defun foo ((a b &optional c) d) \"Return A+B+C+D.\")")
     (checkdoc-defun)))
 
-(ert-deftest checkdoc-cl-defmethod-ok ()
-  "Checkdoc should be happy with a simple correct cl-defmethod."
-  (with-temp-buffer
-    (emacs-lisp-mode)
-    (insert "(cl-defmethod foo (a) \"Return A.\")")
-    (checkdoc-defun)))
-
-(ert-deftest checkdoc-cl-defmethod-with-types-ok ()
-  "Checkdoc should be happy with a cl-defmethod using types."
-  (with-temp-buffer
-    (emacs-lisp-mode)
-    ;; this method matches if A is the symbol `smthg' and if b is a list:
-    (insert "(cl-defmethod foo ((a (eql smthg)) (b list)) \"Return A+B.\")")
-    (checkdoc-defun)))
-
-(ert-deftest checkdoc-cl-defun-with-key-ok ()
-  "Checkdoc should be happy with a cl-defun using &key."
-  (with-temp-buffer
-    (emacs-lisp-mode)
-    (insert "(cl-defun foo (&key a (b 27)) \"Return :A+:B.\")")
-    (checkdoc-defun)))
-
-(ert-deftest checkdoc-cl-defun-with-allow-other-keys-ok ()
-  "Checkdoc should be happy with a cl-defun using &allow-other-keys."
-  (with-temp-buffer
-    (emacs-lisp-mode)
-    (insert "(cl-defun foo (&key a &allow-other-keys) \"Return :A.\")")
-    (checkdoc-defun)))
-
-(ert-deftest checkdoc-cl-defun-with-default-optional-value-ok ()
-  "Checkdoc should be happy with a cl-defun using default values for optional 
args."
-  (with-temp-buffer
-    (emacs-lisp-mode)
-    ;; B is optional and equals 1+a if not provided. HAS-BS is non-nil
-    ;; if B was provided in the call:
-    (insert "(cl-defun foo (a &optional (b (1+ a) has-bs)) \"Return A + B.\")")
-    (checkdoc-defun)))
-
-(ert-deftest checkdoc-cl-defun-with-destructuring-ok ()
-  "Checkdoc should be happy with a cl-defun destructuring its arguments."
-  (with-temp-buffer
-    (emacs-lisp-mode)
-    (insert "(cl-defun foo ((a b &optional c) d) \"Return A+B+C+D.\")")
-    (checkdoc-defun)))
-
 (ert-deftest checkdoc-tests--next-docstring ()
   "Checks that the one-argument form of `defvar' works.
 See the comments in Bug#24998."