【テスト戦略】スピードと品質を両立させプロダクトを安定成長させる

<em># Fat Controllerの例</em>
class HealthCheckupsController < ApplicationController
  def create
    <em># ビジネスロジックがコントローラーに集中</em>
    @checkup = HealthCheckup.new(checkup_params)
    
    <em># 複雑な評価ロジック</em>
    if @checkup.bmi > 25 && @checkup.blood_pressure_high > 140
      @checkup.risk_level = calculate_risk_level(@checkup)
      @checkup.requires_consultation = true
    end
    
    <em># さらに複雑な処理が続く...</em>
    if @checkup.save
      send_notification(@checkup)
      redirect_to @checkup
    else
      render :new
    end
  end
end

<em># 単体テストの例</em>
RSpec.describe HealthCheckupRiskCalculator do
  describe '#calculate_risk_level' do
    context 'BMIが25を超え、血圧が140を超える場合' do
      it 'リスクレベル「高」を返すこと' do
        calculator = described_class.new(bmi: 26, blood_pressure_high: 145)
        expect(calculator.calculate_risk_level).to eq('high')
      end
    end
    
    context 'BMIが正常値の場合' do
      it 'リスクレベル「低」を返すこと' do
        calculator = described_class.new(bmi: 22, blood_pressure_high: 120)
        expect(calculator.calculate_risk_level).to eq('low')
      end
    end
    
    <em># 異常系のテストケース</em>
    context '不正な値が渡された場合' do
      it 'ArgumentErrorを発生させること' do
        expect {
          described_class.new(bmi: -1, blood_pressure_high: 140)
        }.to raise_error(ArgumentError)
      end
    end
  end
end

<em># 統合テストの例</em>
RSpec.describe 'HealthCheckups API', type: :request do
  describe 'POST /health_checkups' do
    context '正常系' do
      it '健康診断データを作成し、適切なレスポンスを返すこと' do
        post '/health_checkups', params: valid_params
        
        expect(response).to have_http_status(:created)
        expect(response.content_type).to include('application/json')
        expect(json_response['risk_level']).to eq('high')
      end
    end
  end
end

<em># システムテストの例</em>
RSpec.describe '健康診断登録フロー', type: :system do
  it 'ユーザーが健康診断データを登録できる' do
    visit new_health_checkup_path
    
    fill_in '身長', with: '170'
    fill_in '体重', with: '75'
    fill_in '血圧（高）', with: '145'
    
    click_button '登録'
    
    expect(page).to have_content('健康診断データを登録しました')
    expect(page).to have_content('リスクレベル: 高')
  end
end

<em># カバレッジは100%でも、バグを見逃す例</em>
def calculate_discount(price, customer_type)
  if customer_type == 'premium'
    price * 0.9  <em># 10%割引</em>
  else
    price
  end
end

<em># このテストはカバレッジ100%だが...</em>
it 'プレミアム顧客に割引を適用する' do
  expect(calculate_discount(1000, 'premium')).to eq(900)
end

<em># 境界値や異常系のテストが抜けている</em>
<em># - priceが0や負の値の場合は？</em>
<em># - customer_typeがnilやその他の値の場合は？</em>

class HealthCheckupsController < ApplicationController
  def create
    @checkup = HealthCheckup.new(checkup_params)
    
    <em># ビジネスロジックがコントローラーに集中</em>
    if @checkup.bmi > 25 && @checkup.blood_pressure_high > 140
      @checkup.risk_level = calculate_risk_level(@checkup)
      @checkup.requires_consultation = true
    end
    
    if @checkup.save
      send_notification(@checkup)
      redirect_to @checkup
    else
      render :new
    end
  end
  
  private
  
  def calculate_risk_level(checkup)
    <em># 複雑な計算ロジック...</em>
  end
end

<em># app/services/health_checkup_risk_calculator.rb</em>
class HealthCheckupRiskCalculator
  def initialize(bmi:, blood_pressure_high:)
    @bmi = bmi
    @blood_pressure_high = blood_pressure_high
  end
  
  def calculate_risk_level
    return 'high' if high_risk?
    return 'medium' if medium_risk?
    'low'
  end
  
  def requires_consultation?
    high_risk?
  end
  
  private
  
  def high_risk?
    @bmi > 25 && @blood_pressure_high > 140
  end
  
  def medium_risk?
    @bmi > 23 || @blood_pressure_high > 130
  end
end

<em># app/controllers/health_checkups_controller.rb</em>
class HealthCheckupsController < ApplicationController
  def create
    @checkup = HealthCheckup.new(checkup_params)
    
    <em># ビジネスロジックは専用のクラスに委譲</em>
    calculator = HealthCheckupRiskCalculator.new(
      bmi: @checkup.bmi,
      blood_pressure_high: @checkup.blood_pressure_high
    )
    
    @checkup.risk_level = calculator.calculate_risk_level
    @checkup.requires_consultation = calculator.requires_consultation?
    
    if @checkup.save
      HealthCheckupNotificationService.new(@checkup).send
      redirect_to @checkup
    else
      render :new
    end
  end
end

<em># 高速で安定した単体テストが書ける</em>
RSpec.describe HealthCheckupRiskCalculator do
  describe '#calculate_risk_level' do
    it 'BMIと血圧が高い場合、highを返す' do
      calculator = described_class.new(bmi: 26, blood_pressure_high: 145)
      expect(calculator.calculate_risk_level).to eq('high')
    end
    
    it 'BMIがやや高い場合、mediumを返す' do
      calculator = described_class.new(bmi: 24, blood_pressure_high: 120)
      expect(calculator.calculate_risk_level).to eq('medium')
    end
    
    it '正常値の場合、lowを返す' do
      calculator = described_class.new(bmi: 22, blood_pressure_high: 120)
      expect(calculator.calculate_risk_level).to eq('low')
    end
  end
  
  describe '#requires_consultation?' do
    it '高リスクの場合、trueを返す' do
      calculator = described_class.new(bmi: 26, blood_pressure_high: 145)
      expect(calculator.requires_consultation?).to be true
    end
  end
end