fix most of warnings

16 files changed, 41 insertions, 39 deletions

diff --git a/spec/contexts/fork_context_spec.rb b/spec/contexts/fork_context_spec.rb
index ed51b0c3f8e..70f650bc83d 100644
--- a/spec/contexts/fork_context_spec.rb
+++ b/spec/contexts/fork_context_spec.rb
@@ -48,7 +48,7 @@ describe Projects::ForkContext do
 
   def fork_project(from_project, user, fork_success = true)
     context = Projects::ForkContext.new(from_project, user)
-    shell = mock("gitlab_shell")
+    shell = double("gitlab_shell")
     shell.stub(fork_repository: fork_success)
     context.stub(gitlab_shell: shell)
     context.execute
diff --git a/spec/controllers/application_controller_spec.rb b/spec/controllers/application_controller_spec.rb
index d528d12c66c..e1c0269b295 100644
--- a/spec/controllers/application_controller_spec.rb
+++ b/spec/controllers/application_controller_spec.rb
@@ -8,7 +8,7 @@ describe ApplicationController do
     it 'should redirect if the user is over their password expiry' do
       user.password_expires_at = Time.new(2002)
       user.ldap_user?.should be_false
-      controller.stub!(:current_user).and_return(user)
+      controller.stub(:current_user).and_return(user)
       controller.should_receive(:redirect_to)
       controller.should_receive(:new_profile_password_path)
       controller.send(:check_password_expiration)
@@ -17,15 +17,15 @@ describe ApplicationController do
     it 'should not redirect if the user is under their password expiry' do
       user.password_expires_at = Time.now + 20010101
       user.ldap_user?.should be_false
-      controller.stub!(:current_user).and_return(user)
+      controller.stub(:current_user).and_return(user)
       controller.should_not_receive(:redirect_to)
       controller.send(:check_password_expiration)
     end
 
     it 'should not redirect if the user is over their password expiry but they are an ldap user' do
       user.password_expires_at = Time.new(2002)
-      user.stub!(:ldap_user?).and_return(true)
-      controller.stub!(:current_user).and_return(user)
+      user.stub(:ldap_user?).and_return(true)
+      controller.stub(:current_user).and_return(user)
       controller.should_not_receive(:redirect_to)
       controller.send(:check_password_expiration)
     end
diff --git a/spec/helpers/application_helper_spec.rb b/spec/helpers/application_helper_spec.rb
index d63a2de8806..21c3e15f57b 100644
--- a/spec/helpers/application_helper_spec.rb
+++ b/spec/helpers/application_helper_spec.rb
@@ -3,7 +3,7 @@ require 'spec_helper'
 describe ApplicationHelper do
   describe 'current_controller?' do
     before do
-      controller.stub!(:controller_name).and_return('foo')
+      controller.stub(:controller_name).and_return('foo')
     end
 
     it "returns true when controller matches argument" do
@@ -22,7 +22,7 @@ describe ApplicationHelper do
 
   describe 'current_action?' do
     before do
-      stub!(:action_name).and_return('foo')
+      allow(self).to receive(:action_name).and_return('foo')
     end
 
     it "returns true when action matches argument" do
@@ -52,7 +52,7 @@ describe ApplicationHelper do
     it "should call gravatar_icon when no avatar is present" do
       user = create(:user)
       user.save!
-      stub!(:gravatar_icon).and_return('gravatar_method_called')
+      allow(self).to receive(:gravatar_icon).and_return('gravatar_method_called')
       avatar_icon(user.email).to_s.should == "gravatar_method_called"
     end
   end
@@ -70,33 +70,33 @@ describe ApplicationHelper do
     end
 
     it "should return default gravatar url" do
-      stub!(:request).and_return(double(:ssl? => false))
+      allow(self).to receive(:request).and_return(double(:ssl? => false))
       gravatar_icon(user_email).should match('http://www.gravatar.com/avatar/b58c6f14d292556214bd64909bcdb118')
     end
 
     it "should use SSL when appropriate" do
-      stub!(:request).and_return(double(:ssl? => true))
+      allow(self).to receive(:request).and_return(double(:ssl? => true))
       gravatar_icon(user_email).should match('https://secure.gravatar.com')
     end
 
     it "should return custom gravatar path when gravatar_url is set" do
