diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-04-21 02:50:22 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-04-21 02:50:22 +0300 |
commit | 9dc93a4519d9d5d7be48ff274127136236a3adb3 (patch) | |
tree | 70467ae3692a0e35e5ea56bcb803eb512a10bedb /spec/lib/gitlab/hook_data | |
parent | 4b0f34b6d759d6299322b3a54453e930c6121ff0 (diff) |
Add latest changes from gitlab-org/gitlab@13-11-stable-eev13.11.0-rc43
Diffstat (limited to 'spec/lib/gitlab/hook_data')
4 files changed, 93 insertions, 0 deletions
diff --git a/spec/lib/gitlab/hook_data/issue_builder_spec.rb b/spec/lib/gitlab/hook_data/issue_builder_spec.rb index 8a2395d70b2..8f898d898de 100644 --- a/spec/lib/gitlab/hook_data/issue_builder_spec.rb +++ b/spec/lib/gitlab/hook_data/issue_builder_spec.rb @@ -5,6 +5,7 @@ require 'spec_helper' RSpec.describe Gitlab::HookData::IssueBuilder do let_it_be(:label) { create(:label) } let_it_be(:issue) { create(:labeled_issue, labels: [label], project: label.project) } + let(:builder) { described_class.new(issue) } describe '#build' do diff --git a/spec/lib/gitlab/hook_data/merge_request_builder_spec.rb b/spec/lib/gitlab/hook_data/merge_request_builder_spec.rb index fede7f273f1..0339faa9fcf 100644 --- a/spec/lib/gitlab/hook_data/merge_request_builder_spec.rb +++ b/spec/lib/gitlab/hook_data/merge_request_builder_spec.rb @@ -4,6 +4,7 @@ require 'spec_helper' RSpec.describe Gitlab::HookData::MergeRequestBuilder do let_it_be(:merge_request) { create(:merge_request) } + let(:builder) { described_class.new(merge_request) } describe '#build' do diff --git a/spec/lib/gitlab/hook_data/release_builder_spec.rb b/spec/lib/gitlab/hook_data/release_builder_spec.rb index b630780b162..449965f5df1 100644 --- a/spec/lib/gitlab/hook_data/release_builder_spec.rb +++ b/spec/lib/gitlab/hook_data/release_builder_spec.rb @@ -4,6 +4,7 @@ require 'spec_helper' RSpec.describe Gitlab::HookData::ReleaseBuilder do let_it_be(:project) { create(:project, :public, :repository) } + let(:release) { create(:release, project: project) } let(:builder) { described_class.new(release) } diff --git a/spec/lib/gitlab/hook_data/user_builder_spec.rb b/spec/lib/gitlab/hook_data/user_builder_spec.rb new file mode 100644 index 00000000000..f971089850b --- /dev/null +++ b/spec/lib/gitlab/hook_data/user_builder_spec.rb @@ -0,0 +1,90 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Gitlab::HookData::UserBuilder do + let_it_be(:user) { create(:user, name: 'John Doe', username: 'johndoe', email: 'john@example.com') } + + describe '#build' do + let(:data) { described_class.new(user).build(event) } + let(:event_name) { data[:event_name] } + let(:attributes) do + [ + :event_name, :created_at, :updated_at, :name, :email, :user_id, :username + ] + end + + context 'data' do + shared_examples_for 'includes the required attributes' do + it 'includes the required attributes' do + expect(data).to include(*attributes) + + expect(data[:name]).to eq('John Doe') + expect(data[:email]).to eq('john@example.com') + expect(data[:user_id]).to eq(user.id) + expect(data[:username]).to eq('johndoe') + expect(data[:created_at]).to eq(user.created_at.xmlschema) + expect(data[:updated_at]).to eq(user.updated_at.xmlschema) + end + end + + shared_examples_for 'does not include old username attributes' do + it 'does not include old username attributes' do + expect(data).not_to include(:old_username) + end + end + + shared_examples_for 'does not include state attributes' do + it 'does not include state attributes' do + expect(data).not_to include(:state) + end + end + + context 'on create' do + let(:event) { :create } + + it { expect(event_name).to eq('user_create') } + it_behaves_like 'includes the required attributes' + it_behaves_like 'does not include old username attributes' + it_behaves_like 'does not include state attributes' + end + + context 'on destroy' do + let(:event) { :destroy } + + it { expect(event_name).to eq('user_destroy') } + it_behaves_like 'includes the required attributes' + it_behaves_like 'does not include old username attributes' + it_behaves_like 'does not include state attributes' + end + + context 'on rename' do + let(:event) { :rename } + + it { expect(event_name).to eq('user_rename') } + it_behaves_like 'includes the required attributes' + it_behaves_like 'does not include state attributes' + + it 'includes old username details' do + allow(user).to receive(:username_before_last_save).and_return('old-username') + + expect(data[:old_username]).to eq(user.username_before_last_save) + end + end + + context 'on failed_login' do + let(:event) { :failed_login } + + it { expect(event_name).to eq('user_failed_login') } + it_behaves_like 'includes the required attributes' + it_behaves_like 'does not include old username attributes' + + it 'includes state details' do + user.ldap_block! + + expect(data[:state]).to eq('ldap_blocked') + end + end + end + end +end |