test: is_activeカラム削除に伴うテストコードの修正

- Factory: is_activeを削除し、inactivated_atとinactiveトレイトを追加
- spec/models/stat_spec.rb: is_active参照を削除
- spec/requests/dojos_spec.rb: is_active参照をinactivated_atに変更
- spec/lib/tasks/dojos_spec.rb: is_activeテストをinactivated_atテストに置き換え
- spec/models/dojo_spec.rb: is_active関連のテストを削除・簡素化

これにより全158個のテストが成功することを確認

Author: yasulab

spec/factories/dojos.rb

@@ -8,7 +8,13 @@
     url           { 'https://example.com' }
     logo          { '/img/dojos/default.webp' }
     counter       { 1 }
-    is_active     { true }
     order         { 131001 }
+    # デフォルトはアクティブ（inactivated_at: nil）
+    inactivated_at { nil }
+    
+    # 非アクティブなDojoを作成するためのtrait
+    trait :inactive do
+      inactivated_at { Time.current }
+    end
   end
 end

spec/lib/tasks/dojos_spec.rb

@@ -54,7 +54,7 @@
       expect(new_records.first.url).to eq('http://tinkerkids.jp/coderdojo.html')
       expect(new_records.first.description).to eq('毎月開催')
       expect(new_records.first.tags).to eq(%w(Scratch ラズベリーパイ Python JavaScript))
-      expect(new_records.first.is_active).to eq(true)
+      expect(new_records.first.active?).to eq(true)
       expect(new_records.first.is_private).to eq(false)
     end
 
@@ -75,65 +75,69 @@
       expect(mod_record.name).to eq('dojo_1(mod)')
     end
 
-    context 'is_active' do
+    context 'inactivated_at (活性状態管理)' do
       let(:dojo_base) do
         @dojo_2.attributes.keep_if { |k,v| %w(id order name prefecture_id logo url description tags).include?(k) }
       end
 
-      it '指定なし ⇒ アクティブ' do
+      it 'inactivated_at 指定なし ⇒ アクティブ' do
         allow(YAML).to receive(:unsafe_load_file).and_return([
           dojo_base
         ])
 
         # before
-        @dojo_2.update_columns(is_active: false)
+        @dojo_2.update_columns(inactivated_at: Time.current)
         expect(Dojo.count).to eq(3)
-        expect(@dojo_2.is_active).to eq(false)
+        expect(@dojo_2.reload.active?).to eq(false)
 
         # exec
         expect(@rake[task].invoke).to be_truthy
 
         # after
         expect(Dojo.count).to eq(3)
         mod_record = Dojo.find_by(id: @dojo_2.id)
-        expect(mod_record.is_active).to eq(true)
+        expect(mod_record.active?).to eq(true)
+        expect(mod_record.inactivated_at).to be_nil
       end
 
-      it 'true 指定 ⇒ アクティブ' do
+      it 'inactivated_at: nil 指定 ⇒ アクティブ' do
         allow(YAML).to receive(:unsafe_load_file).and_return([
-          dojo_base.merge('is_active' => true)
+          dojo_base.merge('inactivated_at' => nil)
         ])
 
         # before
-        @dojo_2.update_columns(is_active: false)
+        @dojo_2.update_columns(inactivated_at: Time.current)
         expect(Dojo.count).to eq(3)
-        expect(@dojo_2.is_active).to eq(false)
+        expect(@dojo_2.reload.active?).to eq(false)
 
         # exec
         expect(@rake[task].invoke).to be_truthy
 
         # after
         expect(Dojo.count).to eq(3)
         mod_record = Dojo.find_by(id: @dojo_2.id)
-        expect(mod_record.is_active).to eq(true)
+        expect(mod_record.active?).to eq(true)
+        expect(mod_record.inactivated_at).to be_nil
       end
 
-      it 'false 指定 ⇒ 非アクティブ' do
+      it 'inactivated_at に日付指定 ⇒ 非アクティブ' do
+        inactivation_date = '2023-01-15'
         allow(YAML).to receive(:unsafe_load_file).and_return([
-          dojo_base.merge('is_active' => false)
+          dojo_base.merge('inactivated_at' => inactivation_date)
         ])
 
         # before
         expect(Dojo.count).to eq(3)
-        expect(@dojo_2.is_active).to eq(true)
+        expect(@dojo_2.active?).to eq(true)
 
         # exec
         expect(@rake[task].invoke).to be_truthy
 
         # after
         expect(Dojo.count).to eq(3)
         mod_record = Dojo.find_by(id: @dojo_2.id)
-        expect(mod_record.is_active).to eq(false)
+        expect(mod_record.active?).to eq(false)
+        expect(mod_record.inactivated_at).to eq(Time.zone.parse(inactivation_date))
       end
     end
 

spec/models/dojo_spec.rb

@@ -86,28 +86,12 @@
     end
   end
 
-  describe 'validate inactivated_at for inactive dojos' do
-    it 'ensures all inactive dojos in YAML have inactivated_at date' do
+  describe 'validate inactivated_at dates' do
+    it 'verifies inactivated_at dates are valid when present' do
       yaml_data = Dojo.load_attributes_from_yaml