-      stub!(:request).and_return(double(:ssl? => false))
+      allow(self).to receive(:request).and_return(double(:ssl? => false))
       Gitlab.config.gravatar.stub(:plain_url).and_return('http://example.local/?s=%{size}&hash=%{hash}')
       gravatar_icon(user_email, 20).should == 'http://example.local/?s=20&hash=b58c6f14d292556214bd64909bcdb118'
     end
 
     it "should accept a custom size" do
-      stub!(:request).and_return(double(:ssl? => false))
+      allow(self).to receive(:request).and_return(double(:ssl? => false))
       gravatar_icon(user_email, 64).should match(/\?s=64/)
     end
 
     it "should use default size when size is wrong" do
-      stub!(:request).and_return(double(:ssl? => false))
+      allow(self).to receive(:request).and_return(double(:ssl? => false))
       gravatar_icon(user_email, nil).should match(/\?s=40/)
     end
 
     it "should be case insensitive" do
-      stub!(:request).and_return(double(:ssl? => false))
+      allow(self).to receive(:request).and_return(double(:ssl? => false))
       gravatar_icon(user_email).should == gravatar_icon(user_email.upcase + " ")
     end
 
@@ -105,7 +105,7 @@ describe ApplicationHelper do
   describe "user_color_scheme_class" do
     context "with current_user is nil" do
       it "should return a string" do
-        stub!(:current_user).and_return(nil)
+        allow(self).to receive(:current_user).and_return(nil)
         user_color_scheme_class.should be_kind_of(String)
       end
     end
@@ -115,7 +115,7 @@ describe ApplicationHelper do
         context "with color_scheme_id == #{color_scheme_id}" do
           it "should return a string" do
             current_user = double(:color_scheme_id => color_scheme_id)
-            stub!(:current_user).and_return(current_user)
+            allow(self).to receive(:current_user).and_return(current_user)
             user_color_scheme_class.should be_kind_of(String)
           end
         end
diff --git a/spec/helpers/gitlab_markdown_helper_spec.rb b/spec/helpers/gitlab_markdown_helper_spec.rb
index 0c25fa66ece..33e69d4326c 100644
--- a/spec/helpers/gitlab_markdown_helper_spec.rb
+++ b/spec/helpers/gitlab_markdown_helper_spec.rb
@@ -435,7 +435,7 @@ describe GitlabMarkdownHelper do
 
   describe "#render_wiki_content" do
     before do
-      @wiki = stub('WikiPage')
+      @wiki = double('WikiPage')
       @wiki.stub(:content).and_return('wiki content')
     end
 
@@ -449,7 +449,7 @@ describe GitlabMarkdownHelper do
 
     it "should use the Gollum renderer for all other file types" do
       @wiki.stub(:format).and_return(:rdoc)
-      formatted_content_stub = stub('formatted_content')
+      formatted_content_stub = double('formatted_content')
       formatted_content_stub.should_receive(:html_safe)
       @wiki.stub(:formatted_content).and_return(formatted_content_stub)
 
diff --git a/spec/helpers/notifications_helper_spec.rb b/spec/helpers/notifications_helper_spec.rb
index 328f66237c3..c1efc1fb2a0 100644
--- a/spec/helpers/notifications_helper_spec.rb
+++ b/spec/helpers/notifications_helper_spec.rb
@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe NotificationsHelper do
   describe 'notification_icon' do
-    let(:notification) { stub(disabled?: false, participating?: false, watch?: false) }
+    let(:notification) { double(disabled?: false, participating?: false, watch?: false) }
 
     context "disabled notification" do
       before { notification.stub(disabled?: true) }
diff --git a/spec/helpers/search_helper_spec.rb b/spec/helpers/search_helper_spec.rb
index 55b6b6b4dad..33ecb980202 100644
--- a/spec/helpers/search_helper_spec.rb
+++ b/spec/helpers/search_helper_spec.rb
@@ -8,7 +8,9 @@ describe SearchHelper do
 
   describe 'search_autocomplete_source' do
     context "with no current user" do
