Code Test Complete

award.rb

@@ -1 +1,52 @@
-Award = Struct.new(:name, :expires_in, :quality)
+class Award
+  attr_accessor :name, :expires_in, :quality
+
+  def initialize(name, expires_in, quality)
+    @name = name
+    @expires_in = expires_in
+    @quality = quality
+  end
+
+  def update_quality
+    return if name == 'Blue Distinction Plus'
+
+    case name
+    when 'Blue First'
+      update_blue_first
+    when 'Blue Compare'
+      update_blue_compare
+    else
+      update_normal
+    end
+
+    self.expires_in -= 1 unless name == 'Blue Distinction Plus'
+    self.quality = [quality, 0].max
+    self.quality = [quality, 50].min unless name == 'Blue Distinction Plus'
+  end
+
+  private
+
+  def update_blue_first
+    return if quality >= 50
+
+    self.quality += 1
+    self.quality += 1 if expires_in <= 0
+  end
+
+  def update_blue_compare
+    return self.quality = 0 if expires_in <= 0
+
+    quality_increase = 1
+    quality_increase += 1 if expires_in <= 10
+    quality_increase += 1 if expires_in <= 5
+    self.quality += quality_increase
+  end
+
+  def update_normal
+    return if quality <= 0
+
+    quality_degrade = expires_in <= 0 ? 2 : 1
+    quality_degrade *= 2 if name == 'Blue Star'
+    self.quality -= quality_degrade
+  end
+end

update_quality.rb

@@ -2,48 +2,6 @@
 
 def update_quality(awards)
   awards.each do |award|
-    if award.name != 'Blue First' && award.name != 'Blue Compare'
-      if award.quality > 0
-        if award.name != 'Blue Distinction Plus'
-          award.quality -= 1
-        end
-      end
-    else
-      if award.quality < 50
-        award.quality += 1
-        if award.name == 'Blue Compare'
-          if award.expires_in < 11
-            if award.quality < 50
-              award.quality += 1
-            end
-          end
-          if award.expires_in < 6
-            if award.quality < 50
-              award.quality += 1
-            end
-          end
-        end
-      end
-    end
-    if award.name != 'Blue Distinction Plus'
-      award.expires_in -= 1
-    end
-    if award.expires_in < 0
-      if award.name != 'Blue First'
-        if award.name != 'Blue Compare'
-          if award.quality > 0
-            if award.name != 'Blue Distinction Plus'
-              award.quality -= 1
-            end
-          end
-        else
-          award.quality = award.quality - award.quality
-        end
-      else
-        if award.quality < 50
-          award.quality += 1
-        end
-      end
-    end
+    award.update_quality
   end
-end
+end

update_quality_spec.rb

@@ -177,7 +177,6 @@
       end
 
       context 'given a Blue Star award' do
-        before { pending }
         let(:name) { 'Blue Star' }
         before { award.expires_in.should == initial_expires_in-1 }
 
@@ -234,4 +233,4 @@
       specify { expect(awards[1].expires_in).to eq(2) }
     end
   end
-end
+end