use TString::Join() and use TStringBuf instead of const char*

commit_hash:451ebc0373719a4e87b0eef9c6278b1fe8eea842

2 files changed, 4 insertions, 5 deletions

diff --git a/library/cpp/testing/unittest/registar.cpp b/library/cpp/testing/unittest/registar.cpp
index d48f13813e3..5c2e9343f99 100644
--- a/library/cpp/testing/unittest/registar.cpp
+++ b/library/cpp/testing/unittest/registar.cpp
@@ -5,7 +5,6 @@
 
 #include <util/generic/yexception.h>
 #include <util/random/fast.h>
-#include <util/string/printf.h>
 #include <util/system/backtrace.h>
 #include <util/system/guard.h>
 #include <util/system/tls.h>
@@ -134,8 +133,8 @@ struct TTraceDiffFormatter {
     }
 };
 
-TString NUnitTest::GetFormatTag(const char* name) {
-    return Sprintf("[[%s]]", name);
+TString NUnitTest::GetFormatTag(TStringBuf name) {
+    return TString::Join("[[", name, "]]");
 }
 
 TString NUnitTest::GetResetTag() {
@@ -156,7 +155,7 @@ TString NUnitTest::ColoredDiff(TStringBuf s1, TStringBuf s2, const TString& deli
 }
 
 static TString MakeTestName(const NUnitTest::ITestSuiteProcessor::TTest& test) {
-    return TStringBuilder() << test.unit->name << "::" << test.name;
+    return TString::Join(test.unit->name, "::", test.name);
 }
 
 static size_t CountTests(const TMap<TString, size_t>& testErrors, bool succeeded) {
diff --git a/library/cpp/testing/unittest/registar.h b/library/cpp/testing/unittest/registar.h
index c65c79e79f4..c6c1281b4db 100644
--- a/library/cpp/testing/unittest/registar.h
+++ b/library/cpp/testing/unittest/registar.h
@@ -42,7 +42,7 @@ namespace NUnitTest {
 
     extern bool ShouldColorizeDiff;
     TString ColoredDiff(TStringBuf s1, TStringBuf s2, const TString& delims = TString(), bool reverse = false);
-    TString GetFormatTag(const char* name);
+    TString GetFormatTag(TStringBuf name);
     TString GetResetTag();
 
     // Raise error handler