diff --git a/aries_cloudagent/protocols/issue_credential/v1_0/manager.py b/aries_cloudagent/protocols/issue_credential/v1_0/manager.py index 75fe7895ac..bc11f4995e 100644 --- a/aries_cloudagent/protocols/issue_credential/v1_0/manager.py +++ b/aries_cloudagent/protocols/issue_credential/v1_0/manager.py @@ -884,7 +884,7 @@ async def send_credential_ack( Delete credential exchange record if set to auto-remove. Returns: - Tuple: cred ex record, credential ack message for tracing. + credential ack message for tracing """ credential_ack_message = CredentialAck() diff --git a/aries_cloudagent/protocols/issue_credential/v1_0/routes.py b/aries_cloudagent/protocols/issue_credential/v1_0/routes.py index fd8c7b3f59..abe70b2d96 100644 --- a/aries_cloudagent/protocols/issue_credential/v1_0/routes.py +++ b/aries_cloudagent/protocols/issue_credential/v1_0/routes.py @@ -1183,10 +1183,9 @@ async def credential_exchange_store(request: web.BaseRequest): ) try: # protocol owes an ack - ( - cred_ex_record, - credential_ack_message, - ) = await credential_manager.send_credential_ack(cred_ex_record) + credential_ack_message = await credential_manager.send_credential_ack( + cred_ex_record + ) result = cred_ex_record.serialize() # pick up state done except ( diff --git a/aries_cloudagent/protocols/issue_credential/v1_0/tests/test_routes.py b/aries_cloudagent/protocols/issue_credential/v1_0/tests/test_routes.py index 8744ffe449..0fe2cf8d78 100644 --- a/aries_cloudagent/protocols/issue_credential/v1_0/tests/test_routes.py +++ b/aries_cloudagent/protocols/issue_credential/v1_0/tests/test_routes.py @@ -1187,8 +1187,7 @@ async def test_credential_exchange_store(self): mock_cred_ex_record ) mock_credential_manager.return_value.send_credential_ack.return_value = ( - mock_cred_ex_record, - async_mock.MagicMock(), + async_mock.MagicMock() ) await test_module.credential_exchange_store(self.request) @@ -1224,8 +1223,7 @@ async def test_credential_exchange_store_bad_cred_id_json(self): mock_cred_ex_record ) mock_credential_manager.return_value.send_credential_ack.return_value = ( - mock_cred_ex_record, - async_mock.MagicMock(), + async_mock.MagicMock() ) await test_module.credential_exchange_store(self.request) @@ -1278,8 +1276,7 @@ async def test_credential_exchange_store_no_conn_record(self): mock_cred_ex ) mock_credential_manager.return_value.send_credential_ack.return_value = ( - mock_cred_ex, - async_mock.MagicMock(), + async_mock.MagicMock() ) with self.assertRaises(test_module.web.HTTPBadRequest): @@ -1339,10 +1336,7 @@ async def test_credential_exchange_store_x(self): return_value=mock_cred_ex_record ), send_credential_ack=async_mock.CoroutineMock( - return_value=( - mock_cred_ex_record, - async_mock.MagicMock(), - ) + return_value=async_mock.MagicMock() ), )