Commit bba8e59a authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Fix test hook and tests

parent 4e7df003
...@@ -24,7 +24,7 @@ class Projects::ServicesController < Projects::ApplicationController ...@@ -24,7 +24,7 @@ class Projects::ServicesController < Projects::ApplicationController
end end
def test def test
data = Gitlab::PushDataBuilder.build(project, current_user) data = Gitlab::PushDataBuilder.build_sample(project, current_user)
@service.execute(data) @service.execute(data)
redirect_to :back redirect_to :back
......
class TestHookService class TestHookService
def execute(hook, current_user) def execute(hook, current_user)
data = Gitlab::PushDataBuilder.build(hook.project, current_user) data = Gitlab::PushDataBuilder.build_sample(hook.project, current_user)
hook.execute(data) hook.execute(data)
end end
end end
...@@ -33,7 +33,7 @@ describe AssemblaService, models: true do ...@@ -33,7 +33,7 @@ describe AssemblaService, models: true do
token: 'verySecret', token: 'verySecret',
subdomain: 'project_name' subdomain: 'project_name'
) )
@sample_data = GitPushService.new.sample_data(project, user) @sample_data = Gitlab::PushDataBuilder.build_sample(project, user)
@api_url = 'https://atlas.assembla.com/spaces/project_name/github_tool?secret_key=verySecret' @api_url = 'https://atlas.assembla.com/spaces/project_name/github_tool?secret_key=verySecret'
WebMock.stub_request(:post, @api_url) WebMock.stub_request(:post, @api_url)
end end
......
...@@ -32,7 +32,7 @@ describe FlowdockService do ...@@ -32,7 +32,7 @@ describe FlowdockService do
service_hook: true, service_hook: true,
token: 'verySecret' token: 'verySecret'
) )
@sample_data = GitPushService.new.sample_data(project, user) @sample_data = Gitlab::PushDataBuilder.build_sample(project, user)
@api_url = 'https://api.flowdock.com/v1/git/verySecret' @api_url = 'https://api.flowdock.com/v1/git/verySecret'
WebMock.stub_request(:post, @api_url) WebMock.stub_request(:post, @api_url)
end end
......
...@@ -33,7 +33,7 @@ describe GemnasiumService do ...@@ -33,7 +33,7 @@ describe GemnasiumService do
token: 'verySecret', token: 'verySecret',
api_key: 'GemnasiumUserApiKey' api_key: 'GemnasiumUserApiKey'
) )
@sample_data = GitPushService.new.sample_data(project, user) @sample_data = Gitlab::PushDataBuilder.build_sample(project, user)
end end
it "should call Gemnasium service" do it "should call Gemnasium service" do
Gemnasium::GitlabService.should_receive(:execute).with(an_instance_of(Hash)).once Gemnasium::GitlabService.should_receive(:execute).with(an_instance_of(Hash)).once
......
...@@ -36,7 +36,7 @@ describe PushoverService do ...@@ -36,7 +36,7 @@ describe PushoverService do
let(:pushover) { PushoverService.new } let(:pushover) { PushoverService.new }
let(:user) { create(:user) } let(:user) { create(:user) }
let(:project) { create(:project) } let(:project) { create(:project) }
let(:sample_data) { GitPushService.new.sample_data(project, user) } let(:sample_data) { Gitlab::PushDataBuilder.build_sample(project, user) }
let(:api_key) { 'verySecret' } let(:api_key) { 'verySecret' }
let(:user_key) { 'verySecret' } let(:user_key) { 'verySecret' }
......
...@@ -34,7 +34,7 @@ describe SlackService do ...@@ -34,7 +34,7 @@ describe SlackService do
let(:slack) { SlackService.new } let(:slack) { SlackService.new }
let(:user) { create(:user) } let(:user) { create(:user) }
let(:project) { create(:project) } let(:project) { create(:project) }
let(:sample_data) { GitPushService.new.sample_data(project, user) } let(:sample_data) { Gitlab::PushDataBuilder.build_sample(project, user) }
let(:webhook_url) { 'https://hooks.slack.com/services/SVRWFV0VVAR97N/B02R25XN3/ZBqu7xMupaEEICInN685' } let(:webhook_url) { 'https://hooks.slack.com/services/SVRWFV0VVAR97N/B02R25XN3/ZBqu7xMupaEEICInN685' }
before do before do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment