バス路線はline_typeをOtherLineTypeに強制

[TinyKitten]

Summary by CodeRabbit

リリースノート

• バグ修正

  • バス路線のデータ変換処理を改善しました。

• テスト

  • バスと鉄道路線の処理パターンに関する包括的なテストを追加しました。

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

stationapi/src/use_case/dto/line.rsにて、Line型からGrpcLine型への変換時に、transport_typeがBusの場合にline_typeを強制的に0（OtherLineType）に設定するバス固有の正規化ロジックを導入しました。これまでのunwrap_or_defaultを計算済みのline_type変数に置き換え、包括的なテストモジュールを追加しました。

Changes

Cohort / File(s)	変更サマリー
バス正規化ロジックとテスト stationapi/src/use_case/dto/line.rs	transport_typeがBusの場合にline_typeを強制的にOtherLineType（0）に設定する条件付き正規化ロジックを実装。Rail型の場合は既存のline_typeを保持またはデフォルト値を使用。複数のバス・レール線種ケース、convert_transport_type関数動作、エッジケース処理、未知のline_type処理、および線路ID変換を網羅した包括的なテストモジュールを追加。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• 後方互換性のためにデフォルトで鉄道駅のみ返すように仕様変更 #1358: Transport-type処理（convert_transport_type動作やテスト含む）を扱っているため、コードレベルでの関連性があります。

Poem

🐰✨ バスの線も、統一の手で
OtherLineTypeへ優しく導かれ
レールは個性を保ちながら
テストが全てを照らすように
正規化の旅、完成だね！🚌🚃

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	バス路線のline_typeをOtherLineTypeに強制する変更を正確に説明しており、プルリクエストの主要な変更内容を明確に示している。
Docstring Coverage	✅ Passed	Docstring coverage is 89.29% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7be4886 and ba2e953.

📒 Files selected for processing (1)

• stationapi/src/use_case/dto/line.rs

🧰 Additional context used

📓 Path-based instructions (3)

stationapi/src/use_case/**/*.rs

📄 CodeRabbit inference engine (AGENTS.md)

Application logic in use case layer should implement contract traits defined in traits/ module (e.g., QueryUseCase in interactor/query.rs)

Files:

• stationapi/src/use_case/dto/line.rs

stationapi/src/**/*.rs

📄 CodeRabbit inference engine (AGENTS.md)

Run cargo test --lib --package stationapi or make test-unit for unit tests focusing on entities and repository mocks without database

Files:

• stationapi/src/use_case/dto/line.rs

**/*.rs

📄 CodeRabbit inference engine (AGENTS.md)

**/*.rs: Before committing, run cargo fmt on all Rust code
Before committing, run cargo clippy --all-targets --all-features and resolve new Clippy warnings unless covered by existing #![allow] attributes

Files:

• stationapi/src/use_case/dto/line.rs

🧠 Learnings (1)

📚 Learning: 2025-11-25T10:50:36.694Z

Learnt from: CR
Repo: TrainLCD/StationAPI PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T10:50:36.694Z
Learning: Applies to stationapi/src/use_case/interactor/query.rs : When modifying `QueryInteractor`, ensure enrichment steps (companies, train types, line symbols) still behave as expected and verify helper methods such as `update_station_vec_with_attributes` and `build_route_tree_map`

Applied to files:

• stationapi/src/use_case/dto/line.rs

🧬 Code graph analysis (1)

stationapi/src/use_case/dto/line.rs (4)

stationapi/src/domain/entity/line.rs (1)

• create_test_line (152-189)

stationapi/src/use_case/interactor/query.rs (1)

• create_test_line (1207-1244)

stationapi/src/domain/entity/train_type.rs (1)

• create_test_line (66-103)

stationapi/src/presentation/controller/grpc.rs (3)

• test_convert_transport_type_rail (406-410)
• convert_transport_type (22-29)
• test_convert_transport_type_bus (413-417)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Test Suite

🔇 Additional comments (4)

stationapi/src/use_case/dto/line.rs (4)

8-14: 実装は正確で、コメントも明確です。

バス路線のline_typeを強制的にOtherLineType(0)に設定するロジックは正しく実装されています。鉄道路線ではunwrap_or_default()を使用して既存の値を保持し、Noneの場合はデフォルト値(0)を返すようになっています。

Also applies to: 25-25

---

39-44: ヘルパー関数の実装は正しいです。

convert_transport_type関数はTransportTypeからGrpcTransportTypeへのマッピングを正確に行っています。

---

46-221: テストカバレッジが非常に包括的です。

バス路線と鉄道路線の両方について、様々なline_type値の変換ロジックを網羅的にテストしています。エッジケース（None値、境界値）も適切にカバーされています。

---

227-366: テスト実装は適切で、コーディングガイドラインに準拠しています。

convert_transport_type、Line から GrpcLine への変換、境界値、エッジケースのテストすべてが正しく実装されています。以下の項目を確認しました：

• フィールドマッピング：id、名前、色などの基本フィールドが正しく変換される
• オプショナルフィールド：None 値が適切にデフォルト値に変換される
• 関連オブジェクト：station、company、train_type が正しく None になる
• 境界値：line_type 0-5 が各 GrpcLineType に正確に変換される
• エッジケース：
  • 鉄道路線の未知の line_type（99）は保持される
  • バス路線の未知の line_type（99）は OtherLineType になる
• 型変換：line_cd が u32 に正しく変換される
• コード形式：cargo fmt チェック合格

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}