emacs-elpa-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[elpa] externals/ebdb 0cb406e 210/350: Expand ebdb-test-save-vars to sav


From: Eric Abrahamsen
Subject: [elpa] externals/ebdb 0cb406e 210/350: Expand ebdb-test-save-vars to save ebdb-db-list as well
Date: Mon, 14 Aug 2017 11:46:38 -0400 (EDT)

branch: externals/ebdb
commit 0cb406e38b0e36cc05fe6db0d2713b8412ff3f49
Author: Eric Abrahamsen <address@hidden>
Commit: Eric Abrahamsen <address@hidden>

    Expand ebdb-test-save-vars to save ebdb-db-list as well
    
    * ebdb-test.el (ebdb-test-save-vars): Save value of ebdb-db-list, too.
      (ebdb-test-with-database): Nest `ebdb-test-save-vars' inside this
      macro.
---
 ebdb-test.el | 34 ++++++++++++++++++----------------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/ebdb-test.el b/ebdb-test.el
index 4687fcf..5467663 100644
--- a/ebdb-test.el
+++ b/ebdb-test.el
@@ -26,35 +26,37 @@
 (require 'ert)
 (require 'ebdb)
 (require 'ebdb-snarf)
-(eval-when-compile
-  (require 'cl-macs))
 
 ;; Testing macros.
 
 (defmacro ebdb-test-with-database (db-and-filename &rest body)
   "Macro providing a temporary database to work with."
   (declare (indent 1) (debug t))
-  `(let ((,(car db-and-filename) (make-instance 'ebdb-db-file
-                                                :file ,(nth 1 db-and-filename)
-                                                :dirty t)))
-     (ebdb-db-save ,(car db-and-filename))
-     (unwind-protect
-        (progn
-          ,@body)
-       (delete-file ,(nth 1 db-and-filename))
-       (setq ebdb-db-list (remove ,(car db-and-filename)
-                                 ebdb-db-list)))))
+  `(ebdb-test-save-vars
+     (let ((,(car db-and-filename) (make-instance 'ebdb-db-file
+                                                 :file ,(nth 1 db-and-filename)
+                                                 :dirty t)))
+       (ebdb-db-save ,(car db-and-filename))
+       (unwind-protect
+          (progn
+            ,@body)
+        (delete-file ,(nth 1 db-and-filename))))))
 
 (defmacro ebdb-test-save-vars (&rest body)
-  "Don't let EBDB tests pollute `ebdb-record-tracker'."
+  "Don't let EBDB tests pollute `ebdb-record-tracker' and
+`ebdb-db-list'."
   (declare (indent 0) (debug t))
-  (let ((old-record-tracker (cl-gensym)))
+  (let ((old-record-tracker (cl-gensym))
+       (old-db-list (cl-gensym)))
     `(let ((,old-record-tracker ebdb-record-tracker)
-          (ebdb-record-tracker nil))
+          (,old-db-list ebdb-db-list)
+          (ebdb-record-tracker nil)
+          (ebdb-db-list nil))
        (unwind-protect
           (progn
             ,@body)
-        (setq ebdb-record-tracker ,old-record-tracker)))))
+        (setq ebdb-record-tracker ,old-record-tracker
+              ebdb-db-list ,old-db-list)))))
 
 ;; Test database file name.
 (defvar ebdb-test-database-1 (make-temp-name



reply via email to

[Prev in Thread] Current Thread [Next in Thread]