tessa 0.1.3 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 255c6add5ff6306866a4a1dee9824a0accc98798
-  data.tar.gz: e501ec81adef345982f8b2c0265d151b032b0048
+  metadata.gz: a4842f3250777ed9f020f3096a04782f1e740a7d
+  data.tar.gz: a1984882ebc543a5a0acf098f54b4c081f61b8b0
 SHA512:
-  metadata.gz: e05ba5fb9523cbf9470a3bc162f98972d4d0b9affc29b8f90127ebc188c107e63e3dec1b558a3c7a295e3adfd03a3db16a0c2eb8e98b32075c40eb70b5759f76
-  data.tar.gz: 9c4f7b70485c409485d1e8750c3321cd2d7be724a8f8f6487e251ed733872caec0ce1176ce996db38322916f8afc347e73ac43d17313efea87c902ff2a279648
+  metadata.gz: ed5894431e9139b5440edb4d5cf30ad8c2f4364771953215a4e60089076e615b910be1bbb43616b35f9fa0a62e1b7b3dba893d06f4623dd59bcf6715e0da0db0
+  data.tar.gz: 5c1d7e34c98f0bc52a7bc01354217be0669346d97eeb1f2a71c2dcd5eb37aaecd9ab320d7f13e513dbba370960e65e23bb2062ea5c735823f12b751b012b9824

data/lib/tessa.rb

@@ -8,8 +8,12 @@ require "json"
 require "tessa/config"
 require "tessa/response_factory"
 require "tessa/asset"
+require "tessa/asset_change"
+require "tessa/asset_change_set"
+require "tessa/model"
 require "tessa/upload"
 
+
 module Tessa
   def self.config
     @config ||= Config.new
@@ -19,5 +23,13 @@ module Tessa
     yield config
   end
 
-  class RequestFailed < StandardError; end
+  class RequestFailed < StandardError
+    attr_reader :response
+
+    def initialize(message=nil, response=nil)
+      super(message)
+      @response = response
+    end
+  end
+
 end

data/lib/tessa/asset_change.rb

@@ -0,0 +1,49 @@
+module Tessa
+  class AssetChange
+    include Virtus.model
+
+    attribute :id, Integer
+    attribute :action, String
+
+    def initialize(args={})
+      case args
+      when Array
+        id, attributes = args
+        super attributes.merge(id: id)
+      else
+        super
+      end
+    end
+
+    def apply
+      if add?
+        asset.complete!
+      elsif remove?
+        asset.delete!
+      end
+    end
+
+    def hash
+      [id, action].hash
+    end
+
+    def eql?(b)
+      self.class == b.class &&
+        self.hash == b.hash
+    end
+
+    def add?
+      action == 'add'
+    end
+
+    def remove?
+      action == 'remove'
+    end
+
+    private
+
+    def asset
+      Tessa::Asset.new(id: id)
+    end
+  end
+end

data/lib/tessa/asset_change_set.rb

@@ -0,0 +1,49 @@
+module Tessa
+  class AssetChangeSet
+    include Virtus.model
+
+    attribute :changes, Array[AssetChange]
+    attribute :scoped_ids, Array[Integer]
+
+    def scoped_ids=(new_ids)
+      super new_ids.compact
+    end
+
+    def scoped_changes
+      changes.select { |change| scoped_ids.include?(change.id) }
+    end
+
+    def apply
+      scoped_changes.uniq.each(&:apply)
+    end
+
+    def +(b)
+      self.changes = (self.changes + b.changes).uniq
+      self.scoped_ids = (self.scoped_ids + b.scoped_ids).uniq
+      self
+    end
+
+    def add(value)
+      id = id_from_asset(value)
+      changes << AssetChange.new(id: id, action: "add")
+      scoped_ids << id
+    end
+
+    def remove(value)
+      id = id_from_asset(value)
+      changes << AssetChange.new(id: id, action: "remove")
+      scoped_ids << id
+    end
+
+    private
+
+    def id_from_asset(value)
+      case value
+      when Asset
+        value.id
+      when Fixnum
+        value
+      end
+    end
+  end
+end

