Fix AllParameters test after CSA API change

tests/executable_src/testAllParameters.cc

@@ -39,13 +39,14 @@ BOOST_FIXTURE_TEST_CASE(testControlSystemVariables, Fixture_t) {
   auto pvManager = testFacility.getPvManager();
 
   for(auto& params : descriptors) {
+    std::cout << "Testing parameter " << params.pvName << std::endl;
     if(not params.readGroup.empty()) readGroups.insert(params.readGroup);
     if(not params.writeGroup.empty()) writeGroups.insert(params.writeGroup);
     if(not params.writeToFlashGroup.empty()) writeToFlashGroups.insert(params.writeToFlashGroup);
 
     // This variable has to exist
-    BOOST_CHECK_NO_THROW(pvManager->getProcessVariable("/TecModule/" + params.pvName + "_rd"));
     auto pv = pvManager->getProcessVariable("/TecModule/" + params.pvName + "_rd");
+    BOOST_CHECK(pv != nullptr);
     BOOST_CHECK(pv->isReadOnly());
     if(params.valueType == ValueType::Int)
       BOOST_CHECK_MESSAGE(pv->getValueType() == typeid(int), "Check if _rd valueType is integer");
@@ -56,18 +57,18 @@ BOOST_FIXTURE_TEST_CASE(testControlSystemVariables, Fixture_t) {
 
 
     // if this variable exists, it has to be
-    try {
-      pv = pvManager->getProcessVariable("/TecModule/" + params.pvName + "_enaPeriodicRead");
+    pv = pvManager->getProcessVariable("/TecModule/" + params.pvName + "_enaPeriodicRead");
+    if (pv != nullptr) {
       BOOST_CHECK(pv->isWriteable());
       BOOST_CHECK_MESSAGE(pv->getValueType() == typeid(int32_t), "Check if _enaPeriodicRead valueType is int32_t");
       BOOST_CHECK(params.direction == Direction::Read);
-    } catch(ChimeraTK::logic_error&) {
+    } else {
       // If this variable does not exist, the parameter has to be read-only
       BOOST_CHECK(params.direction == Direction::ReadWrite);
     }
 
-    try {
-      pv = pvManager->getProcessVariable("/TecModule/" + params.pvName + "_wr");
+    pv = pvManager->getProcessVariable("/TecModule/" + params.pvName + "_wr");
+    if (pv != nullptr){
       BOOST_CHECK(params.direction == Direction::ReadWrite);
       BOOST_CHECK(pv->isWriteable());
       if(params.valueType == ValueType::Int)
@@ -77,7 +78,7 @@ BOOST_FIXTURE_TEST_CASE(testControlSystemVariables, Fixture_t) {
       else
         BOOST_ASSERT(false);
     }
-    catch(ChimeraTK::logic_error&) {
+    else {
       // If this variable does not exist, the parameter has to be read-only
       BOOST_CHECK(params.direction == Direction::Read);
     }
@@ -86,8 +87,8 @@ BOOST_FIXTURE_TEST_CASE(testControlSystemVariables, Fixture_t) {
   // Check if the group accessors exist
   for(auto& group : {readGroups, writeGroups, writeToFlashGroups}) {
     for(auto& entry : group) {
-      BOOST_REQUIRE_NO_THROW(pvManager->getProcessVariable("/TecModule/" + entry + "_wr"));
       auto pv = pvManager->getProcessVariable("/TecModule/" + entry + "_wr");
+      BOOST_REQUIRE(pv != nullptr);
       BOOST_CHECK(pv->isWriteable());
     }
   }