Add more tests for workflows.

2 files changed, 17 insertions, 1 deletions

diff --git a/src/mailman_pgp/workflows/base.py b/src/mailman_pgp/workflows/base.py
index ace8f91..1e6b867 100644
--- a/src/mailman_pgp/workflows/base.py
+++ b/src/mailman_pgp/workflows/base.py
@@ -100,7 +100,6 @@ class PubkeyMixin:
             # The workflow was confirmed but we still dont have an address
             # or the pubkey. So resend request and wait.
             self.push('send_key_request')
-            return
         else:
             self.pubkey = pgp_address.key
             if not self.pubkey_confirmed:
diff --git a/src/mailman_pgp/workflows/tests/test_base.py b/src/mailman_pgp/workflows/tests/test_base.py
index c99c6d6..4f39296 100644
--- a/src/mailman_pgp/workflows/tests/test_base.py
+++ b/src/mailman_pgp/workflows/tests/test_base.py
@@ -189,3 +189,20 @@ class TestPubkeyMixin(unittest.TestCase):
         with patch.object(receive_workflow, '_step_do_subscription') as step:
             next(receive_workflow)
         step.assert_called_once_with()
+
+    def test_exisitng_pgp_address(self):
+        workflow = ConfirmSubscriptionPolicy(self.mlist, self.sender,
+                                             pre_verified=True,
+                                             pre_confirmed=True)
+
+        with transaction() as t:
+            pgp_address = PGPAddress(self.sender)
+            pgp_address.key = self.sender_key.pubkey
+            pgp_address.key_confirmed = True
+            t.add(pgp_address)
+
+        workflow.run_thru('pubkey_checks')
+        next(workflow)
+        with patch.object(workflow, '_step_do_subscription') as step:
+            next(workflow)
+        step.assert_called_once_with()