diff options
Diffstat (limited to 'spec/lib/uploaded_file_spec.rb')
-rw-r--r-- | spec/lib/uploaded_file_spec.rb | 54 |
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 |