checkLogicalPlan parameter adjustment

kuzudb · May 26, 2023 · f2750c7 · f2750c7
1 parent 19368de
commit f2750c7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/test/include/test_runner/test_runner.h b/test/include/test_runner/test_runner.h
@@ -22,7 +22,7 @@ class TestRunner {
     static bool checkLogicalPlans(std::unique_ptr<main::PreparedStatement>& preparedStatement,
         TestStatement* statement, main::Connection& conn);
     static bool checkLogicalPlan(std::unique_ptr<main::PreparedStatement>& preparedStatement,
-        TestStatement* statement, uint32_t planIdx, main::Connection& conn);
+        TestStatement* statement, main::Connection& conn, uint32_t planIdx);
     static std::vector<std::string> convertResultToString(
         main::QueryResult& queryResult, bool checkOutputOrder = false);
     static bool checkPlanResult(std::unique_ptr<main::QueryResult>& result,

diff --git a/test/test_runner/test_runner.cpp b/test/test_runner/test_runner.cpp
@@ -43,18 +43,18 @@ bool TestRunner::checkLogicalPlans(std::unique_ptr<PreparedStatement>& preparedS
     auto numPlans = preparedStatement->logicalPlans.size();
     auto numPassedPlans = 0u;
     if (numPlans == 0) {
-        return checkLogicalPlan(preparedStatement, statement, 0, conn);
+        return checkLogicalPlan(preparedStatement, statement, conn, 0);
     }
     for (auto i = 0u; i < numPlans; ++i) {
-        if (checkLogicalPlan(preparedStatement, statement, i, conn)) {
+        if (checkLogicalPlan(preparedStatement, statement, conn, i)) {
             numPassedPlans++;
         }
     }
     return numPassedPlans == numPlans;
 }
 
 bool TestRunner::checkLogicalPlan(std::unique_ptr<PreparedStatement>& preparedStatement,
-    TestStatement* statement, uint32_t planIdx, Connection& conn) {
+    TestStatement* statement, Connection& conn, uint32_t planIdx) {
     auto result = conn.executeAndAutoCommitIfNecessaryNoLock(preparedStatement.get(), planIdx);
     if (statement->expectedError) {
         if (statement->errorMessage == StringUtils::rtrim(result->getErrorMessage())) {