Merge pull request #23 from tritonuas/feat/test-gh-actions

github action tests

.github/workflows/test.yml

@@ -0,0 +1,31 @@
+name: Tests
+
+on:
+ push:
+ branches:
+ - main
+ pull_request:
+
+jobs:
+ test:
+ name: C++ Tests 
+ runs-on: ubuntu-latest
+
+ steps:
+ - name: Install Cmake
+ run: sudo apt install cmake
+
+ - name: Checkout Repo
+ uses: actions/checkout@v2
+ with:
+ submodules: true
+ token: ${{ secrets.SSH_TOKEN }}
+
+ - name: Set up Cmake
+ run: cmake .
+
+ - name: Run tests
+ run: make test 
+
+
+

CMakeLists.txt

@@ -38,27 +38,6 @@ target_link_libraries(obcpp
 # =============================
 # Unit tests
 
+add_subdirectory(tests)
 add_subdirectory(deps/google-test)
-
-# enable CTest testing
-enable_testing()
-
-# Add a testing executable
-add_executable(hello_test tests/hello_test.cpp)
-add_executable(hello_test2 tests/hello_test2.cpp)
-
-target_link_libraries(
- hello_test
- GTest::GTest 
- GTest::Main
-)
-
-target_link_libraries(
- hello_test2
- GTest::GTest 
- GTest::Main
-)
-
-add_test(test_all ./bin/hello_test)
-add_test(test_all ./bin/hello_test2)
-# =============================
+# =============================

README.md

@@ -109,7 +109,7 @@ Now that everything is installed, here is the process to build and run the appli
 
 11. Verify that the testing framework is set up correctly
  ```sh
- ctest .
+ make test
  ```
 
  If you receive output about certain tests passing/failing, then you are good. Ideally the main branch (which should be what
@@ -130,4 +130,4 @@ Each module has its own folder in `include/` and `src/`. Currently all of the he
 
 ## Style
 
-We are still deciding on a linting/style pipeline. When this is decided, this section should be updated.
+We are still deciding on a linting/style pipeline. When this is decided, this section should be updated.

tests/CMakeLists.txt

@@ -0,0 +1,21 @@
+set(BINARY ${CMAKE_PROJECT_NAME}_tests)
+
+file(GLOB_RECURSE TEST_SOURCES LIST_DIRECTORIES false *.h *.cpp)
+
+set(SOURCES ${TEST_SOURCES})
+
+add_executable(${BINARY} ${TEST_SOURCES})
+
+target_link_libraries(${BINARY} PUBLIC gtest_main)
+
+add_test(NAME ${BINARY} COMMAND ${BINARY})
+
+# setup make target (make test)
+add_custom_target(test)
+add_custom_command(
+ TARGET test
+ COMMAND "${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/${BINARY}"
+ ARGS "--output-on-failure"
+)
+add_dependencies(test ${BINARY})
+

tests/hello_test.cpp

@@ -1,7 +1,7 @@
 #include <gtest/gtest.h>
 
 // Demonstrate some basic assertions.
-TEST(HelloTest, BasicAssertions) {
+TEST(HelloTest, HelloTest1) {
  // Expect two strings not to be equal.
  EXPECT_STRNE("hello", "world");
  // Expect equality.

tests/hello_test2.cpp

@@ -1,11 +1,10 @@
 #include <gtest/gtest.h>
 
 // Demonstrate some basic assertions.
-TEST(HelloTest, BasicAssertions) {
+TEST(HelloTest, HelloTest2) {
  // Expect two strings not to be equal.
  EXPECT_STRNE("hello", "world");
  // Expect equality.
  EXPECT_EQ(7 * 6, 42);
- EXPECT_LT(3, 8);
 }