[#24195] Added operators for ReturnCodeValue

Signed-off-by: danipiza <dpizarrogallego@gmail.com>

Author: Danipiza

cpp_utils/include/cpp_utils/ReturnCode.hpp

@@ -69,10 +69,18 @@ class ReturnCode
     bool operator ==(
             const ReturnCode& c) const noexcept;
 
+    CPP_UTILS_DllAPI
+    bool operator ==(
+            const ReturnCodeValue& c) const noexcept;
+
     CPP_UTILS_DllAPI
     bool operator !=(
             const ReturnCode& c) const noexcept;
 
+    CPP_UTILS_DllAPI
+    bool operator !=(
+            const ReturnCodeValue& c) const noexcept;
+
     CPP_UTILS_DllAPI
     bool operator <(
             const ReturnCode& other) const noexcept;
@@ -97,8 +105,28 @@ class ReturnCode
             std::ostream& os,
             const ReturnCode& code);
 
+    CPP_UTILS_DllAPI
+    friend bool operator ==(
+            const ReturnCodeValue& lhs,
+            const ReturnCode& rhs) noexcept;
+
+    CPP_UTILS_DllAPI
+    friend bool operator !=(
+            const ReturnCodeValue& lhs,
+            const ReturnCode& rhs) noexcept;
+
 };
 
+CPP_UTILS_DllAPI
+bool operator ==(
+        const ReturnCode::ReturnCodeValue& lhs,
+        const ReturnCode& rhs) noexcept;
+
+CPP_UTILS_DllAPI
+bool operator !=(
+        const ReturnCode::ReturnCodeValue& lhs,
+        const ReturnCode& rhs) noexcept;
+
 //! \c ReturnCode to stream serializator
 CPP_UTILS_DllAPI
 std::ostream& operator <<(

cpp_utils/src/cpp/ReturnCode.cpp

@@ -78,12 +78,24 @@ bool ReturnCode::operator ==(
     return value_ == c.value_;
 }
 
+bool ReturnCode::operator ==(
+        const ReturnCode::ReturnCodeValue& c) const noexcept
+{
+    return value_ == c;
+}
+
 bool ReturnCode::operator !=(
         const ReturnCode& c) const noexcept
 {
     return value_ != c.value_;
 }
 
+bool ReturnCode::operator !=(
+        const ReturnCode::ReturnCodeValue& c) const noexcept
+{
+    return value_ != c;
+}
+
 bool ReturnCode::operator <(
         const ReturnCode& other) const noexcept
 {
@@ -95,6 +107,20 @@ bool ReturnCode::operator !() const noexcept
     return value_ != ReturnCode::RETCODE_OK;
 }
 
+bool operator ==(
+        const ReturnCode::ReturnCodeValue& lhs,
+        const ReturnCode& rhs) noexcept
+{
+    return rhs == lhs;
+}
+
+bool operator !=(
+        const ReturnCode::ReturnCodeValue& lhs,
+        const ReturnCode& rhs) noexcept
+{
+    return rhs != lhs;
+}
+
 const std::map<ReturnCode::ReturnCodeValue, std::string> ReturnCode::to_string_conversion_ =
 {
     {ReturnCode::RETCODE_OK, "Ok"},

cpp_utils/test/unittest/return_code/CMakeLists.txt

@@ -21,6 +21,8 @@ set(TEST_SOURCES
 
 set(TEST_LIST
         serializator
+        compare_against_return_code_value_rhs
+        compare_against_return_code_value_lhs
     )
 
 set(TEST_EXTRA_LIBRARIES

cpp_utils/test/unittest/return_code/ReturnCodeTest.cpp

@@ -43,6 +43,34 @@ TEST(ReturnCodeTest, serializator)
     }
 }
 
+/**
+ * Test ReturnCode compares directly against ReturnCodeValue values.
+ */
+TEST(ReturnCodeTest, compare_against_return_code_value_rhs)
+{
+    // Right side of the comparation
+
+    ReturnCode from_fastdds(fastdds::dds::RETCODE_NO_DATA);
+
+    ASSERT_TRUE(from_fastdds == ReturnCode::RETCODE_NO_DATA);
+    ASSERT_FALSE(from_fastdds == ReturnCode::RETCODE_ERROR);
+    ASSERT_TRUE(from_fastdds != ReturnCode::RETCODE_ERROR);
+}
+
+/**
+ * Test ReturnCodeValue compares directly against ReturnCode from lhs.
+ */
+TEST(ReturnCodeTest, compare_against_return_code_value_lhs)
+{
+    // Left side of the comparation
+
+    ReturnCode ret(ReturnCode::RETCODE_NOT_ENABLED);
+
+    ASSERT_TRUE(ReturnCode::RETCODE_NOT_ENABLED == ret);
+    ASSERT_TRUE(ReturnCode::RETCODE_OK != ret);
+    ASSERT_FALSE(ReturnCode::RETCODE_OK == ret);
+}
+
 int main(
         int argc,
         char** argv)