src/merge/tests/log.rs (1)

21-21: Optional: extract a tiny test helper for logger setup.

RunLogger::new(&mut buffer, &behaviours) repeats across most test cases; a helper would reduce duplication and make future API renames cheaper.

♻️ Suggested refactor

+fn new_logger<'a>(
+    buffer: &'a mut Vec<u8>,
+    behaviours: &RunBehaviours,
+) -> RunLogger<&'a mut Vec<u8>> {
+    RunLogger::new(buffer, behaviours)
+}

-    let mut l = RunLogger::new(&mut buffer, &RunBehaviours::default());
+    let mut l = new_logger(&mut buffer, &RunBehaviours::default());

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/merge/tests/log.rs` at line 21, Create a small test helper that
centralizes the repeated logger setup to reduce duplication: add a function
(e.g., make_test_logger or setup_run_logger) that accepts a mutable buffer (or
constructs one) and optional behaviours, and returns the instantiated RunLogger
created via RunLogger::new(&mut buffer, &behaviours) using
RunBehaviours::default when behaviours are not provided; then replace direct
calls to RunLogger::new(&mut buffer, &RunBehaviours::default()) in tests with
the new helper to simplify future API renames and keep tests DRY.