summaryrefslogtreecommitdiff
path: root/spec/lib/uploaded_file_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/lib/uploaded_file_spec.rb')
-rw-r--r--spec/lib/uploaded_file_spec.rb54
1 files changed, 27 insertions, 27 deletions
diff --git a/spec/lib/uploaded_file_spec.rb b/spec/lib/uploaded_file_spec.rb
index a2f5c2e7121..47d77e88067 100644
--- a/spec/lib/uploaded_file_spec.rb
+++ b/spec/lib/uploaded_file_spec.rb
@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
describe UploadedFile do
let(:temp_dir) { Dir.tmpdir }
@@ -23,10 +23,10 @@ describe UploadedFile do
described_class.from_params(params, :file, upload_path)
end
- context 'when valid file is specified' do
- context 'only local path is specified' do
+ context "when valid file is specified" do
+ context "only local path is specified" do
let(:params) do
- { 'file.path' => temp_file.path }
+ {"file.path" => temp_file.path}
end
it "succeeds" do
@@ -38,13 +38,13 @@ describe UploadedFile do
end
end
- context 'all parameters are specified' do
+ context "all parameters are specified" do
let(:params) do
- { 'file.path' => temp_file.path,
- 'file.name' => 'dir/my file&.txt',
- 'file.type' => 'my/type',
- 'file.sha256' => 'sha256',
- 'file.remote_id' => 'remote_id' }
+ {"file.path" => temp_file.path,
+ "file.name" => "dir/my file&.txt",
+ "file.type" => "my/type",
+ "file.sha256" => "sha256",
+ "file.remote_id" => "remote_id",}
end
it "succeeds" do
@@ -52,15 +52,15 @@ describe UploadedFile do
end
it "generates filename from path" do
- expect(subject.original_filename).to eq('my_file_.txt')
- expect(subject.content_type).to eq('my/type')
- expect(subject.sha256).to eq('sha256')
- expect(subject.remote_id).to eq('remote_id')
+ expect(subject.original_filename).to eq("my_file_.txt")
+ expect(subject.content_type).to eq("my/type")
+ expect(subject.sha256).to eq("sha256")
+ expect(subject.remote_id).to eq("remote_id")
end
end
end
- context 'when no params are specified' do
+ context "when no params are specified" do
let(:params) do
{}
end
@@ -70,9 +70,9 @@ describe UploadedFile do
end
end
- context 'when only remote id is specified' do
+ context "when only remote id is specified" do
let(:params) do
- { 'file.remote_id' => 'remote_id' }
+ {"file.remote_id" => "remote_id"}
end
it "raises an error" do
@@ -80,12 +80,12 @@ describe UploadedFile do
end
end
- context 'when verifying allowed paths' do
+ context "when verifying allowed paths" do
let(:params) do
- { 'file.path' => temp_file.path }
+ {"file.path" => temp_file.path}
end
- context 'when file is stored in system temporary folder' do
+ context "when file is stored in system temporary folder" do
let(:temp_dir) { Dir.tmpdir }
it "succeeds" do
@@ -93,7 +93,7 @@ describe UploadedFile do
end
end
- context 'when file is stored in user provided upload path' do
+ context "when file is stored in user provided upload path" do
let(:upload_path) { Dir.mktmpdir }
let(:temp_dir) { upload_path }
@@ -102,7 +102,7 @@ describe UploadedFile do
end
end
- context 'when file is stored outside of user provided upload path' do
+ context "when file is stored outside of user provided upload path" do
let!(:generated_dir) { Dir.mktmpdir }
let!(:temp_dir) { Dir.mktmpdir }
@@ -118,10 +118,10 @@ describe UploadedFile do
end
end
- describe '#sanitize_filename' do
- it { expect(described_class.new(temp_file.path).sanitize_filename('spaced name')).to eq('spaced_name') }
- it { expect(described_class.new(temp_file.path).sanitize_filename('#$%^&')).to eq('_____') }
- it { expect(described_class.new(temp_file.path).sanitize_filename('..')).to eq('_..') }
- it { expect(described_class.new(temp_file.path).sanitize_filename('')).to eq('unnamed') }
+ describe "#sanitize_filename" do
+ it { expect(described_class.new(temp_file.path).sanitize_filename("spaced name")).to eq("spaced_name") }
+ it { expect(described_class.new(temp_file.path).sanitize_filename("\#$%^&")).to eq("_____") }
+ it { expect(described_class.new(temp_file.path).sanitize_filename("..")).to eq("_..") }
+ it { expect(described_class.new(temp_file.path).sanitize_filename("")).to eq("unnamed") }
end
end