Moved all tests regarding debug naming to the new TU debug_naming_tests

Author: GagaLP

test/buffer_manager_tests.cc

@@ -1078,13 +1078,6 @@ namespace detail {
 		}
 	}
 
-	TEST_CASE_METHOD(test_utils::runtime_fixture, "buffer_manager allows to set buffer debug names on  buffers", "[buffer_manager]") {
-		celerity::buffer<int, 1> buff_a(16);
-		std::string buff_name{"my_buffer"};
-		detail::runtime::get_instance().get_buffer_manager().set_debug_name(detail::get_buffer_id(buff_a), buff_name);
-		CHECK(detail::runtime::get_instance().get_buffer_manager().get_debug_name(detail::get_buffer_id(buff_a)) == buff_name);
-	}
-
 	TEST_CASE_METHOD(test_utils::device_queue_fixture, "device_queue allows to allocate device memory and query usage", "[device_queue]") {
 		auto& dq = get_device_queue();
 		const size_t ten_mib = 1024ul * 1024ul * 10ul;

test/debug_naming_tests.cc

@@ -50,4 +50,29 @@ TEST_CASE("debug names can be set and retrieved from tasks", "[debug]") {
 		CHECK(tt.tm.get_task(tid_a)->get_debug_name() == task_name);
 		CHECK_THAT(tt.tm.get_task(tid_b)->get_debug_name(), Catch::Matchers::ContainsSubstring("nd_range_task_unnamed"));
 	}
+}
+
+TEST_CASE_METHOD(test_utils::runtime_fixture, "buffer_manager allows to set buffer debug names on  buffers", "[buffer_manager]") {
+	celerity::buffer<int, 1> buff_a(16);
+	std::string buff_name{"my_buffer"};
+	celerity::detail::runtime::get_instance().get_buffer_manager().set_debug_name(celerity::detail::get_buffer_id(buff_a), buff_name);
+	CHECK(celerity::detail::runtime::get_instance().get_buffer_manager().get_debug_name(celerity::detail::get_buffer_id(buff_a)) == buff_name);
+}
+
+
+namespace foo {
+	class MySecondKernel;
+}
+
+template <typename T>
+class MyThirdKernel;
+
+TEST_CASE("device_compute tasks derive debug name from kernel name", "[task]") {
+	auto tm = celerity::detail::task_manager(1, nullptr, {});
+	const auto t1 = tm.get_task(tm.submit_command_group([](handler& cgh) { cgh.parallel_for<class MyFirstKernel>(range<1>{1}, [](id<1>) {}); }));
+	const auto t2 = tm.get_task(tm.submit_command_group([](handler& cgh) { cgh.parallel_for<foo::MySecondKernel>(range<1>{1}, [](id<1>) {}); }));
+	const auto t3 = tm.get_task(tm.submit_command_group([](handler& cgh) { cgh.parallel_for<MyThirdKernel<int>>(range<1>{1}, [](id<1>) {}); }));
+	CHECK(utils::simplify_task_name(t1->get_debug_name()) == "MyFirstKernel");
+	CHECK(utils::simplify_task_name(t2->get_debug_name()) == "MySecondKernel");
+	CHECK(utils::simplify_task_name(t3->get_debug_name()) == "MyThirdKernel<...>");
 }

test/runtime_tests.cc

@@ -288,23 +288,6 @@ namespace detail {
 		REQUIRE(req == box<3>());
 	}
 
-	namespace foo {
-		class MySecondKernel;
-	}
-
-	template <typename T>
-	class MyThirdKernel;
-
-	TEST_CASE("device_compute tasks derive debug name from kernel name", "[task]") {
-		auto tm = detail::task_manager(1, nullptr, {});
-		const auto t1 = tm.get_task(tm.submit_command_group([](handler& cgh) { cgh.parallel_for<class MyFirstKernel>(range<1>{1}, [](id<1>) {}); }));
-		const auto t2 = tm.get_task(tm.submit_command_group([](handler& cgh) { cgh.parallel_for<foo::MySecondKernel>(range<1>{1}, [](id<1>) {}); }));
-		const auto t3 = tm.get_task(tm.submit_command_group([](handler& cgh) { cgh.parallel_for<MyThirdKernel<int>>(range<1>{1}, [](id<1>) {}); }));
-		CHECK(utils::simplify_task_name(t1->get_debug_name()) == "MyFirstKernel");
-		CHECK(utils::simplify_task_name(t2->get_debug_name()) == "MySecondKernel");
-		CHECK(utils::simplify_task_name(t3->get_debug_name()) == "MyThirdKernel<...>");
-	}
-
 	TEST_CASE_METHOD(test_utils::runtime_fixture, "basic SYNC command functionality", "[distr_queue][sync][control-flow]") {
 		constexpr int N = 10;