ArduPilot / MethodicConfigurator / 23712733005

94%

push

github

refactor(tests): deduplicate capture_safe_write into backend_safe_file_io.py

Move the capture_safe_write helper that was defined 6 times in
test_backend_filesystem_vehicle_components.py into a single
make_capture_safe_write() factory in backend_safe_file_io.py.
Address Copilot review comments:
- Use precise type annotations (dict[str, Any], Callable[[IO[str]], object])
  in make_capture_safe_write for pyright/mypy compatibility
- Restore captured_data in test_save_component_templates_empty_input to
  verify that the written JSON is actually an empty dict
use implicit booleaness for empty dict check (pylint C1803)

Follow-up to PR #1437 as requested by @amilcarlucas.

Signed-off-by: Yash Goel <yashhzd@users.noreply.github.com>

1689 of 1756 branches covered (96.18%)

11135 of 11869 relevant lines covered (93.82%)

3.74 hits per line