Address review comments

Signed-off-by: Xiaodong Ye <[email protected]>

Author: yeahdongcn

tests/test-backend-ops.cpp

@@ -442,13 +442,6 @@ struct test_result {
     }
 };
 
-// Enum for test status
-enum class test_status_t {
-    NOT_SUPPORTED,
-    OK,
-    FAIL
-};
-
 // Forward declarations for the visitor pattern
 struct message_visitor;
 
@@ -470,6 +463,12 @@ struct message_visitor {
 };
 
 // Printer classes for different output formats
+enum class test_status_t {
+    NOT_SUPPORTED,
+    OK,
+    FAIL
+};
+
 struct test_operation_info : public message_data {
     std::string op_name;
     std::string op_params;
@@ -637,6 +636,32 @@ struct printer : public message_visitor {
     void print_message(const message_data& data) {
         data.accept(*this);
     }
+
+    // Default implementations for all visit methods (no-op)
+    // Derived classes can override only the ones they care about
+    void visit(const test_operation_info& info) override {
+        (void)info;
+    }
+
+    void visit(const test_summary_info& info) override {
+        (void)info;
+    }
+
+    void visit(const testing_start_info& info) override {
+        (void)info;
+    }
+
+    void visit(const backend_init_info& info) override {
+        (void)info;
+    }
+
+    void visit(const backend_status_info& info) override {
+        (void)info;
+    }
+
+    void visit(const overall_summary_info& info) override {
+        (void)info;
+    }
 };
 
 struct console_printer : public printer {
@@ -857,31 +882,6 @@ struct sql_printer : public printer {
         }
         fprintf(fout, ");\n");
     }
-
-    // Visitor pattern implementations - SQL printer doesn't need to handle message types for now
-    void visit(const test_operation_info& info) override {
-        (void)info;
-    }
-
-    void visit(const test_summary_info& info) override {
-        (void)info;
-    }
-
-    void visit(const testing_start_info& info) override {
-        (void)info;
-    }
-
-    void visit(const backend_init_info& info) override {
-        (void)info;
-    }
-
-    void visit(const backend_status_info& info) override {
-        (void)info;
-    }
-
-    void visit(const overall_summary_info& info) override {
-        (void)info;
-    }
 };
 
 static std::unique_ptr<printer> create_printer(output_formats format) {