fix: 年フィルタリング時のアクティブ状態を選択年時点の状態に修正

## 問題
- /dojos?year=2023 で2024年に非アクティブ化された道場が灰色表示されていた
- 現在のis_activeを使用していたため、選択年時点の状態と異なっていた

## 解決策
- 選択年末時点でのアクティブ状態を計算するロジックを追加
- inactivated_atと選択年を比較して正しい状態を判定

## テスト
- inactive-item CSSクラスの存在をチェックするテストを追加
- TDDアプローチ：先にテストを書いて失敗を確認してから修正

## 今後の課題（このPRではやらない）
- inactive-item → inactivated-item へのCSS名変更
- is_activeカラムの削除（inactivated_atで代替可能）
- 変数名・コメントのinactive → inactivatedへの統一

Author: yasulab

app/controllers/dojos_controller.rb

@@ -32,14 +32,24 @@ def index
 
     @dojos = []
     dojos_scope.includes(:prefecture).order(is_active: :desc, order: :asc).each do |dojo|
+      # 年が選択されている場合は、その年末時点でのアクティブ状態を判定
+      # 選択されていない場合は、現在の is_active を使用
+      is_active_at_selected_time = if @selected_year
+        # その年末時点でアクティブだったかを判定
+        # inactivated_at が nil（まだアクティブ）または選択年より後に非アクティブ化
+        dojo.inactivated_at.nil? || dojo.inactivated_at > Time.zone.local(@selected_year).end_of_year
+      else
+        dojo.is_active
+      end
+      
       @dojos << {
         id:          dojo.id,
         url:         dojo.url,
         name:        dojo.name,
         logo:        root_url + dojo.logo[1..],
         order:       dojo.order,
         counter:     dojo.counter,
-        is_active:   dojo.is_active,
+        is_active:   is_active_at_selected_time,
         prefecture:  dojo.prefecture.name,
         created_at:  dojo.created_at,
         description: dojo.description,

spec/requests/dojos_spec.rb

@@ -128,6 +128,27 @@
           expect(dojo_ids).not_to include(@dojo_2021_active.id)
         end
 
+        it "does not show inactivated styling for dojos active in 2020" do
+          get dojos_path(year: 2020, format: :html)
+          
+          # HTMLレスポンスを取得
+          html = response.body
+          
+          # 2021年に非アクティブ化された道場（Test Dojo 2020 Inactive）が含まれていることを確認
+          expect(html).to include("Test Dojo 2020 Inactive")
+          
+          # その道場の行を探す（IDで特定）
+          dojo_row_match = html.match(/Test Dojo 2020 Inactive.*?<\/tr>/m)
+          expect(dojo_row_match).not_to be_nil
+          
+          dojo_row = dojo_row_match[0]
+          
+          # 重要: この道場は2021年3月に非アクティブ化されたが、
+          # 2020年末時点ではアクティブだったので、inactive-item クラスを持たないべき
+          # 現在のコードはここで失敗するはず（現在の is_active: false を使っているため）
+          expect(dojo_row).not_to include('class="inactive-item"')
+        end
+        
         it "filters correctly in CSV format" do
           get dojos_path(year: 2020, format: :csv)
 
@@ -154,6 +175,25 @@
           expect(dojo_ids).not_to include(@dojo_2020_inactive.id)
           expect(dojo_ids).not_to include(@dojo_2019_inactive.id)
         end
+        
+        it "does not show any inactivated dojos for year 2021" do
+          get dojos_path(year: 2021, format: :html)
+          
+          html = response.body
+          
+          # 2021年末時点でアクティブな道場のみが含まれる
+          expect(html).to include("Test Dojo 2020")      # アクティブ
+          expect(html).to include("Test Dojo 2021")      # アクティブ
+          expect(html).to include("Multi Branch Dojo")   # アクティブ
+          
+          # 2021年に非アクティブ化された道場は含まれない
+          expect(html).not_to include("Test Dojo 2020 Inactive")
+          expect(html).not_to include("Test Dojo 2019 Inactive")
+          
+          # すべての表示された道場は inactive-item クラスを持たないべき
+          # （2021年末時点ではすべてアクティブなので）
+          expect(html.scan('class="inactive-item"').count).to eq(0)
+        end
       end
     end