data/lib/tessa/model.rb

@@ -0,0 +1,78 @@
+require 'tessa/model/field'
+
+module Tessa
+  module Model
+
+    def self.included(base)
+      base.send :include, InstanceMethods
+      base.extend ClassMethods
+      base.after_commit :apply_tessa_change_sets if base.respond_to?(:after_commit)
+      base.before_destroy :remove_all_tessa_assets if base.respond_to?(:before_destroy)
+    end
+
+    module InstanceMethods
+
+      def pending_tessa_change_sets
+        @pending_tessa_change_sets ||= Hash.new { AssetChangeSet.new }
+      end
+
+      def apply_tessa_change_sets
+        @pending_tessa_change_sets.delete_if do |_, change_set|
+          change_set.apply
+        end
+      end
+
+      def remove_all_tessa_assets
+        self.class.tessa_fields.each do |name, field|
+          change_set = pending_tessa_change_sets[name]
+          field.ids(on: self).each do |asset_id|
+            change_set.remove(asset_id)
+          end
+          pending_tessa_change_sets[name] = change_set
+        end
+      end
+
+    end
+
+    module ClassMethods
+
+      def asset(name, args={})
+        field = tessa_fields[name] = Field.new(args.merge(name: name))
+
+        define_method(name) do
+          if instance_variable_defined?(ivar = "@#{name}")
+            instance_variable_get(ivar)
+          else
+            instance_variable_set(
+              ivar,
+              Tessa::Asset.find(field.id(on: self))
+            )
+          end
+        end
+
+        define_method("#{name}=") do |value|
+          change_set = field.change_set_for(value)
+
+          if !(field.multiple? && value.is_a?(AssetChangeSet))
+            new_ids = change_set.scoped_changes.select(&:add?).map(&:id)
+            change_set += field.difference_change_set(new_ids, on: self)
+          end
+
+          pending_tessa_change_sets[name] += change_set
+
+          field.apply(change_set, on: self)
+        end
+      end
+
+      def tessa_fields
+        @tessa_fields ||= {}
+      end
+
+      def inherited(subclass)
+        subclass.instance_variable_set(:@tessa_fields, @tessa_fields.dup)
+      end
+
+    end
+
+  end
+end

data/lib/tessa/model/field.rb

@@ -0,0 +1,77 @@
+module Tessa
+  module Model
+    class Field
+      include Virtus.model
+
+      attribute :model
+      attribute :name, String
+      attribute :multiple, Boolean, default: false
+      attribute :id_field, String
+
+      def id_field
+        super || "#{name}#{default_id_field_suffix}"
+      end
+
+      def ids(on:)
+        [*id(on: on)]
+      end
+
+      def id(on:)
+        on.public_send(id_field)
+      end
+
+      def apply(set, on:)
+        ids = ids(on: on)
+
+        set.scoped_changes.each do |change|
+          if change.add?
+            ids << change.id
+          elsif change.remove?
+            ids.delete change.id
+          end
+        end
+
+        if multiple?
+          on.public_send(id_field_writer, ids)
+        else
+          on.public_send(id_field_writer, ids.first)
+        end
+      end
+
+      def change_set_for(value)
+        case value
+        when AssetChangeSet
+          value
+        when Array
+          value.map { |item| change_set_for(item) }.reduce(:+)
+        when Asset
+          AssetChangeSet.new.tap { |set| set.add(value) }
+        else
+          AssetChangeSet.new
+        end
+      end
+
+      def difference_change_set(subtrahend_ids, on:)
+        AssetChangeSet.new.tap do |change_set|
+          (ids(on: on) - subtrahend_ids).each do |id|
+            change_set.remove(id)
+          end
+        end
+      end
+
+      private
+
+      def id_field_writer
+        "#{id_field}="
+      end
+
+      def default_id_field_suffix
+        if multiple
+          "_ids"
+        else
+          "_id"
+        end
+      end
+    end
+  end
+end

