[Lldb-commits] [lldb] cea112f - [lldb] Fix a typo in a test name

Walter Erquinigo via lldb-commits lldb-commits at lists.llvm.org
Tue Apr 14 12:19:13 PDT 2020


Author: Walter Erquinigo
Date: 2020-04-14T12:18:58-07:00
New Revision: cea112f422cea3e6fab7cefffef2e09eadec279e

URL: https://github.com/llvm/llvm-project/commit/cea112f422cea3e6fab7cefffef2e09eadec279e
DIFF: https://github.com/llvm/llvm-project/commit/cea112f422cea3e6fab7cefffef2e09eadec279e.diff

LOG: [lldb] Fix a typo in a test name

Summary:
Removing the Test prefix from the file name and its usages. The standard is using only Test as a suffix.
This was correctly pointed out in https://reviews.llvm.org/D77444.

Reviewers: labath, clayborg

Subscribers: mgorny, lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D77878

Added: 
    lldb/unittests/API/SBCommandInterpreterTest.cpp

Modified: 
    lldb/unittests/API/CMakeLists.txt

Removed: 
    lldb/unittests/API/TestSBCommandInterpreterTest.cpp


################################################################################
diff  --git a/lldb/unittests/API/CMakeLists.txt b/lldb/unittests/API/CMakeLists.txt
index 50ec47bfe00f..6f71165a8a70 100644
--- a/lldb/unittests/API/CMakeLists.txt
+++ b/lldb/unittests/API/CMakeLists.txt
@@ -1,5 +1,5 @@
 add_lldb_unittest(APITests
-  TestSBCommandInterpreterTest.cpp
+  SBCommandInterpreterTest.cpp
 
   LINK_LIBS
     liblldb

diff  --git a/lldb/unittests/API/TestSBCommandInterpreterTest.cpp b/lldb/unittests/API/SBCommandInterpreterTest.cpp
similarity index 94%
rename from lldb/unittests/API/TestSBCommandInterpreterTest.cpp
rename to lldb/unittests/API/SBCommandInterpreterTest.cpp
index 6194b6536a27..d117c08c0bf4 100644
--- a/lldb/unittests/API/TestSBCommandInterpreterTest.cpp
+++ b/lldb/unittests/API/SBCommandInterpreterTest.cpp
@@ -1,4 +1,4 @@
-//===-- TestSBCommandInterpreterTest.cpp ----------------------------------===//
+//===-- SBCommandInterpreterTest.cpp ------------------------===----------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
@@ -17,7 +17,7 @@
 
 using namespace lldb;
 
-class TestSBCommandInterpreterTest : public testing::Test {
+class SBCommandInterpreterTest : public testing::Test {
 protected:
   void SetUp() override {
     SBDebugger::Initialize();
@@ -44,7 +44,7 @@ class DummyCommand : public SBCommandPluginInterface {
   std::string m_message;
 };
 
-TEST_F(TestSBCommandInterpreterTest, SingleWordCommand) {
+TEST_F(SBCommandInterpreterTest, SingleWordCommand) {
   // We first test a command without autorepeat
   DummyCommand dummy("It worked");
   m_interp.AddCommand("dummy", &dummy, /*help=*/nullptr);
@@ -79,7 +79,7 @@ TEST_F(TestSBCommandInterpreterTest, SingleWordCommand) {
   }
 }
 
-TEST_F(TestSBCommandInterpreterTest, MultiWordCommand) {
+TEST_F(SBCommandInterpreterTest, MultiWordCommand) {
   auto command = m_interp.AddMultiwordCommand("multicommand", /*help=*/nullptr);
   // We first test a subcommand without autorepeat
   DummyCommand subcommand("It worked again");


        


More information about the lldb-commits mailing list