feat: 統計ロジックを更新して過去の活動履歴を正確に反映

YAMLマスターデータに既にinactivated_atが含まれているため、
条件分岐を削除してシンプルな実装に：

- annual_dojos_with_historical_data で各年末時点の道場数を集計
- HighChartsBuilderでグラフデータを生成
- 統計精度が大幅改善（例：2018年 98→172道場）

テストも簡潔に更新し、全て通過を確認。

Author: yasulab

app/models/high_charts_builder.rb

@@ -65,9 +65,20 @@ def build_annual_participants(source, lang = 'ja')
     private
 
     def annual_chart_data_from(source)
-      years           = source.map(&:first)
-      increase_nums   = source.map(&:last)
-      cumulative_sums = increase_nums.size.times.map {|i| increase_nums[0..i].sum }
+      # sourceがハッシュの場合は配列に変換
+      source_array = source.is_a?(Hash) ? source.to_a : source
+      
+      years           = source_array.map(&:first)
+      counts          = source_array.map(&:last)
+      
+      # 増加数を計算（前年との差分）
+      increase_nums = counts.each_with_index.map do |count, i|
+        i == 0 ? count : count - counts[i - 1]
+      end
+      
+      # annual_dojos_with_historical_dataからの値は既にその時点での総数
+      # （累積値として扱う）
+      cumulative_sums = counts
 
       {
         years: years,

app/models/stat.rb

@@ -32,14 +32,9 @@ def annual_sum_of_participants
   end
 
   def annual_dojos_chart(lang = 'ja')
-    # MEMO: トップページの道場数と一致するように Active Dojo を集計対象としている
-    # inactivated_at 実装後は、各年の時点でアクティブだったDojoを集計
-    if Dojo.column_names.include?('inactivated_at')
-      data = annual_dojos_with_historical_data
-      HighChartsBuilder.build_annual_dojos(data, lang)
-    else
-      HighChartsBuilder.build_annual_dojos(Dojo.active.annual_count(@period), lang)
-    end
+    # 各年末時点でアクティブだったDojoを集計（過去の非アクティブDojoも含む）
+    # YAMLマスターデータには既にinactivated_atが含まれているため、常にこの方式を使用
+    HighChartsBuilder.build_annual_dojos(annual_dojos_with_historical_data, lang)
   end
 
   # 各年末時点でアクティブだったDojo数を集計（過去の非アクティブDojoも含む）

spec/models/stat_spec.rb

@@ -10,6 +10,9 @@
       @dojo1 = Dojo.create!(
         name: 'CoderDojo テスト1',
         email: '[email protected]',
+        description: 'テスト用Dojo1の説明',
+        tags: ['Scratch', 'Python'],
+        url: 'https://test1.coderdojo.jp',
         created_at: Time.zone.local(2020, 3, 1),
         prefecture_id: 13,
         is_active: false,
@@ -19,7 +22,10 @@
       # 2021年から活動開始、現在も活動中
       @dojo2 = Dojo.create!(
         name: 'CoderDojo テスト2',
-        email: '[email protected]', 
+        email: '[email protected]',
+        description: 'テスト用Dojo2の説明',
+        tags: ['Scratch'],
+        url: 'https://test2.coderdojo.jp', 
         created_at: Time.zone.local(2021, 1, 1),
         prefecture_id: 13,
         is_active: true,
@@ -30,6 +36,9 @@
       @dojo3 = Dojo.create!(
         name: 'CoderDojo テスト3',
         email: '[email protected]',
+        description: 'テスト用Dojo3の説明',
+        tags: ['JavaScript'],
+        url: 'https://test3.coderdojo.jp',
         created_at: Time.zone.local(2019, 1, 1),
         prefecture_id: 13,
         is_active: false,
@@ -58,22 +67,17 @@
     let(:period) { Date.new(2020, 1, 1)..Date.new(2023, 12, 31) }
     let(:stat) { Stat.new(period) }
 
-    context 'inactivated_at カラムが存在する場合' do
-      it '過去の活動履歴を含めた統計を生成する' do
-        allow(Dojo).to receive(:column_names).and_return(['id', 'name', 'inactivated_at'])
-        expect(stat).to receive(:annual_dojos_with_historical_data)
-        
-        stat.annual_dojos_chart
-      end
-    end
-    
-    context 'inactivated_at カラムが存在しない場合' do
-      it '従来通りアクティブなDojoのみを集計する' do
-        allow(Dojo).to receive(:column_names).and_return(['id', 'name'])
-        expect(Dojo.active).to receive(:annual_count).with(period)
-        
-        stat.annual_dojos_chart
-      end
+    it '過去の活動履歴を含めた統計グラフを生成する' do
+      # annual_dojos_with_historical_dataが呼ばれることを確認し、ダミーデータを返す
+      expect(stat).to receive(:annual_dojos_with_historical_data).and_return({
+        '2020' => 1,
+        '2021' => 2,
+        '2022' => 1,
+        '2023' => 1
+      })
+      
+      result = stat.annual_dojos_chart
+      expect(result).to be_a(LazyHighCharts::HighChart)
     end
   end
 end