-      inactive_dojos = yaml_data.select { |dojo| dojo['is_active'] == false }
-      
-      missing_dates = inactive_dojos.select { |dojo| dojo['inactivated_at'].nil? }
-      
-      if missing_dates.any?
-        missing_info = missing_dates.map { |d| "ID: #{d['id']} (#{d['name']})" }.join(", ")
-        fail "以下の非アクティブDojoにinactivated_atが設定されていません: #{missing_info}"
-      end
-      
-      expect(inactive_dojos.all? { |dojo| dojo['inactivated_at'].present? }).to be true
-    end
-    
-    it 'verifies inactivated_at dates are valid' do
-      yaml_data = Dojo.load_attributes_from_yaml
-      inactive_dojos = yaml_data.select { |dojo| dojo['is_active'] == false }
+      dojos_with_inactivated_at = yaml_data.select { |dojo| dojo['inactivated_at'].present? }
 
-      inactive_dojos.each do |dojo|
-        next if dojo['inactivated_at'].nil?
-        
+      dojos_with_inactivated_at.each do |dojo|
         # 日付が正しくパースできることを確認
         expect {
           Time.zone.parse(dojo['inactivated_at'])
@@ -124,32 +108,6 @@
         end
       end
     end
-    
-    it 'ensures all dojos with inactivated_at have is_active column' do
-      yaml_data = Dojo.load_attributes_from_yaml
-      dojos_with_inactivated_at = yaml_data.select { |dojo| dojo['inactivated_at'].present? }
-      
-      dojos_with_inactivated_at.each do |dojo|
-        # inactivated_atがあるDojoは必ずis_activeカラムを持つべき
-        # （再活性化されたDojoはis_active: trueの可能性があるため、値は問わない）
-        unless dojo.key?('is_active')
-          fail "ID: #{dojo['id']} (#{dojo['name']}) はinactivated_atを持っていますが、is_activeカラムがありません"
-        end
-      end
-      
-      # 統計情報として表示
-      if dojos_with_inactivated_at.any?
-        reactivated_count = dojos_with_inactivated_at.count { |d| d['is_active'] == true }
-        inactive_count = dojos_with_inactivated_at.count { |d| d['is_active'] == false }
-        
-        # テスト出力には表示されないが、デバッグ時に有用
-        # puts "inactivated_atを持つDojo数: #{dojos_with_inactivated_at.count}"
-        # puts "  - 現在非アクティブ: #{inactive_count}"
-        # puts "  - 再活性化済み: #{reactivated_count}"
-        
-        expect(dojos_with_inactivated_at.count).to eq(inactive_count + reactivated_count)
-      end
-    end
   end
 
   # inactivated_at カラムの基本的なテスト

spec/models/stat_spec.rb

@@ -15,7 +15,6 @@
         url: 'https://test1.coderdojo.jp',
         created_at: Time.zone.local(2020, 3, 1),
         prefecture_id: 13,
-        is_active: false,
         inactivated_at: Time.zone.local(2022, 6, 15)
       )
 
@@ -28,7 +27,6 @@
         url: 'https://test2.coderdojo.jp', 
         created_at: Time.zone.local(2021, 1, 1),
         prefecture_id: 13,
-        is_active: true,
         inactivated_at: nil
       )
 
@@ -41,7 +39,6 @@
         url: 'https://test3.coderdojo.jp',
         created_at: Time.zone.local(2019, 1, 1),
         prefecture_id: 13,
-        is_active: false,
         inactivated_at: Time.zone.local(2020, 3, 1)
       )
     end
@@ -95,7 +92,7 @@
         url: 'https://test1.coderdojo.jp',
         created_at: Time.zone.local(2012, 4, 1),
         prefecture_id: 13,
-        is_active: true
+        inactivated_at: nil
       )
 
       # 2022年に非アクティブ化される道場
@@ -107,7 +104,6 @@
         url: 'https://test2.coderdojo.jp',
         created_at: Time.zone.local(2019, 1, 1),
         prefecture_id: 14,
-        is_active: false,
         inactivated_at: Time.zone.local(2022, 6, 1)
       )
 
@@ -120,7 +116,6 @@
         url: 'https://test3.coderdojo.jp',
         created_at: Time.zone.local(2020, 1, 1),
         prefecture_id: 27,
-        is_active: false,
         inactivated_at: Time.zone.local(2023, 3, 1)
       )
 

spec/requests/dojos_spec.rb

@@ -8,37 +8,35 @@
       @dojo_2020_active = create(:dojo, 
         name: "Test Dojo 2020",
         created_at: "2020-06-01",
-        is_active: true
+        inactivated_at: nil
       )
 
       # 2020年に作成、2021年に非アクティブ化
       @dojo_2020_inactive = create(:dojo,
         name: "Test Dojo 2020 Inactive",
         created_at: "2020-01-01",
-        is_active: false,
         inactivated_at: "2021-03-01"
       )
 
       # 2021年に作成されたアクティブな道場
       @dojo_2021_active = create(:dojo,
         name: "Test Dojo 2021",
         created_at: "2021-01-01",
-        is_active: true
+        inactivated_at: nil
       )
 
       # 2019年に作成、2020年に非アクティブ化（2020年末時点では非アクティブ）
       @dojo_2019_inactive = create(:dojo,
         name: "Test Dojo 2019 Inactive",
         created_at: "2019-01-01",
-        is_active: false,
         inactivated_at: "2020-06-01"
       )
 
       # counter > 1 の道場
       @dojo_multi_branch = create(:dojo,
         name: "Multi Branch Dojo",
         created_at: "2020-01-01",
-        is_active: true,
+        inactivated_at: nil,
         counter: 3
       )
     end
@@ -151,7 +149,6 @@
 
           # 重要: この道場は2021年3月に非アクティブ化されたが、
           # 2020年末時点ではアクティブだったので、inactive-item クラスを持たないべき
-          # 現在のコードはここで失敗するはず（現在の is_active: false を使っているため）
           expect(dojo_row).not_to include('class="inactive-item"')
         end