diff options
| author | Barry Warsaw | 2016-01-04 08:24:12 -0500 |
|---|---|---|
| committer | Barry Warsaw | 2016-01-04 08:24:12 -0500 |
| commit | e93affa947002959138cf8a45f33bea5fa06d3f7 (patch) | |
| tree | 27b818860238de5a5d4d8aa18b7ac5c151551f4d /src/mailman/rest/sub_moderation.py | |
| parent | ff31bf58804fc984c694f6acfaa302042bc85d89 (diff) | |
| download | mailman-e93affa947002959138cf8a45f33bea5fa06d3f7.tar.gz mailman-e93affa947002959138cf8a45f33bea5fa06d3f7.tar.zst mailman-e93affa947002959138cf8a45f33bea5fa06d3f7.zip | |
More coverage.
Diffstat (limited to 'src/mailman/rest/sub_moderation.py')
| -rw-r--r-- | src/mailman/rest/sub_moderation.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/mailman/rest/sub_moderation.py b/src/mailman/rest/sub_moderation.py index 92ee7ff10..b0ffdc8c1 100644 --- a/src/mailman/rest/sub_moderation.py +++ b/src/mailman/rest/sub_moderation.py @@ -79,7 +79,7 @@ class IndividualRequest(_ModerationBase): bad_request(response, str(error)) return action = arguments['action'] - if action is Action.defer: + if action in (Action.defer, Action.hold): # At least see if the token is in the database. pendable = self._pendings.confirm(self._token, expunge=False) if pendable is None: @@ -100,7 +100,8 @@ class IndividualRequest(_ModerationBase): not_found(response) else: no_content(response) - elif action is Action.reject: + else: + assert action is Action.reject, action # Like discard but sends a rejection notice to the user. pendable = self._pendings.confirm(self._token, expunge=True) if pendable is None: |