data/lib/tessa/response_factory.rb

@@ -2,7 +2,7 @@ module Tessa
   module ResponseFactory
 
     def new_from_response(response)
-      raise RequestFailed unless response.success?
+      raise RequestFailed.new("Tessa responded with #{response.status}", response) unless response.success?
       case json = JSON.parse(response.body)
       when Array
         json.map { |record| new record }

data/lib/tessa/version.rb

@@ -1,3 +1,3 @@
 module Tessa
-  VERSION = "0.1.3"
+  VERSION = "0.2.0"
 end

data/spec/support/remote_call_macro.rb

@@ -31,7 +31,10 @@ RSpec.shared_examples_for "remote call macro" do |method, path, return_type|
     }
 
     it "raises Tessa::RequestFailed" do
-      expect{ call }.to raise_error(Tessa::RequestFailed)
+      expect{ call }.to raise_error { |error|
+        expect(error).to be_a(Tessa::RequestFailed)
+        expect(error.response).to be_a(Faraday::Response)
+      }
     end
   end
 

data/spec/tessa/asset_change_set_spec.rb

@@ -0,0 +1,198 @@
+require 'spec_helper'
+
+RSpec.describe Tessa::AssetChangeSet do
+  subject(:set) { described_class.new(args) }
+  let(:args) {
+    {
+      changes: Array.new(2) { Tessa::AssetChange.new },
+      scoped_ids: [1, 2],
+    }
+  }
+
+  describe "#initialize" do
+    it "sets :changes to attribute" do
+      expect(set.changes).to eq(args[:changes])
+    end
+
+    it "sets :scoped_ids to attribute" do
+      expect(set.scoped_ids).to eq([1, 2])
+    end
+
+    context "with various scoped_id input types" do
+      before { args[:scoped_ids] = ["1", 2, nil] }
+
+      it "ensures all values are integers and ignores nils" do
+        expect(set.scoped_ids).to eq([1, 2])
+      end
+    end
+
+    context "with hash format for changes" do
+      before do
+        args[:changes] = { "123" => { "action" => "add" }, "456" => { "action" => "remove" } }
+      end
+
+      it "initializes AssetChange objects for each hash" do
+        expect(set.changes[0].id).to eq(123)
+        expect(set.changes[0].action).to eq("add")
+        expect(set.changes[1].id).to eq(456)
+        expect(set.changes[1].action).to eq("remove")
+      end
+    end
+  end
+
+  describe "#scoped_changes" do
+    context "with changes not contained in scoped ids" do
+      let(:args) {
+        {
+          changes: [
+            Tessa::AssetChange.new(id: 1),
+            Tessa::AssetChange.new(id: 2),
+            Tessa::AssetChange.new(id: 3),
+            Tessa::AssetChange.new(id: 4)],
+          scoped_ids: [1, 2],
+        }
+      }
+
+      it "only returns the scoped changes" do
+        expect(set.scoped_changes.size).to eq(2)
+        expect(set.scoped_changes.map(&:id)).to eq([1, 2])
+      end
+    end
+  end
+
+  describe "#apply" do
+
+    context "with unscoped changes" do
+      let(:args) {
+        {
+          changes: [
+            Tessa::AssetChange.new(id: 1),
+            Tessa::AssetChange.new(id: 2)],
+          scoped_ids: [1],
+        }
+      }
+
+      it "calls apply on each scoped change" do
+        expect(set.changes[0]).to receive(:apply)
+        expect(set.changes[1]).not_to receive(:apply)
+        set.apply
+      end
+    end
+
+    context "with duplicate changes" do
+      let(:args) {
+        {
+          changes: [
+            Tessa::AssetChange.new(id: 1, action: 'remove'),
+            Tessa::AssetChange.new(id: 1, action: 'remove')],
+          scoped_ids: [1],
+        }
+      }
+
+      it "only calls apply on unique elements" do
+        expect(set.changes[0]).to receive(:apply)
+        expect(set.changes[1]).not_to receive(:apply)
+        set.apply
+      end
+    end
+  end
+
+  describe "#+" do
+    let(:a) {
+      described_class.new(
+        changes: [Tessa::AssetChange.new(id: 1, action: 'add')],
+        scoped_ids: [1],
+      )
+    }
+    let(:b) {
+      described_class.new(
+        changes: [Tessa::AssetChange.new(id: 2, action: 'add')],
+        scoped_ids: [2],
+      )
+    }
+    subject(:sum) { a + b }
+
+    it "concatenates the values of changes" do
+      expect(sum.changes.collect(&:id)).to eq([1, 2])
+    end
+
+    it "concatenates the values of scoped_ids" do
+      expect(sum.scoped_ids).to eq([1, 2])
+    end
+
+    context "with duplicate entries" do
+      subject(:sum) { a + b + b }
+
+      it "only includes unique values" do
+        expect(sum.changes.collect(&:id)).to eq([1, 2])
+        expect(sum.scoped_ids).to eq([1, 2])
+      end
+    end
+  end
+
+  describe "#add" do
+    let(:args) { {} }
+    let(:asset) { Tessa::Asset.new(id: 1) }
+
+    shared_examples_for "adds the represented asset" do
+      it "adds an 'add' change for this asset" do
+        expect(set.changes[0].id).to eq(1)
+        expect(set.changes[0].action).to eq("add")
+      end
+
+      it "adds the asset id to the scoped_ids array" do
+        expect(set.scoped_ids).to eq([1])
+      end
+    end
+
+    context "when passed an asset" do
+      before do
+        set.add(asset)
+      end
+
+      it_behaves_like "adds the represented asset"
+    end
+
+    context "when passed an integer id" do
+      before do
+        set.add(1)
+      end
+
+      it_behaves_like "adds the represented asset"
+    end
+  end
+
+  describe "#remove" do
+    let(:args) { {} }
+    let(:asset) { Tessa::Asset.new(id: 1) }
+
+    shared_examples_for "removes the represented asset" do
+      it "adds a 'remove' change for this asset" do
+        expect(set.changes[0].id).to eq(1)
+        expect(set.changes[0].action).to eq("remove")
+      end
+
+      it "adds the asset id to the scoped_ids array" do
+        expect(set.scoped_ids).to eq([1])
+      end
+    end
+
+    context "when passed an asset" do
+      before do
+        set.remove(asset)
+      end
+
+      it_behaves_like "removes the represented asset"
+    end
+
+    context "when passed an integer id" do
+      before do
+        set.remove(1)
+      end
+
+      it_behaves_like "removes the represented asset"
+    end
+
+  end
+
+end

