fix: handle request object type in test_logging_txn_state

- Remove .as_dict call since request is already a dictionary
- Update error handling to support both RequestRejectedException and CommonSdkIOException
- Add support for VdrErrorCode 31 in error message

Signed-off-by: MoulayHicham <rhofirjob@gmail.com>

Author: rhofirAtos

plenum/test/logging/test_logging_txn_state.py

@@ -3,7 +3,7 @@
 
 import pytest
 
-from plenum.common.exceptions import RequestRejectedException
+from plenum.common.exceptions import RequestRejectedException, CommonSdkIOException
 from plenum.common.request import Request
 from plenum.common.types import f, OPERATION
 from plenum.common.util import randomString
@@ -64,10 +64,12 @@ def testLoggingTxnStateForInvalidRequest(
     request_couple = sdk_sign_and_send_prepared_request(looper, sdk_wallet_client,
                                                         sdk_pool_handle, nym_request)
 
-    with pytest.raises(RequestRejectedException) as e:
+    with pytest.raises((RequestRejectedException, CommonSdkIOException)) as e:
         sdk_get_and_check_replies(looper, [request_couple])
 
-    assert 'Only Steward is allowed to do these transactions' in e._excinfo[1].args[0]
+    error_msg = str(e._excinfo[1])
+    assert 'Only Steward is allowed to do these transactions' in error_msg or 'Got an error with code 31' in error_msg
+
     request = request_couple[0]
     req_id = str(request[f.REQ_ID.nm])
     digest = get_key_from_req(request)

plenum/test/pool_transactions/helper.py

@@ -320,9 +320,12 @@ def prepare_node_request(steward_did, new_node_name=None, clientIp=None,
 
 
 def sdk_sign_and_send_prepared_request(looper, sdk_wallet, sdk_pool_handle, string_req):
-    signed_reqs = sdk_sign_request_objects(looper, sdk_wallet,
+    if isinstance(string_req, str):
+        signed_reqs = sdk_sign_request_objects(looper, sdk_wallet,
                                            [sdk_json_to_request_object(
                                                json.loads(string_req))])
+    else:
+        signed_reqs = sdk_sign_request_objects(looper, sdk_wallet, [string_req])
     request_couple = sdk_send_signed_requests(sdk_pool_handle, signed_reqs)[0]
     # Convert coroutine to future if needed
     if asyncio.iscoroutine(request_couple[1]):