Skip to content

Commit 64d004f

Browse files
authored
two tests translated from boost test to catch2 (#2222)
* two tests translated from boost test to catch2 * if we try out CustomParameters::operator<<, check if it does not throw
1 parent 1003026 commit 64d004f

3 files changed

Lines changed: 108 additions & 119 deletions

File tree

Framework/CMakeLists.txt

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -232,8 +232,10 @@ add_executable(o2-qc-test-core
232232
test/testActivityHelpers.cxx
233233
test/testAggregatorInterface.cxx
234234
test/testAggregatorRunner.cxx
235+
test/testCheck.cxx
235236
test/testCheckInterface.cxx
236237
test/testCheckRunner.cxx
238+
test/testCustomParameters.cxx
237239
test/testInfrastructureGenerator.cxx
238240
test/testMonitorObject.cxx
239241
test/testPolicyManager.cxx
@@ -265,7 +267,6 @@ set(TEST_SRCS
265267
test/testPublisher.cxx
266268
test/testQcInfoLogger.cxx
267269
test/testTaskRunner.cxx
268-
test/testCheck.cxx
269270
test/testObjectsManager.cxx
270271
test/testCcdbDatabase.cxx
271272
test/testCcdbDatabaseExtra.cxx
@@ -279,7 +280,6 @@ set(TEST_SRCS
279280
test/testRepoPathUtils.cxx
280281
test/testQualitiesToTRFCollectionConverter.cxx
281282
test/testUserCodeInterface.cxx
282-
test/testCustomParameters.cxx
283283
)
284284

285285
set(TEST_ARGS
@@ -295,13 +295,11 @@ set(TEST_ARGS
295295
""
296296
""
297297
""
298-
""
299298
"-b --run"
300299
"-b --run"
301300
""
302301
""
303302
""
304-
""
305303
)
306304

307305
list(LENGTH TEST_SRCS count)
@@ -325,7 +323,7 @@ endforeach()
325323

326324
foreach(t testWorkflow testTaskRunner testCheckWorkflow
327325
testPostProcessingConfig testPostProcessingInterface
328-
testCheck testTrendingTask testCustomParameters)
326+
testTrendingTask)
329327
target_sources(${t} PRIVATE
330328
${CMAKE_BINARY_DIR}/getTestDataDirectory.cxx)
331329
target_include_directories(${t} PRIVATE ${CMAKE_SOURCE_DIR})

Framework/test/testCheck.cxx

Lines changed: 20 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,7 @@
2525
#include <TH1F.h>
2626
#include <Configuration/ConfigurationFactory.h>
2727
#include <Configuration/ConfigurationInterface.h>
28-
29-
#define BOOST_TEST_MODULE Check test
30-
#define BOOST_TEST_MAIN
31-
#define BOOST_TEST_DYN_LINK
32-
33-
#include <boost/test/unit_test.hpp>
28+
#include <catch_amalgamated.hpp>
3429

3530
using namespace o2::quality_control::checker;
3631
using namespace o2::quality_control::core;
@@ -57,28 +52,28 @@ CheckConfig getCheckConfig(const std::string& configFilePath, const std::string&
5752
}
5853
}
5954

60-
BOOST_AUTO_TEST_CASE(test_check_specs)
55+
TEST_CASE("test_check_specs")
6156
{
6257
std::string configFilePath = std::string("json://") + getTestDataDirectory() + "testSharedConfig.json";
6358

6459
Check check(getCheckConfig(configFilePath, "singleCheck"));
6560

66-
BOOST_REQUIRE_EQUAL(check.getInputs().size(), 1);
67-
BOOST_CHECK_EQUAL(check.getInputs()[0], (InputSpec{ { "mo" }, "QTST", "skeletonTask", 0, Lifetime::Sporadic }));
61+
REQUIRE(check.getInputs().size() == 1);
62+
CHECK(check.getInputs()[0] == (InputSpec{ { "mo" }, "QTST", "skeletonTask", 0, Lifetime::Sporadic }));
6863

69-
BOOST_CHECK_EQUAL(check.getOutputSpec(), (OutputSpec{ "CTST", "singleCheck", 0, Lifetime::Sporadic }));
64+
CHECK(check.getOutputSpec() == (OutputSpec{ "CTST", "singleCheck", 0, Lifetime::Sporadic }));
7065
}
7166

72-
BOOST_AUTO_TEST_CASE(test_check_long_description)
67+
TEST_CASE("test_check_long_description")
7368
{
7469
std::string configFilePath = std::string("json://") + getTestDataDirectory() + "testSharedConfig.json";
7570

7671
Check check(getCheckConfig(configFilePath, "singleCheckLongDescription"));
7772

78-
BOOST_REQUIRE_EQUAL(check.getInputs().size(), 1);
79-
BOOST_CHECK_EQUAL(check.getInputs()[0], (InputSpec{ { "mo" }, "QTST", "skeletonTask", 0, Lifetime::Sporadic }));
73+
REQUIRE(check.getInputs().size() == 1);
74+
CHECK(check.getInputs()[0] == (InputSpec{ { "mo" }, "QTST", "skeletonTask", 0, Lifetime::Sporadic }));
8075

81-
BOOST_CHECK_EQUAL(check.getOutputSpec(), (OutputSpec{ "CTST", "singleCheckL9fdb", 0, Lifetime::Sporadic }));
76+
CHECK(check.getOutputSpec() == (OutputSpec{ "CTST", "singleCheckL9fdb", 0, Lifetime::Sporadic }));
8277
}
8378

8479
std::shared_ptr<MonitorObject> dummyMO(const std::string& objName)
@@ -88,7 +83,7 @@ std::shared_ptr<MonitorObject> dummyMO(const std::string& objName)
8883
return obj;
8984
}
9085

91-
BOOST_AUTO_TEST_CASE(test_check_empty_mo)
86+
TEST_CASE("test_check_empty_mo")
9287
{
9388
std::string configFilePath = std::string("json://") + getTestDataDirectory() + "testSharedConfig.json";
9489

@@ -102,7 +97,7 @@ BOOST_AUTO_TEST_CASE(test_check_empty_mo)
10297
};
10398

10499
auto qos = check.check(moMap);
105-
BOOST_CHECK_EQUAL(qos.size(), 0);
100+
CHECK(qos.size() == 0);
106101
}
107102

