From 1fca5703c14a2e3efccba057fd0d6f0180a065b5 Mon Sep 17 00:00:00 2001
From: Athanasius <github@miggy.org>
Date: Fri, 19 Mar 2021 19:19:45 +0000
Subject: [PATCH] Add self.locked asserts as check on _obtain_lock() side
effects
---
tests/journal_lock.py/test_journal_lock.py | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/journal_lock.py/test_journal_lock.py b/tests/journal_lock.py/test_journal_lock.py
index 9c4a274f..f25cb8ca 100644
--- a/tests/journal_lock.py/test_journal_lock.py
+++ b/tests/journal_lock.py/test_journal_lock.py
@@ -116,6 +116,7 @@ class TestJournalLock:
# Check that an actual journaldir is handled correctly.
locked = jlock.obtain_lock()
assert locked == JournalLockResult.LOCKED
+ assert jlock.locked is True
def test_obtain_lock_with_tmpdir_ro(self, mock_journaldir: py_path_local_LocalPath):
"""Test JournalLock.obtain_lock() with read-only tmpdir."""
@@ -189,6 +190,7 @@ class TestJournalLock:
locked = jlock.obtain_lock()
assert locked == JournalLockResult.LOCKED
+ assert jlock.locked is True
# Now attempt to lock again, but only that.
second_attempt = jlock._obtain_lock()
assert second_attempt == JournalLockResult.ALREADY_LOCKED