Merge pull request #9 from mizzy/set-build-output-and-status-before-a…

…fter_executes

Set build output and status before after executes

app/models/job.rb

@@ -114,7 +114,10 @@ def execute_without_before_executes
   end
 
   def execute_script
-    Altria::Executer.execute(script)
+    result = Altria::Executer.execute(script)
+    return unless result
+    current_build.update_attributes!(output: result[:output], status: result[:status])
+    result
   end
 
   def execute_before_enqueues

spec/factories/build.rb

@@ -3,7 +3,5 @@
 FactoryGirl.define do
   factory(:build) do
     job { FactoryGirl.create(:job) }
-    started_at { Time.now }
-    finished_at { Time.now }
   end
 end

spec/requests/builds_spec.rb

@@ -14,7 +14,7 @@
   end
 
   let(:build) do
-    FactoryGirl.create(:build, job: job)
+    FactoryGirl.create(:build, job: job, finished_at: Time.now)
   end
 
   describe "GET /jobs/:job_id/builds" do
@@ -31,7 +31,7 @@
     context "with page" do
       before do
         params[:page] = 2
-        10.times { FactoryGirl.create(:build, job: job) }
+        10.times { FactoryGirl.create(:build, job: job, finished_at: Time.now) }
       end
 
       it "paginates builds" do