BigW Consortium Gitlab

project_spec.rb 60.1 KB
Newer Older
gitlabhq committed
1 2
require 'spec_helper'

Douwe Maan committed
3
describe Project, models: true do
4
  describe 'associations' do
5 6 7 8
    it { is_expected.to belong_to(:group) }
    it { is_expected.to belong_to(:namespace) }
    it { is_expected.to belong_to(:creator).class_name('User') }
    it { is_expected.to have_many(:users) }
ubudzisz committed
9
    it { is_expected.to have_many(:services) }
10 11 12 13 14
    it { is_expected.to have_many(:events).dependent(:destroy) }
    it { is_expected.to have_many(:merge_requests).dependent(:destroy) }
    it { is_expected.to have_many(:issues).dependent(:destroy) }
    it { is_expected.to have_many(:milestones).dependent(:destroy) }
    it { is_expected.to have_many(:project_members).dependent(:destroy) }
15 16
    it { is_expected.to have_many(:users).through(:project_members) }
    it { is_expected.to have_many(:requesters).dependent(:destroy) }
17 18 19 20 21 22 23
    it { is_expected.to have_many(:notes).dependent(:destroy) }
    it { is_expected.to have_many(:snippets).class_name('ProjectSnippet').dependent(:destroy) }
    it { is_expected.to have_many(:deploy_keys_projects).dependent(:destroy) }
    it { is_expected.to have_many(:deploy_keys) }
    it { is_expected.to have_many(:hooks).dependent(:destroy) }
    it { is_expected.to have_many(:protected_branches).dependent(:destroy) }
    it { is_expected.to have_one(:forked_project_link).dependent(:destroy) }
24
    it { is_expected.to have_one(:slack_service).dependent(:destroy) }
Tiago Botelho committed
25
    it { is_expected.to have_one(:microsoft_teams_service).dependent(:destroy) }
26
    it { is_expected.to have_one(:mattermost_service).dependent(:destroy) }
27 28
    it { is_expected.to have_one(:pushover_service).dependent(:destroy) }
    it { is_expected.to have_one(:asana_service).dependent(:destroy) }
29
    it { is_expected.to have_many(:boards).dependent(:destroy) }
ubudzisz committed
30 31 32
    it { is_expected.to have_one(:campfire_service).dependent(:destroy) }
    it { is_expected.to have_one(:drone_ci_service).dependent(:destroy) }
    it { is_expected.to have_one(:emails_on_push_service).dependent(:destroy) }
33
    it { is_expected.to have_one(:pipelines_email_service).dependent(:destroy) }
ubudzisz committed
34 35 36 37 38
    it { is_expected.to have_one(:irker_service).dependent(:destroy) }
    it { is_expected.to have_one(:pivotaltracker_service).dependent(:destroy) }
    it { is_expected.to have_one(:hipchat_service).dependent(:destroy) }
    it { is_expected.to have_one(:flowdock_service).dependent(:destroy) }
    it { is_expected.to have_one(:assembla_service).dependent(:destroy) }
Kamil Trzcinski committed
39
    it { is_expected.to have_one(:slack_slash_commands_service).dependent(:destroy) }
40
    it { is_expected.to have_one(:mattermost_slash_commands_service).dependent(:destroy) }
ubudzisz committed
41 42 43 44 45 46 47 48 49 50 51
    it { is_expected.to have_one(:gemnasium_service).dependent(:destroy) }
    it { is_expected.to have_one(:buildkite_service).dependent(:destroy) }
    it { is_expected.to have_one(:bamboo_service).dependent(:destroy) }
    it { is_expected.to have_one(:teamcity_service).dependent(:destroy) }
    it { is_expected.to have_one(:jira_service).dependent(:destroy) }
    it { is_expected.to have_one(:redmine_service).dependent(:destroy) }
    it { is_expected.to have_one(:custom_issue_tracker_service).dependent(:destroy) }
    it { is_expected.to have_one(:bugzilla_service).dependent(:destroy) }
    it { is_expected.to have_one(:gitlab_issue_tracker_service).dependent(:destroy) }
    it { is_expected.to have_one(:external_wiki_service).dependent(:destroy) }
    it { is_expected.to have_one(:project_feature).dependent(:destroy) }
52
    it { is_expected.to have_one(:statistics).class_name('ProjectStatistics').dependent(:delete) }
ubudzisz committed
53 54 55
    it { is_expected.to have_one(:import_data).class_name('ProjectImportData').dependent(:destroy) }
    it { is_expected.to have_one(:last_event).class_name('Event') }
    it { is_expected.to have_one(:forked_from_project).through(:forked_project_link) }
Kamil Trzcinski committed
56
    it { is_expected.to have_many(:commit_statuses) }
57
    it { is_expected.to have_many(:pipelines) }
58 59 60
    it { is_expected.to have_many(:builds) }
    it { is_expected.to have_many(:runner_projects) }
    it { is_expected.to have_many(:runners) }
Kamil Trzcinski committed
61
    it { is_expected.to have_many(:active_runners) }
62 63
    it { is_expected.to have_many(:variables) }
    it { is_expected.to have_many(:triggers) }
64
    it { is_expected.to have_many(:pages_domains) }
65
    it { is_expected.to have_many(:labels).class_name('ProjectLabel').dependent(:destroy) }
ubudzisz committed
66
    it { is_expected.to have_many(:users_star_projects).dependent(:destroy) }
67 68
    it { is_expected.to have_many(:environments).dependent(:destroy) }
    it { is_expected.to have_many(:deployments).dependent(:destroy) }
69
    it { is_expected.to have_many(:todos).dependent(:destroy) }
ubudzisz committed
70 71 72 73 74
    it { is_expected.to have_many(:releases).dependent(:destroy) }
    it { is_expected.to have_many(:lfs_objects_projects).dependent(:destroy) }
    it { is_expected.to have_many(:project_group_links).dependent(:destroy) }
    it { is_expected.to have_many(:notification_settings).dependent(:destroy) }
    it { is_expected.to have_many(:forks).through(:forked_project_links) }
75
    it { is_expected.to have_many(:uploads).dependent(:destroy) }
76

77 78
    context 'after initialized' do
      it "has a project_feature" do
79
        expect(Project.new.project_feature).to be_present
80 81 82
      end
    end

83
    describe '#members & #requesters' do
84
      let(:project) { create(:empty_project, :public, :access_requestable) }
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
      let(:requester) { create(:user) }
      let(:developer) { create(:user) }
      before do
        project.request_access(requester)
        project.team << [developer, :developer]
      end

      describe '#members' do
        it 'includes members and exclude requesters' do
          member_user_ids = project.members.pluck(:user_id)

          expect(member_user_ids).to include(developer.id)
          expect(member_user_ids).not_to include(requester.id)
        end
      end

      describe '#requesters' do
        it 'does not include requesters' do
          requester_user_ids = project.requesters.pluck(:user_id)

          expect(requester_user_ids).to include(requester.id)
          expect(requester_user_ids).not_to include(developer.id)
        end
      end
    end
110 111 112 113 114

    describe '#boards' do
      it 'raises an error when attempting to add more than one board to the project' do
        subject.boards.build

115
        expect { subject.boards.build }.to raise_error(Project::BoardLimitExceeded, 'Number of permitted boards exceeded')
116 117 118
        expect(subject.boards.size).to eq 1
      end
    end
gitlabhq committed
119 120
  end

121 122 123 124 125 126
  describe 'modules' do
    subject { described_class }

    it { is_expected.to include_module(Gitlab::ConfigHelper) }
    it { is_expected.to include_module(Gitlab::ShellAdapter) }
    it { is_expected.to include_module(Gitlab::VisibilityLevel) }
127
    it { is_expected.to include_module(Gitlab::CurrentSettings) }
128 129
    it { is_expected.to include_module(Referable) }
    it { is_expected.to include_module(Sortable) }
130 131
  end

132
  describe 'validation' do
133
    let!(:project) { create(:empty_project) }
134

135 136
    it { is_expected.to validate_presence_of(:name) }
    it { is_expected.to validate_uniqueness_of(:name).scoped_to(:namespace_id) }
137
    it { is_expected.to validate_length_of(:name).is_at_most(255) }
138

139 140
    it { is_expected.to validate_presence_of(:path) }
    it { is_expected.to validate_uniqueness_of(:path).scoped_to(:namespace_id) }
141 142 143 144
    it { is_expected.to validate_length_of(:path).is_at_most(255) }

    it { is_expected.to validate_length_of(:description).is_at_most(2000) }

145
    it { is_expected.to validate_presence_of(:creator) }
146