108103
{
@@ -111,11 +106,11 @@ BOOST_AUTO_TEST_CASE(test_check_empty_mo)
111106
};
112107

113108
auto qos = check.check(moMap);
114-
BOOST_CHECK_EQUAL(qos.size(), 0);
109+
CHECK(qos.size() == 0);
115110
}
116111
}
117112

118-
BOOST_AUTO_TEST_CASE(test_check_invoke_check)
113+
TEST_CASE("test_check_invoke_check")
119114
{
120115
std::string configFilePath = std::string("json://") + getTestDataDirectory() + "testSharedConfig.json";
121116

@@ -128,10 +123,10 @@ BOOST_AUTO_TEST_CASE(test_check_invoke_check)
128123
};
129124

130125
auto qos = check.check(moMap);
131-
BOOST_CHECK_EQUAL(qos.size(), 1);
126+
CHECK(qos.size() == 1);
132127
}
133128

134-
BOOST_AUTO_TEST_CASE(test_check_postprocessing)
129+
TEST_CASE("test_check_postprocessing")
135130
{
136131
std::string configFilePath = std::string("json://") + getTestDataDirectory() + "testSharedConfig.json";
137132

@@ -144,10 +139,10 @@ BOOST_AUTO_TEST_CASE(test_check_postprocessing)
144139
};
145140

146141
auto qos = check.check(moMap);
147-
BOOST_CHECK_EQUAL(qos.size(), 1);
142+
CHECK(qos.size() == 1);
148143
}
149144

150-
BOOST_AUTO_TEST_CASE(test_check_activity)
145+
TEST_CASE("test_check_activity")
151146
{
152147
Check check({ "test",
153148
"QcSkeleton",
@@ -170,7 +165,7 @@ BOOST_AUTO_TEST_CASE(test_check_activity)
170165
check.startOfActivity(Activity());
171166
auto qos = check.check(moMap);
172167

173-
BOOST_REQUIRE_EQUAL(qos.size(), 1);
168+
REQUIRE(qos.size() == 1);
174169
ValidityInterval correctValidity{ 1, 15 };
175-
BOOST_CHECK(qos[0]->getActivity().mValidity == correctValidity);
176-
}
170+
CHECK(qos[0]->getActivity().mValidity == correctValidity);
171+
}

0 commit comments

Comments
 (0)