fix: alert-info always displayed

[takaishi]

No description provided.

[claude]

Claude finished @takaishi's task —— View job

---

コードレビュー結果

このPull Requestをレビューしました。以下がフィードバックです：

レビュータスク

• 変更されたファイルを読み込んで変更内容を理解する
• コード品質とベストプラクティスをレビューする
• 潜在的なバグや問題をチェックする
• パフォーマンスとセキュリティの観点から確認する
• テストカバレッジについて確認する
• フィードバックをコメントに投稿する

1. 修正内容の理解

このPRは、app/views/proposals/show.html.erbにおいて、alert-infoのdiv要素が常に表示されてしまう問題を修正しています。

変更前:

<div class="alert alert-info %> show">
  <% if @conference.cfp_result_visible %>
    <%= link_to 'CFPは終了しました。採択結果はこちら', talks_path %>
  <% end %>
</div>

変更後:

<% if @conference.cfp_result_visible %>
  <div class="alert alert-info">
    <%= link_to 'CFPは終了しました。採択結果はこちら', talks_path %>
  </div>
<% end %>

2. 問題点の詳細

元のコードには以下の問題がありました：

• <div class="alert alert-info %> show"> という無効なHTMLクラス構文がありました（%> が誤って含まれていた）
• div要素が条件分岐の外側にあったため、@conference.cfp_result_visibleがfalseでも空のalert-infoボックスが表示されていました

3. 評価

✅ コード品質とベストプラクティス

• 修正は適切で、ERBの条件分岐が正しく適用されています
• HTMLの構文エラーが修正されています

✅ 潜在的なバグ

• 修正により、CFP結果が非表示の場合にalertボックスが正しく非表示になります
• 新たなバグは導入されていません

✅ パフォーマンス

• この変更によるパフォーマンスへの影響はありません

✅ セキュリティ

• セキュリティ上の懸念はありません

✅ テストカバレッジ

• 既存のテスト（spec/requests/proposals/proposals_show_spec.rb）が、この条件分岐をカバーしています
• includes_link_to_talksとnot_includes_link_to_talksのshared examplesが、修正された動作を正しく検証します

4. 総評

このPRは、明らかなバグを適切に修正しており、承認を推奨します。修正は最小限で、既存のテストによってカバーされているため、安全にマージできます。

[github-actions]

Review app

• https://dreamkast-dk-2613.dev.cloudnativedays.jp

[github-actions]

Simplecov Report

Covered	Threshold
64.36%	60%