add functionality to calculate bonus for spare and strike. Tests green

lib/scorecard.rb

@@ -1,16 +1,46 @@
-class Scorecard
-  attr_accessor :scores, :total
-
+class Scorecard 
   def initialize
     @scores = []
   end
 
+  def scores
+    @scores
+  end
+
   def add_frame(rolls)
+    calculate_bonus(rolls) 
+
+    return @scores << { rolls: rolls } if rolls.sum == 10
+
     if @scores.empty?
       total = rolls.sum
     else
       total = @scores.last[:score] + rolls.sum
     end
     @scores << { rolls: rolls, score: total }
   end
+
+  private
+
+  def calculate_bonus(rolls)
+    return if @scores.empty?
+
+    if @scores.last[:rolls].include?(10)
+      strike_bonus(rolls)
+    elsif @scores.last[:rolls].sum == 10
+      spare_bonus(rolls)
+    end
+  end
+
+  def strike_bonus(rolls)
+    @scores.last[:score] = prev_score + 10 + rolls.sum
+  end
+
+  def spare_bonus(rolls)
+    @scores.last[:score] = prev_score + 10 + rolls[0]
+  end
+
+  def prev_score
+    return @scores[-2] ? @scores[-2][:score] : 0
+  end
 end

spec/scorecard_spec.rb

@@ -12,7 +12,7 @@
     expect(scorecard.scores).to eq [{ rolls: [3, 4], score: 7 }]
   end
 
-  it 'adds multiple frames' do
+  it 'adds multiple "open" frames' do
     scorecard = Scorecard.new
     scorecard.add_frame([3, 4])
     scorecard.add_frame([2, 0])
@@ -23,4 +23,65 @@
       { rolls: [6, 3], score: 18 }
     ]
   end
+
+  it 'adds a single strike' do
+    scorecard = Scorecard.new
+    scorecard.add_frame([10])
+    expect(scorecard.scores).to eq [{ rolls: [10] }]
+  end
+
+  it 'adds a single spare' do
+    scorecard = Scorecard.new
+    scorecard.add_frame([9, 1])
+    expect(scorecard.scores).to eq [{ rolls: [9, 1] }]
+  end
+
+  it 'adds multiple frames including a strike' do
+    scorecard = Scorecard.new
+    scorecard.add_frame([8, 1])
+    scorecard.add_frame([10])
+    scorecard.add_frame([3, 4])
+    expect(scorecard.scores).to eq [
+      { rolls: [8, 1], score: 9 },
+      { rolls: [10], score: 26 },
+      { rolls: [3, 4], score: 33 },
+    ]
+  end
+
+  it 'adds multiple frames including a spare' do
+    scorecard = Scorecard.new
+    scorecard.add_frame([9, 1])
+    scorecard.add_frame([3, 4])
+    expect(scorecard.scores).to eq [
+      { rolls: [9, 1], score: 13 },
+      { rolls: [3, 4], score: 20 },
+    ]
+  end
+
+  it 'correctly scores a gutter game' do
+    scorecard = Scorecard.new
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    scorecard.add_frame([0, 0])
+    expect(scorecard.scores.length).to eq 10
+    expect(scorecard.scores).to eq [
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 },
+      { rolls: [0, 0], score: 0 }
+    ]
+  end
 end