saviour 0.2.3 → 0.3.0

data/spec/feature/persisted_path_spec.rb

@@ -1,34 +0,0 @@
-require 'spec_helper'
-
-describe "persisted path" do
-  before { allow(Saviour::Config).to receive(:storage).and_return(Saviour::LocalStorage.new(local_prefix: @tmpdir, public_url_prefix: "http://domain.com")) }
-
-  context "can change the default_path on the uploader and previous instances are not affected" do
-    it do
-      uploader = Class.new(Saviour::BaseUploader) { store_dir { "/store/dir" } }
-      klass = Class.new(Test) { include Saviour }
-      klass.attach_file :file, uploader
-
-      with_test_file("example.xml") do |example|
-        a = klass.create!
-        expect(a.update_attributes(file: example)).to be_truthy
-        expect(Saviour::Config.storage.exists?(a[:file])).to be_truthy
-        expect(File.dirname(a[:file])).to eq "/store/dir"
-
-
-        uploader.class_eval { store_dir { "/another/dir" } }
-
-        with_test_file("camaloon.jpg") do |example_2|
-          b = klass.create!
-          expect(b.update_attributes(file: example_2)).to be_truthy
-
-          expect(Saviour::Config.storage.exists?(b[:file])).to be_truthy
-          expect(Saviour::Config.storage.exists?(a[:file])).to be_truthy
-
-          expect(File.dirname(b[:file])).to eq "/another/dir"
-          expect(File.dirname(a[:file])).to eq "/store/dir"
-        end
-      end
-    end
-  end
-end

data/spec/feature/reload_model_spec.rb

@@ -1,24 +0,0 @@
-require 'spec_helper'
-
-describe "reload model" do
-  before { allow(Saviour::Config).to receive(:storage).and_return(Saviour::LocalStorage.new(local_prefix: @tmpdir, public_url_prefix: "http://domain.com")) }
-
-  context "updates the Saviour::File instance" do
-    it do
-      uploader = Class.new(Saviour::BaseUploader) { store_dir { "/store/dir" } }
-      klass = Class.new(Test) { include Saviour }
-      klass.attach_file :file, uploader
-      a = klass.create!
-      b = klass.find(a.id)
-
-      with_test_file("example.xml") do |example|
-        a.update_attributes! file: example
-        expect(a.file.exists?).to be_truthy
-        expect(b.file.exists?).to be_falsey
-
-        b.reload
-        expect(b.file.exists?).to be_truthy
-      end
-    end
-  end
-end

data/spec/feature/validations_spec.rb

@@ -1,171 +0,0 @@
-require 'spec_helper'
-
-describe "validations saving a new file" do
-  before { allow(Saviour::Config).to receive(:storage).and_return(Saviour::LocalStorage.new(local_prefix: @tmpdir, public_url_prefix: "http://domain.com")) }
-
-  let(:uploader) {
-    Class.new(Saviour::BaseUploader) {
-      store_dir { "/store/dir" }
-    }
-  }
-
-  let(:base_klass) {
-    a = Class.new(Test) { include Saviour }
-    a.attach_file :file, uploader
-    a
-  }
-
-  it "fails at block validation" do
-    klass = Class.new(base_klass) do
-      attach_validation(:file) do |contents, _|
-        errors.add(:file, "Cannot start with X") if contents[0] == 'X'
-      end
-    end
-
-    with_test_file("example.xml") do |example|
-      allow(example).to receive(:read).and_return("X-Extra contents for the file")
-      a = klass.new
-      a.file = example
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Cannot start with X"
-    end
-
-    with_test_file("example.xml") do |example|
-      a = klass.new
-      a.file = example
-      expect(a).to be_valid
-      expect(a.save).to be_truthy
-    end
-  end
-
-
-  it "fails at method validation" do
-    klass = Class.new(base_klass) do
-      attach_validation :file, :check_filesize
-
-      def check_filesize(contents, _)
-        errors.add(:file, "Filesize must be less than 10 bytes") if contents.bytesize >= 10
-      end
-    end
-
-    with_test_file("example.xml") do |example|
-      allow(example).to receive(:read).and_return("1234567890")
-      a = klass.new
-      a.file = example
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Filesize must be less than 10 bytes"
-    end
-
-    with_test_file("example.xml") do |example|
-      allow(example).to receive(:read).and_return("123456789")
-      a = klass.new
-      a.file = example
-      expect(a).to be_valid
-    end
-  end
-
-  it "combined validatinos" do
-    klass = Class.new(base_klass) do
-      attach_validation :file, :check_filesize
-      attach_validation(:file) do |contents, _|
-        errors.add(:file, "Cannot start with X") if contents[0] == 'X'
-      end
-
-      def check_filesize(contents, _)
-        errors.add(:file, "Filesize must be less than 10 bytes") if contents.bytesize >= 10
-      end
-    end
-
-    with_test_file("example.xml") do |example|
-      allow(example).to receive(:read).and_return("X-Ex")
-      a = klass.new
-      a.file = example
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Cannot start with X"
-    end
-
-    with_test_file("example.xml") do |example|
-      allow(example).to receive(:read).and_return("Ex too long content")
-      a = klass.new
-      a.file = example
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Filesize must be less than 10 bytes"
-    end
-
-    # Consistent order
-    with_test_file("example.xml") do |example|
-      allow(example).to receive(:read).and_return("X-Ex too long content")
-      a = klass.new
-      a.file = example
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Filesize must be less than 10 bytes"
-      expect(a.errors[:file][1]).to eq "Cannot start with X"
-    end
-  end
-
-  it "validates by filename" do
-    klass = Class.new(base_klass) do
-      attach_validation :file, :check_filename
-
-      def check_filename(_, filename)
-        errors.add(:file, "Only .jpg files") unless filename =~ /\.jpg/
-      end
-    end
-
-    with_test_file("example.xml") do |example|
-      allow(example).to receive(:read).and_return("X-Ex")
-      a = klass.new
-      a.file = example
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Only .jpg files"
-    end
-
-    with_test_file("camaloon.jpg") do |example|
-      allow(example).to receive(:read).and_return("X-Ex")
-      a = klass.new
-      a.file = example
-      expect(a).to be_valid
-    end
-  end
-
-  it "receives the attached_as information" do
-    klass = Class.new(base_klass) do
-      attach_validation :file, :check_filename
-
-      def check_filename(_, _, opts)
-        errors.add(:file, "Received error in #{opts[:attached_as]}")
-      end
-    end
-
-    with_test_file("example.xml") do |example|
-      a = klass.new file: example
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Received error in file"
-    end
-  end
-
-  context "versions are validated" do
-    let(:uploader) {
-      Class.new(Saviour::BaseUploader) {
-        store_dir { "/store/dir" }
-        version(:thumb)
-      }
-    }
-    let(:klass) {
-      Class.new(base_klass) do
-        attach_validation(:file) do |contents, _, opts|
-          errors.add(:file, "Cannot start with X in version #{opts[:version]}") if contents[0] == 'X'
-        end
-      end
-    }
-
-    it do
-      a = klass.create!
-      a.file.assign Saviour::StringSource.new("correct contents")
-      a.file(:thumb).assign Saviour::StringSource.new("X Incorrect contents")
-
-      expect(a).not_to be_valid
-      expect(a.errors[:file][0]).to eq "Cannot start with X in version thumb"
-    end
-  end
-end

