diff options
Diffstat (limited to 'spec/controllers/api/v1/statuses/bookmarks_controller_spec.rb')
-rw-r--r-- | spec/controllers/api/v1/statuses/bookmarks_controller_spec.rb | 67 |
1 files changed, 18 insertions, 49 deletions
diff --git a/spec/controllers/api/v1/statuses/bookmarks_controller_spec.rb b/spec/controllers/api/v1/statuses/bookmarks_controller_spec.rb index aa5ca433f..b79853718 100644 --- a/spec/controllers/api/v1/statuses/bookmarks_controller_spec.rb +++ b/spec/controllers/api/v1/statuses/bookmarks_controller_spec.rb @@ -21,67 +21,36 @@ describe Api::V1::Statuses::BookmarksController do post :create, params: { status_id: status.id } end - context 'with public status' do - it 'returns http success' do - expect(response).to have_http_status(:success) - end - - it 'updates the bookmarked attribute' do - expect(user.account.bookmarked?(status)).to be true - end - - it 'returns json with updated attributes' do - hash_body = body_as_json + it 'returns http success' do + expect(response).to have_http_status(:success) + end - expect(hash_body[:id]).to eq status.id.to_s - expect(hash_body[:bookmarked]).to be true - end + it 'updates the bookmarked attribute' do + expect(user.account.bookmarked?(status)).to be true end - context 'with private status of not-followed account' do - let(:status) { Fabricate(:status, visibility: :private) } + it 'return json with updated attributes' do + hash_body = body_as_json - it 'returns http not found' do - expect(response).to have_http_status(404) - end + expect(hash_body[:id]).to eq status.id.to_s + expect(hash_body[:bookmarked]).to be true end end describe 'POST #destroy' do - context 'with public status' do - let(:status) { Fabricate(:status, account: user.account) } - - before do - Bookmark.find_or_create_by!(account: user.account, status: status) - post :destroy, params: { status_id: status.id } - end - - it 'returns http success' do - expect(response).to have_http_status(:success) - end - - it 'updates the bookmarked attribute' do - expect(user.account.bookmarked?(status)).to be false - end - - it 'returns json with updated attributes' do - hash_body = body_as_json + let(:status) { Fabricate(:status, account: user.account) } - expect(hash_body[:id]).to eq status.id.to_s - expect(hash_body[:bookmarked]).to be false - end + before do + Bookmark.find_or_create_by!(account: user.account, status: status) + post :destroy, params: { status_id: status.id } end - context 'with private status that was not bookmarked' do - let(:status) { Fabricate(:status, visibility: :private) } - - before do - post :destroy, params: { status_id: status.id } - end + it 'returns http success' do + expect(response).to have_http_status(:success) + end - it 'returns http not found' do - expect(response).to have_http_status(404) - end + it 'updates the bookmarked attribute' do + expect(user.account.bookmarked?(status)).to be false end end end |