From aafd42c49d516d301d72a4f43d7e56d7bf168d80 Mon Sep 17 00:00:00 2001
From: Lin Jen-Shin <godfat@godfat.org>
Date: Wed, 14 Sep 2016 22:12:31 +0800
Subject: [PATCH] How could I forget this?

---
 spec/models/ci/pipeline_spec.rb | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/spec/models/ci/pipeline_spec.rb b/spec/models/ci/pipeline_spec.rb
index f564a2d4fdd..dc1d1de12a1 100644
--- a/spec/models/ci/pipeline_spec.rb
+++ b/spec/models/ci/pipeline_spec.rb
@@ -528,6 +528,7 @@ describe Ci::Pipeline, models: true do
     before do
       ActionMailer::Base.deliveries.clear
 
+      pipeline.update(user: create(:user))
       pipeline.enqueue
       pipeline.run
     end
@@ -548,9 +549,11 @@ describe Ci::Pipeline, models: true do
     context 'with success pipeline' do
       before do
         perform_enqueued_jobs do
-          pipeline.success
+          pipeline.succeed
         end
       end
+
+      it_behaves_like 'sending a notification'
     end
 
     context 'with failed pipeline' do
@@ -559,6 +562,8 @@ describe Ci::Pipeline, models: true do
           pipeline.drop
         end
       end
+
+      it_behaves_like 'sending a notification'
     end
 
     context 'with skipped pipeline' do
@@ -567,6 +572,8 @@ describe Ci::Pipeline, models: true do
           pipeline.skip
         end
       end
+
+      it_behaves_like 'not sending any notification'
     end
 
     context 'with cancelled pipeline' do
@@ -575,6 +582,8 @@ describe Ci::Pipeline, models: true do
           pipeline.cancel
         end
       end
+
+      it_behaves_like 'not sending any notification'
     end
   end
 end
-- 
GitLab