Add test for 'txn.user = None'.

Restores 100% coverage.
zopefoundation · Mar 16, 2017 · f8d611a · f8d611a
1 parent 2bb7d05
commit f8d611a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/transaction/_transaction.py b/transaction/_transaction.py
@@ -138,6 +138,8 @@ def user(self):
     def user(self, v):
         if v is not None:
             self._user = text_or_warn(v)
+        else:
+            self._user = u""
 
     @property
     def description(self):

diff --git a/transaction/tests/test__transaction.py b/transaction/tests/test__transaction.py
@@ -1060,6 +1060,12 @@ def test_setUser_explicit_path(self):
         txn.setUser(u'phreddy', u'/bedrock')
         self.assertEqual(txn.user, u'/bedrock phreddy')
 
+    def test_user_w_none(self):
+        txn = self._makeOne()
+        txn.user = b'phreddy'
+        txn.user = None  # resets to empty text
+        self.assertEqual(txn.user, u'')
+
     def _test_user_non_text(self, user, path, expect, both=False):
         txn = self._makeOne()
         with warnings.catch_warnings(record=True) as w: