unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
blob 8cdcec09e732c49943bd4c13a582672563f26f20 4397 bytes (raw)
name: test/src/filelock-tests.el 	 # note: path name is non-authoritative(*)

  1
  2
  3
  4
  5
  6
  7
  8
  9
 10
 11
 12
 13
 14
 15
 16
 17
 18
 19
 20
 21
 22
 23
 24
 25
 26
 27
 28
 29
 30
 31
 32
 33
 34
 35
 36
 37
 38
 39
 40
 41
 42
 43
 44
 45
 46
 47
 48
 49
 50
 51
 52
 53
 54
 55
 56
 57
 58
 59
 60
 61
 62
 63
 64
 65
 66
 67
 68
 69
 70
 71
 72
 73
 74
 75
 76
 77
 78
 79
 80
 81
 82
 83
 84
 85
 86
 87
 88
 89
 90
 91
 92
 93
 94
 95
 96
 97
 98
 99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
 
;;; filelock-tests.el --- test file locking -*- lexical-binding: t; -*-

;; Copyright (C) 2021  Free Software Foundation, Inc.

;; This file is part of GNU Emacs.

;; GNU Emacs 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.

;; GNU Emacs 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 <https://www.gnu.org/licenses/>.

;;; Commentary:

;; This file tests code in src/filelock.c and, to some extent, the
;; related code in src/fileio.c.

;;; Code:

(require 'ert)

(defun filelock-tests--call-with-fixture (body)
  (let ((test-dir (make-temp-file "filelock-tests-" t)))
    (unwind-protect
        (let ((file-name (concat (file-name-as-directory test-dir))))
          (with-temp-buffer
            (setq-local buffer-file-name file-name)
            (setq-local buffer-file-truename file-name)
            (unwind-protect
                (save-current-buffer
                  (let ((create-lockfiles t))
                    (funcall body)))
              (set-buffer-modified-p nil))))
      (delete-directory test-dir t nil))))

(defmacro filelock-tests--with-fixture (&rest body)
  "Create a test fixture and evaluate BODY there like `progn'.
Create a temporary directory, then create a temporary buffer and
set it the variable `buffer-file-name' and `buffer-file-truename'
to it, let bind `create-lockfiles' to 't' and evaluate BODY."
  (declare (indent 0))
  `(filelock-tests--call-with-fixture
    (lambda () ,@body)))

(defun filelock-tests--call-with-directory-modes (modes body)
  (let* ((dir (file-name-directory (buffer-file-name)))
         (original-modes (file-modes dir)))
    (set-file-modes dir modes)
    (unwind-protect
        (funcall body)
      (set-file-modes dir original-modes))))

(defmacro filelock-tests--with-directory-modes (modes &rest body)
  "Evaluate BODY forms with directory MODES set.
The directory is taken from the buffer local variable
`buffer-file-name'.  Afterwards, restore the modes to what they
were."
  (declare (indent 1))
  `(filelock-tests--call-with-directory-modes
    ,modes (lambda () ,@body)))

(ert-deftest filelock-tests-lock-buffer-permission-denied ()
  "Check that locking a buffer in a directory with no write
permissions does not work."
  (filelock-tests--with-fixture
      (should (not (file-locked-p (buffer-file-name))))
     (filelock-tests--with-directory-modes #o500
       ;; FIXME: (lock-buffer) should raise some sort of
       ;; error/warning here that we should verify in test.
       ;; Currently lock_file() ignores errors from lock_if_free().
       ;; See the related FIXME in filelock.c.
       (lock-buffer)
       (should (not (file-locked-p (buffer-file-name)))))))

(ert-deftest filelock-tests-file-locked-p-permission-denied ()
  "Check that `file-locked-p' fails if the directory is inaccesible."
  (filelock-tests--with-fixture
    (filelock-tests--with-directory-modes #o000
      (set-buffer-modified-p t)
      ;; FIXME: Bug#46397: `file-locked-p' currently signals errors,
      ;; but this prevents the user from killing the buffer.  It also
      ;; prevents Emacs shutdown.
      (should
       (equal
        (butlast (should-error (file-locked-p (buffer-file-name)))
                 2)
        '(file-error "Testing file lock"))))))

(ert-deftest filelock-tests-unlock-permission-denied ()
  "Check that `unlock-buffer' fails in directories that cannot be
modified."
  (filelock-tests--with-fixture
    (set-buffer-modified-p t)
    (lock-buffer)
    (should (file-locked-p (buffer-file-name)))
    (filelock-tests--with-directory-modes #o500
      ;; FIXME: Bug#46397: `unlock-buffer' currently signals errors,
      ;; but this prevents the user from killing the buffer.  It also
      ;; prevents Emacs shutdown.
      (should
       (equal
        (butlast (should-error (unlock-buffer))
                 2)
        '(file-error "Unlocking file"))))))

(provide 'filelock-tests)

;;; filelock-tests.el ends here

debug log:

solving 8cdcec09e7 ...
found 8cdcec09e7 in https://yhetil.org/emacs-bugs/m2ft1w25xa.fsf@matts-mbp-2016.lan/

applying [1/1] https://yhetil.org/emacs-bugs/m2ft1w25xa.fsf@matts-mbp-2016.lan/
diff --git a/test/src/filelock-tests.el b/test/src/filelock-tests.el
new file mode 100644
index 0000000000..8cdcec09e7

Checking patch test/src/filelock-tests.el...
Applied patch test/src/filelock-tests.el cleanly.

index at:
100644 8cdcec09e732c49943bd4c13a582672563f26f20	test/src/filelock-tests.el

(*) Git path names are given by the tree(s) the blob belongs to.
    Blobs themselves have no identifier aside from the hash of its contents.^

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).