fix: 1日の境界にあるイベントの表示が正しくない

[Hirotaka-Hanai]

チケット

66

実装内容

• タイムテーブルの日付の初期値を1日の開始時刻にする
• タイムテーブルの日付をDIコンテナから取得
• イベント・アクティビティの取得方法を、取得期間と重なる時間があれば取得するように変更
• 1日の境界にあるイベント・アクティビティの高さの計算方法を修正
• イベント取得のテストの実装
• WindowLog取得のテストの実装

未実装

• ~イベント取得のテストが成功しない~
  • ~テストではイベントが取得できていない(アプリケーションでは取得できて正しい表示になっている)~
• ~アクティビティ取得のテスト~

[msato-ok]

LGTM

座標の計算のところ、ややこしかったと思うけど、よく短期間で修正できましたね。 あと、単体テストの実装も、よいと思います。 この実装のやり方は、パラメタライズドテストなどと言われてます。 テストデータをfixtureとして別に実装しておいて、テストロジックと分離すると、テストコードが見やすくなるという利点もあります。fixture はデフォルト値をあらかじめ埋め込んでおいて、Options ObjectパターンでPartial で値を置き替えることで、テストで確認すべきポイントが明確になるというのを狙ってます。