Add RpcV2ErrorMarshaller tests

Author: sbaluja

src/aws-cpp-sdk-core/source/client/AWSErrorMarshaller.cpp

@@ -497,6 +497,8 @@ AWSError<CoreErrors> RpcV2ErrorMarshaller::Marshall(const Aws::Http::HttpRespons
     error = AWSError<CoreErrors>(CoreErrors::UNKNOWN, "", "Failed to parse CBOR error payload", isRetryable);
   }
 
+  MarshallError(error, httpResponse);
+
   error.SetRequestId(httpResponse.HasHeader(REQUEST_ID_HEADER) ? httpResponse.GetHeader(REQUEST_ID_HEADER) : "");
   return error;
 }

src/aws-cpp-sdk-core/source/utils/cbor/CborValue.cpp

@@ -8,6 +8,8 @@
 using namespace Aws::Utils::Cbor;
 using namespace Aws::Crt::Cbor;
 
+static const char AWS_CBOR_VALUE_LOG_TAG[] = "CborValue";
+
 CborValue::CborValue() : m_decoder(nullptr)
 {
 }
@@ -18,29 +20,29 @@ CborValue::CborValue(Aws::IStream& istream) : m_decoder(nullptr)
     ss << istream.rdbuf();
     m_body = ss.str();
     auto cursor = Aws::Crt::ByteCursorFromArray(reinterpret_cast<const uint8_t*>(m_body.c_str()), m_body.length());
-    m_decoder = std::make_shared<CborDecoder>(cursor);
+    m_decoder = Aws::MakeShared<CborDecoder>(AWS_CBOR_VALUE_LOG_TAG, cursor);
 }
 
 CborValue::CborValue(const Aws::String& value) : m_decoder(nullptr)
 {
   m_body = value;
   auto cursor = Aws::Crt::ByteCursorFromArray(reinterpret_cast<const uint8_t*>(m_body.c_str()), m_body.length());
-  m_decoder = std::make_shared<CborDecoder>(cursor);
+  m_decoder = Aws::MakeShared<CborDecoder>(AWS_CBOR_VALUE_LOG_TAG, cursor);
 }
 
 CborValue::CborValue(const CborValue& other) :
     m_body(other.m_body)
 {
     auto cursor = Aws::Crt::ByteCursorFromArray(reinterpret_cast<const uint8_t*>(m_body.c_str()), m_body.length());
-    m_decoder = std::make_shared<CborDecoder>(cursor);
+    m_decoder = Aws::MakeShared<CborDecoder>(AWS_CBOR_VALUE_LOG_TAG, cursor);
 }
 
 CborValue& CborValue::operator=(const CborValue& other)
 {
     if (this != &other) {
         m_body = other.m_body;
         auto cursor = Aws::Crt::ByteCursorFromArray(reinterpret_cast<const uint8_t*>(m_body.c_str()), m_body.length());
-        m_decoder = std::make_shared<CborDecoder>(cursor);
+        m_decoder = Aws::MakeShared<CborDecoder>(AWS_CBOR_VALUE_LOG_TAG, cursor);
     }
     return *this;
 }
@@ -49,15 +51,15 @@ CborValue::CborValue(CborValue&& value) noexcept :
     m_body(std::move(value.m_body))
 {
     auto cursor = Aws::Crt::ByteCursorFromArray(reinterpret_cast<const uint8_t*>(m_body.c_str()), m_body.length());
-    m_decoder = std::make_shared<CborDecoder>(cursor);
+    m_decoder = Aws::MakeShared<CborDecoder>(AWS_CBOR_VALUE_LOG_TAG, cursor);
 }
 
 CborValue& CborValue::operator=(CborValue&& other) noexcept
 {
     if (this != &other) {
         m_body = std::move(other.m_body);
         auto cursor = Aws::Crt::ByteCursorFromArray(reinterpret_cast<const uint8_t*>(m_body.c_str()), m_body.length());
-        m_decoder = std::make_shared<CborDecoder>(cursor);
+        m_decoder = Aws::MakeShared<CborDecoder>(AWS_CBOR_VALUE_LOG_TAG, cursor);
     }
     return *this;
 }