[Emacs-diffs] master 55086ef: Fix obsolete ‘test/automated’ references

[Paul Eggert]

branch: master
commit 55086ef6517bd7294783d08d2b3cd50c330a9e96
Author: Paul Eggert <address@hidden>
Commit: Paul Eggert <address@hidden>

    Fix obsolete ‘test/automated’ references
    
    * Makefile.in (mostlyclean, clean, maybeclean_dirs, distclean)
    (bootstrap-clean, maintainer-clean):
    Clean ‘test’, not ‘test/automated’.  Test for existence of
    subdirectory only for ‘test’, not for directories that should
    always exist.
    * admin/MAINTAINERS, etc/TODO, lisp/emacs-lisp/bytecomp.el:
    * lisp/emacs-lisp/seq.el, lisp/emacs-lisp/thunk.el:
    * lisp/man.el (Man-parse-man-k):
    * lisp/url/url-domsuf.el, make-dist:
    * test/file-organization.org:
    Fix obsolete references to test/automated.
---
 Makefile.in                 | 26 +++++++++++++-------------
 admin/MAINTAINERS           | 12 ++++++------
 etc/TODO                    |  2 +-
 lisp/emacs-lisp/bytecomp.el |  2 +-
 lisp/emacs-lisp/seq.el      |  2 +-
 lisp/emacs-lisp/thunk.el    |  2 --
 lisp/man.el                 |  5 +----
 lisp/url/url-domsuf.el      |  2 +-
 make-dist                   |  9 +++------
 test/file-organization.org  |  6 +++---
 10 files changed, 30 insertions(+), 38 deletions(-)

diff --git a/Makefile.in b/Makefile.in
index cf8ff84..06909c3 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -821,9 +821,7 @@ mostlyclean_dirs = src oldXMenu lwlib lib lib-src nt 
doc/emacs doc/misc \
 $(foreach dir,$(mostlyclean_dirs),$(eval $(call 
submake_template,$(dir),mostlyclean)))
 
 mostlyclean: $(mostlyclean_dirs:=_mostlyclean)
-       for dir in test/automated; do \
-         [ ! -d $$dir ] || $(MAKE) -C $$dir mostlyclean; \
-       done
+       [ ! -d test ] || $(MAKE) -C test $@
 
 ### 'clean'
 ###      Delete all files from the current directory that are normally
@@ -838,9 +836,8 @@ clean_dirs = $(mostlyclean_dirs) nextstep
 $(foreach dir,$(clean_dirs),$(eval $(call submake_template,$(dir),clean)))
 
 clean: $(clean_dirs:=_clean)
