Merge pull request #9 from DCas/bug/status-image-format

Bug/status image format

Author: mlocher

lib/codeship/status.rb

@@ -14,7 +14,7 @@ def initialize(uuid, options = {})
     end
 
     def status
-      image.scan(/status_(.*).png/).flatten.first.to_sym
+      image.scan(/status_(.*).(png|gif)/).flatten.first.to_sym
     end
 
     private

spec/status_spec.rb

@@ -2,28 +2,59 @@
 
 RSpec.describe Codeship::Status do
   context 'parsing the project status' do
-    Codeship::Status::STATES.each do |state|
-      it "should parse #{state}" do
+    context 'of a project with completed build' do
+      Codeship::Status::STATES.each do |state|
+        it "should parse #{state}" do
 
-        stub_request(:head, "https://codeship.com/projects/#{state}/status").
-                 with(headers: {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
-                 to_return(status: 200, body: "", headers: {'Content-Disposition' => "inline; filename=\"status_#{state}.png\""})
+          stub_request(:head, "https://codeship.com/projects/#{state}/status").
+                   with(headers: {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
+                   to_return(status: 200, body: "", headers: {'Content-Disposition' => "inline; filename=\"status_#{state}.png\""})
 
-        project_status = Codeship::Status.new(state)
-        expect(project_status.status).to eq(state)
+          project_status = Codeship::Status.new(state)
+          expect(project_status.status).to eq(state)
+        end
+      end
+    end
+
+    context 'of a project with active build' do
+      Codeship::Status::STATES.each do |state|
+        it "should parse #{state}" do
+
+          stub_request(:head, "https://codeship.com/projects/#{state}/status").
+                   with(headers: {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
+                   to_return(status: 200, body: "", headers: {'Content-Disposition' => "inline; filename=\"status_#{state}.gif\""})
+
+          project_status = Codeship::Status.new(state)
+          expect(project_status.status).to eq(state)
+        end
       end
     end
   end
 
   context 'parsing the project status on a certain branch' do
-    Codeship::Status::STATES.each do |state|
-      it "should parse #{state}" do
-        stub_request(:head, "https://codeship.com/projects/#{state}/status?branch=master").
-                 with(headers: {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
-                 to_return(status: 200, body: "", headers: {'Content-Disposition' => "inline; filename=\"status_#{state}.png\""})
-
-        project_status = Codeship::Status.new(state, branch: 'master')
-        expect(project_status.status).to eq(state)
+    context 'of a project with completed build' do
+      Codeship::Status::STATES.each do |state|
+        it "should parse #{state}" do
+          stub_request(:head, "https://codeship.com/projects/#{state}/status?branch=master").
+                   with(headers: {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
+                   to_return(status: 200, body: "", headers: {'Content-Disposition' => "inline; filename=\"status_#{state}.png\""})
+
+          project_status = Codeship::Status.new(state, branch: 'master')
+          expect(project_status.status).to eq(state)
+        end
+      end
+    end
+
+    context 'of a project with active build' do
+      Codeship::Status::STATES.each do |state|
+        it "should parse #{state}" do
+          stub_request(:head, "https://codeship.com/projects/#{state}/status?branch=master").
+                   with(headers: {'Accept'=>'*/*', 'User-Agent'=>'Ruby'}).
+                   to_return(status: 200, body: "", headers: {'Content-Disposition' => "inline; filename=\"status_#{state}.gif\""})
+
+          project_status = Codeship::Status.new(state, branch: 'master')
+          expect(project_status.status).to eq(state)
+        end
       end
     end
   end