-      before { stub!(:current_user).and_return(nil) }
+      before do
+        allow(self).to receive(:current_user).and_return(nil)
+      end
 
       it "it returns nil" do
         search_autocomplete_source.should be_nil
@@ -20,7 +22,7 @@ describe SearchHelper do
       let(:result) { JSON.parse(search_autocomplete_source) }
 
       before do
-        stub!(:current_user).and_return(user)
+        allow(self).to receive(:current_user).and_return(user)
       end
 
       it "includes Help sections" do
diff --git a/spec/helpers/tab_helper_spec.rb b/spec/helpers/tab_helper_spec.rb
index ef8e4cf6375..fa8a3f554f7 100644
--- a/spec/helpers/tab_helper_spec.rb
+++ b/spec/helpers/tab_helper_spec.rb
@@ -5,8 +5,8 @@ describe TabHelper do
 
   describe 'nav_link' do
     before do
-      controller.stub!(:controller_name).and_return('foo')
-      stub!(:action_name).and_return('foo')
+      controller.stub(:controller_name).and_return('foo')
+      allow(self).to receive(:action_name).and_return('foo')
     end
 
     it "captures block output" do
diff --git a/spec/lib/extracts_path_spec.rb b/spec/lib/extracts_path_spec.rb
index aac72c63ea5..7b3818ea5c8 100644
--- a/spec/lib/extracts_path_spec.rb
+++ b/spec/lib/extracts_path_spec.rb
@@ -7,7 +7,7 @@ describe ExtractsPath do
 
   before do
     @project = project
-    project.stub(repository: stub(ref_names: ['master', 'foo/bar/baz', 'v1.0.0', 'v2.0.0']))
+    project.stub(repository: double(ref_names: ['master', 'foo/bar/baz', 'v1.0.0', 'v2.0.0']))
     project.stub(path_with_namespace: 'gitlab/gitlab-ci')
   end
 
diff --git a/spec/lib/gitlab/ldap/ldap_user_auth_spec.rb b/spec/lib/gitlab/ldap/ldap_user_auth_spec.rb
index b1c583c0476..b7d7bbaad2e 100644
--- a/spec/lib/gitlab/ldap/ldap_user_auth_spec.rb
+++ b/spec/lib/gitlab/ldap/ldap_user_auth_spec.rb
@@ -6,7 +6,7 @@ describe Gitlab::LDAP do
   before do
     Gitlab.config.stub(omniauth: {})
 