147
    it { is_expected.to validate_presence_of(:namespace) }
148

149
    it { is_expected.to validate_presence_of(:repository_storage) }
150

151
    it 'does not allow new projects beyond user limits' do
152
      project2 = build(:empty_project)
153 154
      allow(project2).to receive(:creator).and_return(double(can_create_project?: false, projects_limit: 0).as_null_object)
      expect(project2).not_to be_valid
155
      expect(project2.errors[:limit_reached].first).to match(/Personal project creation is not allowed/)
156
    end
157 158 159

    describe 'wiki path conflict' do
      context "when the new path has been used by the wiki of other Project" do
160
        it 'has an error on the name attribute' do
161
          new_project = build_stubbed(:empty_project, namespace_id: project.namespace_id, path: "#{project.path}.wiki")
162 163 164 165 166 167 168

          expect(new_project).not_to be_valid
          expect(new_project.errors[:name].first).to eq('has already been taken')
        end
      end

      context "when the new wiki path has been used by the path of other Project" do
169
        it 'has an error on the name attribute' do
170 171
          project_with_wiki_suffix = create(:empty_project, path: 'foo.wiki')
          new_project = build_stubbed(:empty_project, namespace_id: project_with_wiki_suffix.namespace_id, path: 'foo')
172 173 174 175 176 177

          expect(new_project).not_to be_valid
          expect(new_project.errors[:name].first).to eq('has already been taken')
        end
      end
    end
178 179

    context 'repository storages inclussion' do
180
      let(:project2) { build(:empty_project, repository_storage: 'missing') }
181 182

      before do
183
        storages = { 'custom' => { 'path' => 'tmp/tests/custom_repositories' } }
184 185 186
        allow(Gitlab.config.repositories).to receive(:storages).and_return(storages)
      end

187
      it "does not allow repository storages that don't match a label in the configuration" do
188 189 190 191
        expect(project2).not_to be_valid
        expect(project2.errors[:repository_storage].first).to match(/is not included in the list/)
      end
    end
192

193
    it 'does not allow an invalid URI as import_url' do
194
      project2 = build(:empty_project, import_url: 'invalid://')
195 196 197 198

      expect(project2).not_to be_valid
    end

199
    it 'does allow a valid URI as import_url' do
200
      project2 = build(:empty_project, import_url: 'ssh://test@gitlab.com/project.git')
201

202 203
      expect(project2).to be_valid
    end
204

205
    it 'allows an empty URI' do
206
      project2 = build(:empty_project, import_url: '')
207

208
      expect(project2).to be_valid
209 210 211
    end

    it 'does not produce import data on an empty URI' do
212
      project2 = build(:empty_project, import_url: '')
213 214 215 216 217

      expect(project2.import_data).to be_nil
    end

    it 'does not produce import data on an invalid URI' do
218
      project2 = build(:empty_project, import_url: 'test://')
219 220 221

      expect(project2.import_data).to be_nil
    end
222

223 224 225 226 227 228 229 230 231 232 233 234 235 236
    it "does not allow blocked import_url localhost" do
      project2 = build(:empty_project, import_url: 'http://localhost:9000/t.git')

      expect(project2).to be_invalid
      expect(project2.errors[:import_url]).to include('imports are not allowed from that URL')
    end

    it "does not allow blocked import_url port" do
      project2 = build(:empty_project, import_url: 'http://github.com:25/t.git')

      expect(project2).to be_invalid
      expect(project2.errors[:import_url]).to include('imports are not allowed from that URL')
    end

237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255
    describe 'project pending deletion' do
      let!(:project_pending_deletion) do
        create(:empty_project,
               pending_delete: true)
      end
      let(:new_project) do
        build(:empty_project,
              name: project_pending_deletion.name,
              namespace: project_pending_deletion.namespace)
      end

      before do
        new_project.validate
      end

      it 'contains errors related to the project being deleted' do
        expect(new_project.errors.full_messages.first).to eq('The project is still being deleted. Please try again later.')
      end
    end
gitlabhq committed
256
  end
257

258 259 260 261 262 263 264 265 266
  describe 'default_scope' do
    it 'excludes projects pending deletion from the results' do
      project = create(:empty_project)
      create(:empty_project, pending_delete: true)

      expect(Project.all).to eq [project]
    end
  end

267
  describe 'project token' do
268
    it 'sets an random token if none provided' do
269 270
      project = FactoryGirl.create :empty_project, runners_token: ''
      expect(project.runners_token).not_to eq('')
271 272
    end

ubudzisz committed
273
    it 'does not set an random token if one provided' do
274 275
      project = FactoryGirl.create :empty_project, runners_token: 'my-token'
      expect(project.runners_token).to eq('my-token')
276 277
    end
  end
gitlabhq committed
278

279
  describe 'Respond to' do
280 281 282 283 284
    it { is_expected.to respond_to(:url_to_repo) }
    it { is_expected.to respond_to(:repo_exists?) }
    it { is_expected.to respond_to(:execute_hooks) }
    it { is_expected.to respond_to(:owner) }
    it { is_expected.to respond_to(:path_with_namespace) }
gitlabhq committed
285 286
  end

287 288 289 290 291 292 293
  describe 'delegation' do
    it { is_expected.to delegate_method(:add_guest).to(:team) }
    it { is_expected.to delegate_method(:add_reporter).to(:team) }
    it { is_expected.to delegate_method(:add_developer).to(:team) }
    it { is_expected.to delegate_method(:add_master).to(:team) }
  end

294
  describe '#to_reference' do
295
    let(:owner)     { create(:user, name: 'Gitlab') }
296
    let(:namespace) { create(:namespace, path: 'sample-namespace', owner: owner) }
297 298
    let(:project)   { create(:empty_project, path: 'sample-project', namespace: namespace) }
    let(:group)     { create(:group, name: 'Group', path: 'sample-group', owner: owner) }
299

300
    context 'when nil argument' do
301 302 303 304 305
      it 'returns nil' do
        expect(project.to_reference).to be_nil
      end
    end

306
    context 'when full is true' do
307
      it 'returns complete path to the project' do
308 309 310
        expect(project.to_reference(full: true)).to          eq 'sample-namespace/sample-project'
        expect(project.to_reference(project, full: true)).to eq 'sample-namespace/sample-project'
        expect(project.to_reference(group, full: true)).to   eq 'sample-namespace/sample-project'
311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330
      end
    end

    context 'when same project argument' do
      it 'returns nil' do
        expect(project.to_reference(project)).to be_nil
      end
    end

    context 'when cross namespace project argument' do
      let(:another_namespace_project) { create(:empty_project, name: 'another-project') }

      it 'returns complete path to the project' do
        expect(project.to_reference(another_namespace_project)).to eq 'sample-namespace/sample-project'
      end
    end

    context 'when same namespace / cross-project argument' do
      let(:another_project) { create(:empty_project, namespace: namespace) }

331
      it 'returns path to the project' do
332 333 334
        expect(project.to_reference(another_project)).to eq 'sample-project'
      end
    end
335

336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355
    context 'when different namespace / cross-project argument' do
      let(:another_namespace) { create(:namespace, path: 'another-namespace', owner: owner) }
      let(:another_project)   { create(:empty_project, path: 'another-project', namespace: another_namespace) }

      it 'returns full path to the project' do
        expect(project.to_reference(another_project)).to eq 'sample-namespace/sample-project'
      end
    end

    context 'when argument is a namespace' do
      context 'with same project path' do
        it 'returns path to the project' do
          expect(project.to_reference(namespace)).to eq 'sample-project'
        end
      end

      context 'with different project path' do
        it 'returns full path to the project' do
          expect(project.to_reference(group)).to eq 'sample-namespace/sample-project'
        end
356 357
      end
    end
358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390
  end

  describe '#to_human_reference' do
    let(:owner) { create(:user, name: 'Gitlab') }
    let(:namespace) { create(:namespace, name: 'Sample namespace', owner: owner) }
    let(:project) { create(:empty_project, name: 'Sample project', namespace: namespace) }

    context 'when nil argument' do
      it 'returns nil' do
        expect(project.to_human_reference).to be_nil
      end
    end

    context 'when same project argument' do
      it 'returns nil' do
        expect(project.to_human_reference(project)).to be_nil
      end
    end

    context 'when cross namespace project argument' do
      let(:another_namespace_project) { create(:empty_project, name: 'another-project') }

      it 'returns complete name with namespace of the project' do
        expect(project.to_human_reference(another_namespace_project)).to eq 'Gitlab / Sample project'
      end
    end

    context 'when same namespace / cross-project argument' do
      let(:another_project) { create(:empty_project, namespace: namespace) }

      it 'returns name of the project' do
        expect(project.to_human_reference(another_project)).to eq 'Sample project'
      end
