iox-eclipse-iceoryx#751 Add unnamed semaphore tests

Signed-off-by: Christian Eltzschig <me@elchris.org>
ApexAI · May 23, 2022 · 55b71ba · 55b71ba
1 parent 12a2afb
commit 55b71ba
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/iceoryx_hoofs/include/iceoryx_hoofs/posix_wrapper/unnamed_semaphore.hpp b/iceoryx_hoofs/include/iceoryx_hoofs/posix_wrapper/unnamed_semaphore.hpp
@@ -34,11 +34,11 @@ class UnnamedSemaphore final : public internal::SemaphoreInterface<UnnamedSemaph
     UnnamedSemaphore& operator=(UnnamedSemaphore&&) noexcept = delete;
     ~UnnamedSemaphore() noexcept;
 
+  private:
     friend class UnnamedSemaphoreBuilder;
     friend class iox::cxx::optional<UnnamedSemaphore>;
     friend class SemaphoreInterface<UnnamedSemaphore>;
 
-  private:
     UnnamedSemaphore() noexcept = default;
     iox_sem_t* getHandle() noexcept;
 

diff --git a/iceoryx_hoofs/test/moduletests/test_posix_unnamed_semaphore.cpp b/iceoryx_hoofs/test/moduletests/test_posix_unnamed_semaphore.cpp
@@ -38,8 +38,19 @@ class UnnamedSemaphoreTest : public Test
     optional<UnnamedSemaphore> sut;
 };
 
-TEST_F(UnnamedSemaphoreTest, asd)
+TEST_F(UnnamedSemaphoreTest, DefaultInitialValueIsZero)
 {
+    ASSERT_FALSE(UnnamedSemaphoreBuilder().create(sut).has_error());
+    EXPECT_THAT(0U, sut->getState().expect("Failed to access semaphore").value);
+}
+
+TEST_F(UnnamedSemaphoreTest, InitialValueIsSetOnCreation)
+{
+    for (uint32_t initialValue = 313; initialValue < 10000; initialValue *= 3)
+    {
+        ASSERT_FALSE(UnnamedSemaphoreBuilder().initialValue(initialValue).create(sut).has_error());
+        EXPECT_THAT(initialValue, sut->getState().expect("Failed to access semaphore").value);
+    }
 }
 } // namespace