Skip to content

Commit

Permalink
[ci] Refactored spec to use rspec change operator.
Browse files Browse the repository at this point in the history
  • Loading branch information
evanrolfe committed Aug 24, 2017
1 parent 733defa commit 80343b1
Showing 1 changed file with 5 additions and 11 deletions.
16 changes: 5 additions & 11 deletions src/api/spec/controllers/person_controller_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,20 +8,18 @@
let(:user) { create(:confirmed_user) }
let(:admin_user) { create(:admin_user) }

let!(:old_realname) { user.realname }
let!(:old_email) { user.email }

shared_examples "not allowed to change user details" do
it 'sets an error code' do
subject
expect(response.header['X-Opensuse-Errorcode']).to eq('change_userinfo_no_permission')
end

it 'does not change users real name' do
expect(user.realname).to eq(old_realname)
expect { subject }.not_to(change { user.realname })
end

it 'does not change users email address' do
expect(user.email).to eq(old_email)
expect { subject }.not_to(change { user.email })
end
end

Expand Down Expand Up @@ -55,12 +53,11 @@
request.env["RAW_POST_DATA"] = xml
end

subject { put :put_userinfo, params: { login: user.login, format: :xml } }

context 'as an admin' do
before do
login admin_user

put :put_userinfo, params: { login: user.login, format: :xml }
user.reload
end

it_should_behave_like "not allowed to change user details"
Expand All @@ -69,9 +66,6 @@
context 'as a user' do
before do
login user

put :put_userinfo, params: { login: user.login, format: :xml }
user.reload
end

it_should_behave_like "not allowed to change user details"
Expand Down

0 comments on commit 80343b1

Please sign in to comment.