391 392 393
    end
  end

394
  describe '#repository_storage_path' do
395
    let(:project) { create(:empty_project, repository_storage: 'custom') }
396 397 398

    before do
      FileUtils.mkdir('tmp/tests/custom_repositories')
399
      storages = { 'custom' => { 'path' => 'tmp/tests/custom_repositories' } }
400 401 402 403 404 405 406 407 408 409 410 411
      allow(Gitlab.config.repositories).to receive(:storages).and_return(storages)
    end

    after do
      FileUtils.rm_rf('tmp/tests/custom_repositories')
    end

    it 'returns the repository storage path' do
      expect(project.repository_storage_path).to eq('tmp/tests/custom_repositories')
    end
  end

412
  it 'returns valid url to repo' do
413
    project = Project.new(path: 'somewhere')
414
    expect(project.url_to_repo).to eq(Gitlab.config.gitlab_shell.ssh_path_prefix + 'somewhere.git')
gitlabhq committed
415 416
  end

Douwe Maan committed
417 418 419 420
  describe "#web_url" do
    let(:project) { create(:empty_project, path: "somewhere") }

    it 'returns the full web URL for this repo' do
421
      expect(project.web_url).to eq("#{Gitlab.config.gitlab.url}/#{project.namespace.full_path}/somewhere")
Douwe Maan committed
422
    end
423 424
  end

425
  describe "#new_issue_address" do
426 427 428
    let(:project) { create(:empty_project, path: "somewhere") }
    let(:user) { create(:user) }

429 430 431 432 433 434
    context 'incoming email enabled' do
      before do
        stub_incoming_email_setting(enabled: true, address: "p+%{key}@gl.ab")
      end

      it 'returns the address to create a new issue' do
435
        address = "p+#{project.path_with_namespace}+#{user.incoming_email_token}@gl.ab"
436 437 438 439 440 441 442 443 444

        expect(project.new_issue_address(user)).to eq(address)
      end
    end

    context 'incoming email disabled' do
      before do
        stub_incoming_email_setting(enabled: false)
      end
445

446 447 448
      it 'returns nil' do
        expect(project.new_issue_address(user)).to be_nil
      end
449 450 451
    end
  end

452
  describe 'last_activity methods' do
453 454
    let(:timestamp) { 2.hours.ago }
    # last_activity_at gets set to created_at upon creation
455
    let(:project) { create(:empty_project, created_at: timestamp, updated_at: timestamp) }
gitlabhq committed
456

457
    describe 'last_activity' do
458
      it 'alias last_activity to last_event' do
459 460
        last_event = create(:event, project: project)

461
        expect(project.last_activity).to eq(last_event)
462
      end
gitlabhq committed
463 464
    end

465 466
    describe 'last_activity_date' do
      it 'returns the creation date of the project\'s last event if present' do
467 468
        new_event = create(:event, project: project, created_at: Time.now)

469
        project.reload
470
        expect(project.last_activity_at.to_i).to eq(new_event.created_at.to_i)
471
      end
472

473
      it 'returns the project\'s last update date if it has no events' do
474
        expect(project.last_activity_date).to eq(project.updated_at)
475
      end
476 477
    end
  end
478

479 480
  describe '#get_issue' do
    let(:project) { create(:empty_project) }
481
    let!(:issue)  { create(:issue, project: project) }
482 483 484 485 486
    let(:user)    { create(:user) }

    before do
      project.team << [user, :developer]
    end
487 488 489

    context 'with default issues tracker' do
      it 'returns an issue' do
490
        expect(project.get_issue(issue.iid, user)).to eq issue
491 492
      end

493 494 495 496
      it 'returns count of open issues' do
        expect(project.open_issues_count).to eq(1)
      end

497
      it 'returns nil when no issue found' do
498 499 500 501 502 503
        expect(project.get_issue(999, user)).to be_nil
      end

      it "returns nil when user doesn't have access" do
        user = create(:user)
        expect(project.get_issue(issue.iid, user)).to eq nil
504 505 506 507 508 509 510 511 512
      end
    end

    context 'with external issues tracker' do
      before do
        allow(project).to receive(:default_issues_tracker?).and_return(false)
      end

      it 'returns an ExternalIssue' do
513
        issue = project.get_issue('FOO-1234', user)
514 515 516 517 518 519 520 521 522 523 524 525 526 527 528 529 530 531 532 533 534
        expect(issue).to be_kind_of(ExternalIssue)
        expect(issue.iid).to eq 'FOO-1234'
        expect(issue.project).to eq project
      end
    end
  end

  describe '#issue_exists?' do
    let(:project) { create(:empty_project) }

    it 'is truthy when issue exists' do
      expect(project).to receive(:get_issue).and_return(double)
      expect(project.issue_exists?(1)).to be_truthy
    end

    it 'is falsey when issue does not exist' do
      expect(project).to receive(:get_issue).and_return(nil)
      expect(project.issue_exists?(1)).to be_falsey
    end
  end

535
  describe '#to_param' do
536 537 538
    context 'with namespace' do
      before do
        @group = create :group, name: 'gitlab'
539
        @project = create(:empty_project, name: 'gitlabhq', namespace: @group)
540 541
      end

Vinnie Okada committed
542
      it { expect(@project.to_param).to eq('gitlabhq') }
543
    end
544 545 546 547 548 549 550 551 552 553 554 555 556 557 558 559 560 561

    context 'with invalid path' do
      it 'returns previous path to keep project suitable for use in URLs when persisted' do
        project = create(:empty_project, path: 'gitlab')
        project.path = 'foo&bar'

        expect(project).not_to be_valid
        expect(project.to_param).to eq 'gitlab'
      end

      it 'returns current path when new record' do
        project = build(:empty_project, path: 'gitlab')
        project.path = 'foo&bar'

        expect(project).not_to be_valid
        expect(project.to_param).to eq 'foo&bar'
      end
    end
562
  end
Dmitriy Zaporozhets committed
563

564
  describe '#repository' do
565
    let(:project) { create(:project, :repository) }
Dmitriy Zaporozhets committed
566

567
    it 'returns valid repo' do
568
      expect(project.repository).to be_kind_of(Repository)
Dmitriy Zaporozhets committed
569 570
    end
  end
571

572
  describe '#default_issues_tracker?' do
573
    it "is true if used internal tracker" do
574 575
      project = build(:empty_project)

576
      expect(project.default_issues_tracker?).to be_truthy
577 578
    end

579
    it "is false if used other tracker" do
580 581 582 583
      # NOTE: The current nature of this factory requires persistence
      project = create(:redmine_project)

      expect(project.default_issues_tracker?).to be_falsey
584 585 586
    end
  end

587
  describe '#external_issue_tracker' do
588
    let(:project) { create(:empty_project) }
589 590 591 592 593 594 595 596 597 598 599 600 601 602 603 604 605 606 607 608 609 610 611 612 613 614 615 616 617 618 619 620 621
    let(:ext_project) { create(:redmine_project) }

    context 'on existing projects with no value for has_external_issue_tracker' do
      before(:each) do
        project.update_column(:has_external_issue_tracker, nil)
        ext_project.update_column(:has_external_issue_tracker, nil)
      end

      it 'updates the has_external_issue_tracker boolean' do
        expect do
          project.external_issue_tracker
        end.to change { project.reload.has_external_issue_tracker }.to(false)

        expect do
          ext_project.external_issue_tracker
        end.to change { ext_project.reload.has_external_issue_tracker }.to(true)
      end
    end

    it 'returns nil and does not query services when there is no external issue tracker' do
      expect(project).not_to receive(:services)

      expect(project.external_issue_tracker).to eq(nil)
    end

    it 'retrieves external_issue_tracker querying services and cache it when there is external issue tracker' do
      ext_project.reload # Factory returns a project with changed attributes
      expect(ext_project).to receive(:services).once.and_call_original

      2.times { expect(ext_project.external_issue_tracker).to be_a_kind_of(RedmineService) }
    end
  end

622
  describe '#cache_has_external_issue_tracker' do
623
    let(:project) { create(:empty_project, has_external_issue_tracker: nil) }
624 625 626 627 628 629 630 631 632 633 634 635 636 637 638 639 640 641 642 643

    it 'stores true if there is any external_issue_tracker' do
      services = double(:service, external_issue_trackers: [RedmineService.new])
      expect(project).to receive(:services).and_return(services)

      expect do
        project.cache_has_external_issue_tracker
      end.to change { project.has_external_issue_tracker}.to(true)
    end

    it 'stores false if there is no external_issue_tracker' do
      services = double(:service, external_issue_trackers: [])
      expect(project).to receive(:services).and_return(services)

      expect do
        project.cache_has_external_issue_tracker
      end.to change { project.has_external_issue_tracker}.to(false)
    end
  end

644
  describe '#has_wiki?' do
645
    let(:no_wiki_project)       { create(:empty_project, :wiki_disabled, has_external_wiki: false) }
646 647
    let(:wiki_enabled_project)  { create(:empty_project) }
    let(:external_wiki_project) { create(:empty_project, has_external_wiki: true) }
648 649 650 651 652 653 654 655

    it 'returns true if project is wiki enabled or has external wiki' do
      expect(wiki_enabled_project).to have_wiki
      expect(external_wiki_project).to have_wiki
      expect(no_wiki_project).not_to have_wiki
    end
  end

656
  describe '#external_wiki' do
657
    let(:project) { create(:empty_project) }
658

659 660 661 662 663
    context 'with an active external wiki' do
      before do
        create(:service, project: project, type: 'ExternalWikiService', active: true)
        project.external_wiki
      end
664

665 666 667
      it 'sets :has_external_wiki as true' do
        expect(project.has_external_wiki).to be(true)
      end
668

669 670
      it 'sets :has_external_wiki as false if an external wiki service is destroyed later' do
        expect(project.has_external_wiki).to be(true)
671

672 673 674 675
        project.services.external_wikis.first.destroy

        expect(project.has_external_wiki).to be(false)
      end
676 677
    end

678 679 680 681
    context 'with an inactive external wiki' do
      before do
        create(:service, project: project, type: 'ExternalWikiService', active: false)
      end
682

683 684 685
      it 'sets :has_external_wiki as false' do
        expect(project.has_external_wiki).to be(false)
      end
686 687
    end

688 689 690 691
    context 'with no external wiki' do
      before do
        project.external_wiki
      end
692

693 694 695 696 697 698 699 700 701 702 703
      it 'sets :has_external_wiki as false' do
        expect(project.has_external_wiki).to be(false)
      end

      it 'sets :has_external_wiki as true if an external wiki service is created later' do
        expect(project.has_external_wiki).to be(false)

        create(:service, project: project, type: 'ExternalWikiService', active: true)

        expect(project.has_external_wiki).to be(true)
      end
704 705 706
    end
  end

707 708
  describe '#star_count' do
    it 'counts stars from multiple users' do
Ciro Santilli committed
709 710
      user1 = create :user
      user2 = create :user
711
      project = create(:empty_project, :public)
Ciro Santilli committed
712 713

      expect(project.star_count).to eq(0)
714

Ciro Santilli committed
715
      user1.toggle_star(project)
716 717
      expect(project.reload.star_count).to eq(1)

Ciro Santilli committed
718
      user2.toggle_star(project)
719 720 721
      project.reload
      expect(project.reload.star_count).to eq(2)

Ciro Santilli committed
722
      user1.toggle_star(project)
723 724 725
      project.reload
      expect(project.reload.star_count).to eq(1)

Ciro Santilli committed
726
      user2.toggle_star(project)
727 728 729 730
      project.reload
      expect(project.reload.star_count).to eq(0)
    end

731
    it 'counts stars on the right project' do
732
      user = create :user
733 734
      project1 = create(:empty_project, :public)
      project2 = create(:empty_project, :public)
735 736 737 738 739 740 741 742 743 744 745 746 747 748 749 750 751 752 753 754 755 756 757 758 759 760 761

      expect(project1.star_count).to eq(0)
      expect(project2.star_count).to eq(0)

      user.toggle_star(project1)
      project1.reload
      project2.reload
      expect(project1.star_count).to eq(1)
      expect(project2.star_count).to eq(0)

      user.toggle_star(project1)
      project1.reload
      project2.reload
      expect(project1.star_count).to eq(0)
      expect(project2.star_count).to eq(0)

      user.toggle_star(project2)
      project1.reload
      project2.reload
      expect(project1.star_count).to eq(0)
      expect(project2.star_count).to eq(1)

      user.toggle_star(project2)
      project1.reload
      project2.reload
      expect(project1.star_count).to eq(0)
      expect(project2.star_count).to eq(0)
Ciro Santilli committed
762 763
    end
  end
764

765
  describe '#avatar_type' do
766
    let(:project) { create(:empty_project) }
767

768
    it 'is true if avatar is image' do
769
      project.update_attribute(:avatar, 'uploads/avatar.png')
770
      expect(project.avatar_type).to be_truthy
771 772
    end

773
    it 'is false if avatar is html page' do
774
      project.update_attribute(:avatar, 'uploads/avatar.html')
775
      expect(project.avatar_type).to eq(['only images allowed'])
776 777
    end
  end
778

779
  describe '#avatar_url' do
780 781
    subject { project.avatar_url }

782
    let(:project) { create(:empty_project) }
783 784 785 786 787 788 789 790 791 792 793

    context 'When avatar file is uploaded' do
      before do
        project.update_columns(avatar: 'uploads/avatar.png')
        allow(project.avatar).to receive(:present?) { true }
      end

      let(:avatar_path) do
        "/uploads/project/avatar/#{project.id}/uploads/avatar.png"
      end

794
      it { should eq "http://#{Gitlab.config.gitlab.host}#{avatar_path}" }
795 796 797 798 799 800 801 802
    end

    context 'When avatar file in git' do
      before do
        allow(project).to receive(:avatar_in_git) { true }
      end

      let(:avatar_path) do
803
        "/#{project.full_path}/avatar"
804 805
      end

806
      it { should eq "http://#{Gitlab.config.gitlab.host}#{avatar_path}" }
807
    end
808 809 810 811 812 813

    context 'when git repo is empty' do
      let(:project) { create(:empty_project) }

      it { should eq nil }
    end
814
  end
815

816
  describe '#pipeline_for' do
817
    let(:project) { create(:project, :repository) }
818
    let!(:pipeline) { create_pipeline }
819

820 821
    shared_examples 'giving the correct pipeline' do
      it { is_expected.to eq(pipeline) }
822

823 824
      context 'return latest' do
        let!(:pipeline2) { create_pipeline }
825

826
        it { is_expected.to eq(pipeline2) }
827
      end
828 829 830 831 832 833 834 835 836 837 838 839 840
    end

    context 'with explicit sha' do
      subject { project.pipeline_for('master', pipeline.sha) }

      it_behaves_like 'giving the correct pipeline'
    end

    context 'with implicit sha' do
      subject { project.pipeline_for('master') }

      it_behaves_like 'giving the correct pipeline'
    end
841

842 843 844 845 846
    def create_pipeline
      create(:ci_pipeline,
             project: project,
             ref: 'master',
             sha: project.commit('master').sha)
847
    end
848
  end
849

850
  describe '#builds_enabled' do
851
    let(:project) { create(:empty_project) }
852

853 854 855
    subject { project.builds_enabled }

    it { expect(project.builds_enabled?).to be_truthy }
856
  end
857

858 859 860 861 862 863 864 865 866 867 868 869 870 871 872 873 874 875 876 877
  describe '.with_shared_runners' do
    subject { Project.with_shared_runners }

    context 'when shared runners are enabled for project' do
      let!(:project) { create(:empty_project, shared_runners_enabled: true) }

      it "returns a project" do
        is_expected.to eq([project])
      end
    end

    context 'when shared runners are disabled for project' do
      let!(:project) { create(:empty_project, shared_runners_enabled: false) }

      it "returns an empty array" do
        is_expected.to be_empty
      end
    end
  end

878 879 880 881 882 883 884 885 886 887 888 889 890 891
  describe '.cached_count', caching: true do
    let(:group)     { create(:group, :public) }
    let!(:project1) { create(:empty_project, :public, group: group) }
    let!(:project2) { create(:empty_project, :public, group: group) }

    it 'returns total project count' do
      expect(Project).to receive(:count).once.and_call_original

      3.times do
        expect(Project.cached_count).to eq(2)
      end
    end
  end

892
  describe '.trending' do
Felipe Artur committed
893
    let(:group)    { create(:group, :public) }
894 895 896 897 898 899 900 901 902 903
    let(:project1) { create(:empty_project, :public, group: group) }
    let(:project2) { create(:empty_project, :public, group: group) }

    before do
      2.times do
        create(:note_on_commit, project: project1)
      end

      create(:note_on_commit, project: project2)

904
      TrendingProject.refresh!
905 906
    end

907
    subject { described_class.trending.to_a }
908

909 910
    it 'sorts projects by the amount of notes in descending order' do
      expect(subject).to eq([project1, project2])
