From 8a0aeab846f5c97ba56d34644834e38f2378c7ce Mon Sep 17 00:00:00 2001
From: Lin Jen-Shin <godfat@godfat.org>
Date: Thu, 16 Jun 2016 23:33:53 +0800
Subject: [PATCH] Use active tense, feedback from:

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4093#note_12501303
---
 spec/models/ci/runner_spec.rb | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/spec/models/ci/runner_spec.rb b/spec/models/ci/runner_spec.rb
index 2b21c3561db..c7248ef1384 100644
--- a/spec/models/ci/runner_spec.rb
+++ b/spec/models/ci/runner_spec.rb
@@ -20,17 +20,17 @@ describe Ci::Runner, models: true do
   end
 
   describe '#display_name' do
-    it 'should return the description if it has a value' do
+    it 'returns the description if it has a value' do
       runner = FactoryGirl.build(:ci_runner, description: 'Linux/Ruby-1.9.3-p448')
       expect(runner.display_name).to eq 'Linux/Ruby-1.9.3-p448'
     end
 
-    it 'should return the token if it does not have a description' do
+    it 'returns the token if it does not have a description' do
       runner = FactoryGirl.create(:ci_runner)
       expect(runner.display_name).to eq runner.description
     end
 
-    it 'should return the token if the description is an empty string' do
+    it 'returns the token if the description is an empty string' do
       runner = FactoryGirl.build(:ci_runner, description: '', token: 'token')
       expect(runner.display_name).to eq runner.token
     end
@@ -277,13 +277,13 @@ describe Ci::Runner, models: true do
         runner.update(is_shared: true)
       end
 
-      context 'should not give owned runner' do
+      context 'does not give owned runner' do
         subject { Ci::Runner.available_for(project) }
 
         it { is_expected.to be_empty }
       end
 
-      context 'should not give shared runner' do
+      context 'does not give shared runner' do
         subject { Ci::Runner.available_for(another_project) }
 
         it { is_expected.to be_empty }
@@ -291,13 +291,13 @@ describe Ci::Runner, models: true do
     end
 
     context 'with unlocked runner' do
-      context 'should not give owned runner' do
+      context 'does not give owned runner' do
         subject { Ci::Runner.available_for(project) }
 
         it { is_expected.to be_empty }
       end
 
-      context 'should give a specific runner' do
+      context 'does give a specific runner' do
         subject { Ci::Runner.available_for(another_project) }
 
         it { is_expected.to contain_exactly(runner) }
@@ -309,13 +309,13 @@ describe Ci::Runner, models: true do
         runner.update(locked: true)
       end
 
-      context 'should not give owned runner' do
+      context 'does not give owned runner' do
         subject { Ci::Runner.available_for(project) }
 
         it { is_expected.to be_empty }
       end
 
-      context 'should not give a locked runner' do
+      context 'does not give a locked runner' do
         subject { Ci::Runner.available_for(another_project) }
 
         it { is_expected.to be_empty }