Not to skip executable specs (#9753)

* Not to skip executable specs

* Combine specs

Combine specs to one to reduce multiple slow http post.
signup-info-prompt
ysksn 2019-01-10 23:12:31 +09:00 committed by Eugen Rochko
parent af74491bc8
commit 61ecda1575
1 changed files with 5 additions and 7 deletions

View File

@ -84,19 +84,17 @@ RSpec.describe Api::V1::MediaController, type: :controller do
post :create, params: { file: fixture_file_upload('files/attachment.webm', 'video/webm') } post :create, params: { file: fixture_file_upload('files/attachment.webm', 'video/webm') }
end end
xit 'returns http success' do it do
# returns http success
expect(response).to have_http_status(200) expect(response).to have_http_status(200)
end
xit 'creates a media attachment' do # creates a media attachment
expect(MediaAttachment.first).to_not be_nil expect(MediaAttachment.first).to_not be_nil
end
xit 'uploads a file' do # uploads a file
expect(MediaAttachment.first).to have_attached_file(:file) expect(MediaAttachment.first).to have_attached_file(:file)
end
xit 'returns media ID in JSON' do # returns media ID in JSON
expect(body_as_json[:id]).to eq MediaAttachment.first.id.to_s expect(body_as_json[:id]).to eq MediaAttachment.first.id.to_s
end end
end end