-       for dir in test/automated admin/charsets; do \
-         [ ! -d $$dir ] || $(MAKE) -C $$dir clean; \
-       done
+       $(MAKE) -C admin/charsets $@
+       [ ! -d test ] || $(MAKE) -C test $@
        -rm -f *.tmp etc/*.tmp*
        -rm -rf info-dir.*
 
@@ -864,12 +861,13 @@ distclean_dirs = $(clean_dirs) leim lisp
 
 $(foreach dir,$(distclean_dirs),$(eval $(call 
submake_template,$(dir),distclean)))
 
-maybeclean_dirs = test/automated admin/grammars admin/unidata admin/charsets
+maybeclean_dirs = test admin/grammars admin/unidata admin/charsets
 
 distclean: $(distclean_dirs:=_distclean)
-       for dir in ${maybeclean_dirs}; do \
-         [ ! -d $$dir ] || $(MAKE) -C $$dir distclean; \
+       for dir in $(filter-out test,$(maybeclean_dirs)); do \
+         $(MAKE) -C $$dir $@ || exit; \
        done
+       [ ! -d test ] || $(MAKE) -C test $@
        ${top_distclean}
 
 ### 'bootstrap-clean'
@@ -878,9 +876,10 @@ distclean: $(distclean_dirs:=_distclean)
 $(foreach dir,$(distclean_dirs),$(eval $(call 
submake_template,$(dir),bootstrap-clean)))
 
 bootstrap-clean: $(distclean_dirs:=_bootstrap-clean)
-       for dir in ${maybeclean_dirs}; do \
-         [ ! -d $$dir ] || $(MAKE) -C $$dir bootstrap-clean; \
+       for dir in $(filter-out test,$(maybeclean_dirs)); do \
+         $(MAKE) -C $$dir $@ || exit; \
        done
+       [ ! -d test ] || $(MAKE) -C test $@
        [ ! -f config.log ] || mv -f config.log config.log~
        rm -rf ${srcdir}/info
        rm -f ${srcdir}/etc/refcards/emacsver.tex
@@ -906,9 +905,10 @@ maintainer_clean_dirs = src leim lisp
 $(foreach dir,$(maintainer_clean_dirs),$(eval $(call 
submake_template,$(dir),maintainer-clean)))
 
 maintainer-clean: bootstrap-clean $(maintainer_clean_dirs:=_maintainer-clean)
-       for dir in ${maybeclean_dirs}; do \
-         [ ! -d $$dir ] || $(MAKE) -C $$dir maintainer-clean; \
+       for dir in $(filter-out test,$(maybeclean_dirs)); do \
+         $(MAKE) -C $$dir $@ || exit; \
        done
+       [ ! -d test ] || $(MAKE) -C test $@
        ${top_maintainer_clean}
 
 ### This doesn't actually appear in the coding standards, but Karl
diff --git a/admin/MAINTAINERS b/admin/MAINTAINERS
index 5aff824..c13cb55 100644
--- a/admin/MAINTAINERS
+++ b/admin/MAINTAINERS
@@ -82,11 +82,11 @@ Simen Heggestøyl
 
 Nicolas Petton
        lisp/emacs-lisp/map.el
-       test/automated/map-tests.el
+       test/lisp/emac-lisp/map-tests.el
        lisp/emacs-lisp/seq.el
-       test/automated/seq-tests.el
+       test/lisp/emac-lisp/seq-tests.el
        lisp/emacs-lisp/thunk.el
-       test/automated/thunk-tests.el
+       test/lisp/emac-lisp/thunk-tests.el
 
 The GNU AUCTeX maintainers (address@hidden)
        RefTeX
@@ -104,7 +104,7 @@ The GNU AUCTeX maintainers (address@hidden)
 
 Dmitry Gutov
        lisp/progmodes/ruby-mode.el
-       test/automated/ruby-mode-tests.el
+       test/lisp/progmodes/ruby-mode-tests.el
        test/indent/ruby.rb
        lisp/progmodes/xref.el
        lisp/progmodes/project.el
@@ -119,11 +119,11 @@ Ulf Jasper
            lisp/net/newst-ticker.el
            lisp/net/newst-treeview.el
            lisp/net/newsticker.el
-           test/automated/newsticker-tests.el
+           test/lisp/net/newsticker-tests.el
 
        Icalendar
            lisp/calendar/icalendar.el
-           test/automated/icalendar-tests.el
+           test/lisp/calendar/icalendar-tests.el
 
 ==============================================================================
 2. Areas that someone is willing to maintain, although he would not
diff --git a/etc/TODO b/etc/TODO
index 71704f0..af2f41b 100644
--- a/etc/TODO
+++ b/etc/TODO
@@ -139,7 +139,7 @@ things in their .emacs.
 case to make sure it stays fixed.  Or pick your favorite programming
 major-mode, and write a test for its indentation.  Or a version
 control backend, and write a test for its status parser.  Etc.
-See test/automated for examples.
+See the 'test' directory for examples.
 
 * Small but important fixes needed in existing features:
 
diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el
index f45ae35..2c2996e 100644
--- a/lisp/emacs-lisp/bytecomp.el
+++ b/lisp/emacs-lisp/bytecomp.el
@@ -443,7 +443,7 @@ Return the compile-time value of FORM."
   ;; Macroexpand (not macroexpand-all!) form at toplevel in case it
   ;; expands into a toplevel-equivalent `progn'.  See CLHS section
   ;; 3.2.3.1, "Processing of Top Level Forms".  The semantics are very
-  ;; subtle: see test/automated/bytecomp-tests.el for interesting
+  ;; subtle: see test/lisp/emacs-lisp/bytecomp-tests.el for interesting
   ;; cases.
   (setf form (macroexp-macroexpand form byte-compile-macro-environment))
   (if (eq (car-safe form) 'progn)
diff --git a/lisp/emacs-lisp/seq.el b/lisp/emacs-lisp/seq.el
index 4118764..10de248 100644
--- a/lisp/emacs-lisp/seq.el
+++ b/lisp/emacs-lisp/seq.el
@@ -52,7 +52,7 @@
 ;; - `seq-copy'
 ;; - `seq-into'
 ;;
-;; All functions are tested in test/automated/seq-tests.el
+;; All functions are tested in test/lisp/emacs-lisp/seq-tests.el
 
 ;;; Code:
 
diff --git a/lisp/emacs-lisp/thunk.el b/lisp/emacs-lisp/thunk.el
index be0a90f..f4c075d 100644
--- a/lisp/emacs-lisp/thunk.el
+++ b/lisp/emacs-lisp/thunk.el
@@ -42,8 +42,6 @@
 ;;
 ;;    (thunk-force delayed)
 
-;; Tests are located at test/automated/thunk-tests.el
-
 ;;; Code:
 
 (defmacro thunk-delay (&rest body)
diff --git a/lisp/man.el b/lisp/man.el
index 4f6e118..0e1c929 100644
--- a/lisp/man.el
+++ b/lisp/man.el
@@ -832,10 +832,7 @@ indicating optional parts and whitespace being interpreted
 somewhat loosely.
 
 foo[, bar [, ...]] [other stuff] (sec) - description
-foo(sec)[, bar(sec) [, ...]] [other stuff] - description
-
-For more details and some regression tests, please see
-test/automated/man-tests.el in the emacs repository."
+foo(sec)[, bar(sec) [, ...]] [other stuff] - description"
   (goto-char (point-min))
   ;; See man-tests for data about which systems use which format (hopefully we
   ;; will be able to simplify the code if/when some of those formats aren't
diff --git a/lisp/url/url-domsuf.el b/lisp/url/url-domsuf.el
index 25a6a42..b24f245 100644
--- a/lisp/url/url-domsuf.el
+++ b/lisp/url/url-domsuf.el
@@ -89,7 +89,7 @@
 
 ;; Tests:
 
-;; TODO convert to a proper test/automated test.
+;; TODO convert to a proper test.
 ;; (url-domsuf-cookie-allowed-p "com") => nil
 ;; (url-domsuf-cookie-allowed-p "foo.bar.bd") => t
 ;; (url-domsuf-cookie-allowed-p "bar.bd") => nil
diff --git a/make-dist b/make-dist
index 96e0a73..7e1f0df 100755
--- a/make-dist
+++ b/make-dist
@@ -538,12 +538,9 @@ done
 
 if [ "$with_tests" = "yes" ]; then
   echo "Making links to 'test' and its subdirectories"
-  for f in `find test -type f`; do
-    case $f in
-      test/automated/*.log) continue ;;
-      test/automated/flymake/warnpred/a.out) continue ;;
-      test/automated/Makefile) continue ;;
-    esac
+  for f in `find test -type f ! -name '*.log' ! -name a.out \
+             ! -path test/Makefile
+  `; do
     ln $f $tempdir/$f
   done
 fi
diff --git a/test/file-organization.org b/test/file-organization.org
index 8d9c671..4d76c00 100644
--- a/test/file-organization.org
+++ b/test/file-organization.org
@@ -50,9 +50,9 @@ not run by the "make check" command and its derivatives.
 Resource files for tests (containing test data) should reside in a
 directory named after the feature with a ~-resources~ suffix, and
 located in the same directory as the feature.  Hence, the lisp file
-~flymake.el~ should have test files in
-~/test/automated/lisp/progmodes/flymake-tests.el~ should reside in a
-directory called ~/test/automated/lisp/progmodes/flymake-resources~.
+~lisp/progmodes/flymake.el~, with tests in
+~test/lisp/progmodes/flymake-tests.el~, should have resources in a
+directory called ~test/lisp/progmodes/flymake-resources~.
 
 No guidance is given for the organization of resource files inside the
 ~-resource~ directory; files can be organized at the author's