From a676b44d41121e09c2a3995387ee53228e5b8ede Mon Sep 17 00:00:00 2001 From: JayantiTA Date: Sun, 29 Oct 2023 20:26:55 +0700 Subject: [PATCH 1/2] fix: fix keisan unit tests --- test/angle/euler_test.cpp | 8 ++++---- test/angle/quaternion_test.cpp | 6 +++--- test/matrix/matrix_inverse_test.cpp | 4 +--- 3 files changed, 8 insertions(+), 10 deletions(-) diff --git a/test/angle/euler_test.cpp b/test/angle/euler_test.cpp index 730ccc5..c231b25 100644 --- a/test/angle/euler_test.cpp +++ b/test/angle/euler_test.cpp @@ -54,9 +54,9 @@ TEST(EulerTest, AssignmentConstructor) { \ ksn::Euler 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_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 index 7df6c17..08f4d9d 100644 --- a/test/angle/quaternion_test.cpp +++ b/test/angle/quaternion_test.cpp @@ -52,9 +52,9 @@ TEST(QuaternionTest, AssignmentConstructor) { \ ksn::Quaternion 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_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 index dab7b33..51f843a 100644 --- a/test/matrix/matrix_inverse_test.cpp +++ b/test/matrix/matrix_inverse_test.cpp @@ -51,9 +51,7 @@ class MatrixInverseTest : public ::testing::TestWithParam Date: Mon, 30 Oct 2023 21:50:57 +0700 Subject: [PATCH 2/2] ci: update workflow --- .github/workflows/build-and-test.yml | 2 ++ .github/workflows/deploy-debian-nightly.yml | 1 + .github/workflows/deploy-debian-stable.yml | 2 ++ 3 files changed, 5 insertions(+) diff --git a/.github/workflows/build-and-test.yml b/.github/workflows/build-and-test.yml index 73c6a83..1f95ec3 100644 --- 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 index 42e0608..8915fba 100644 --- 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 index a30194a..af29b8a 100644 --- 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