mirror of
https://github.com/wahyd4/gitlabhq.git
synced 2026-08-21 10:36:04 +10:00
Hannes RosenöggerandDouwe Maan
9bf8480b4a
This commit generalizes the image upload via drag and drop so it supports all files. It also adds access control for these files.
69 lines
2.3 KiB
Ruby
69 lines
2.3 KiB
Ruby
require('spec_helper')
|
|
|
|
describe ProjectsController do
|
|
let(:project) { create(:project) }
|
|
let(:public_project) { create(:project, :public) }
|
|
let(:user) { create(:user) }
|
|
let(:jpg) { fixture_file_upload(Rails.root + 'spec/fixtures/rails_sample.jpg', 'image/jpg') }
|
|
let(:txt) { fixture_file_upload(Rails.root + 'spec/fixtures/doc_sample.txt', 'text/plain') }
|
|
|
|
describe 'POST #upload_file' do
|
|
before do
|
|
sign_in(user)
|
|
project.team << [user, :developer]
|
|
end
|
|
|
|
context "without params['markdown_file']" do
|
|
it 'returns an error' do
|
|
post :upload_file, id: project.to_param, format: :json
|
|
expect(response.status).to eq(422)
|
|
end
|
|
end
|
|
|
|
context 'with valid image' do
|
|
before do
|
|
post :upload_file,
|
|
id: project.to_param,
|
|
markdown_file: jpg,
|
|
format: :json
|
|
end
|
|
|
|
it 'returns a content with original filename, new link, and correct type.' do
|
|
expect(response.body).to match '\"alt\":\"rails_sample\"'
|
|
expect(response.body).to match "\"url\":\"http://test.host/uploads/#{project.path_with_namespace}"
|
|
expect(response.body).to match '\"is_image\":true'
|
|
end
|
|
end
|
|
|
|
context 'with valid non-image file' do
|
|
before do
|
|
post :upload_file, id: project.to_param, markdown_file: txt, format: :json
|
|
end
|
|
|
|
it 'returns a content with original filename, new link, and correct type.' do
|
|
expect(response.body).to match '\"alt\":\"doc_sample.txt\"'
|
|
expect(response.body).to match "\"url\":\"http://test.host/uploads/#{project.path_with_namespace}"
|
|
expect(response.body).to match '\"is_image\":false'
|
|
end
|
|
end
|
|
end
|
|
|
|
describe 'POST #toggle_star' do
|
|
it 'toggles star if user is signed in' do
|
|
sign_in(user)
|
|
expect(user.starred?(public_project)).to be_falsey
|
|
post :toggle_star, id: public_project.to_param
|
|
expect(user.starred?(public_project)).to be_truthy
|
|
post :toggle_star, id: public_project.to_param
|
|
expect(user.starred?(public_project)).to be_falsey
|
|
end
|
|
|
|
it 'does nothing if user is not signed in' do
|
|
post :toggle_star, id: public_project.to_param
|
|
expect(user.starred?(public_project)).to be_falsey
|
|
post :toggle_star, id: public_project.to_param
|
|
expect(user.starred?(public_project)).to be_falsey
|
|
end
|
|
end
|
|
end
|