911
    end
912 913 914 915 916 917 918 919

    it 'does not take system notes into account' do
      10.times do
        create(:note_on_commit, project: project2, system: true)
      end

      expect(described_class.trending.to_a).to eq([project1, project2])
    end
920
  end
921 922

  describe '.visible_to_user' do
923
    let!(:project) { create(:empty_project, :private) }
924 925 926 927 928 929
    let!(:user)    { create(:user) }

    subject { described_class.visible_to_user(user) }

    describe 'when a user has access to a project' do
      before do
930
        project.add_user(user, Gitlab::Access::MASTER)
931 932 933 934 935 936 937 938 939
      end

      it { is_expected.to eq([project]) }
    end

    describe 'when a user does not have access to any projects' do
      it { is_expected.to eq([]) }
    end
  end
940

941 942 943 944
  context 'repository storage by default' do
    let(:project) { create(:empty_project) }

    before do
945
      storages = {
946 947
        'default' => { 'path' => 'tmp/tests/repositories' },
        'picked'  => { 'path' => 'tmp/tests/repositories' },
948
      }
949 950 951
      allow(Gitlab.config.repositories).to receive(:storages).and_return(storages)
    end

952 953 954 955 956
    it 'picks storage from ApplicationSetting' do
      expect_any_instance_of(ApplicationSetting).to receive(:pick_repository_storage).and_return('picked')

      expect(project.repository_storage).to eq('picked')
    end
957 958
  end

959 960 961 962 963 964 965 966 967 968 969 970 971 972 973 974 975 976
  context 'shared runners by default' do
    let(:project) { create(:empty_project) }

    subject { project.shared_runners_enabled }

    context 'are enabled' do
      before { stub_application_setting(shared_runners_enabled: true) }

      it { is_expected.to be_truthy }
    end

    context 'are disabled' do
      before { stub_application_setting(shared_runners_enabled: false) }

      it { is_expected.to be_falsey }
    end
  end

977
  describe '#any_runners' do
978
    let(:project) { create(:empty_project, shared_runners_enabled: shared_runners_enabled) }
979 980
    let(:specific_runner) { create(:ci_runner) }
    let(:shared_runner) { create(:ci_runner, :shared) }
981 982 983

    context 'for shared runners disabled' do
      let(:shared_runners_enabled) { false }
984

985
      it 'has no runners available' do
986 987
        expect(project.any_runners?).to be_falsey
      end
988

989
      it 'has a specific runner' do
990
        project.runners << specific_runner
991 992
        expect(project.any_runners?).to be_truthy
      end
993

994
      it 'has a shared runner, but they are prohibited to use' do
995 996 997
        shared_runner
        expect(project.any_runners?).to be_falsey
      end
998

999
      it 'checks the presence of specific runner' do
1000
        project.runners << specific_runner
1001 1002 1003
        expect(project.any_runners? { |runner| runner == specific_runner }).to be_truthy
      end
    end
1004

1005 1006
    context 'for shared runners enabled' do
      let(:shared_runners_enabled) { true }
1007

1008
      it 'has a shared runner' do
1009 1010 1011 1012 1013 1014 1015 1016 1017 1018
        shared_runner
        expect(project.any_runners?).to be_truthy
      end

      it 'checks the presence of shared runner' do
        shared_runner
        expect(project.any_runners? { |runner| runner == shared_runner }).to be_truthy
      end
    end
  end
1019

1020 1021 1022 1023 1024 1025 1026 1027 1028 1029 1030 1031 1032 1033 1034 1035 1036 1037 1038 1039 1040 1041
  describe '#shared_runners' do
    let!(:runner) { create(:ci_runner, :shared) }

    subject { project.shared_runners }

    context 'when shared runners are enabled for project' do
      let!(:project) { create(:empty_project, shared_runners_enabled: true) }

      it "returns a list of shared runners" do
        is_expected.to eq([runner])
      end
    end

    context 'when shared runners are disabled for project' do
      let!(:project) { create(:empty_project, shared_runners_enabled: false) }

      it "returns a empty list" do
        is_expected.to be_empty
      end
    end
  end

1042
  describe '#visibility_level_allowed?' do
1043
    let(:project) { create(:empty_project, :internal) }
1044 1045 1046 1047 1048 1049 1050 1051

    context 'when checking on non-forked project' do
      it { expect(project.visibility_level_allowed?(Gitlab::VisibilityLevel::PRIVATE)).to be_truthy }
      it { expect(project.visibility_level_allowed?(Gitlab::VisibilityLevel::INTERNAL)).to be_truthy }
      it { expect(project.visibility_level_allowed?(Gitlab::VisibilityLevel::PUBLIC)).to be_truthy }
    end

    context 'when checking on forked project' do
1052 1053
      let(:project)        { create(:empty_project, :internal) }
      let(:forked_project) { create(:empty_project, forked_from_project: project) }
1054 1055 1056 1057 1058

      it { expect(forked_project.visibility_level_allowed?(Gitlab::VisibilityLevel::PRIVATE)).to be_truthy }
      it { expect(forked_project.visibility_level_allowed?(Gitlab::VisibilityLevel::INTERNAL)).to be_truthy }
      it { expect(forked_project.visibility_level_allowed?(Gitlab::VisibilityLevel::PUBLIC)).to be_falsey }
    end
1059
  end
1060

1061 1062 1063 1064 1065 1066
  describe '#pages_deployed?' do
    let(:project) { create :empty_project }

    subject { project.pages_deployed? }

    context 'if public folder does exist' do
1067
      before { allow(Dir).to receive(:exist?).with(project.public_pages_path).and_return(true) }
1068 1069 1070 1071 1072 1073 1074 1075 1076

      it { is_expected.to be_truthy }
    end

    context "if public folder doesn't exist" do
      it { is_expected.to be_falsey }
    end
  end

1077
  describe '.search' do
1078
    let(:project) { create(:empty_project, description: 'kitten mittens') }
1079

1080 1081 1082
    it 'returns projects with a matching name' do
      expect(described_class.search(project.name)).to eq([project])
    end
1083

1084 1085 1086
    it 'returns projects with a partially matching name' do
      expect(described_class.search(project.name[0..2])).to eq([project])
    end
1087

1088 1089 1090 1091 1092 1093 1094 1095 1096 1097 1098 1099 1100 1101 1102 1103 1104 1105 1106 1107 1108 1109 1110 1111 1112 1113 1114 1115 1116 1117 1118 1119 1120 1121 1122 1123 1124 1125 1126
    it 'returns projects with a matching name regardless of the casing' do
      expect(described_class.search(project.name.upcase)).to eq([project])
    end

    it 'returns projects with a matching description' do
      expect(described_class.search(project.description)).to eq([project])
    end

    it 'returns projects with a partially matching description' do
      expect(described_class.search('kitten')).to eq([project])
    end

    it 'returns projects with a matching description regardless of the casing' do
      expect(described_class.search('KITTEN')).to eq([project])
    end

    it 'returns projects with a matching path' do
      expect(described_class.search(project.path)).to eq([project])
    end

    it 'returns projects with a partially matching path' do
      expect(described_class.search(project.path[0..2])).to eq([project])
    end

    it 'returns projects with a matching path regardless of the casing' do
      expect(described_class.search(project.path.upcase)).to eq([project])
    end

    it 'returns projects with a matching namespace name' do
      expect(described_class.search(project.namespace.name)).to eq([project])
    end

    it 'returns projects with a partially matching namespace name' do
      expect(described_class.search(project.namespace.name[0..2])).to eq([project])
    end

    it 'returns projects with a matching namespace name regardless of the casing' do
      expect(described_class.search(project.namespace.name.upcase)).to eq([project])
    end
1127 1128 1129 1130 1131

    it 'returns projects when eager loading namespaces' do
      relation = described_class.all.includes(:namespace)

      expect(relation.search(project.namespace.name)).to eq([project])
1132
    end
1133
  end
1134 1135

  describe '#rename_repo' do
1136
    let(:project) { create(:project, :repository) }
1137 1138 1139 1140 1141 1142 1143
    let(:gitlab_shell) { Gitlab::Shell.new }

    before do
      # Project#gitlab_shell returns a new instance of Gitlab::Shell on every
      # call. This makes testing a bit easier.
      allow(project).to receive(:gitlab_shell).and_return(gitlab_shell)
      allow(project).to receive(:previous_changes).and_return('path' => ['foo'])
1144
    end
1145

1146
    it 'renames a repository' do
1147 1148
      stub_container_registry_config(enabled: false)

1149 1150
      expect(gitlab_shell).to receive(:mv_repository).
        ordered.
