minor cleanup

Author: SatyaRanjanDas-tomtom

README.md

@@ -49,12 +49,11 @@ Below is the program written as unit-test for validating the correctness of gene
 
 namespace fs = boost::filesystem;
 
-TEST_CASE(" Parsing hello world program")
+TEST_CASE("Parsing hello world program")
 {
-  CppParser parser;
-  parser.parseFunctionBodyAsBlob(false);
-  auto testFilePath = fs::path(__FILE__).parent_path() / "test-files/hello-world.cpp";
-  auto ast          = parser.parseFile(testFilePath.string());
+  CppParser  parser;
+  const auto testFilePath = fs::path(__FILE__).parent_path() / "test-files/hello-world.cpp";
+  const auto ast          = parser.parseFile(testFilePath.string());
   REQUIRE(ast != nullptr);
 
   const auto& members = ast->members();

test/unit/disabled-code-test.cpp

@@ -15,7 +15,7 @@ class DisabledCodeTest : public EmbeddedSnippetTestBase
   }
 };
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #if")
+TEST_CASE_METHOD(DisabledCodeTest, "Code disabled using #if")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -43,7 +43,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #if")
   CHECK(params->size() == 1);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #if !")
+TEST_CASE_METHOD(DisabledCodeTest, "Code disabled using #if !")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -71,7 +71,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #if !")
   CHECK(params->size() == 2);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code enabled in #else part of #if")
+TEST_CASE_METHOD(DisabledCodeTest, "Code enabled in #else part of #if")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -101,7 +101,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code enabled in #else part of #if")
   CHECK(params->size() == 2);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code enabled using #if {ID} >= {NUM}")
+TEST_CASE_METHOD(DisabledCodeTest, "Code enabled using #if {ID} >= {NUM}")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -129,7 +129,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code enabled using #if {ID} >= {NUM}")
   CHECK(params->size() == 2);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #ifdef")
+TEST_CASE_METHOD(DisabledCodeTest, "Code disabled using #ifdef")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -157,7 +157,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #ifdef")
   CHECK(params->size() == 1);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code enabled in #else part of #ifdef")
+TEST_CASE_METHOD(DisabledCodeTest, "Code enabled in #else part of #ifdef")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -187,7 +187,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code enabled in #else part of #ifdef")
   CHECK(params->size() == 2);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #ifndef")
+TEST_CASE_METHOD(DisabledCodeTest, "Code disabled using #ifndef")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -216,7 +216,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code disabled using #ifndef")
   CHECK(params->size() == 1);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Code enabled in #else part of #ifndef")
+TEST_CASE_METHOD(DisabledCodeTest, "Code enabled in #else part of #ifndef")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
   void FunctionWithDisabledParams(int normalParam
@@ -246,7 +246,7 @@ TEST_CASE_METHOD(DisabledCodeTest, " Code enabled in #else part of #ifndef")
   CHECK(params->size() == 2);
 }
 
-TEST_CASE_METHOD(DisabledCodeTest, " Enabled code section has disabled subsection")
+TEST_CASE_METHOD(DisabledCodeTest, "Enabled code section has disabled subsection")
 {
 #if TEST_CASE_SNIPPET_STARTS_FROM_NEXT_LINE
 #  if CPPPARSER_TEST_DEFINED_MACRO

test/unit/namespace-test.cpp

@@ -21,7 +21,7 @@ auto p = new char*[5];
 #  endif
 #endif
 
-TEST_CASE_METHOD(NamespaceTest, " C++17 style nested namespace")
+TEST_CASE_METHOD(NamespaceTest, "C++17 style nested namespace")
 {
   auto testSnippet = getTestSnippetParseStream(__LINE__ - 5);
 

test/unit/test-hello-world.cpp

@@ -6,12 +6,11 @@
 
 namespace fs = boost::filesystem;
 
-TEST_CASE(" Parsing hello world program")
+TEST_CASE("Parsing hello world program")
 {
-  CppParser parser;
-  parser.parseFunctionBodyAsBlob(false);
-  auto testFilePath = fs::path(__FILE__).parent_path() / "test-files/hello-world.cpp";
-  auto ast          = parser.parseFile(testFilePath.string());
+  CppParser  parser;
+  const auto testFilePath = fs::path(__FILE__).parent_path() / "test-files/hello-world.cpp";
+  const auto ast          = parser.parseFile(testFilePath.string());
   REQUIRE(ast != nullptr);
 
   const auto& members = ast->members();