data/spec/tessa/asset_change_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+RSpec.describe Tessa::AssetChange do
+  subject(:change) { described_class.new(args) }
+  let(:args) {
+    {
+      id: 123,
+      action: "add",
+    }
+  }
+
+  describe "#initialize" do
+    context "with hash args" do
+      it "sets id" do
+        expect(subject.id).to eq(123)
+      end
+
+      it "sets action" do
+        expect(subject.action).to eq("add")
+      end
+    end
+
+    context "with array arg" do
+      let(:args) { [123, { "action" => "add" }] }
+
+      it "sets id" do
+        expect(subject.id).to eq(123)
+      end
+
+      it "sets action" do
+        expect(subject.action).to eq("add")
+      end
+    end
+  end
+
+  describe "#apply" do
+    let(:asset) { instance_spy(Tessa::Asset) }
+    before do
+      expect(Tessa::Asset).to receive(:new).with(id: 123).and_return(asset)
+    end
+
+    context "with action 'add'" do
+      before { args[:action] = "add" }
+      it "calls complete! on asset with :id" do
+        change.apply
+        expect(asset).to have_received(:complete!)
+      end
+    end
+
+    context "with action 'remove'" do
+      before { args[:action] = "remove" }
+      it "calls delete! on asset with :id" do
+        change.apply
+        expect(asset).to have_received(:delete!)
+      end
+    end
+  end
+
+  describe "#add?" do
+    subject(:add?) { change.add? }
+
+    context "action = 'add'" do
+      before { change.action = 'add' }
+      it { is_expected.to eq(true) }
+    end
+
+    context "action != 'add'" do
+      before { change.action = 'something' }
+      it { is_expected.to eq(false) }
+    end
+  end
+
+  describe "#remove?" do
+    subject(:remove?) { change.remove? }
+
+    context "action = 'remove'" do
+      before { change.action = 'remove' }
+      it { is_expected.to eq(true) }
+    end
+
+    context "action != 'remove'" do
+      before { change.action = 'something' }
+      it { is_expected.to eq(false) }
+    end
+  end
+end