data/spec/models/processors/digest_spec.rb

@@ -1,22 +0,0 @@
-require 'spec_helper'
-
-describe Saviour::Processors::Digest do
-  describe "#digest_filename" do
-    subject {
-      Class.new { include Saviour::Processors::Digest }.new
-    }
-
-    let(:filename) { "name.jpg" }
-    let(:contents) { "bynary contents for a file" }
-
-    it do
-      _, new_name = subject.digest_filename(contents, filename)
-      expect(new_name).to eq "name-ab54d187b7909ff4bba34777073d4654.jpg"
-    end
-
-    it do
-      _, new_name = subject.digest_filename(contents, filename, separator: '/')
-      expect(new_name).to eq "name/ab54d187b7909ff4bba34777073d4654.jpg"
-    end
-  end
-end

data/spec/models/saviour_spec.rb

@@ -1,80 +0,0 @@
-require 'spec_helper'
-
-describe Saviour do
-  it "raises error if included in a non active record class" do
-    expect {
-      Class.new do
-        include Saviour
-      end
-    }.to raise_error(Saviour::NoActiveRecordDetected)
-  end
-
-  it "error if column not present" do
-    expect {
-      Class.new(Test) do
-        include Saviour
-
-        attach_file :not_present, Saviour::BaseUploader
-      end
-    }.to raise_error(RuntimeError)
-  end
-
-  context do
-    it "error if column not present on version" do
-      uploader = Class.new(Saviour::BaseUploader) do
-        store_dir { "/store/dir" }
-
-        version(:thumb) do
-          store_dir { "/versions/store/dir" }
-        end
-
-        version(:not_present)
-      end
-
-      expect {
-        Class.new(Test) do
-          include Saviour
-
-          attach_file :file, uploader
-        end
-      }.to raise_error(RuntimeError)
-    end
-  end
-
-  it "does not raise error if table is not present" do
-    allow(Test).to receive(:table_exists?).and_return(false)
-
-    expect {
-      Class.new(Test) do
-        include Saviour
-
-        attach_file :not_present, Saviour::BaseUploader
-      end
-    }.to_not raise_error
-  end
-
-  describe ".attached_files" do
-    it "includes a mapping of the currently attached files and their versions" do
-      uploader = Class.new(Saviour::BaseUploader) do
-        store_dir { "/store/dir" }
-
-        version(:thumb)
-        version(:thumb_2)
-      end
-
-      klass = Class.new(Test) do
-        include Saviour
-        attach_file :file, uploader
-      end
-
-      expect(klass.attached_files).to eq({file: [:thumb, :thumb_2]})
-
-      klass2 = Class.new(Test) do
-        include Saviour
-        attach_file :file, Saviour::BaseUploader
-      end
-
-      expect(klass2.attached_files).to eq({file: []})
-    end
-  end
-end

data/spec/support/schema.rb

@@ -1,9 +0,0 @@
-ActiveRecord::Schema.define do
-  create_table :tests do |t|
-    t.string :file
-    t.string :file_thumb
-    t.string :file_thumb_2
-    t.string :name
-    t.timestamps null: false
-  end
-end