1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
|
# frozen_string_literal: true
require 'spec_helper'
RSpec.describe Import::FogbugzService, feature_category: :importers do
let_it_be(:user) { create(:user) }
let(:base_uri) { "https://test:7990" }
let(:token) { "asdasd12345" }
let(:repo_id) { "fogbugz_id" }
let(:repo) { instance_double(Gitlab::FogbugzImport::Repository, name: 'test', raw_data: nil) }
let(:client) { instance_double(Gitlab::FogbugzImport::Client) }
let(:credentials) { { uri: base_uri, token: token } }
let(:params) { { repo_id: repo_id } }
subject { described_class.new(client, user, params) }
before do
allow(subject).to receive(:authorized?).and_return(true)
end
context 'when no repo is found' do
before do
allow(client).to receive(:repo).with(repo_id).and_return(nil)
end
it 'returns an error' do
result = subject.execute(credentials)
expect(result).to include(
message: "Project #{repo_id} could not be found",
status: :error,
http_status: :unprocessable_entity
)
end
end
context 'when import source is disabled' do
before do
stub_application_setting(import_sources: nil)
allow(client).to receive(:repo).with(repo_id).and_return(repo)
end
it 'returns forbidden' do
result = subject.execute(credentials)
expect(result).to include(
status: :error,
http_status: :forbidden
)
end
end
context 'when user is unauthorized' do
before do
allow(subject).to receive(:authorized?).and_return(false)
end
it 'returns an error' do
result = subject.execute(credentials)
expect(result).to include(
message: "You don't have permissions to import this project",
status: :error,
http_status: :unauthorized
)
end
end
context 'verify url' do
shared_examples 'denies local request' do
before do
allow(client).to receive(:repo).with(repo_id).and_return(repo)
end
it 'does not allow requests' do
result = subject.execute(credentials)
expect(result[:status]).to eq(:error)
expect(result[:message]).to include("Invalid URL:")
end
end
context 'when host is localhost' do
let(:base_uri) { 'http://localhost:3000' }
include_examples 'denies local request'
end
context 'when host is on local network' do
let(:base_uri) { 'https://192.168.0.191' }
include_examples 'denies local request'
end
context 'when host is ftp protocol' do
let(:base_uri) { 'ftp://testing' }
include_examples 'denies local request'
end
end
context 'when import starts succesfully' do
before do
allow(client).to receive(:repo).with(repo_id).and_return(
instance_double(Gitlab::FogbugzImport::Repository, name: 'test', raw_data: nil)
)
end
it 'returns success' do
result = subject.execute(credentials)
expect(result[:status]).to eq(:success)
expect(result[:project].name).to eq('test')
end
end
context 'when import fails to start' do
let(:error_messages_array) { instance_double(Array, join: "something went wrong") }
let(:errors_double) { instance_double(ActiveModel::Errors, full_messages: error_messages_array, :[] => nil) }
let(:project_double) { instance_double(Project, persisted?: false, errors: errors_double) }
let(:project_creator) { instance_double(Gitlab::FogbugzImport::ProjectCreator, execute: project_double) }
before do
allow(Gitlab::FogbugzImport::ProjectCreator).to receive(:new).and_return(project_creator)
allow(client).to receive(:repo).with(repo_id).and_return(
instance_double(Gitlab::FogbugzImport::Repository, name: 'test', raw_data: nil)
)
end
it 'returns error' do
result = subject.execute(credentials)
expect(result[:status]).to eq(:error)
expect(result[:message]).to eq("something went wrong")
end
end
it 'returns error for unknown error causes' do
message = 'Not Implemented'
exception = StandardError.new(message)
allow(client).to receive(:repo).and_raise(exception)
expect(subject.execute(credentials)).to include({
status: :error,
message: "Fogbugz import failed due to an error: #{message}"
})
end
end
|
