Add time synchronization feature to BSB-Lan integration

[liudger]

Breaking change

Proposed change

This pull request adds a new sync_time service to the BSB-LAN integration in Home Assistant, allowing users to synchronize the time on their BSB-LAN device with Home Assistant. The implementation includes service registration, validation, error handling, user-facing descriptions, and comprehensive tests.

Key changes:

New Service Implementation:

• Added the sync_time service, which updates the BSB-LAN device time to match Home Assistant, only performing the update if the times differ. This includes schema validation, device/config entry checks, and error handling for device communication issues. [1] [2] [3] [4] [5]

• Registered an appropriate Material Design Icon for the new service in icons.json.

User Interface and Documentation:

• Updated services.yaml and strings.json to document the new service, including its description, fields, and error messages for failed sync attempts. [1] [2] [3]

Testing:

• Added extensive tests for the sync_time service, covering successful syncs, no-op when times match, error conditions (such as device or entry not found, or device communication failures), and service registration. Also added a device time fixture for test consistency. [1] [2] [3] [4] [5] [6]

These changes ensure that users can reliably keep their BSB-LAN device clocks in sync with Home Assistant, with clear feedback and robust error handling.

Type of change

• Dependency upgrade
• Bugfix (non-breaking change which fixes an issue)
• New integration (thank you!)
• New feature (which adds functionality to an existing integration)
• Deprecation (breaking change to happen in the future)
• Breaking change (fix/feature causing existing functionality to break)
• Code quality improvements to existing code or addition of tests

Additional information

• This PR fixes or closes issue: fixes #
• This PR is related to issue:
• Link to documentation pull request: BSB-Lan: Add timesync action integration documentation home-assistant.io#41789
• Link to developer documentation pull request:
• Link to frontend pull request:

Checklist

• I understand the code I am submitting and can explain how it works.
• The code change is tested and works locally.
• Local tests pass. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.
• I have followed the development checklist
• I have followed the perfect PR recommendations
• The code has been formatted using Ruff (ruff format homeassistant tests)
• Tests have been added to verify that the new code works.
• Any generated code has been carefully reviewed for correctness and compliance with project standards.

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

• The manifest file has all fields filled out correctly.
  Updated and included derived files by running: python3 -m script.hassfest.
• New or updated dependencies have been added to requirements_all.txt.
  Updated by running python3 -m script.gen_requirements_all.
• For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.

To help with the load of incoming pull requests:

• I have reviewed two other open pull requests in this repository.

[Copilot]

Pull Request Overview

This PR adds a time synchronization feature to the BSB-Lan integration, allowing Home Assistant to synchronize its time with the BSB-Lan device once per hour when enabled via a new configuration option.

Key Changes

• Adds user-configurable time synchronization option (sync_time) in the options flow
• Implements hourly time synchronization logic with graceful error handling
• Includes comprehensive test coverage for the new feature

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
homeassistant/components/bsblan/__init__.py	Implements time sync logic with async_setup_time_sync function and update listener for option changes
homeassistant/components/bsblan/const.py	Adds new constants for sync time option, default value, and hourly sync interval
homeassistant/components/bsblan/config_flow.py	Adds options flow handler with schema for time sync configuration
homeassistant/components/bsblan/strings.json	Adds user-facing strings for the time sync option
tests/components/bsblan/test_init.py	Adds comprehensive tests covering enabled/disabled states, time synchronization behavior, error handling, and periodic sync
tests/components/bsblan/test_config_flow.py	Adds tests for the options flow configuration
tests/components/bsblan/fixtures/time.json	Adds test fixture for device time data

[joostlek]

Wouldn't it be better to have this as a:

1. Button
2. Service

Instead of something that runs on boot?

[home-assistant]

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

[liudger]

Wouldn't it be better to have this as a:

1. Button
2. Service

Instead of something that runs on boot?

it's an option