[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] externals/detached 274993d9c9 20/38: Remove 'started state
From: |
ELPA Syncer |
Subject: |
[elpa] externals/detached 274993d9c9 20/38: Remove 'started state |
Date: |
Thu, 17 Nov 2022 17:57:55 -0500 (EST) |
branch: externals/detached
commit 274993d9c9b6f535bb4d441794ee2b8fc1e82159
Author: Niklas Eklund <niklas.eklund@posteo.net>
Commit: Niklas Eklund <niklas.eklund@posteo.net>
Remove 'started state
---
detached.el | 15 +++++++--------
test/detached-test.el | 11 ++++-------
2 files changed, 11 insertions(+), 15 deletions(-)
diff --git a/detached.el b/detached.el
index cf3b3857c1..e341c85ec9 100644
--- a/detached.el
+++ b/detached.el
@@ -938,7 +938,6 @@ This function uses the `notifications' library."
"-n"
,(number-to-string detached-session-context-lines)
,log)))))
- (detached--set-session-state session 'started)
(pcase type
('string (string-join command " "))
('list command)
@@ -1113,7 +1112,7 @@ This function uses the `notifications' library."
(defun detached-session-started-p (session)
"Return t if SESSION has been started."
- (not (eq 'unknown (detached-session-state session))))
+ (eq 'active (detached-session-state session)))
(defun detached-session-validated-p (session)
"Return t if SESSION has been validated."
@@ -1314,7 +1313,7 @@ It can take some time for a dtach socket to be created.
Therefore all
sessions are created with state unknown. This function creates a
function to verify that a session was created correctly. If the
session is missing its deleted from the database."
- (setf (alist-get (detached-session-id session)
detached--unvalidated-sessions)
+ (setf (alist-get (detached--session-id session)
detached--unvalidated-sessions)
session)
(run-with-timer detached-dtach-socket-creation-delay
nil
@@ -1444,10 +1443,10 @@ Optionally make the path LOCAL to host."
(seq-length durations)))))
`(:durations ,durations :mean ,mean :std ,std)))
-(defun detached--set-session-state (session state)
- "Update SESSION with STATE."
- (setf (detached--session-state session) state)
- (detached--db-update-entry session))
+;; (defun detached--set-session-state (session state)
+;; "Update SESSION with STATE."
+;; (setf (detached--session-state session) state)
+;; (detached--db-update-entry session))
;;;;; Database
@@ -1489,7 +1488,7 @@ Optionally make the path LOCAL to host."
(defun detached--db-update-entry (session)
"Update SESSION in `detached--sessions' and the database."
- (setf (alist-get (detached-session-id session) detached--sessions) session)
+ (setf (alist-get (detached--session-id session) detached--sessions) session)
(when detached--update-database
(detached--db-update-sessions)))
diff --git a/test/detached-test.el b/test/detached-test.el
index 3a8e5fcb91..10dd888c3a 100644
--- a/test/detached-test.el
+++ b/test/detached-test.el
@@ -68,15 +68,12 @@
;;;;; Session interface
-(ert-deftest detached-test-set-session-state ()
+(ert-deftest detached-test-session-started-p ()
(detached-test--with-temp-database
(cl-letf* ((session (detached-create-session "foo")))
- (should (eq 'unknown (detached-session-state session)))
- (detached--set-session-state session 'started)
- (should (eq 'started
- (detached-session-state
- (detached--db-get-session
- (detached-session-id session))))))))
+ (should (not (detached-session-started-p session)))
+ (setf (detached--session-state session) 'active)
+ (should (detached-session-started-p session)))))
(ert-deftest detached-test-session-status ()
(let ((failed-session (detached--session-create :status `(failure . 128))))
- [elpa] externals/detached fec1a4c7bd 24/38: Update function signatures, (continued)
- [elpa] externals/detached fec1a4c7bd 24/38: Update function signatures, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 473fcc2d43 23/38: Rename detached-session-modes, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 5c212b2609 05/38: Update start-detached-session, ELPA Syncer, 2022/11/17
- [elpa] externals/detached aca448c773 19/38: Revert marks in list when widening, ELPA Syncer, 2022/11/17
- [elpa] externals/detached f70ac98dc5 21/38: Remove dtach--arg dependency, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 18c388bb48 36/38: Update attach shell command, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 88f925ee15 14/38: Add missing string-join, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 4cc2c4e40f 12/38: Add select host session completion function, ELPA Syncer, 2022/11/17
- [elpa] externals/detached d1c3861e8e 16/38: Add public function for session directory, ELPA Syncer, 2022/11/17
- [elpa] externals/detached ca0ee7e465 37/38: Add detach key to session map, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 274993d9c9 20/38: Remove 'started state,
ELPA Syncer <=
- [elpa] externals/detached 51858bceb7 26/38: Begin rework of how to start a session, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 65c661bc9c 27/38: Change detached env to text-mode, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 3dc3651e98 18/38: Add session working directory function, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 88618a9cee 17/38: Add public function for session command, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 4b3609385d 01/38: Start session rewrite, ELPA Syncer, 2022/11/17
- [elpa] externals/detached f0f9728e2c 31/38: Make local predicate private, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 09b37f681c 30/38: Add detached-with-session macro, ELPA Syncer, 2022/11/17
- [elpa] externals/detached a57a728382 22/38: Remove detached-session-mode 'attach, ELPA Syncer, 2022/11/17
- [elpa] externals/detached 4e36d9de69 10/38: Make current session public, ELPA Syncer, 2022/11/17
- [elpa] externals/detached f896cd9777 32/38: Reformat code, ELPA Syncer, 2022/11/17