Skip to content

Commit

Permalink
Try to fix flaky tests
Browse files Browse the repository at this point in the history
  • Loading branch information
SiarheiFedartsou committed Nov 3, 2024
1 parent 21c9f5d commit 3d9f0c6
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/osrm-backend.yml
Original file line number Diff line number Diff line change
Expand Up @@ -575,7 +575,7 @@ jobs:
# All tests assume to be run from the build directory
pushd ${OSRM_BUILD_DIR}
for i in $(seq 1 1000); do
for i in $(seq 1 100); do
echo "Iteration $i"
for test in ./unit_tests/*-tests; do
echo "Running $test"
Expand Down
24 changes: 12 additions & 12 deletions unit_tests/library/extract.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -47,18 +47,18 @@ BOOST_AUTO_TEST_CASE(test_extract_with_invalid_config)
oneapi::tbb::finalize(handle);
}

// BOOST_AUTO_TEST_CASE(test_extract_with_valid_config)
// {
// oneapi::tbb::task_scheduler_handle handle{tbb::attach{}};
// osrm::ExtractorConfig config;
// config.input_path = OSRM_TEST_DATA_DIR "/monaco.osm.pbf";
// config.UseDefaultOutputNames(OSRM_TEST_DATA_DIR "/monaco.osm.pbf");
// config.profile_path = OSRM_TEST_DATA_DIR "/../../profiles/car.lua";
// config.small_component_size = 1000;
// config.requested_num_threads = std::thread::hardware_concurrency();
// BOOST_CHECK_NO_THROW(osrm::extract(config));
// oneapi::tbb::finalize(handle);
// }
BOOST_AUTO_TEST_CASE(test_extract_with_valid_config)
{
oneapi::tbb::task_scheduler_handle handle{tbb::attach{}};
osrm::ExtractorConfig config;
config.input_path = OSRM_TEST_DATA_DIR "/monaco.osm.pbf";
config.UseDefaultOutputNames(OSRM_TEST_DATA_DIR "/monaco.osm.pbf");
config.profile_path = OSRM_TEST_DATA_DIR "/../../profiles/car.lua";
config.small_component_size = 1000;
config.requested_num_threads = std::thread::hardware_concurrency();
BOOST_CHECK_NO_THROW(osrm::extract(config));
oneapi::tbb::finalize(handle);
}

// BOOST_AUTO_TEST_CASE(test_setup_runtime_error)
// {
Expand Down

0 comments on commit 3d9f0c6

Please sign in to comment.