From fdab3fbcca606ee99e57ff9d95065ac37628a812 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phane=20Del=20Pino?= <stephane.delpino44@gmail.com>
Date: Tue, 3 Nov 2020 11:46:56 +0100
Subject: [PATCH] Add missing test for Messenger

Also rename test file (getting rid of the 'mpi_' prefix)
---
 CMakeLists.txt                                       | 2 +-
 tests/CMakeLists.txt                                 | 2 +-
 tests/{mpi_test_Messenger.cpp => test_Messenger.cpp} | 7 +++++++
 3 files changed, 9 insertions(+), 2 deletions(-)
 rename tests/{mpi_test_Messenger.cpp => test_Messenger.cpp} (98%)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 0a4883b37..3f62e1b88 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -447,7 +447,7 @@ if("${CMAKE_BUILD_TYPE}" STREQUAL "Coverage")
 
       COMMAND ${FASTCOV} -q --gcov "${GCOV_BIN}"
       --include "${PUGS_SOURCE_DIR}/src"
-      --exclude "${PUGS_SOURCE_DIR}/src/main.cpp" "${PUGS_SOURCE_DIR}/src/utils/BacktraceManager.*" "${PUGS_SOURCE_DIR}/src/utils/FPEManager.*"
+      --exclude "${PUGS_SOURCE_DIR}/src/main.cpp" "${PUGS_SOURCE_DIR}/src/utils/BacktraceManager.*" "${PUGS_SOURCE_DIR}/src/utils/FPEManager.*" "${PUGS_SOURCE_DIR}/src/utils/SignalManager.*"
       --lcov -o coverage.info -n
 
       COMMAND ${LCOV} --gcov "${GCOV_BIN}" --list coverage.info
diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
index f46beee09..6d032f7a6 100644
--- a/tests/CMakeLists.txt
+++ b/tests/CMakeLists.txt
@@ -93,7 +93,7 @@ add_executable (unit_tests
 
 add_executable (mpi_unit_tests
   mpi_test_main.cpp
-  mpi_test_Messenger.cpp
+  test_Messenger.cpp
   )
 
 add_library(test_Pugs_MeshDataBase
diff --git a/tests/mpi_test_Messenger.cpp b/tests/test_Messenger.cpp
similarity index 98%
rename from tests/mpi_test_Messenger.cpp
rename to tests/test_Messenger.cpp
index 19d89851c..022a5c07e 100644
--- a/tests/mpi_test_Messenger.cpp
+++ b/tests/test_Messenger.cpp
@@ -471,4 +471,11 @@ TEST_CASE("Messenger", "[mpi]")
 
     std::remove("barrier_test");
   }
+
+  SECTION("errors")
+  {
+    int argc    = 0;
+    char** argv = nullptr;
+    REQUIRE_THROWS_WITH((parallel::Messenger::create(argc, argv)), "unexpected error: Messenger already created");
+  }
 }
-- 
GitLab