Merge pull request #262 from Crown-Commercial-Service/add-method-to-u…

…ndo-countersign

Add a new method to the api client to allow it to undo the countersigning

dmapiclient/__init__.py

@@ -1,4 +1,4 @@
-__version__ = '23.1.2'
+__version__ = '23.2.0'
 
 from .errors import APIError, HTTPError, InvalidResponse  # noqa
 from .errors import REQUEST_ERROR_STATUS_CODE, REQUEST_ERROR_MESSAGE  # noqa

dmapiclient/data.py

@@ -1064,6 +1064,13 @@ def update_framework_agreement(self, framework_agreement_id, framework_agreement
             user=user,
         )
 
+    def update_framework_agreement_undo_countersign(self, framework_agreement_id, user=None):
+        return self._post_with_updated_by(
+            "/agreements/{}/undo-countersign".format(framework_agreement_id),
+            data={},
+            user=user,
+        )
+
     def sign_framework_agreement(self, framework_agreement_id, user, signed_agreement_details=None):
         data = {"agreement": {"signedAgreementDetails": signed_agreement_details}} if signed_agreement_details else {}
         return self._post_with_updated_by(

tests/test_data_api_client.py

@@ -2386,6 +2386,19 @@ def test_update_framework_agreement(self, data_client, rmock):
             "updated_by": "[email protected]",
         }
 
+    def test_update_framework_agreement_undo_countersign(self, data_client, rmock):
+        rmock.post(
+            "http://baseurl/agreements/12345/undo-countersign",
+            json={},
+            status_code=200)
+
+        result = data_client.update_framework_agreement_undo_countersign(12345, "[email protected]")
+
+        assert result == {}
+        assert rmock.last_request.json() == {
+            "updated_by": "[email protected]",
+        }
+
     def test_sign_framework_agreement_with_no_signed_agreement_details(self, data_client, rmock):
         rmock.post(
             "http://baseurl/agreements/12345/sign",