Merge pull request #65 from ichiro-its/hotfix/fix-keisan-unit-tests

[Sprint 22 / PD-373] - [Hotfix] Fix Keisan Unit Tests
ichiro-its · Nov 3, 2023 · 1b77713 · 1b77713
2 parents 0ce3ac2 + 7dd64d0
commit 1b77713
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 10 deletions.
diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml
@@ -16,3 +16,5 @@ jobs:
 
       - name: Build and test workspace
         uses: ichiro-its/ros2-build-and-test-action@main
+        with:
+          ros2-distro: rolling
diff --git a/.github/workflows/deploy-debian-nightly.yml b/.github/workflows/deploy-debian-nightly.yml
@@ -15,6 +15,7 @@ jobs:
       - name: Build nightly Debian package
         uses: ichiro-its/ros2-build-debian-action@main
         with:
+          ros2-distro: rolling
           unique-version: true
 
       - name: Deploy nightly Debian package to server

diff --git a/.github/workflows/deploy-debian-stable.yml b/.github/workflows/deploy-debian-stable.yml
@@ -14,6 +14,8 @@ jobs:
 
       - name: Build stable Debian package
         uses: ichiro-its/ros2-build-debian-action@main
+        with:
+          ros2-distro: rolling
 
       - name: Deploy stable Debian package to server
         uses: appleboy/scp-action@master

diff --git a/test/angle/euler_test.cpp b/test/angle/euler_test.cpp
@@ -54,9 +54,9 @@ TEST(EulerTest, AssignmentConstructor)
   { \
     ksn::Euler<TYPE> a(SOURCE), b = SOURCE, c; \
     c = SOURCE; \
-    EXPECT_EQ(a, SOURCE); \
-    EXPECT_EQ(b, SOURCE); \
-    EXPECT_EQ(c, SOURCE); \
+    EXPECT_TRUE(a == SOURCE); \
+    EXPECT_TRUE(b == SOURCE); \
+    EXPECT_TRUE(c == SOURCE); \
   }
 
   ksn::Euler<float> float_euler(90_deg, 90_deg, 90_deg);
@@ -96,5 +96,5 @@ TEST(EulerTest, QuaternionConversion)
 
   auto quaternion = euler.quaternion();
 
-  ASSERT_EQ(quaternion.euler(), euler);
+  ASSERT_TRUE(quaternion.euler() == euler);
 }
diff --git a/test/angle/quaternion_test.cpp b/test/angle/quaternion_test.cpp
@@ -52,9 +52,9 @@ TEST(QuaternionTest, AssignmentConstructor)
   { \
     ksn::Quaternion<TYPE> a(SOURCE), b = SOURCE, c; \
     c = SOURCE; \
-    EXPECT_EQ(a, SOURCE); \
-    EXPECT_EQ(b, SOURCE); \
-    EXPECT_EQ(c, SOURCE); \
+    EXPECT_TRUE(a == SOURCE); \
+    EXPECT_TRUE(b == SOURCE); \
+    EXPECT_TRUE(c == SOURCE); \
   }
 
   ksn::Quaternion<float> float_quaternion(1.0f, 1.0f, 1.0f, 1.0f);

diff --git a/test/matrix/matrix_inverse_test.cpp b/test/matrix/matrix_inverse_test.cpp
@@ -51,9 +51,7 @@ class MatrixInverseTest : public ::testing::TestWithParam<matrix_3d_inverse_args
     }
 
     if (((*matrix) == params.inverse) != params.is_valid) {
-      return testing::AssertionFailure() << "the result validation is failed\n" <<
-             "result = " << (*matrix) << "\n" <<
-             "target = " << params.inverse;
+      return testing::AssertionFailure() << "the result validation is failed";
     }
 
     return testing::AssertionSuccess();