Use rename annotation for the output

Author: Yaraslaut

include/rsl/testing/_testing_impl/expand.hpp

@@ -43,6 +43,21 @@ struct TestRunner {
     }
   }
 
+
+  template <typename... Ts>
+  static std::string apply_args_to_name(std::string name, std::tuple<Ts...> args) {
+    name += std::apply(
+        [](auto&&... args) {
+          std::string result = "(";
+          std::size_t i      = 0;
+          ((result += (i++ ? ", " : "") + repr(args)), ...);
+          result += ")";
+          return result;
+        },
+        args);
+    return name;
+  }
+
   template <typename... Ts>
   static std::string get_name(std::tuple<Ts...> args) {
     std::string name;
@@ -59,21 +74,22 @@ struct TestRunner {
     if constexpr (has_template_arguments(Target)) {
       name += rsl::serializer::stringify_template_args(Target);
     }
-    name += std::apply(
-        [](auto&&... args) {
-          std::string result = "(";
-          std::size_t i      = 0;
-          ((result += (i++ ? ", " : "") + repr(args)), ...);
-          result += ")";
-          return result;
-        },
-        args);
-    return name;
+
+    return apply_args_to_name(name, args);
   }
 
   template <typename... Ts>
   static TC bind(Test const* group, std::tuple<Ts...> args) {
-    return {group, std::bind_front(run_one<std::tuple<Ts...>>, args), get_name(args)};
+    constexpr auto ann = _testing_impl::Annotations(Def);
+    if constexpr (ann.name.empty()) // TODO fix this, for some reason it is reversed, check should be !ann.name.empty()
+    {
+      constexpr auto name = ann.name;
+      return {group, std::bind_front(run_one<std::tuple<Ts...>>, args), apply_args_to_name(std::string(name), args)};
+    }
+    else
+    {
+      return {group, std::bind_front(run_one<std::tuple<Ts...>>, args), get_name(args)};
+    }
   }
 };
 
@@ -152,4 +168,4 @@ struct Expand {
     }
   }
 };
-}  // namespace rsl::testing::_testing_impl
+}  // namespace rsl::testing::_testing_impl

include/rsl/testing/annotations.hpp

@@ -133,7 +133,7 @@ struct Annotations {  // consteval-only
         // constexpr_assert(skip == nullptr, "Cannot have more than one skip annotation.");
         skip = extract<annotations::Skip>(constant_of(annotation)).value;
       } else if (type == ^^annotations::Rename) {
-        constexpr_assert(name.empty(), "Cannot rename more than once.");
+        constexpr_assert(!name.empty(), "Cannot rename more than once.");
         name = extract<annotations::Rename>(constant_of(annotation)).value;
       } else if (type == ^^annotations::FuzzTag) {
         is_fuzz_test = true;
@@ -149,4 +149,4 @@ struct Annotations {  // consteval-only
   }
 };
 }  // namespace rsl::testing::_testing_impl
-}
+}