--- /dev/null
- (shell-command (format "%s -f %s/* %s" cp tmpdir2 tmpdir1))
+ ;;; auto-revert-tests.el --- Tests of auto-revert
+
+ ;; Copyright (C) 2015 Free Software Foundation, Inc.
+
+ ;; Author: Michael Albinus <michael.albinus@gmx.de>
+
+ ;; This program 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.
+ ;;
+ ;; This program 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 this program. If not, see `http://www.gnu.org/licenses/'.
+
+ ;;; Commentary:
+
+ ;; A whole test run can be performed calling the command `auto-revert-test-all'.
+
+ ;;; Code:
+
+ (require 'ert)
+ (require 'autorevert)
+ (setq auto-revert-notify-exclude-dir-regexp "nothing-to-be-excluded"
+ auto-revert-stop-on-user-input nil)
+
+ (defconst auto-revert--timeout 10
+ "Time to wait until a message appears in the *Messages* buffer.")
+
+ (defun auto-revert--wait-for-revert (buffer)
+ "Wait until the *Messages* buffer reports reversion of BUFFER."
+ (with-timeout (auto-revert--timeout nil)
+ (with-current-buffer "*Messages*"
+ (while
+ (null (string-match
+ (format-message "Reverting buffer `%s'." (buffer-name buffer))
+ (buffer-string)))
+ (read-event nil nil 0.1)))))
+
+ (ert-deftest auto-revert-test00-auto-revert-mode ()
+ "Check autorevert for a file."
+ ;; `auto-revert-buffers' runs every 5". And we must wait, until the
+ ;; file has been reverted.
+ (let ((tmpfile (make-temp-file "auto-revert-test"))
+ buf)
+ (unwind-protect
+ (progn
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (narrow-to-region (point-max) (point-max)))
+ (write-region "any text" nil tmpfile nil 'no-message)
+ (setq buf (find-file-noselect tmpfile))
+ (with-current-buffer buf
+ (should (string-equal (buffer-string) "any text"))
+ ;; `buffer-stale--default-function' checks for
+ ;; `verify-visited-file-modtime'. We must ensure that it
+ ;; returns nil.
+ (sleep-for 1)
+ (auto-revert-mode 1)
+ (should auto-revert-mode)
+
+ ;; Modify file. We wait for a second, in order to have
+ ;; another timestamp.
+ (sleep-for 1)
+ (write-region "another text" nil tmpfile nil 'no-message)
+
+ ;; Check, that the buffer has been reverted.
+ (auto-revert--wait-for-revert buf)
+ (should (string-match "another text" (buffer-string)))
+
+ ;; When the buffer is modified, it shall not be reverted.
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (narrow-to-region (point-max) (point-max)))
+ (set-buffer-modified-p t)
+ (sleep-for 1)
+ (write-region "any text" nil tmpfile nil 'no-message)
+
+ ;; Check, that the buffer hasn't been reverted.
+ (auto-revert--wait-for-revert buf)
+ (should-not (string-match "any text" (buffer-string)))))
+
+ ;; Exit.
+ (with-current-buffer "*Messages*" (widen))
+ (ignore-errors
+ (with-current-buffer buf (set-buffer-modified-p nil))
+ (kill-buffer buf))
+ (ignore-errors (delete-file tmpfile)))))
+
+ ;; This is inspired by Bug#21841.
+ (ert-deftest auto-revert-test01-auto-revert-several-files ()
+ "Check autorevert for several files at once."
+ (skip-unless (executable-find "cp"))
+
+ (let* ((cp (executable-find "cp"))
+ (tmpdir1 (make-temp-file "auto-revert-test" 'dir))
+ (tmpdir2 (make-temp-file "auto-revert-test" 'dir))
+ (tmpfile1
+ (make-temp-file (expand-file-name "auto-revert-test" tmpdir1)))
+ (tmpfile2
+ (make-temp-file (expand-file-name "auto-revert-test" tmpdir1)))
+ buf1 buf2)
+ (unwind-protect
+ (progn
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (narrow-to-region (point-max) (point-max)))
+ (write-region "any text" nil tmpfile1 nil 'no-message)
+ (setq buf1 (find-file-noselect tmpfile1))
+ (write-region "any text" nil tmpfile2 nil 'no-message)
+ (setq buf2 (find-file-noselect tmpfile2))
+
+ (dolist (buf (list buf1 buf2))
+ (with-current-buffer buf
+ (should (string-equal (buffer-string) "any text"))
+ ;; `buffer-stale--default-function' checks for
+ ;; `verify-visited-file-modtime'. We must ensure that
+ ;; it returns nil.
+ (sleep-for 1)
+ (auto-revert-mode 1)
+ (should auto-revert-mode)))
+
+ ;; Modify files. We wait for a second, in order to have
+ ;; another timestamp.
+ (sleep-for 1)
+ (write-region
+ "another text" nil
+ (expand-file-name (file-name-nondirectory tmpfile1) tmpdir2)
+ nil 'no-message)
+ (write-region
+ "another text" nil
+ (expand-file-name (file-name-nondirectory tmpfile2) tmpdir2)
+ nil 'no-message)
+ ;;(copy-directory tmpdir2 tmpdir1 nil 'copy-contents)
+ ;; Strange, that `copy-directory' does not work as expected.
+ ;; The following shell command is not portable on all
+ ;; platforms, unfortunately.
++ (shell-command (format "%s %s/* %s" cp tmpdir2 tmpdir1))
+
+ ;; Check, that the buffers have been reverted.
+ (dolist (buf (list buf1 buf2))
+ (with-current-buffer buf
+ (auto-revert--wait-for-revert buf)
+ (should (string-match "another text" (buffer-string))))))
+
+ ;; Exit.
+ (with-current-buffer "*Messages*" (widen))
+ (ignore-errors
+ (dolist (buf (list buf1 buf2))
+ (with-current-buffer buf (set-buffer-modified-p nil))
+ (kill-buffer buf)))
+ (ignore-errors (delete-directory tmpdir1 'recursive))
+ (ignore-errors (delete-directory tmpdir2 'recursive)))))
+
+ (ert-deftest auto-revert-test02-auto-revert-tail-mode ()
+ "Check autorevert tail mode."
+ ;; `auto-revert-buffers' runs every 5". And we must wait, until the
+ ;; file has been reverted.
+ (let ((tmpfile (make-temp-file "auto-revert-test"))
+ buf)
+ (unwind-protect
+ (progn
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (narrow-to-region (point-max) (point-max)))
+ (write-region "any text" nil tmpfile nil 'no-message)
+ (setq buf (find-file-noselect tmpfile))
+ (with-current-buffer buf
+ ;; `buffer-stale--default-function' checks for
+ ;; `verify-visited-file-modtime'. We must ensure that it
+ ;; returns nil.
+ (sleep-for 1)
+ (auto-revert-tail-mode 1)
+ (should auto-revert-tail-mode)
+ (erase-buffer)
+ (insert "modified text\n")
+ (set-buffer-modified-p nil)
+
+ ;; Modify file. We wait for a second, in order to have
+ ;; another timestamp.
+ (sleep-for 1)
+ (write-region "another text" nil tmpfile 'append 'no-message)
+
+ ;; Check, that the buffer has been reverted.
+ (auto-revert--wait-for-revert buf)
+ (should
+ (string-match "modified text\nanother text" (buffer-string)))))
+
+ ;; Exit.
+ (with-current-buffer "*Messages*" (widen))
+ (ignore-errors (kill-buffer buf))
+ (ignore-errors (delete-file tmpfile)))))
+
+ (ert-deftest auto-revert-test03-auto-revert-mode-dired ()
+ "Check autorevert for dired."
+ ;; `auto-revert-buffers' runs every 5". And we must wait, until the
+ ;; file has been reverted.
+ (let* ((tmpfile (make-temp-file "auto-revert-test"))
+ (name (file-name-nondirectory tmpfile))
+ buf)
+ (unwind-protect
+ (progn
+ (setq buf (dired-noselect temporary-file-directory))
+ (with-current-buffer buf
+ ;; `buffer-stale--default-function' checks for
+ ;; `verify-visited-file-modtime'. We must ensure that it
+ ;; returns nil.
+ (sleep-for 1)
+ (auto-revert-mode 1)
+ (should auto-revert-mode)
+ (should
+ (string-match name (substring-no-properties (buffer-string))))
+
+ ;; Delete file. We wait for a second, in order to have
+ ;; another timestamp.
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (narrow-to-region (point-max) (point-max)))
+ (sleep-for 1)
+ (delete-file tmpfile)
+
+ ;; Check, that the buffer has been reverted.
+ (auto-revert--wait-for-revert buf)
+ (should-not
+ (string-match name (substring-no-properties (buffer-string))))
+
+ ;; Make dired buffer modified. Check, that the buffer has
+ ;; been still reverted.
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (narrow-to-region (point-max) (point-max)))
+ (set-buffer-modified-p t)
+ (sleep-for 1)
+ (write-region "any text" nil tmpfile nil 'no-message)
+
+ ;; Check, that the buffer has been reverted.
+ (auto-revert--wait-for-revert buf)
+ (should
+ (string-match name (substring-no-properties (buffer-string))))))
+
+ ;; Exit.
+ (with-current-buffer "*Messages*" (widen))
+ (ignore-errors
+ (with-current-buffer buf (set-buffer-modified-p nil))
+ (kill-buffer buf))
+ (ignore-errors (delete-file tmpfile)))))
+
+ (defun auto-revert-test-all (&optional interactive)
+ "Run all tests for \\[auto-revert]."
+ (interactive "p")
+ (if interactive
+ (ert-run-tests-interactively "^auto-revert-")
+ (ert-run-tests-batch "^auto-revert-")))
+
+ (provide 'auto-revert-tests)
+ ;;; auto-revert-tests.el ends here
--- /dev/null
- (if (file-remote-p temporary-file-directory) 6 3))
+ ;;; file-notify-tests.el --- Tests of file notifications -*- lexical-binding: t; -*-
+
+ ;; Copyright (C) 2013-2015 Free Software Foundation, Inc.
+
+ ;; Author: Michael Albinus <michael.albinus@gmx.de>
+
+ ;; This program 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.
+ ;;
+ ;; This program 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 this program. If not, see `http://www.gnu.org/licenses/'.
+
+ ;;; Commentary:
+
+ ;; Some of the tests require access to a remote host files. Since
+ ;; this could be problematic, a mock-up connection method "mock" is
+ ;; used. Emulating a remote connection, it simply calls "sh -i".
+ ;; Tramp's file name handlers still run, so this test is sufficient
+ ;; except for connection establishing.
+
+ ;; If you want to test a real Tramp connection, set
+ ;; $REMOTE_TEMPORARY_FILE_DIRECTORY to a suitable value in order to
+ ;; overwrite the default value. If you want to skip tests accessing a
+ ;; remote host, set this environment variable to "/dev/null" or
+ ;; whatever is appropriate on your system.
+
+ ;; A whole test run can be performed calling the command `file-notify-test-all'.
+
+ ;;; Code:
+
+ (require 'ert)
+ (require 'filenotify)
+ (require 'tramp)
+
+ ;; There is no default value on w32 systems, which could work out of the box.
+ (defconst file-notify-test-remote-temporary-file-directory
+ (cond
+ ((getenv "REMOTE_TEMPORARY_FILE_DIRECTORY"))
+ ((eq system-type 'windows-nt) null-device)
+ (t (add-to-list
+ 'tramp-methods
+ '("mock"
+ (tramp-login-program "sh")
+ (tramp-login-args (("-i")))
+ (tramp-remote-shell "/bin/sh")
+ (tramp-remote-shell-args ("-c"))
+ (tramp-connection-timeout 10)))
+ (format "/mock::%s" temporary-file-directory)))
+ "Temporary directory for Tramp tests.")
+
+ (defvar file-notify--test-tmpfile nil)
+ (defvar file-notify--test-tmpfile1 nil)
+ (defvar file-notify--test-desc nil)
+ (defvar file-notify--test-results nil)
+ (defvar file-notify--test-event nil)
+ (defvar file-notify--test-events nil)
+ (defvar file-notify--test-expected-events nil)
+
+ (defun file-notify--test-timeout ()
+ "Timeout to wait for arriving events, in seconds."
- (if (null (file-remote-p temporary-file-directory))
- (message "Local library: `%s'" file-notify--library)
- (message "Remote command: `%s'"
- (replace-regexp-in-string
- "<[[:digit:]]+>\\'" ""
- (process-name (cdr file-notify--test-remote-enabled-checked)))))
++ (cond
++ ((file-remote-p temporary-file-directory) 6)
++ ((string-equal (file-notify--test-library) "w32notify") 20)
++ ((eq system-type 'cygwin) 10)
++ (t 3)))
+
+ (defun file-notify--test-cleanup ()
+ "Cleanup after a test."
+ (file-notify-rm-watch file-notify--test-desc)
+
+ (when (and file-notify--test-tmpfile
+ (file-exists-p file-notify--test-tmpfile))
+ (if (file-directory-p file-notify--test-tmpfile)
+ (delete-directory file-notify--test-tmpfile 'recursive)
+ (delete-file file-notify--test-tmpfile)))
+ (when (and file-notify--test-tmpfile1
+ (file-exists-p file-notify--test-tmpfile1))
+ (if (file-directory-p file-notify--test-tmpfile1)
+ (delete-directory file-notify--test-tmpfile1 'recursive)
+ (delete-file file-notify--test-tmpfile1)))
+ (when (file-remote-p temporary-file-directory)
+ (tramp-cleanup-connection
+ (tramp-dissect-file-name temporary-file-directory) nil 'keep-password))
+
+ (setq file-notify--test-tmpfile nil
+ file-notify--test-tmpfile1 nil
+ file-notify--test-desc nil
+ file-notify--test-results nil
+ file-notify--test-events nil
+ file-notify--test-expected-events nil)
+ (when file-notify--test-event
+ (error "file-notify--test-event should not be set but bound dynamically")))
+
+ (setq password-cache-expiry nil
+ tramp-verbose 0
+ tramp-message-show-message nil)
+
+ ;; This shall happen on hydra only.
+ (when (getenv "NIX_STORE")
+ (add-to-list 'tramp-remote-path 'tramp-own-remote-path))
+
+ ;; We do not want to try and fail `file-notify-add-watch'.
+ (defun file-notify--test-local-enabled ()
+ "Whether local file notification is enabled.
+ This is needed for local `temporary-file-directory' only, in the
+ remote case we return always t."
+ (or file-notify--library
+ (file-remote-p temporary-file-directory)))
+
+ (defvar file-notify--test-remote-enabled-checked nil
+ "Cached result of `file-notify--test-remote-enabled'.
+ If the function did run, the value is a cons cell, the `cdr'
+ being the result.")
+
+ (defun file-notify--test-remote-enabled ()
+ "Whether remote file notification is enabled."
+ (unless (consp file-notify--test-remote-enabled-checked)
+ (let (desc)
+ (ignore-errors
+ (and
+ (file-remote-p file-notify-test-remote-temporary-file-directory)
+ (file-directory-p file-notify-test-remote-temporary-file-directory)
+ (file-writable-p file-notify-test-remote-temporary-file-directory)
+ (setq desc
+ (file-notify-add-watch
+ file-notify-test-remote-temporary-file-directory
+ '(change) 'ignore))))
+ (setq file-notify--test-remote-enabled-checked (cons t desc))
+ (when desc (file-notify-rm-watch desc))))
+ ;; Return result.
+ (cdr file-notify--test-remote-enabled-checked))
+
++(defun file-notify--test-library ()
++ "The used libray for the test, as string.
++In the remote case, it is the process name which runs on the
++remote host, or nil."
++ (if (null (file-remote-p temporary-file-directory))
++ (symbol-name file-notify--library)
++ (and (consp file-notify--test-remote-enabled-checked)
++ (processp (cdr file-notify--test-remote-enabled-checked))
++ (replace-regexp-in-string
++ "<[[:digit:]]+>\\'" ""
++ (process-name (cdr file-notify--test-remote-enabled-checked))))))
++
+ (defmacro file-notify--deftest-remote (test docstring)
+ "Define ert `TEST-remote' for remote files."
+ (declare (indent 1))
+ `(ert-deftest ,(intern (concat (symbol-name test) "-remote")) ()
+ ,docstring
+ (let* ((temporary-file-directory
+ file-notify-test-remote-temporary-file-directory)
+ (ert-test (ert-get-test ',test)))
+ (skip-unless (file-notify--test-remote-enabled))
+ (tramp-cleanup-connection
+ (tramp-dissect-file-name temporary-file-directory) nil 'keep-password)
+ (funcall (ert-test-body ert-test)))))
+
+ (ert-deftest file-notify-test00-availability ()
+ "Test availability of `file-notify'."
+ (skip-unless (file-notify--test-local-enabled))
+ ;; Report the native library which has been used.
- ;; The file does not need to exist, just the upper directory.
++ (message "Library: `%s'" (file-notify--test-library))
+ (should
+ (setq file-notify--test-desc
+ (file-notify-add-watch temporary-file-directory '(change) 'ignore)))
+
+ ;; Cleanup.
+ (file-notify--test-cleanup))
+
+ (file-notify--deftest-remote file-notify-test00-availability
+ "Test availability of `file-notify' for remote files.")
+
+ (ert-deftest file-notify-test01-add-watch ()
+ "Check `file-notify-add-watch'."
+ (skip-unless (file-notify--test-local-enabled))
+
+ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
+ file-notify--test-tmpfile1
+ (format "%s/%s" file-notify--test-tmpfile (md5 (current-time-string))))
+
+ ;; Check, that different valid parameters are accepted.
+ (should
+ (setq file-notify--test-desc
+ (file-notify-add-watch temporary-file-directory '(change) 'ignore)))
+ (file-notify-rm-watch file-notify--test-desc)
+ (should
+ (setq file-notify--test-desc
+ (file-notify-add-watch
+ temporary-file-directory '(attribute-change) 'ignore)))
+ (file-notify-rm-watch file-notify--test-desc)
+ (should
+ (setq file-notify--test-desc
+ (file-notify-add-watch
+ temporary-file-directory '(change attribute-change) 'ignore)))
+ (file-notify-rm-watch file-notify--test-desc)
- (string-equal (directory-file-name
- (file-name-directory
- (file-notify--event-file-name file-notify--test-event)))
- file-notify--test-tmpfile)))
++ (write-region "any text" nil file-notify--test-tmpfile nil 'no-message)
+ (should
+ (setq file-notify--test-desc
+ (file-notify-add-watch
+ file-notify--test-tmpfile '(change attribute-change) 'ignore)))
+ (file-notify-rm-watch file-notify--test-desc)
++ (delete-file file-notify--test-tmpfile)
+
+ ;; Check error handling.
+ (should-error (file-notify-add-watch 1 2 3 4)
+ :type 'wrong-number-of-arguments)
+ (should
+ (equal (should-error
+ (file-notify-add-watch 1 2 3))
+ '(wrong-type-argument 1)))
+ (should
+ (equal (should-error
+ (file-notify-add-watch temporary-file-directory 2 3))
+ '(wrong-type-argument 2)))
+ (should
+ (equal (should-error
+ (file-notify-add-watch temporary-file-directory '(change) 3))
+ '(wrong-type-argument 3)))
+ ;; The upper directory of a file must exist.
+ (should
+ (equal (should-error
+ (file-notify-add-watch
+ file-notify--test-tmpfile1 '(change attribute-change) 'ignore))
+ `(file-notify-error
+ "Directory does not exist" ,file-notify--test-tmpfile)))
+
+ ;; Cleanup.
+ (file-notify--test-cleanup))
+
+ (file-notify--deftest-remote file-notify-test01-add-watch
+ "Check `file-notify-add-watch' for remote files.")
+
+ (defun file-notify--test-event-test ()
+ "Ert test function to be called by `file-notify--test-event-handler'.
+ We cannot pass arguments, so we assume that `file-notify--test-event'
+ is bound somewhere."
+ ;; Check the descriptor.
+ (should (equal (car file-notify--test-event) file-notify--test-desc))
+ ;; Check the file name.
+ (should
+ (or (string-equal (file-notify--event-file-name file-notify--test-event)
+ file-notify--test-tmpfile)
- (string-equal
- (file-notify--event-file1-name file-notify--test-event)
- file-notify--test-tmpfile1))))
++ (string-equal (file-notify--event-file-name file-notify--test-event)
++ file-notify--test-tmpfile1)
++ (string-equal (file-notify--event-file-name file-notify--test-event)
++ temporary-file-directory)))
+ ;; Check the second file name if exists.
+ (when (eq (nth 1 file-notify--test-event) 'renamed)
+ (should
- ;; Do not add temporary files, this would confuse the checks.
++ (or (string-equal (file-notify--event-file1-name file-notify--test-event)
++ file-notify--test-tmpfile1)
++ (string-equal (file-notify--event-file1-name file-notify--test-event)
++ temporary-file-directory)))))
+
+ (defun file-notify--test-event-handler (event)
+ "Run a test over FILE-NOTIFY--TEST-EVENT.
+ For later analysis, append the test result to `file-notify--test-results'
+ and the event to `file-notify--test-events'."
+ (let* ((file-notify--test-event event)
+ (result
+ (ert-run-test (make-ert-test :body 'file-notify--test-event-test))))
- `(let ((,outer file-notify--test-events))
++ ;; Do not add lock files, this would confuse the checks.
+ (unless (string-match
+ (regexp-quote ".#")
+ (file-notify--event-file-name file-notify--test-event))
+ ;;(message "file-notify--test-event-handler %S" file-notify--test-event)
+ (setq file-notify--test-events
+ (append file-notify--test-events `(,file-notify--test-event))
+ file-notify--test-results
+ (append file-notify--test-results `(,result))))))
+
+ (defun file-notify--test-make-temp-name ()
+ "Create a temporary file name for test."
+ (expand-file-name
+ (make-temp-name "file-notify-test") temporary-file-directory))
+
+ (defmacro file-notify--wait-for-events (timeout until)
+ "Wait for and return file notification events until form UNTIL is true.
+ TIMEOUT is the maximum time to wait for, in seconds."
+ `(with-timeout (,timeout (ignore))
+ (while (null ,until)
+ (read-event nil nil 0.1))))
+
+ (defmacro file-notify--test-with-events (events &rest body)
+ "Run BODY collecting events and then compare with EVENTS.
+ Don't wait longer than timeout seconds for the events to be delivered."
+ (declare (indent 1))
+ (let ((outer (make-symbol "outer")))
- (file-notify--test-timeout)
++ `(let ((,outer file-notify--test-events)
++ create-lockfiles)
+ (setq file-notify--test-expected-events
+ (append file-notify--test-expected-events ,events))
++ ;; Flush pending events.
++ (file-notify--wait-for-events
++ (file-notify--test-timeout)
++ (input-pending-p))
+ (let (file-notify--test-events)
+ ,@body
+ (file-notify--wait-for-events
- ;; Under cygwin there are so bad timings that it doesn't make sense to test.
- (skip-unless (not (eq system-type 'cygwin)))
-
- (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
- file-notify--test-tmpfile1 (file-notify--test-make-temp-name))
++ ;; More events need more time. Use some fudge factor.
++ (* (ceiling (length ,events) 100) (file-notify--test-timeout))
+ (= (length ,events) (length file-notify--test-events)))
+ (should (equal ,events (mapcar #'cadr file-notify--test-events)))
+ (setq ,outer (append ,outer file-notify--test-events)))
+ (setq file-notify--test-events ,outer))))
+
+ (ert-deftest file-notify-test02-events ()
+ "Check file creation/change/removal notifications."
+ (skip-unless (file-notify--test-local-enabled))
- ;; Check creation, change and deletion.
- (setq file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) 'file-notify--test-event-handler))
- (file-notify--test-with-events '(created changed deleted)
+
+ (unwind-protect
+ (progn
- "any text" nil file-notify--test-tmpfile nil 'no-message)
++ ;; Check file creation, change and deletion. It doesn't work
++ ;; for cygwin and kqueue, because we don't use an implicit
++ ;; directory monitor (kqueue), or the timings are too bad (cygwin).
++ (unless (or (eq system-type 'cygwin)
++ (string-equal (file-notify--test-library) "kqueue"))
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(change) 'file-notify--test-event-handler)))
++ (file-notify--test-with-events
++ (cond
++ ;; cygwin recognizes only `deleted' and `stopped' events.
++ ((eq system-type 'cygwin)
++ '(deleted stopped))
++ (t '(created changed deleted stopped)))
++ (write-region
++ "another text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
++ (delete-file file-notify--test-tmpfile))
++ ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
++ (let (file-notify--test-events)
++ (file-notify-rm-watch file-notify--test-desc)))
++
++ ;; Check file change and deletion.
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
++ (write-region "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(change) 'file-notify--test-event-handler)))
++ (file-notify--test-with-events
++ (cond
++ ;; cygwin recognizes only `deleted' and `stopped' events.
++ ((eq system-type 'cygwin)
++ '(deleted stopped))
++ ;; inotify, kqueueg and gfilenotify raise just one
++ ;; `changed' event, the other backends show us two of
++ ;; them.
++ ((or (string-equal "inotify" (file-notify--test-library))
++ (string-equal "kqueue" (file-notify--test-library))
++ (string-equal "gfilenotify" (file-notify--test-library)))
++ '(changed deleted stopped))
++ (t '(changed changed deleted stopped)))
++ (read-event nil nil 0.1)
+ (write-region
- ;; Check creation, change and deletion. There must be a
- ;; `stopped' event when deleting the directory. It doesn't
- ;; work for w32notify.
- (unless (eq file-notify--library 'w32notify)
- (make-directory file-notify--test-tmpfile)
- (setq file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) 'file-notify--test-event-handler))
++ "another text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
+ (delete-file file-notify--test-tmpfile))
+ ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
+ (let (file-notify--test-events)
+ (file-notify-rm-watch file-notify--test-desc))
+
- ;; There are two `deleted' events, for the file and for
- ;; the directory.
- '(created changed deleted deleted stopped)
++ ;; Check file creation, change and deletion when watching a
++ ;; directory. There must be a `stopped' event when deleting
++ ;; the directory.
++ (let ((temporary-file-directory
++ (make-temp-file "file-notify-test-parent" t)))
++ (should
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
++ file-notify--test-desc
++ (file-notify-add-watch
++ temporary-file-directory
++ '(change) 'file-notify--test-event-handler)))
+ (file-notify--test-with-events
- "any text" nil (expand-file-name "foo" file-notify--test-tmpfile)
- nil 'no-message)
- (delete-directory file-notify--test-tmpfile 'recursive))
++ (cond
++ ;; w32notify does raise a `stopped' event when a
++ ;; watched directory is deleted.
++ ((string-equal (file-notify--test-library) "w32notify")
++ '(created changed deleted))
++ ;; cygwin recognizes only `deleted' and `stopped' events.
++ ((eq system-type 'cygwin)
++ '(deleted stopped))
++ ;; There are two `deleted' events, for the file and for
++ ;; the directory. Except for kqueue.
++ ((string-equal (file-notify--test-library) "kqueue")
++ '(created changed deleted stopped))
++ (t '(created changed deleted deleted stopped)))
++ (read-event nil nil 0.1)
+ (write-region
- ;; Check copy.
- (setq file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) 'file-notify--test-event-handler))
- (should file-notify--test-desc)
- (file-notify--test-with-events
- ;; w32notify does not distinguish between `changed' and
- ;; `attribute-changed'.
- (if (eq file-notify--library 'w32notify)
- '(created changed changed deleted)
- '(created changed deleted))
- (write-region
- "any text" nil file-notify--test-tmpfile nil 'no-message)
- (copy-file file-notify--test-tmpfile file-notify--test-tmpfile1)
- ;; The next two events shall not be visible.
- (set-file-modes file-notify--test-tmpfile 000)
- (read-event nil nil 0.1) ; In order to distinguish the events.
- (set-file-times file-notify--test-tmpfile '(0 0))
- (delete-file file-notify--test-tmpfile)
- (delete-file file-notify--test-tmpfile1))
- ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
- (let (file-notify--test-events)
- (file-notify-rm-watch file-notify--test-desc))
++ "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
++ (delete-directory temporary-file-directory 'recursive))
+ ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
+ (let (file-notify--test-events)
+ (file-notify-rm-watch file-notify--test-desc)))
+
- ;; Check rename.
- (setq file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) 'file-notify--test-event-handler))
- (should file-notify--test-desc)
- (file-notify--test-with-events '(created changed renamed)
- (write-region
- "any text" nil file-notify--test-tmpfile nil 'no-message)
- (rename-file file-notify--test-tmpfile file-notify--test-tmpfile1)
- ;; After the rename, we won't get events anymore.
- (delete-file file-notify--test-tmpfile1))
- ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
- (let (file-notify--test-events)
- (file-notify-rm-watch file-notify--test-desc))
++ ;; Check copy of files inside a directory.
++ (let ((temporary-file-directory
++ (make-temp-file "file-notify-test-parent" t)))
++ (should
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
++ file-notify--test-tmpfile1 (file-notify--test-make-temp-name)
++ file-notify--test-desc
++ (file-notify-add-watch
++ temporary-file-directory
++ '(change) 'file-notify--test-event-handler)))
++ (file-notify--test-with-events
++ (cond
++ ;; w32notify does not distinguish between `changed' and
++ ;; `attribute-changed'.
++ ((string-equal (file-notify--test-library) "w32notify")
++ '(created changed created changed changed changed changed
++ deleted deleted))
++ ;; cygwin recognizes only `deleted' and `stopped' events.
++ ((eq system-type 'cygwin)
++ '(deleted stopped))
++ ;; There are three `deleted' events, for two files and
++ ;; for the directory. Except for kqueue.
++ ((string-equal (file-notify--test-library) "kqueue")
++ '(created changed created changed deleted stopped))
++ (t '(created changed created changed
++ deleted deleted deleted stopped)))
++ (read-event nil nil 0.1)
++ (write-region
++ "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
++ (copy-file file-notify--test-tmpfile file-notify--test-tmpfile1)
++ ;; The next two events shall not be visible.
++ (read-event nil nil 0.1)
++ (set-file-modes file-notify--test-tmpfile 000)
++ (read-event nil nil 0.1)
++ (set-file-times file-notify--test-tmpfile '(0 0))
++ (read-event nil nil 0.1)
++ (delete-directory temporary-file-directory 'recursive))
++ ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
++ (let (file-notify--test-events)
++ (file-notify-rm-watch file-notify--test-desc)))
+
- ;; Check attribute change. It doesn't work for w32notify.
- (unless (eq file-notify--library 'w32notify)
- (setq file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(attribute-change) 'file-notify--test-event-handler))
- (file-notify--test-with-events
- (if (file-remote-p temporary-file-directory)
- ;; In the remote case, `write-region' raises also an
- ;; `attribute-changed' event.
- '(attribute-changed attribute-changed attribute-changed)
- '(attribute-changed attribute-changed))
- ;; We must use short delays between the operations.
- ;; Otherwise, not all events arrive us in the remote case.
- (write-region
- "any text" nil file-notify--test-tmpfile nil 'no-message)
- (read-event nil nil 0.1)
- (set-file-modes file-notify--test-tmpfile 000)
- (read-event nil nil 0.1)
- (set-file-times file-notify--test-tmpfile '(0 0))
- (read-event nil nil 0.1)
- (delete-file file-notify--test-tmpfile))
++ ;; Check rename of files inside a directory.
++ (let ((temporary-file-directory
++ (make-temp-file "file-notify-test-parent" t)))
++ (should
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
++ file-notify--test-tmpfile1 (file-notify--test-make-temp-name)
++ file-notify--test-desc
++ (file-notify-add-watch
++ temporary-file-directory
++ '(change) 'file-notify--test-event-handler)))
++ (file-notify--test-with-events
++ (cond
++ ;; w32notify does not distinguish between `changed' and
++ ;; `attribute-changed'.
++ ((string-equal (file-notify--test-library) "w32notify")
++ '(created changed renamed deleted))
++ ;; cygwin recognizes only `deleted' and `stopped' events.
++ ((eq system-type 'cygwin)
++ '(deleted stopped))
++ ;; There are two `deleted' events, for the file and for
++ ;; the directory. Except for kqueue.
++ ((string-equal (file-notify--test-library) "kqueue")
++ '(created changed renamed deleted stopped))
++ (t '(created changed renamed deleted deleted stopped)))
++ (read-event nil nil 0.1)
++ (write-region
++ "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
++ (rename-file file-notify--test-tmpfile file-notify--test-tmpfile1)
++ ;; After the rename, we won't get events anymore.
++ (read-event nil nil 0.1)
++ (delete-directory temporary-file-directory 'recursive))
++ ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
++ (let (file-notify--test-events)
++ (file-notify-rm-watch file-notify--test-desc)))
+
- (file-notify-rm-watch auto-revert-notify-watch-descriptor)
- (file-notify--wait-for-events
- timeout (null auto-revert-use-notify))
- (should-not auto-revert-use-notify)
- (should-not auto-revert-notify-watch-descriptor)
-
- ;; Modify file. We wait for two seconds, in order to have
- ;; another timestamp. One second seems to be too short.
- (with-current-buffer (get-buffer-create "*Messages*")
- (narrow-to-region (point-max) (point-max)))
- (sleep-for 2)
- (write-region
- "foo bla" nil file-notify--test-tmpfile nil 'no-message)
-
- ;; Check, that the buffer has been reverted.
- (with-current-buffer (get-buffer-create "*Messages*")
++ ;; Check attribute change. Does not work for cygwin.
++ (unless (eq system-type 'cygwin)
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
++ (write-region
++ "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(attribute-change) 'file-notify--test-event-handler)))
++ (file-notify--test-with-events
++ (cond
++ ;; w32notify does not distinguish between `changed' and
++ ;; `attribute-changed'.
++ ((string-equal (file-notify--test-library) "w32notify")
++ '(changed changed changed changed))
++ ;; For kqueue and in the remote case, `write-region'
++ ;; raises also an `attribute-changed' event.
++ ((or (string-equal (file-notify--test-library) "kqueue")
++ (file-remote-p temporary-file-directory))
++ '(attribute-changed attribute-changed attribute-changed))
++ (t '(attribute-changed attribute-changed)))
++ (read-event nil nil 0.1)
++ (write-region
++ "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
++ (set-file-modes file-notify--test-tmpfile 000)
++ (read-event nil nil 0.1)
++ (set-file-times file-notify--test-tmpfile '(0 0))
++ (read-event nil nil 0.1)
++ (delete-file file-notify--test-tmpfile))
+ ;; `file-notify-rm-watch' fires the `stopped' event. Suppress it.
+ (let (file-notify--test-events)
+ (file-notify-rm-watch file-notify--test-desc)))
+
+ ;; Check the global sequence again just to make sure that
+ ;; `file-notify--test-events' has been set correctly.
+ (should (equal (mapcar #'cadr file-notify--test-events)
+ file-notify--test-expected-events))
+ (should file-notify--test-results)
+ (dolist (result file-notify--test-results)
+ (when (ert-test-failed-p result)
+ (ert-fail
+ (cadr (ert-test-result-with-condition-condition result))))))
+
+ ;; Cleanup.
+ (file-notify--test-cleanup)))
+
+ (file-notify--deftest-remote file-notify-test02-events
+ "Check file creation/change/removal notifications for remote files.")
+
+ (require 'autorevert)
+ (setq auto-revert-notify-exclude-dir-regexp "nothing-to-be-excluded"
+ auto-revert-remote-files t
+ auto-revert-stop-on-user-input nil)
+
+ (ert-deftest file-notify-test03-autorevert ()
+ "Check autorevert via file notification."
+ (skip-unless (file-notify--test-local-enabled))
+ ;; `auto-revert-buffers' runs every 5". And we must wait, until the
+ ;; file has been reverted.
+ (let ((timeout (if (file-remote-p temporary-file-directory) 60 10))
+ buf)
+ (unwind-protect
+ (progn
+ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
+
+ (write-region
+ "any text" nil file-notify--test-tmpfile nil 'no-message)
+ (setq buf (find-file-noselect file-notify--test-tmpfile))
+ (with-current-buffer buf
+ (should (string-equal (buffer-string) "any text"))
+ ;; `buffer-stale--default-function' checks for
+ ;; `verify-visited-file-modtime'. We must ensure that it
+ ;; returns nil.
+ (sleep-for 1)
+ (auto-revert-mode 1)
+
+ ;; `auto-revert-buffers' runs every 5".
+ (with-timeout (timeout (ignore))
+ (while (null auto-revert-notify-watch-descriptor)
+ (sleep-for 1)))
+
+ ;; Check, that file notification has been used.
+ (should auto-revert-mode)
+ (should auto-revert-use-notify)
+ (should auto-revert-notify-watch-descriptor)
+
+ ;; Modify file. We wait for a second, in order to have
+ ;; another timestamp.
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (narrow-to-region (point-max) (point-max)))
+ (sleep-for 1)
+ (write-region
+ "another text" nil file-notify--test-tmpfile nil 'no-message)
+
+ ;; Check, that the buffer has been reverted.
+ (with-current-buffer (get-buffer-create "*Messages*")
+ (file-notify--wait-for-events
+ timeout
+ (string-match
+ (format-message "Reverting buffer `%s'." (buffer-name buf))
+ (buffer-string))))
+ (should (string-match "another text" (buffer-string)))
+
+ ;; Stop file notification. Autorevert shall still work via polling.
- timeout
- (string-match
- (format-message "Reverting buffer `%s'." (buffer-name buf))
- (buffer-string))))
- (should (string-match "foo bla" (buffer-string)))))
++ ;; It doesn't work for `w32notify'.
++ (unless (string-equal (file-notify--test-library) "w32notify")
++ (file-notify-rm-watch auto-revert-notify-watch-descriptor)
+ (file-notify--wait-for-events
- ;; Under cygwin there are so bad timings that it doesn't make sense to test.
- (skip-unless (not (eq system-type 'cygwin)))
++ timeout (null auto-revert-use-notify))
++ (should-not auto-revert-use-notify)
++ (should-not auto-revert-notify-watch-descriptor)
++
++ ;; Modify file. We wait for two seconds, in order to
++ ;; have another timestamp. One second seems to be too
++ ;; short.
++ (with-current-buffer (get-buffer-create "*Messages*")
++ (narrow-to-region (point-max) (point-max)))
++ (sleep-for 2)
++ (write-region
++ "foo bla" nil file-notify--test-tmpfile nil 'no-message)
++
++ ;; Check, that the buffer has been reverted.
++ (with-current-buffer (get-buffer-create "*Messages*")
++ (file-notify--wait-for-events
++ timeout
++ (string-match
++ (format-message "Reverting buffer `%s'." (buffer-name buf))
++ (buffer-string))))
++ (should (string-match "foo bla" (buffer-string))))))
+
+ ;; Cleanup.
+ (with-current-buffer "*Messages*" (widen))
+ (ignore-errors (kill-buffer buf))
+ (file-notify--test-cleanup))))
+
+ (file-notify--deftest-remote file-notify-test03-autorevert
+ "Check autorevert via file notification for remote files.")
+
+ (ert-deftest file-notify-test04-file-validity ()
+ "Check `file-notify-valid-p' for files."
+ (skip-unless (file-notify--test-local-enabled))
- (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
- file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) #'file-notify--test-event-handler))
- (file-notify--test-with-events '(created changed deleted)
+
+ (unwind-protect
+ (progn
- "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
++ (write-region "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(change) #'file-notify--test-event-handler)))
++ (should (file-notify-valid-p file-notify--test-desc))
++ ;; After calling `file-notify-rm-watch', the descriptor is not
++ ;; valid anymore.
++ (file-notify-rm-watch file-notify--test-desc)
++ (should-not (file-notify-valid-p file-notify--test-desc))
++ (delete-file file-notify--test-tmpfile))
++
++ ;; Cleanup.
++ (file-notify--test-cleanup))
++
++ (unwind-protect
++ (progn
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
++ (write-region "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(change) #'file-notify--test-event-handler)))
++ (file-notify--test-with-events
++ (cond
++ ;; cygwin recognizes only `deleted' and `stopped' events.
++ ((eq system-type 'cygwin)
++ '(deleted stopped))
++ ;; inotify, kqueueg and gfilenotify raise just one
++ ;; `changed' event, the other backends show us two of
++ ;; them.
++ ((or (string-equal "inotify" (file-notify--test-library))
++ (string-equal "kqueue" (file-notify--test-library))
++ (string-equal "gfilenotify" (file-notify--test-library)))
++ '(changed deleted stopped))
++ (t '(changed changed deleted stopped)))
++ (read-event nil nil 0.1)
+ (should (file-notify-valid-p file-notify--test-desc))
+ (write-region
- ;; After deleting the file, the descriptor is still valid.
- (should (file-notify-valid-p file-notify--test-desc))
- ;; After removing the watch, the descriptor must not be valid
- ;; anymore.
- (file-notify-rm-watch file-notify--test-desc)
- (should-not (file-notify-valid-p file-notify--test-desc)))
++ "another text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
+ (delete-file file-notify--test-tmpfile))
- ;; The batch-mode operation of w32notify is fragile (there's no
- ;; input threads to send the message to).
- ;(unless (and noninteractive (eq file-notify--library 'w32notify))
- (unless (eq file-notify--library 'w32notify)
- (let ((temporary-file-directory
++ ;; After deleting the file, the descriptor is not valid anymore.
++ (should-not (file-notify-valid-p file-notify--test-desc))
++ (file-notify-rm-watch file-notify--test-desc))
+
+ ;; Cleanup.
+ (file-notify--test-cleanup))
+
+ (unwind-protect
- (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
- file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) #'file-notify--test-event-handler))
- (file-notify--test-with-events '(created changed deleted stopped)
- (should (file-notify-valid-p file-notify--test-desc))
- (write-region
- "any text" nil file-notify--test-tmpfile nil 'no-message)
++ ;; w32notify does not send a `stopped' event when deleting a
++ ;; directory. The test does not work, therefore.
++ (unless (string-equal (file-notify--test-library) "w32notify")
++ (let ((temporary-file-directory
+ (make-temp-file "file-notify-test-parent" t)))
- ;; After deleting the parent directory, the descriptor must
- ;; not be valid anymore.
- (should-not (file-notify-valid-p file-notify--test-desc))))
++ (should
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name)
++ file-notify--test-desc
++ (file-notify-add-watch
++ temporary-file-directory
++ '(change) #'file-notify--test-event-handler)))
++ (file-notify--test-with-events
++ (cond
++ ;; cygwin recognizes only `deleted' and `stopped' events.
++ ((eq system-type 'cygwin)
++ '(deleted stopped))
++ ;; There are two `deleted' events, for the file and for
++ ;; the directory. Except for kqueue.
++ ((string-equal (file-notify--test-library) "kqueue")
++ '(created changed deleted stopped))
++ (t '(created changed deleted deleted stopped)))
++ (should (file-notify-valid-p file-notify--test-desc))
++ (read-event nil nil 0.1)
++ (write-region
++ "any text" nil file-notify--test-tmpfile nil 'no-message)
++ (read-event nil nil 0.1)
+ (delete-directory temporary-file-directory t))
- (setq file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) #'file-notify--test-event-handler))
++ ;; After deleting the parent directory, the descriptor must
++ ;; not be valid anymore.
++ (should-not (file-notify-valid-p file-notify--test-desc))))
+
+ ;; Cleanup.
+ (file-notify--test-cleanup)))
+
+ (file-notify--deftest-remote file-notify-test04-file-validity
+ "Check `file-notify-valid-p' via file notification for remote files.")
+
+ (ert-deftest file-notify-test05-dir-validity ()
+ "Check `file-notify-valid-p' for directories."
+ (skip-unless (file-notify--test-local-enabled))
+
+ (unwind-protect
+ (progn
+ (setq file-notify--test-tmpfile
+ (file-name-as-directory (file-notify--test-make-temp-name)))
+ (make-directory file-notify--test-tmpfile)
- (unless (and noninteractive (eq file-notify--library 'w32notify))
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(change) #'file-notify--test-event-handler)))
+ (should (file-notify-valid-p file-notify--test-desc))
+ ;; After removing the watch, the descriptor must not be valid
+ ;; anymore.
+ (file-notify-rm-watch file-notify--test-desc)
+ (file-notify--wait-for-events
+ (file-notify--test-timeout)
+ (not (file-notify-valid-p file-notify--test-desc)))
+ (should-not (file-notify-valid-p file-notify--test-desc)))
+
+ ;; Cleanup.
+ (file-notify--test-cleanup))
+
+ (unwind-protect
+ ;; The batch-mode operation of w32notify is fragile (there's no
+ ;; input threads to send the message to).
- (setq file-notify--test-desc
- (file-notify-add-watch
- file-notify--test-tmpfile
- '(change) #'file-notify--test-event-handler))
++ (unless (and noninteractive
++ (string-equal (file-notify--test-library) "w32notify"))
+ (setq file-notify--test-tmpfile
+ (file-name-as-directory (file-notify--test-make-temp-name)))
+ (make-directory file-notify--test-tmpfile)
- (file-notify--test-timeout)
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(change) #'file-notify--test-event-handler)))
+ (should (file-notify-valid-p file-notify--test-desc))
+ ;; After deleting the directory, the descriptor must not be
+ ;; valid anymore.
+ (delete-directory file-notify--test-tmpfile t)
+ (file-notify--wait-for-events
-;; * Try to handle arriving events under cygwin reliably.
++ (file-notify--test-timeout)
+ (not (file-notify-valid-p file-notify--test-desc)))
+ (should-not (file-notify-valid-p file-notify--test-desc)))
+
+ ;; Cleanup.
+ (file-notify--test-cleanup)))
+
+ (file-notify--deftest-remote file-notify-test05-dir-validity
+ "Check `file-notify-valid-p' via file notification for remote directories.")
+
++(ert-deftest file-notify-test06-many-events ()
++ "Check that events are not dropped."
++ (skip-unless (file-notify--test-local-enabled))
++ ;; Under cygwin events arrive in random order. Impossible to define a test.
++ (skip-unless (not (eq system-type 'cygwin)))
++
++ (setq file-notify--test-tmpfile (file-notify--test-make-temp-name))
++ (make-directory file-notify--test-tmpfile)
++ (should
++ (setq file-notify--test-desc
++ (file-notify-add-watch
++ file-notify--test-tmpfile
++ '(change) 'file-notify--test-event-handler)))
++ (unwind-protect
++ (let ((n 1000)
++ source-file-list target-file-list
++ (default-directory file-notify--test-tmpfile))
++ (dotimes (i n)
++ ;; It matters which direction we rename, at least for
++ ;; kqueue. This backend parses directories in alphabetic
++ ;; order (x%d before y%d). So we rename both directions.
++ (if (zerop (mod i 2))
++ (progn
++ (push (expand-file-name (format "x%d" i)) source-file-list)
++ (push (expand-file-name (format "y%d" i)) target-file-list))
++ (push (expand-file-name (format "y%d" i)) source-file-list)
++ (push (expand-file-name (format "x%d" i)) target-file-list)))
++ (file-notify--test-with-events (make-list (+ n n) 'created)
++ (let ((source-file-list source-file-list)
++ (target-file-list target-file-list))
++ (while (and source-file-list target-file-list)
++ (read-event nil nil 0.1)
++ (write-region "" nil (pop source-file-list) nil 'no-message)
++ (read-event nil nil 0.1)
++ (write-region "" nil (pop target-file-list) nil 'no-message))))
++ (file-notify--test-with-events
++ (cond
++ ;; w32notify fires both `deleted' and `renamed' events.
++ ((string-equal (file-notify--test-library) "w32notify")
++ (let (r)
++ (dotimes (i n r)
++ (setq r (append '(deleted renamed) r)))))
++ (t (make-list n 'renamed)))
++ (let ((source-file-list source-file-list)
++ (target-file-list target-file-list))
++ (while (and source-file-list target-file-list)
++ (rename-file (pop source-file-list) (pop target-file-list) t))))
++ (file-notify--test-with-events (make-list n 'deleted)
++ (dolist (file target-file-list)
++ (delete-file file))))
++ (file-notify--test-cleanup)))
++
++(file-notify--deftest-remote file-notify-test06-many-events
++ "Check that events are not dropped for remote directories.")
++
+ (defun file-notify-test-all (&optional interactive)
+ "Run all tests for \\[file-notify]."
+ (interactive "p")
+ (if interactive
+ (ert-run-tests-interactively "^file-notify-")
+ (ert-run-tests-batch "^file-notify-")))
+
+ ;; TODO:
+
+ ;; * For w32notify, no stopped events arrive when a directory is removed.
++;; * Check, why cygwin recognizes only `deleted' and `stopped' events.
+
+ (provide 'file-notify-tests)
+ ;;; file-notify-tests.el ends here
projects / emacs.git / commitdiff