1151
        with(project.repository_storage_path, "#{project.namespace.full_path}/foo", "#{project.full_path}").
1152 1153 1154 1155
        and_return(true)

      expect(gitlab_shell).to receive(:mv_repository).
        ordered.
1156
        with(project.repository_storage_path, "#{project.namespace.full_path}/foo.wiki", "#{project.full_path}.wiki").
1157 1158 1159 1160 1161 1162 1163 1164
        and_return(true)

      expect_any_instance_of(SystemHooksService).
        to receive(:execute_hooks_for).
        with(project, :rename)

      expect_any_instance_of(Gitlab::UploadsTransfer).
        to receive(:rename_project).
1165
        with('foo', project.path, project.namespace.full_path)
1166 1167 1168 1169 1170

      expect(project).to receive(:expire_caches_before_rename)

      project.rename_repo
    end
1171

1172
    context 'container registry with images' do
1173
      let(:container_repository) { create(:container_repository) }
1174

1175 1176
      before do
        stub_container_registry_config(enabled: true)
1177
        stub_container_registry_tags(repository: :any, tags: ['tag'])
1178
        project.container_repositories << container_repository
1179 1180 1181 1182 1183 1184
      end

      subject { project.rename_repo }

      it { expect{subject}.to raise_error(Exception) }
    end
1185 1186 1187
  end

  describe '#expire_caches_before_rename' do
1188
    let(:project) { create(:project, :repository) }
1189 1190 1191 1192 1193 1194 1195 1196 1197 1198 1199 1200
    let(:repo)    { double(:repo, exists?: true) }
    let(:wiki)    { double(:wiki, exists?: true) }

    it 'expires the caches of the repository and wiki' do
      allow(Repository).to receive(:new).
        with('foo', project).
        and_return(repo)

      allow(Repository).to receive(:new).
        with('foo.wiki', project).
        and_return(wiki)

1201 1202
      expect(repo).to receive(:before_delete)
      expect(wiki).to receive(:before_delete)
1203 1204 1205 1206

      project.expire_caches_before_rename('foo')
    end
  end
1207 1208

  describe '.search_by_title' do
1209
    let(:project) { create(:empty_project, name: 'kittens') }
1210 1211 1212 1213 1214 1215 1216 1217 1218 1219 1220 1221 1222

    it 'returns projects with a matching name' do
      expect(described_class.search_by_title(project.name)).to eq([project])
    end

    it 'returns projects with a partially matching name' do
      expect(described_class.search_by_title('kitten')).to eq([project])
    end

    it 'returns projects with a matching name regardless of the casing' do
      expect(described_class.search_by_title('KITTENS')).to eq([project])
    end
  end
1223 1224 1225 1226 1227

  context 'when checking projects from groups' do
    let(:private_group)    { create(:group, visibility_level: 0)  }
    let(:internal_group)   { create(:group, visibility_level: 10) }

1228 1229
    let(:private_project)  { create :empty_project, :private, group: private_group }
    let(:internal_project) { create :empty_project, :internal, group: internal_group }
1230 1231 1232 1233 1234 1235 1236 1237 1238

    context 'when group is private project can not be internal' do
      it { expect(private_project.visibility_level_allowed?(Gitlab::VisibilityLevel::INTERNAL)).to be_falsey }
    end

    context 'when group is internal project can not be public' do
      it { expect(internal_project.visibility_level_allowed?(Gitlab::VisibilityLevel::PUBLIC)).to be_falsey }
    end
  end
1239

1240
  describe '#create_repository' do
1241
    let(:project) { create(:project, :repository) }
1242 1243 1244 1245 1246 1247 1248 1249 1250
    let(:shell) { Gitlab::Shell.new }

    before do
      allow(project).to receive(:gitlab_shell).and_return(shell)
    end

    context 'using a regular repository' do
      it 'creates the repository' do
        expect(shell).to receive(:add_repository).
1251
          with(project.repository_storage_path, project.path_with_namespace).
1252 1253 1254 1255 1256 1257 1258 1259 1260
          and_return(true)

        expect(project.repository).to receive(:after_create)

        expect(project.create_repository).to eq(true)
      end

      it 'adds an error if the repository could not be created' do
        expect(shell).to receive(:add_repository).
1261
          with(project.repository_storage_path, project.path_with_namespace).
1262 1263 1264 1265 1266 1267 1268 1269 1270 1271 1272 1273 1274 1275 1276 1277 1278 1279
          and_return(false)

        expect(project.repository).not_to receive(:after_create)

        expect(project.create_repository).to eq(false)
        expect(project.errors).not_to be_empty
      end
    end

    context 'using a forked repository' do
      it 'does nothing' do
        expect(project).to receive(:forked?).and_return(true)
        expect(shell).not_to receive(:add_repository)

        project.create_repository
      end
    end
  end
1280

1281
  describe '#user_can_push_to_empty_repo?' do
1282
    let(:project) { create(:empty_project) }
1283
    let(:user)    { create(:user) }
1284

1285 1286 1287
    it 'returns false when default_branch_protection is in full protection and user is developer' do
      project.team << [user, :developer]
      stub_application_setting(default_branch_protection: Gitlab::Access::PROTECTION_FULL)
1288

1289
      expect(project.user_can_push_to_empty_repo?(user)).to be_falsey
1290 1291
    end

1292 1293 1294
    it 'returns false when default_branch_protection only lets devs merge and user is dev' do
      project.team << [user, :developer]
      stub_application_setting(default_branch_protection: Gitlab::Access::PROTECTION_DEV_CAN_MERGE)
1295

1296
      expect(project.user_can_push_to_empty_repo?(user)).to be_falsey
1297 1298
    end

1299 1300 1301
    it 'returns true when default_branch_protection lets devs push and user is developer' do
      project.team << [user, :developer]
      stub_application_setting(default_branch_protection: Gitlab::Access::PROTECTION_DEV_CAN_PUSH)
1302

1303 1304
      expect(project.user_can_push_to_empty_repo?(user)).to be_truthy
    end
1305

1306 1307 1308
    it 'returns true when default_branch_protection is unprotected and user is developer' do
      project.team << [user, :developer]
      stub_application_setting(default_branch_protection: Gitlab::Access::PROTECTION_NONE)
1309

1310
      expect(project.user_can_push_to_empty_repo?(user)).to be_truthy
1311
    end
1312

1313 1314
    it 'returns true when user is master' do
      project.team << [user, :master]
1315

1316
      expect(project.user_can_push_to_empty_repo?(user)).to be_truthy
1317 1318 1319
    end
  end

1320
  describe '#container_registry_url' do
1321 1322
    let(:project) { create(:empty_project) }

1323
    subject { project.container_registry_url }
1324

1325
    before { stub_container_registry_config(**registry_settings) }
1326 1327 1328

    context 'for enabled registry' do
      let(:registry_settings) do
1329 1330
        { enabled: true,
          host_port: 'example.com' }
1331 1332
      end

1333
      it { is_expected.not_to be_nil }
1334 1335 1336 1337
    end

    context 'for disabled registry' do
      let(:registry_settings) do
1338
        { enabled: false }
1339 1340 1341 1342 1343 1344
      end

      it { is_expected.to be_nil }
    end
  end

1345 1346 1347 1348 1349 1350 1351 1352 1353 1354 1355 1356 1357 1358 1359 1360 1361 1362 1363 1364 1365 1366 1367 1368 1369 1370 1371 1372 1373 1374 1375 1376 1377 1378 1379 1380 1381 1382 1383 1384 1385 1386 1387 1388 1389 1390 1391 1392 1393 1394 1395 1396 1397 1398 1399 1400 1401 1402 1403 1404
  describe '#has_container_registry_tags?' do
    let(:project) { create(:empty_project) }

    context 'when container registry is enabled' do
      before { stub_container_registry_config(enabled: true) }

      context 'when tags are present for multi-level registries' do
        before do
          create(:container_repository, project: project, name: 'image')

          stub_container_registry_tags(repository: /image/,
                                       tags: %w[latest rc1])
        end

        it 'should have image tags' do
          expect(project).to have_container_registry_tags
        end
      end

      context 'when tags are present for root repository' do
        before do
          stub_container_registry_tags(repository: project.full_path,
                                       tags: %w[latest rc1 pre1])
        end

        it 'should have image tags' do
          expect(project).to have_container_registry_tags
        end
      end

      context 'when there are no tags at all' do
        before do
          stub_container_registry_tags(repository: :any, tags: [])
        end

        it 'should not have image tags' do
          expect(project).not_to have_container_registry_tags
        end
      end
    end

    context 'when container registry is disabled' do
      before { stub_container_registry_config(enabled: false) }

      it 'should not have image tags' do
        expect(project).not_to have_container_registry_tags
      end

      it 'should not check root repository tags' do
        expect(project).not_to receive(:full_path)
        expect(project).not_to have_container_registry_tags
      end

      it 'should iterate through container repositories' do
        expect(project).to receive(:container_repositories)
        expect(project).not_to have_container_registry_tags
      end
    end
  end

