Fix: PEP8 spacing in logic factory + tests for string time_resolution_minutes

[Copilot]

Addresses review comments from PR #305. The logic factory had a PEP8 violation in its signature, and there were no tests covering the string→int coercion of time_resolution_minutes (e.g. when provided by Home Assistant).

Changes

src/batcontrol/logic/logic.py

• Fixed missing space after comma in create_logic signature: int,config → int, config

tests/batcontrol/test_core.py

• Added TestTimeResolutionString with parametrized coverage for '60' and '15' string inputs:
  • test_string_time_resolution_initialises_without_error — asserts Batcontrol.__init__ accepts string resolution and stores it as int
  • test_logic_factory_accepts_string_resolution_as_int — asserts LogicFactory.create_logic produces a DefaultLogic with the correct interval_minutes

# Previously would raise TypeError if HA provided time_resolution_minutes as a string
bc = Batcontrol({..., 'time_resolution_minutes': '60'})
assert isinstance(bc.time_resolution, int)  # passes — coercion happens in __init__
assert bc.time_resolution == 60

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.