kimata / my-py-lib / 26347514655

61%

Build Type

push

github

Committed by kimata

Commit Message

refactor: BP35A1 をイベント駆動アーキテクチャに再構築

連続して 3 つのバグ (継承漏れ・PAN キャッシュ判定・MODE 3 パース) が
発生したのを契機に、 シリアル通信の処理を 3 レイヤに整理した:

  Layer 0: pyserial (外部)
  Layer 1: EventParser   — 行ストリームを Event オブジェクトに分類
  Layer 2: BP35A1Session — コマンド送信 + イベント待機 + 行プッシュバック
  Layer 3: BP35A1        — public API (内部実装は新セッション経由に置換)

従来の「期待文字列を順に read してマッチ」方式から、 構造化された
Event を待つ方式に変更。 これにより以下の脆さが解消される:

- 想定外イベント (EVENT 1F や ERXUDP 等) の割り込みでパースが壊れる
- 複数行ブロックのフィールド構成差分 (例: PairID の有無) に弱い
- read の二重消費による上位ループの timeout 待ち
- 行レベルの厳密一致 (__expect) で仕様差分に弱い

BP35A1 の public API (ping, scan_channel, connect, disconnect,
send_udp, recv_udp, set_id, set_password, reset) は維持しているため、
echonetenergy.py 等の上位コードは変更不要。

ユニットテスト test_sensor_bp35a1_session.py を 18 ケース追加。
実機 (BP35A1 1.2.10 + メーター実体) で scan_channel → PanDescriptor
取得まで動作確認済み。

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Coverage Stats

144 of 214 new or added lines in 2 files covered. (67.29%)

4 existing lines in 1 file now uncovered.

4029 of 6639 relevant lines covered (60.69%)

0.61 hits per line