From 61ecda15751a73262903f22c0ac163476b7ae75f Mon Sep 17 00:00:00 2001
From: ysksn <bluewhale1982@gmail.com>
Date: Thu, 10 Jan 2019 23:12:31 +0900
Subject: [PATCH] Not to skip executable specs (#9753)
* Not to skip executable specs
* Combine specs
Combine specs to one to reduce multiple slow http post.
---
spec/controllers/api/v1/media_controller_spec.rb | 12 +++++-------
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/spec/controllers/api/v1/media_controller_spec.rb b/spec/controllers/api/v1/media_controller_spec.rb
index f01fcd942..4e3037208 100644
--- a/spec/controllers/api/v1/media_controller_spec.rb
+++ b/spec/controllers/api/v1/media_controller_spec.rb
@@ -84,19 +84,17 @@ RSpec.describe Api::V1::MediaController, type: :controller do
post :create, params: { file: fixture_file_upload('files/attachment.webm', 'video/webm') }
end
- xit 'returns http success' do
+ it do
+ # returns http success
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
- end
- xit 'uploads a file' do
+ # uploads a 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
end
end
--
GitLab