1405
  describe '#latest_successful_builds_for' do
Lin Jen-Shin committed
1406
    def create_pipeline(status = 'success')
1407
      create(:ci_pipeline, project: project,
Lin Jen-Shin committed
1408
                           sha: project.commit.sha,
1409
                           ref: project.default_branch,
Lin Jen-Shin committed
1410
                           status: status)
1411 1412
    end

Lin Jen-Shin committed
1413 1414 1415
    def create_build(new_pipeline = pipeline, name = 'test')
      create(:ci_build, :success, :artifacts,
             pipeline: new_pipeline,
Lin Jen-Shin committed
1416
             status: new_pipeline.status,
Lin Jen-Shin committed
1417
             name: name)
1418 1419
    end

1420
    let(:project) { create(:project, :repository) }
Lin Jen-Shin committed
1421
    let(:pipeline) { create_pipeline }
Lin Jen-Shin committed
1422 1423

    context 'with many builds' do
1424
      it 'gives the latest builds from latest pipeline' do
1425 1426
        pipeline1 = create_pipeline
        pipeline2 = create_pipeline
1427
        build1_p2 = create_build(pipeline2, 'test')
1428 1429
        create_build(pipeline1, 'test')
        create_build(pipeline1, 'test2')
1430
        build2_p2 = create_build(pipeline2, 'test2')
Lin Jen-Shin committed
1431 1432 1433

        latest_builds = project.latest_successful_builds_for

1434
        expect(latest_builds).to contain_exactly(build2_p2, build1_p2)
Lin Jen-Shin committed
1435 1436
      end
    end
Lin Jen-Shin committed
1437

Lin Jen-Shin committed
1438
    context 'with succeeded pipeline' do
Lin Jen-Shin committed
1439
      let!(:build) { create_build }
1440

Lin Jen-Shin committed
1441
      context 'standalone pipeline' do
1442 1443 1444 1445 1446 1447 1448 1449
        it 'returns builds for ref for default_branch' do
          builds = project.latest_successful_builds_for

          expect(builds).to contain_exactly(build)
        end

        it 'returns empty relation if the build cannot be found' do
          builds = project.latest_successful_builds_for('TAIL')
1450

1451 1452 1453
          expect(builds).to be_kind_of(ActiveRecord::Relation)
          expect(builds).to be_empty
        end
1454 1455
      end

Lin Jen-Shin committed
1456
      context 'with some pending pipeline' do
1457
        before do
Lin Jen-Shin committed
1458
          create_build(create_pipeline('pending'))
1459 1460
        end

Lin Jen-Shin committed
1461 1462
        it 'gives the latest build from latest pipeline' do
          latest_build = project.latest_successful_builds_for
1463

Lin Jen-Shin committed
1464
          expect(latest_build).to contain_exactly(build)
1465
        end
1466 1467 1468 1469 1470 1471
      end
    end

    context 'with pending pipeline' do
      before do
        pipeline.update(status: 'pending')
1472
        create_build(pipeline)
1473 1474 1475 1476 1477 1478 1479 1480 1481 1482 1483
      end

      it 'returns empty relation' do
        builds = project.latest_successful_builds_for

        expect(builds).to be_kind_of(ActiveRecord::Relation)
        expect(builds).to be_empty
      end
    end
  end

1484 1485 1486 1487 1488 1489 1490 1491 1492
  describe '#add_import_job' do
    context 'forked' do
      let(:forked_project_link) { create(:forked_project_link) }
      let(:forked_from_project) { forked_project_link.forked_from_project }
      let(:project) { forked_project_link.forked_to_project }

      it 'schedules a RepositoryForkWorker job' do
        expect(RepositoryForkWorker).to receive(:perform_async).
          with(project.id, forked_from_project.repository_storage_path,
1493
              forked_from_project.path_with_namespace, project.namespace.full_path)
1494 1495 1496 1497 1498 1499

        project.add_import_job
      end
    end

    context 'not forked' do
1500
      let(:project) { create(:empty_project) }
1501 1502 1503 1504 1505 1506 1507 1508 1509

      it 'schedules a RepositoryImportWorker job' do
        expect(RepositoryImportWorker).to receive(:perform_async).with(project.id)

        project.add_import_job
      end
    end
  end

1510
  describe '#gitlab_project_import?' do
1511
    subject(:project) { build(:empty_project, import_type: 'gitlab_project') }
1512 1513 1514 1515 1516

    it { expect(project.gitlab_project_import?).to be true }
  end

  describe '#gitea_import?' do
1517
    subject(:project) { build(:empty_project, import_type: 'gitea') }
1518 1519 1520 1521

    it { expect(project.gitea_import?).to be true }
  end

1522
  describe '#lfs_enabled?' do
1523
    let(:project) { create(:empty_project) }
1524 1525 1526 1527 1528 1529 1530 1531 1532 1533 1534 1535 1536 1537 1538 1539 1540 1541 1542 1543 1544 1545 1546 1547 1548 1549 1550 1551 1552 1553 1554 1555 1556 1557 1558 1559 1560 1561 1562 1563 1564 1565 1566 1567 1568 1569 1570 1571 1572 1573 1574 1575 1576 1577 1578 1579 1580 1581 1582 1583

    shared_examples 'project overrides group' do
      it 'returns true when enabled in project' do
        project.update_attribute(:lfs_enabled, true)

        expect(project.lfs_enabled?).to be_truthy
      end

      it 'returns false when disabled in project' do
        project.update_attribute(:lfs_enabled, false)

        expect(project.lfs_enabled?).to be_falsey
      end

      it 'returns the value from the namespace, when no value is set in project' do
        expect(project.lfs_enabled?).to eq(project.namespace.lfs_enabled?)
      end
    end

    context 'LFS disabled in group' do
      before do
        project.namespace.update_attribute(:lfs_enabled, false)
        enable_lfs
      end

      it_behaves_like 'project overrides group'
    end

    context 'LFS enabled in group' do
      before do
        project.namespace.update_attribute(:lfs_enabled, true)
        enable_lfs
      end

      it_behaves_like 'project overrides group'
    end

    describe 'LFS disabled globally' do
      shared_examples 'it always returns false' do
        it do
          expect(project.lfs_enabled?).to be_falsey
          expect(project.namespace.lfs_enabled?).to be_falsey
        end
      end

      context 'when no values are set' do
        it_behaves_like 'it always returns false'
      end

      context 'when all values are set to true' do
        before do
          project.namespace.update_attribute(:lfs_enabled, true)
          project.update_attribute(:lfs_enabled, true)
        end

        it_behaves_like 'it always returns false'
      end
    end
  end

1584
  describe '#change_head' do
1585
    let(:project) { create(:project, :repository) }
1586

1587
    it 'calls the before_change_head and after_change_head methods' do
1588
      expect(project.repository).to receive(:before_change_head)
1589 1590
      expect(project.repository).to receive(:after_change_head)

1591 1592 1593 1594 1595 1596 1597 1598 1599 1600 1601 1602 1603 1604 1605 1606 1607 1608 1609 1610
      project.change_head(project.default_branch)
    end

    it 'creates the new reference with rugged' do
      expect(project.repository.rugged.references).to receive(:create).with('HEAD',
                                                                            "refs/heads/#{project.default_branch}",
                                                                            force: true)
      project.change_head(project.default_branch)
    end

    it 'copies the gitattributes' do
      expect(project.repository).to receive(:copy_gitattributes).with(project.default_branch)
      project.change_head(project.default_branch)
    end

    it 'reloads the default branch' do
      expect(project).to receive(:reload_default_branch)
      project.change_head(project.default_branch)
    end
  end
1611 1612

  describe '#pushes_since_gc' do
1613
    let(:project) { create(:empty_project) }
1614 1615 1616 1617 1618 1619 1620 1621 1622 1623 1624 1625 1626 1627 1628 1629 1630 1631 1632 1633 1634

    after do
      project.reset_pushes_since_gc
    end

    context 'without any pushes' do
      it 'returns 0' do
        expect(project.pushes_since_gc).to eq(0)
      end
    end

    context 'with a number of pushes' do
      it 'returns the number of pushes' do
        3.times { project.increment_pushes_since_gc }

        expect(project.pushes_since_gc).to eq(3)
      end
    end
  end

  describe '#increment_pushes_since_gc' do