-    @info = mock(
+    @info = double(
       uid: '12djsak321',
       name: 'John',
       email: 'john@mail.com'
@@ -15,7 +15,7 @@ describe Gitlab::LDAP do
 
   describe :find_for_ldap_auth do
     before do
-      @auth = mock(
+      @auth = double(
         uid: '12djsak321',
         info: @info,
         provider: 'ldap'
diff --git a/spec/lib/oauth_spec.rb b/spec/lib/oauth_spec.rb
index e21074554b6..3dfe95a8e38 100644
--- a/spec/lib/oauth_spec.rb
+++ b/spec/lib/oauth_spec.rb
@@ -6,7 +6,7 @@ describe Gitlab::OAuth::User do
   before do
     Gitlab.config.stub(omniauth: {})
 
-    @info = mock(
+    @info = double(
       uid: '12djsak321',
       name: 'John',
       email: 'john@mail.com'
@@ -15,7 +15,7 @@ describe Gitlab::OAuth::User do
 
   describe :create do
     it "should create user from LDAP" do
-      @auth = mock(info: @info, provider: 'ldap')
+      @auth = double(info: @info, provider: 'ldap')
       user = gl_auth.create(@auth)
 
       user.should be_valid
@@ -24,7 +24,7 @@ describe Gitlab::OAuth::User do
     end
 
     it "should create user from Omniauth" do
-      @auth = mock(info: @info, provider: 'twitter')
+      @auth = double(info: @info, provider: 'twitter')
       user = gl_auth.create(@auth)
 
       user.should be_valid
@@ -33,7 +33,7 @@ describe Gitlab::OAuth::User do
     end
 
     it "should apply defaults to user" do
-      @auth = mock(info: @info, provider: 'ldap')
+      @auth = double(info: @info, provider: 'ldap')
       user = gl_auth.create(@auth)
 
       user.should be_valid
diff --git a/spec/models/concerns/issuable_spec.rb b/spec/models/concerns/issuable_spec.rb
index 852146ebaec..0827e4f162b 100644
--- a/spec/models/concerns/issuable_spec.rb
+++ b/spec/models/concerns/issuable_spec.rb
@@ -34,7 +34,7 @@ describe Issue, "Issuable" do
     let!(:searchable_issue) { create(:issue, title: "Searchable issue") }
 
     it "matches by title" do
-      described_class.search('able').all.should == [searchable_issue]
+      described_class.search('able').should == [searchable_issue]
     end
   end
 
diff --git a/spec/models/event_spec.rb b/spec/models/event_spec.rb
index 85bdf08ae64..53ede0d5ee9 100644
--- a/spec/models/event_spec.rb
+++ b/spec/models/event_spec.rb
@@ -67,12 +67,12 @@ describe Event do
   end
 
   describe 'Team events' do
-    let(:user_project) { stub.as_null_object }
+    let(:user_project) { double.as_null_object }
     let(:observer) { UsersProjectObserver.instance }
 
     before {
       Event.should_receive :create
-      observer.stub(notification: stub.as_null_object)
+      observer.stub(notification: double.as_null_object)
     }
 
     describe "Joined project team" do
diff --git a/spec/models/forked_project_link_spec.rb b/spec/models/forked_project_link_spec.rb
index 44b8c6155be..472ddf1b59d 100644
--- a/spec/models/forked_project_link_spec.rb
+++ b/spec/models/forked_project_link_spec.rb
@@ -59,7 +59,7 @@ end
 
 def fork_project(from_project, user)
   context = Projects::ForkContext.new(from_project, user)
-  shell = mock("gitlab_shell")
+  shell = double("gitlab_shell")
   shell.stub(fork_repository: true)
   context.stub(gitlab_shell: shell)
   context.execute
diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb
index 3a524158a47..039a0c087ba 100644
--- a/spec/models/merge_request_spec.rb
+++ b/spec/models/merge_request_spec.rb
@@ -107,9 +107,9 @@ describe MergeRequest do
   describe 'detection of issues to be closed' do
     let(:issue0) { create :issue, project: subject.project }
     let(:issue1) { create :issue, project: subject.project }
-    let(:commit0) { mock('commit0', closes_issues: [issue0]) }
-    let(:commit1) { mock('commit1', closes_issues: [issue0]) }
-    let(:commit2) { mock('commit2', closes_issues: [issue1]) }
+    let(:commit0) { double('commit0', closes_issues: [issue0]) }
+    let(:commit1) { double('commit1', closes_issues: [issue0]) }
+    let(:commit2) { double('commit2', closes_issues: [issue1]) }
 
     before do
       subject.stub(commits: [commit0, commit1, commit2])
diff --git a/spec/observers/note_observer_spec.rb b/spec/observers/note_observer_spec.rb
index f9b96c255c1..f8693355b23 100644
--- a/spec/observers/note_observer_spec.rb
+++ b/spec/observers/note_observer_spec.rb
@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe NoteObserver do
   subject { NoteObserver.instance }
-  before { subject.stub(notification: mock('NotificationService').as_null_object) }
+  before { subject.stub(notification: double('NotificationService').as_null_object) }
 
   let(:team_without_author) { (1..2).map { |n| double :user, id: n } }
   let(:note) { double(:note).as_null_object }
diff --git a/spec/requests/api/groups_spec.rb b/spec/requests/api/groups_spec.rb
index 25b9a10bd8c..1a5f11038b7 100644
--- a/spec/requests/api/groups_spec.rb
+++ b/spec/requests/api/groups_spec.rb
@@ -147,7 +147,7 @@ describe API::API do
   describe "POST /groups/:id/projects/:project_id" do
     let(:project) { create(:project) }
     before(:each) do
-      project.stub!(:transfer).and_return(true)
+      project.stub(:transfer).and_return(true)
       Project.stub(:find).and_return(project)
     end