data/spec/tessa/model_field_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+RSpec.describe Tessa::Model::Field do
+  subject(:field) { described_class.new(attrs) }
+  let(:attrs) { {} }
+
+  describe "#initialize" do
+    context "with all fields set" do
+      let(:attrs) {
+        {
+          model: :model,
+          name: "name",
+          multiple: true,
+          id_field: "my_field",
+        }
+      }
+
+      it "sets :model to attribute" do
+        expect(field.model).to eq(:model)
+      end
+
+      it "sets :name to attribute" do
+        expect(field.name).to eq("name")
+      end
+
+      it "sets :multiple to attribute" do
+        expect(field.multiple).to eq(true)
+      end
+
+      it "sets :id_field to attribute" do
+        expect(field.id_field).to eq("my_field")
+      end
+    end
+
+    context "with no fields set" do
+      it "defaults model to nil" do
+        expect(field.model).to be_nil
+      end
+
+      it "defaults name to nil" do
+        expect(field.name).to be_nil
+      end
+
+      it "defaults multiple to false" do
+        expect(field.multiple).to eq(false)
+      end
+
+      context "when multiple true" do
+        before do
+          attrs[:name] = "my_name"
+          attrs[:multiple] = false
+        end
+
+        it "defaults id_field to name + '_id'" do
+          expect(field.id_field).to eq("my_name_id")
+        end
+      end
+
+      context "when multiple false" do
+        before do
+          attrs[:name] = "my_name"
+          attrs[:multiple] = true
+        end
+
+        it "defaults id_field to name + '_ids'" do
+          expect(field.id_field).to eq("my_name_ids")
+        end
+      end
+    end
+  end
+
+end

data/spec/tessa/model_spec.rb