1635
    let(:project) { create(:empty_project) }
1636 1637 1638 1639 1640 1641 1642 1643 1644 1645 1646 1647 1648

    after do
      project.reset_pushes_since_gc
    end

    it 'increments the number of pushes since the last GC' do
      3.times { project.increment_pushes_since_gc }

      expect(project.pushes_since_gc).to eq(3)
    end
  end

  describe '#reset_pushes_since_gc' do
1649
    let(:project) { create(:empty_project) }
1650 1651 1652 1653 1654 1655 1656 1657 1658 1659 1660 1661 1662

    after do
      project.reset_pushes_since_gc
    end

    it 'resets the number of pushes since the last GC' do
      3.times { project.increment_pushes_since_gc }

      project.reset_pushes_since_gc

      expect(project.pushes_since_gc).to eq(0)
    end
  end
1663

1664 1665 1666 1667 1668 1669 1670 1671 1672 1673 1674 1675 1676 1677 1678 1679 1680 1681 1682 1683
  describe '#deployment_variables' do
    context 'when project has no deployment service' do
      let(:project) { create(:empty_project) }

      it 'returns an empty array' do
        expect(project.deployment_variables).to eq []
      end
    end

    context 'when project has a deployment service' do
      let(:project) { create(:kubernetes_project) }

      it 'returns variables from this service' do
        expect(project.deployment_variables).to include(
          { key: 'KUBE_TOKEN', value: project.kubernetes_service.token, public: false }
        )
      end
    end
  end

1684 1685 1686 1687 1688 1689 1690 1691 1692 1693 1694 1695 1696 1697 1698 1699 1700 1701 1702 1703
  describe '#update_project_statistics' do
    let(:project) { create(:empty_project) }

    it "is called after creation" do
      expect(project.statistics).to be_a ProjectStatistics
      expect(project.statistics).to be_persisted
    end

    it "copies the namespace_id" do
      expect(project.statistics.namespace_id).to eq project.namespace_id
    end

    it "updates the namespace_id when changed" do
      namespace = create(:namespace)
      project.update(namespace: namespace)

      expect(project.statistics.namespace_id).to eq namespace.id
    end
  end

1704
  describe 'inside_path' do
1705
    let!(:project1) { create(:empty_project, namespace: create(:namespace, path: 'name_pace')) }
1706
    let!(:project2) { create(:empty_project) }
1707
    let!(:project3) { create(:empty_project, namespace: create(:namespace, path: 'namespace')) }
1708
    let!(:path) { project1.namespace.full_path }
1709

1710 1711 1712
    it 'returns correct project' do
      expect(Project.inside_path(path)).to eq([project1])
    end
1713 1714
  end

Douwe Maan committed
1715 1716 1717 1718 1719 1720 1721 1722 1723 1724
  describe '#route_map_for' do
    let(:project) { create(:project) }
    let(:route_map) do
      <<-MAP.strip_heredoc
      - source: /source/(.*)/
        public: '\\1'
      MAP
    end

    before do
1725
      project.repository.create_file(User.last, '.gitlab/route-map.yml', route_map, message: 'Add .gitlab/route-map.yml', branch_name: 'master')
Douwe Maan committed
1726 1727 1728 1729 1730 1731 1732 1733 1734 1735 1736 1737 1738 1739 1740 1741 1742 1743 1744 1745 1746 1747 1748 1749 1750 1751 1752 1753 1754 1755 1756 1757 1758 1759 1760 1761 1762 1763 1764 1765 1766 1767 1768 1769 1770 1771 1772 1773 1774 1775 1776 1777 1778 1779 1780 1781 1782 1783 1784 1785 1786 1787 1788 1789 1790
    end

    context 'when there is a .gitlab/route-map.yml at the commit' do
      context 'when the route map is valid' do
        it 'returns a route map' do
          map = project.route_map_for(project.commit.sha)
          expect(map).to be_a_kind_of(Gitlab::RouteMap)
        end
      end

      context 'when the route map is invalid' do
        let(:route_map) { 'INVALID' }

        it 'returns nil' do
          expect(project.route_map_for(project.commit.sha)).to be_nil
        end
      end
    end

    context 'when there is no .gitlab/route-map.yml at the commit' do
      it 'returns nil' do
        expect(project.route_map_for(project.commit.parent.sha)).to be_nil
      end
    end
  end

  describe '#public_path_for_source_path' do
    let(:project) { create(:project) }
    let(:route_map) do
      Gitlab::RouteMap.new(<<-MAP.strip_heredoc)
        - source: /source/(.*)/
          public: '\\1'
      MAP
    end
    let(:sha) { project.commit.id }

    context 'when there is a route map' do
      before do
        allow(project).to receive(:route_map_for).with(sha).and_return(route_map)
      end

      context 'when the source path is mapped' do
        it 'returns the public path' do
          expect(project.public_path_for_source_path('source/file.html', sha)).to eq('file.html')
        end
      end

      context 'when the source path is not mapped' do
        it 'returns nil' do
          expect(project.public_path_for_source_path('file.html', sha)).to be_nil
        end
      end
    end

    context 'when there is no route map' do
      before do
        allow(project).to receive(:route_map_for).with(sha).and_return(nil)
      end

      it 'returns nil' do
        expect(project.public_path_for_source_path('source/file.html', sha)).to be_nil
      end
    end
  end

1791 1792 1793 1794 1795 1796 1797 1798 1799 1800 1801 1802 1803 1804
  describe '#parent' do
    let(:project) { create(:empty_project) }

    it { expect(project.parent).to eq(project.namespace) }
  end

  describe '#parent_changed?' do
    let(:project) { create(:empty_project) }

    before { project.namespace_id = 7 }

    it { expect(project.parent_changed?).to be_truthy }
  end

1805 1806 1807
  def enable_lfs
    allow(Gitlab.config.lfs).to receive(:enabled).and_return(true)
  end
1808

1809
  describe '#pages_url' do
1810 1811
    let(:group) { create :group, name: 'Group' }
    let(:nested_group) { create :group, parent: group }
1812 1813 1814 1815 1816 1817 1818 1819 1820
    let(:domain) { 'Example.com' }

    subject { project.pages_url }

    before do
      allow(Settings.pages).to receive(:host).and_return(domain)
      allow(Gitlab.config.pages).to receive(:url).and_return('http://example.com')
    end

1821 1822
    context 'top-level group' do
      let(:project) { create :empty_project, namespace: group, name: project_name }
1823

1824 1825 1826 1827 1828 1829 1830 1831 1832 1833 1834
      context 'group page' do
        let(:project_name) { 'group.example.com' }

        it { is_expected.to eq("http://group.example.com") }
      end

      context 'project page' do
        let(:project_name) { 'Project' }

        it { is_expected.to eq("http://group.example.com/project") }
      end
1835 1836
    end

1837 1838 1839
    context 'nested group' do
      let(:project) { create :empty_project, namespace: nested_group, name: project_name }
      let(:expected_url) { "http://group.example.com/#{nested_group.path}/#{project.path}" }
1840

1841 1842 1843 1844 1845 1846 1847 1848 1849 1850 1851
      context 'group page' do
        let(:project_name) { 'group.example.com' }

        it { is_expected.to eq(expected_url) }
      end

      context 'project page' do
        let(:project_name) { 'Project' }

        it { is_expected.to eq(expected_url) }
      end
1852 1853
    end
  end
1854 1855 1856 1857 1858 1859

  describe '#http_url_to_repo' do
    let(:project) { create :empty_project }

    context 'when no user is given' do
      it 'returns the url to the repo without a username' do
1860 1861
        expect(project.http_url_to_repo).to eq("#{project.web_url}.git")
        expect(project.http_url_to_repo).not_to include('@')
1862 1863 1864 1865 1866 1867 1868
      end
    end

    context 'when user is given' do
      it 'returns the url to the repo with the username' do
        user = build_stubbed(:user)

1869
        expect(project.http_url_to_repo(user)).to start_with("http://#{user.username}@")
1870 1871 1872
      end
    end
  end
1873 1874 1875 1876

  describe '#pipeline_status' do
    let(:project) { create(:project) }
    it 'builds a pipeline status' do
1877
      expect(project.pipeline_status).to be_a(Gitlab::Cache::Ci::ProjectPipelineStatus)
1878 1879 1880 1881 1882 1883
    end

    it 'hase a loaded pipeline status' do
      expect(project.pipeline_status).to be_loaded
    end
  end
gitlabhq committed
1884
end