diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2022-02-10 00:15:30 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-02-10 00:15:30 +0100 |
commit | 63002cde03a836b4510aca5da564504ecaedb5e9 (patch) | |
tree | f6749dd6bafae30c8708559da037668f5585b16e /spec/controllers/api/v1/media_controller_spec.rb | |
parent | 20a3564ab280a004cc7c075c00f63e70b1d65e07 (diff) |
Add editing for published statuses (#17320)
* Add editing for published statuses * Fix change of multiple-choice boolean in poll not resetting votes * Remove the ability to update existing media attachments for now
Diffstat (limited to 'spec/controllers/api/v1/media_controller_spec.rb')
-rw-r--r-- | spec/controllers/api/v1/media_controller_spec.rb | 21 |
1 files changed, 12 insertions, 9 deletions
diff --git a/spec/controllers/api/v1/media_controller_spec.rb b/spec/controllers/api/v1/media_controller_spec.rb index d8d732630..a1f6ddb24 100644 --- a/spec/controllers/api/v1/media_controller_spec.rb +++ b/spec/controllers/api/v1/media_controller_spec.rb @@ -110,21 +110,24 @@ RSpec.describe Api::V1::MediaController, type: :controller do end end - context 'when not attached to a status' do - let(:media) { Fabricate(:media_attachment, status: nil, account: user.account) } + context 'when the author \'s' do + let(:status) { nil } + let(:media) { Fabricate(:media_attachment, status: status, account: user.account) } - it 'updates the description' do + before do put :update, params: { id: media.id, description: 'Lorem ipsum!!!' } + end + + it 'updates the description' do expect(media.reload.description).to eq 'Lorem ipsum!!!' end - end - context 'when attached to a status' do - let(:media) { Fabricate(:media_attachment, status: Fabricate(:status), account: user.account) } + context 'when already attached to a status' do + let(:status) { Fabricate(:status, account: user.account) } - it 'returns http not found' do - put :update, params: { id: media.id, description: 'Lorem ipsum!!!' } - expect(response).to have_http_status(:not_found) + it 'returns http not found' do + expect(response).to have_http_status(:not_found) + end end end end |