From 9d491352d161ee629cc47459537344ba0ea4bb35 Mon Sep 17 00:00:00 2001
From: 魏曹先生 <1992414357@qq.com>
Date: Sat, 20 Jun 2026 01:11:25 +0800
Subject: Add shared `MockProgramCollect` and conditional `Groupped` bounds

Extract duplicate `MockCollect` implementations into a reusable
`MockProgramCollect` type. Conditionally require `Serialize` on the
`Groupped` trait when the `general_renderer` feature is enabled.
---
 mingling_core/src/program/setup.rs | 63 +++-----------------------------------
 1 file changed, 4 insertions(+), 59 deletions(-)

(limited to 'mingling_core/src/program/setup.rs')

diff --git a/mingling_core/src/program/setup.rs b/mingling_core/src/program/setup.rs
index 2bfced1..f248fb6 100644
--- a/mingling_core/src/program/setup.rs
+++ b/mingling_core/src/program/setup.rs
@@ -20,69 +20,14 @@ where
 #[cfg(test)]
 mod tests {
     use super::*;
-    use crate::{AnyOutput, ChainProcess, Groupped, RenderResult};
-
-    /// Minimal mock collector that satisfies `C: ProgramCollect<Enum = C>`
-    /// by setting `Enum = Self`.
-    #[derive(Debug, Clone, PartialEq)]
-    struct MockCollect;
-
-    impl Groupped<MockCollect> for MockCollect {
-        fn member_id() -> MockCollect {
-            MockCollect
-        }
-    }
-
-    impl ProgramCollect for MockCollect {
-        type Enum = MockCollect;
-        type ErrorDispatcherNotFound = MockCollect;
-        type ErrorRendererNotFound = MockCollect;
-        type ResultEmpty = MockCollect;
-
-        fn build_renderer_not_found(_member_id: MockCollect) -> AnyOutput<MockCollect> {
-            unimplemented!()
-        }
-        fn build_dispatcher_not_found(_args: Vec<String>) -> AnyOutput<MockCollect> {
-            unimplemented!()
-        }
-        fn build_empty_result() -> AnyOutput<MockCollect> {
-            unimplemented!()
-        }
-        fn render(_any: AnyOutput<MockCollect>, _r: &mut RenderResult) {
-            unimplemented!()
-        }
-        fn render_help(_any: AnyOutput<MockCollect>, _r: &mut RenderResult) {
-            unimplemented!()
-        }
-        fn do_chain(_any: AnyOutput<MockCollect>) -> ChainProcess<MockCollect> {
-            unimplemented!()
-        }
-        #[cfg(feature = "comp")]
-        fn do_comp(_any: &AnyOutput<MockCollect>, _ctx: &crate::ShellContext) -> crate::Suggest {
-            unimplemented!()
-        }
-        fn has_renderer(_any: &AnyOutput<MockCollect>) -> bool {
-            unimplemented!()
-        }
-        fn has_chain(_any: &AnyOutput<MockCollect>) -> bool {
-            unimplemented!()
-        }
-
-        #[cfg(feature = "general_renderer")]
-        fn general_render(
-            _any: AnyOutput<MockCollect>,
-            _setting: &crate::GeneralRendererSetting,
-        ) -> Result<RenderResult, crate::error::GeneralRendererSerializeError> {
-            unimplemented!()
-        }
-    }
+    use crate::MockProgramCollect;
 
     struct TestSetup {
         called: std::rc::Rc<std::cell::Cell<bool>>,
     }
 
-    impl ProgramSetup<MockCollect> for TestSetup {
-        fn setup(self, _program: &mut Program<MockCollect>) {
+    impl ProgramSetup<MockProgramCollect> for TestSetup {
+        fn setup(self, _program: &mut Program<MockProgramCollect>) {
             self.called.set(true);
         }
     }
@@ -93,7 +38,7 @@ mod tests {
         let setup = TestSetup {
             called: std::rc::Rc::clone(&called),
         };
-        let mut program: Program<MockCollect> = Program::new_with_args(["test"]);
+        let mut program: Program<MockProgramCollect> = Program::new_with_args(["test"]);
         program.with_setup(setup);
         assert!(called.get());
     }
-- 
cgit