diff options
Diffstat (limited to 'spec/services/organizations')
-rw-r--r-- | spec/services/organizations/create_service_spec.rb | 43 | ||||
-rw-r--r-- | spec/services/organizations/update_service_spec.rb | 76 |
2 files changed, 111 insertions, 8 deletions
diff --git a/spec/services/organizations/create_service_spec.rb b/spec/services/organizations/create_service_spec.rb index 7d9bf64ddd3..aae89517c15 100644 --- a/spec/services/organizations/create_service_spec.rb +++ b/spec/services/organizations/create_service_spec.rb @@ -7,7 +7,10 @@ RSpec.describe Organizations::CreateService, feature_category: :cell do let_it_be(:user) { create(:user) } let(:current_user) { user } - let(:params) { attributes_for(:organization) } + let(:params) { attributes_for(:organization).merge(extra_params) } + let(:avatar_filename) { nil } + let(:extra_params) { {} } + let(:created_organization) { response.payload[:organization] } subject(:response) { described_class.new(current_user: current_user, params: params).execute } @@ -23,17 +26,41 @@ RSpec.describe Organizations::CreateService, feature_category: :cell do end context 'when user has permission' do - it 'creates an organization' do - expect { response }.to change { Organizations::Organization.count } + shared_examples 'creating an organization' do + it 'creates the organization' do + expect { response }.to change { Organizations::Organization.count } + expect(response).to be_success + expect(created_organization.name).to eq(params[:name]) + expect(created_organization.path).to eq(params[:path]) + expect(created_organization.description).to eq(params[:description]) + expect(created_organization.avatar.filename).to eq(avatar_filename) + end + end + + it_behaves_like 'creating an organization' - expect(response).to be_success + context 'with description' do + let(:description) { 'Organization description' } + let(:extra_params) { { description: description } } + + it_behaves_like 'creating an organization' end - it 'returns an error when the organization is not persisted' do - params[:name] = nil + context 'with avatar' do + let(:avatar_filename) { 'dk.png' } + let(:avatar) { fixture_file_upload("spec/fixtures/#{avatar_filename}") } + let(:extra_params) { { avatar: avatar } } - expect(response).to be_error - expect(response.message).to match_array(["Name can't be blank"]) + it_behaves_like 'creating an organization' + end + + context 'when the organization is not persisted' do + let(:extra_params) { { name: nil } } + + it 'returns an error when the organization is not persisted' do + expect(response).to be_error + expect(response.message).to match_array(["Name can't be blank"]) + end end end end diff --git a/spec/services/organizations/update_service_spec.rb b/spec/services/organizations/update_service_spec.rb new file mode 100644 index 00000000000..148840770db --- /dev/null +++ b/spec/services/organizations/update_service_spec.rb @@ -0,0 +1,76 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Organizations::UpdateService, feature_category: :cell do + describe '#execute' do + let_it_be(:user) { create(:user) } + let_it_be_with_reload(:organization) { create(:organization) } + + let_it_be(:current_user) { user } # due to use in before_all + let(:name) { 'Name' } + let(:path) { 'path' } + let(:description) { nil } + let(:avatar_filename) { nil } + let(:params) { { name: name, path: path }.merge(extra_params) } + let(:extra_params) { {} } + let(:updated_organization) { response.payload[:organization] } + + subject(:response) do + described_class.new(organization, current_user: current_user, params: params).execute + end + + context 'when user does not have permission' do + let(:current_user) { nil } + + it 'returns an error' do + expect(response).to be_error + + expect(response.message).to match_array(['You have insufficient permissions to update the organization']) + end + end + + context 'when user has permission' do + before_all do + create(:organization_user, organization: organization, user: current_user) + end + + shared_examples 'updating an organization' do + it 'updates the organization' do + expect(response).to be_success + expect(updated_organization.name).to eq(name) + expect(updated_organization.path).to eq(path) + expect(updated_organization.description).to eq(description) + expect(updated_organization.avatar.filename).to eq(avatar_filename) + end + end + + context 'with description' do + let(:description) { 'Organization description' } + let(:extra_params) { { description: description } } + + it_behaves_like 'updating an organization' + end + + context 'with avatar' do + let(:avatar_filename) { 'dk.png' } + let(:avatar) { fixture_file_upload("spec/fixtures/#{avatar_filename}") } + let(:extra_params) { { avatar: avatar } } + + it_behaves_like 'updating an organization' + end + + include_examples 'updating an organization' + + context 'when the organization is not updated' do + let(:extra_params) { { name: nil } } + + it 'returns an error' do + expect(response).to be_error + expect(updated_organization).to be_instance_of Organizations::Organization + expect(response.message).to match_array(["Name can't be blank"]) + end + end + end + end +end |