@@ -0,0 +1,366 @@
+require 'spec_helper'
+
+RSpec.describe Tessa::Model do
+  subject(:described_module) { described_class }
+  let(:model) { Class.new.tap { |c| c.send(:include, described_module) } }
+
+  it { is_expected.to be_a(Module) }
+
+  describe "::asset" do
+    it "creates ModelField and sets it by name to @tessa_fields" do
+      model.asset :new_field
+      expect(model.tessa_fields[:new_field]).to be_a(Tessa::Model::Field)
+    end
+
+    context "with a field named :avatar" do
+      subject(:instance) { model.new }
+      before do
+        model.asset :avatar
+      end
+
+      it "creates an #avatar method" do
+        expect(instance).to respond_to(:avatar)
+      end
+
+      it "creates an #avatar= method" do
+        expect(instance).to respond_to(:avatar=)
+      end
+    end
+
+    context "with customized field" do
+      before do
+        model.asset :custom_field, multiple: true, id_field: "another_place"
+      end
+
+      it "sets all attributes on ModelField properly" do
+        field = model.tessa_fields[:custom_field]
+        expect(field.name).to eq("custom_field")
+        expect(field.multiple).to eq(true)
+        expect(field.id_field).to eq("another_place")
+      end
+    end
+
+    context "with inheritance hierarchy" do
+      let(:submodel) { Class.new(model) }
+      before do
+        model.asset :field1
+        submodel.asset :field2
+        model.asset :field3
+      end
+
+      it "submodel has its own list of fields" do
+        expect(submodel.tessa_fields.keys).to eq([:field1, :field2])
+      end
+
+      it "does not alter parent class fields" do
+        expect(model.tessa_fields.keys).to eq([:field1, :field3])
+      end
+    end
+  end
+
+  describe "#asset_setter" do
+    let(:instance) { model.new }
+
+    context "with change set present for a field" do
+      let(:a) { Tessa::Asset.new(id: 1) }
+      let(:b) { Tessa::Asset.new(id: 2) }
+      let(:set) { instance.pending_tessa_change_sets[:field] }
+      before do
+        model.send :attr_accessor, :field_id
+        model.asset :field
+        instance.field = a
+      end
+
+      it "combines the changes with the new set" do
+        instance.field = b
+        instance.field = nil
+        changes = set.changes.map { |change| [change.id, change.action.to_sym] }
+        expect(changes).to eq([
+          [1, :add],
+          [2, :add],
+          [1, :remove],
+          [2, :remove],
+        ])
+      end
+    end
+
+    context "with a multiple typed field" do
+      before do
+        model.send(:attr_accessor, :file_ids)
+        model.asset :file, multiple: true
+      end
+
+      context "when set with array of assets" do
+        let(:assets) {
+          [
+            Tessa::Asset.new(id: 1),
+            Tessa::Asset.new(id: 2),
+          ]
+        }
+
+        it "sets field to list of ids from assets" do
+          instance.file = assets
+          expect(instance.file_ids).to eq([1, 2])
+        end
+
+        it "removes any ids that aren't in list" do
+          instance.file_ids = [3]
+          instance.file = assets
+          expect(instance.file_ids).to eq([1, 2])
+        end
+
+        it "adds an AssetChangeSet to pending queue for this field" do
+          instance.file = assets
+          expect(instance.pending_tessa_change_sets).to be_a(Hash)
+          expect(instance.pending_tessa_change_sets[:file]).to be_a(Tessa::AssetChangeSet)
+        end
+
+        describe "the added change set" do
+          subject(:set) { instance.pending_tessa_change_sets[:file] }
+
+          it "has an 'add' action for each new asset" do
+            instance.file = assets
+            ids = set.changes.select { |c| c.action == 'add' }.collect(&:id)
+            expect(ids).to eq([1, 2])
+          end
+
+          it "has a 'remove' action for each missing asset" do
+            instance.file_ids = [3]
+            instance.file = assets
+            ids = set.changes.select { |c| c.action == 'remove' }.collect(&:id)
+            expect(ids).to eq([3])
+          end
+
+          it "adds each of the ids to the scoped_ids list" do
+            instance.file_ids = [3]
+            instance.file = assets
+            expect(set.scoped_ids).to eq([1, 2, 3])
+          end
+        end
+      end
+
+      context "when set with an AssetChangeSet" do
+        let(:set) { Tessa::AssetChangeSet.new }
+        before do
+          set.add(1)
+          set.add(2)
+          set.changes << Tessa::AssetChange.new(id: 0, action: "add")
+        end
+
+        it "sets field to list of ids from scoped_changes" do
+          instance.file = set
+          expect(instance.file_ids).to eq([1, 2])
+        end
+
+        it "leaves any ids that are not touched by the set" do
+          instance.file_ids = [3]
+          instance.file = set
+          expect(instance.file_ids).to include(1, 2, 3)
+        end
+
+        it "removes any ids from field that are scoped as removals" do
+          instance.file_ids = [3]
+          set.remove(3)
+          instance.file = set
+          expect(instance.file_ids).to eq([1, 2])
+        end
+
+        it "adds the AssetChangeSet to pending queue for this field" do
+          instance.file = set
+          new_set = instance.pending_tessa_change_sets[:file]
+          expect(new_set.changes).to eq(set.changes)
+          expect(new_set.scoped_ids).to eq(set.scoped_ids)
+        end
+      end
+    end
+
+    context "with a singular typed field" do
+      before do
+        model.send(:attr_accessor, :file_id)
+        model.asset :file
+      end
+
+      context "when set with an Asset" do
+        let(:asset) { Tessa::Asset.new(id: 1) }
+
+        it "sets field to id from asset" do
+          instance.file = asset
+          expect(instance.file_id).to eq(1)
+        end
+
+        it "adds an AssetChangeSet to pending queue for this field" do
+          instance.file = asset
+          expect(instance.pending_tessa_change_sets[:file]).to be_a(Tessa::AssetChangeSet)
+        end
+
+        describe "the added change set" do
+          let(:set) { instance.pending_tessa_change_sets[:file] }
+
+          it "has an 'add' action for the new asset" do
+            instance.file = asset
+            expect(set.changes.select(&:add?).first.id).to eq(1)
+          end
+
+          it "has a 'remove' action for the previous asset" do
+            instance.file_id = 2
+            instance.file = asset
+            expect(set.changes.select(&:remove?).first.id).to eq(2)
+          end
+        end
+      end
+
+      context "when set with an AssetChangeSet" do
+        let(:set) { Tessa::AssetChangeSet.new }
+        before do
+          set.changes << Tessa::AssetChange.new(id: 0, action: "add")
+          set.add(1)
+        end
+
+        it "sets field to asset id of first 'add' action in scoped_changes" do
+          instance.file = set
+          expect(instance.file_id).to eq(1)
+        end
+
+        it "adds the AssetChangeSet to pending queue for this field" do
+          instance.file = set
+          new_set = instance.pending_tessa_change_sets[:file]
+          expect(new_set.changes).to eq(set.changes)
+          expect(new_set.scoped_ids).to eq(set.scoped_ids)
+        end
+
+        context "with no remove in change set" do
+          it "ensures there is a 'remove' action for previous value" do
+            instance.file_id = 2
+            instance.file = set
+            expect(set.scoped_changes.select(&:remove?).map(&:id)).to eq([2])
+          end
+        end
+      end
+    end
+  end
+
+  describe "#asset_getter" do
+    let(:instance) { model.new }
+    subject(:getter) { instance.file }
+
+    context "with a multiple typed field" do
+      before do
+        model.send(:attr_accessor, :file_ids)
+        model.asset :file, multiple: true
+      end
+
+      it "calls find for each of the file_ids and returns result" do
+        instance.file_ids = [1, 2, 3]
+        expect(Tessa::Asset).to receive(:find).with([1, 2, 3]).and_return([:a1, :a2, :a3])
+        expect(getter).to eq([:a1, :a2, :a3])
+      end
+
+      it "caches the result" do
+        expect(Tessa::Asset).to receive(:find).and_return(:val).once
+        instance.file
+        instance.file
+      end
+    end
+
+    context "with a singular typed field" do
+      before do
+        model.send(:attr_accessor, :file_id)
+        model.asset :file
+      end
+
+      it "calls find for file_id and returns result" do
+        instance.file_id = 1
+        expect(Tessa::Asset).to receive(:find).with(1).and_return(:a1)
+        expect(getter).to eq(:a1)
+      end
+
+      it "caches the result" do
+        expect(Tessa::Asset).to receive(:find).and_return(:val).once
+        instance.file
+        instance.file
+      end
+    end
+  end
+
+  describe "#apply_tessa_change_sets" do
+    let(:instance) { model.new }
+    let(:sets) { Array.new(2) { instance_spy(Tessa::AssetChangeSet) } }
+
+    before do
+      model.asset :field1
+      model.asset :field2
+      instance.instance_variable_set(
+        :@pending_tessa_change_sets,
+        {
+          field1: sets[0],
+          field2: sets[1],
+        }
+      )
+      instance.apply_tessa_change_sets
+    end
+
+    it "iterates over all pending changesets calling apply" do
+      expect(sets[0]).to have_received(:apply)
+      expect(sets[1]).to have_received(:apply)
+    end
+
+    it "removes all changesets from list" do
+      expect(instance.pending_tessa_change_sets).to be_empty
+    end
+  end
+
+  describe "#remove_all_tessa_assets" do
+    let(:instance) { model.new }
+    before do
+      model.send :attr_accessor, :field1_id, :field2_ids
+      model.asset :field1
+      model.asset :field2, multiple: true
+      instance.field1_id = 1
+      instance.field2_ids = [2, 3]
+    end
+
+    it "adds pending change sets for each field removing all current assets" do
+      instance.remove_all_tessa_assets
+      changes = instance.pending_tessa_change_sets.values
+        .reduce(Tessa::AssetChangeSet.new, :+)
+        .changes
+        .map { |change| [change.id, change.action.to_sym] }
+        expect(changes).to eq([
+          [1, :remove],
+          [2, :remove],
+          [3, :remove],
+        ])
+    end
+  end
+
+  describe "adds callbacks" do
+    context "model responds to after_commit" do
+      let(:model) {
+        Class.new do
+          def self.after_commit(arg=nil)
+            @after_commit ||= arg
+          end
+        end.tap { |c| c.send(:include, described_module) }
+      }
+
+      it "calls it with :apply_tessa_change_sets" do
+        expect(model.after_commit).to eq(:apply_tessa_change_sets)
+      end
+    end
+
+    context "model responds to before_destroy" do
+      let(:model) {
+        Class.new do
+          def self.before_destroy(arg=nil)
+            @before_destroy ||= arg
+          end
+        end.tap { |c| c.send(:include, described_module) }
+      }
+
+      it "calls it with :remove_all_tessa_assets" do
+        expect(model.before_destroy).to eq(:remove_all_tessa_assets)
+      end
+    end
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tessa
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.0
 platform: ruby
 authors:
 - Justin Powell
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-13 00:00:00.000000000 Z
+date: 2015-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -113,14 +113,22 @@ files:
 - Rakefile
 - lib/tessa.rb
 - lib/tessa/asset.rb
