db97f9d08c
We were very inconsistent regarding these rules. Personally I prefer no empty lines around blocks, clases, etc... as recommended by the Ruby style guide [1], and they're the default values in rubocop, so those are the settings I'm applying. The exception is the `private` access modifier, since we were leaving empty lines around it most of the time. That's the default rubocop rule as well. Personally I don't have a strong preference about this one. [1] https://rubystyle.guide/#empty-lines-around-bodies
59 lines
2.5 KiB
Ruby
59 lines
2.5 KiB
Ruby
require "rails_helper"
|
|
|
|
describe DirectUpload do
|
|
context "configurations" do
|
|
it "is valid for different kind of combinations when attachment is valid" do
|
|
expect(build(:direct_upload, :proposal, :documents)).to be_valid
|
|
expect(build(:direct_upload, :proposal, :image)).to be_valid
|
|
expect(build(:direct_upload, :budget_investment, :documents)).to be_valid
|
|
expect(build(:direct_upload, :budget_investment, :image)).to be_valid
|
|
end
|
|
|
|
it "is not valid for different kind of combinations with invalid atttachment content types" do
|
|
expect(build(:direct_upload, :proposal, :documents, attachment: File.new("spec/fixtures/files/clippy.png"))).not_to be_valid
|
|
expect(build(:direct_upload, :proposal, :image, attachment: File.new("spec/fixtures/files/empty.pdf"))).not_to be_valid
|
|
expect(build(:direct_upload, :budget_investment, :documents, attachment: File.new("spec/fixtures/files/clippy.png"))).not_to be_valid
|
|
expect(build(:direct_upload, :budget_investment, :image, attachment: File.new("spec/fixtures/files/empty.pdf"))).not_to be_valid
|
|
end
|
|
|
|
it "is not valid without resource_type" do
|
|
expect(build(:direct_upload, :proposal, :documents, resource_type: nil)).not_to be_valid
|
|
end
|
|
|
|
it "is not valid without resource_relation" do
|
|
expect(build(:direct_upload, :proposal, :documents, resource_relation: nil)).not_to be_valid
|
|
end
|
|
|
|
it "is not valid without attachment" do
|
|
expect(build(:direct_upload, :proposal, :documents, attachment: nil)).not_to be_valid
|
|
end
|
|
|
|
it "is not valid without user" do
|
|
expect(build(:direct_upload, :proposal, :documents, user: nil)).not_to be_valid
|
|
end
|
|
end
|
|
|
|
context "save_attachment" do
|
|
it "saves uploaded file" do
|
|
proposal_document_direct_upload = build(:direct_upload, :proposal, :documents)
|
|
|
|
proposal_document_direct_upload.save_attachment
|
|
|
|
expect(File.exist?(proposal_document_direct_upload.relation.attachment.path)).to eq(true)
|
|
expect(proposal_document_direct_upload.relation.attachment.path).to include("cached_attachments")
|
|
end
|
|
end
|
|
|
|
context "destroy_attachment" do
|
|
it "removes uploaded file" do
|
|
proposal_document_direct_upload = build(:direct_upload, :proposal, :documents)
|
|
|
|
proposal_document_direct_upload.save_attachment
|
|
uploaded_path = proposal_document_direct_upload.relation.attachment.path
|
|
proposal_document_direct_upload.destroy_attachment
|
|
|
|
expect(File.exist?(uploaded_path)).to eq(false)
|
|
end
|
|
end
|
|
end
|