+- lib/tessa/asset_change.rb
+- lib/tessa/asset_change_set.rb
 - lib/tessa/config.rb
+- lib/tessa/model.rb
+- lib/tessa/model/field.rb
 - lib/tessa/response_factory.rb
 - lib/tessa/upload.rb
 - lib/tessa/version.rb
 - spec/spec_helper.rb
 - spec/support/remote_call_macro.rb
+- spec/tessa/asset_change_set_spec.rb
+- spec/tessa/asset_change_spec.rb
 - spec/tessa/asset_spec.rb
 - spec/tessa/config_spec.rb
+- spec/tessa/model_field_spec.rb
+- spec/tessa/model_spec.rb
 - spec/tessa/upload_spec.rb
 - tessa.gemspec
 homepage: https://github.com/watermarkchurch/tessa
@@ -150,6 +158,10 @@ summary: Manage your assets.
 test_files:
 - spec/spec_helper.rb
 - spec/support/remote_call_macro.rb
+- spec/tessa/asset_change_set_spec.rb
+- spec/tessa/asset_change_spec.rb
 - spec/tessa/asset_spec.rb
 - spec/tessa/config_spec.rb
+- spec/tessa/model_field_spec.rb
+- spec/tessa/model_spec.rb
 - spec/tessa/upload_spec.rb