muve 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d2c799eb30443546d3054c9dfb9b04069fa8ed21
-  data.tar.gz: e47754d72235cafaaaf41e5075319e0b7aab1c84
+  metadata.gz: 86078fd9dbc468dc0837688bd28c9ffbad043681
+  data.tar.gz: 3de1f46815f7ec2cdb1511b8c24445277a06df7a
 SHA512:
-  metadata.gz: 58c9bd11811f3b4181e8ff738963f275fffe8b36244d73a3013e0d5fdaaeef7bcdc8e1ca94d9d9f018995274e79680b6471388f0dd3d7dd556fb1f7b07eb7d55
-  data.tar.gz: abd202a42a8d6da4ebac3409657f9239f4313656028ab64cac55657e57cfd6bf2a632dc02e3453426985232a33e6d811177c3c10948c2c6c077055c4183155ad
+  metadata.gz: b83a8681897f5ef214d6aaa3ed729972c38e263bd91e62fa12cc555775fa2cb6e413bad80911d704eb027c553d06c08e2e3ef9b9628fb1e2a5669d11a9a475db
+  data.tar.gz: b0fb6001de7b85bb2b60e9fea97f32466e5365d058bcfba4aa65548b1456fa4b14ec9202270c4c04bc4260c633ed2bc33b9a8d38475b1251ac261841d1251c57

data/lib/muve/errors.rb

@@ -1,17 +1,19 @@
-module MuveError
-  class MuveStandardError < StandardError; end
-
-  class MuveIncompleteImplementation < MuveStandardError; end
-
-  class MuveInvalidAttributes < MuveStandardError; end
-
-  class MuveInvalidQuery < MuveStandardError; end
-
-  class MuveNotConfigured < MuveStandardError; end
-
-  class MuveNotFound < MuveStandardError; end
-
-  class MuveValidationError < MuveStandardError; end
-
-  class MuveSaveError < MuveStandardError; end
+module Muve
+  module Error
+    class StandardError < StandardError; end
+  
+    class IncompleteImplementation < StandardError; end
+  
+    class InvalidAttribute < StandardError; end
+  
+    class InvalidQuery < StandardError; end
+  
+    class NotConfigured < StandardError; end
+  
+    class NotFound < StandardError; end
+  
+    class ValidationError < StandardError; end
+  
+    class SaveError < StandardError; end
+  end
 end

data/lib/muve/location.rb

@@ -19,6 +19,14 @@ module Muve
       true
     end
   
+    def latitude=(value)
+      @latitude = value.to_f
+    end
+
+    def longitude=(value)
+      @longitude = value.to_f
+    end
+
     def random(center, range)
     end
   end

data/lib/muve/model.rb

@@ -13,7 +13,7 @@ module Muve
   # implementation
   # ++
   module Model
-    include MuveError
+    include Muve::Error
     include Muve::Helper
     
     def initialize(params={})
@@ -27,10 +27,20 @@ module Muve
       @destroyed = false
     end
 
+    def reload
+      self.send(:populate, extract(adaptor.get(self.class, id))) if id
+      self
+    end
+
     def self.included(base)
       base.extend ClassMethods
     end
 
+    def ==(rival)
+      return false unless rival.kind_of? self.class
+      self.attributes == rival.attributes
+    end
+
     # Initializes the +Muve::Model+ class. Use the +Muve::Model::init+ method 
     # to set a adaptor to take care of the retrieval and storage of resources.
     def self.init(handler=nil)
@@ -53,28 +63,29 @@ module Muve
       hash = {}
       attributes.map { |k, v|
         if v.respond_to? :to_hash
-          (hash[k] = v.to_hash(level+1, limit)) if level < limit
+          (hash[k.to_sym] = v.to_hash(level+1, limit)) if level < limit
         else
-          #(raise MuveAssocError, "#Associated #{v.class} for #{k} must respond to #to_hash or be a Hash") unless v.kind_of? Hash
-          hash[k] = v
+          #(raise AssocError, "#Associated #{v.class} for #{k} must respond to #to_hash or be a Hash") unless v.kind_of? Hash
+          hash[k.to_sym] = v
         end
       }
       hash
     end
 
-    # Save a resource and raises an MuveSaveError on failure
+    # Save a resource and raises an SaveError on failure
     def save!
+      raise ValidationError, "validation failed" unless valid?
       create_or_update
     rescue => e
-      e.backtrace.each { |err| p err }
-      raise MuveSaveError, "Save failed because #{e} was raised"
+      raise SaveError, "Save failed because #{e} was raised"
     end
   
     # Save a resource
     def save
       # TODO: be more verbose about the nature of the failure, if any
-      raise MuveValidationError, "validation failed" unless valid?
-      create_or_update
+      (create_or_update if valid?) or false
+    rescue => e
+      false
     end
 
     # Destroy a resource
@@ -115,8 +126,30 @@ module Muve
     def id
       @id
     end
+
+    # The parameterized identifier of the resource
+    def to_param
+      id && id.to_s
+    end
+
+    def attributes=(data)
+      Helper.symbolize_keys(data).each { |k, v| self.public_send("#{k}=", v) }
+      self
+    end
+
+    # Returns a Hash of the attributes for the current resource.
+    def attributes
+      self.class.extract_attributes(
+        resource: self,
+        fields: fields,
+        invalid_attributes: invalid_attributes,
+        id: self.id
+      )
+    end
   
     private
+    # Reserved attributes. These keys are not available to the user because 
+    # they are used to handle plumbing (library internal operations).
     def invalid_attributes
       %w(id adaptor)
     end
@@ -134,25 +167,17 @@ module Muve
       @new_record = false if details.key? :id
     end
 
-    def serialized_attributes
-      to_hash
+    def storeable_attributes
+      hash = {}
+      attributes.map { |k, v| hash[k] = convert(v) }
+      hash
     end
 
     def create_or_update
-      result = new_record? ? create(serialized_attributes) : update(serialized_attributes)
+      result = new_record? ? create(storeable_attributes) : update(storeable_attributes)
       self
     end
 
-    # NOTE: not sure we need this
-    def attributes
-      self.class.extract_attributes(
-        resource: self,
-        fields: fields,
-        invalid_attributes: invalid_attributes,
-        id: self.id
-      )
-    end
-
     # A manifest of the fields known to the model. The model logic seeks 
     # counsel from this resource to determine which properties to write and 
     # read from the repository.
@@ -164,6 +189,7 @@ module Muve
     # the new id and un-marking the new_record?
     def create(attr)
       @id = adaptor.create(self.class, attr)
+      # TODO: deal with unsuccessful #create
       @new_record = false
     end
 
@@ -176,9 +202,17 @@ module Muve
       self.class.adaptor
     end
 
+    def extract(storeable)
+      self.class.extract(storeable)
+    end
+
+    def convert(resource)
+      self.class.convert(resource)
+    end
+
     # Class methods exposed to all Muve models
     module ClassMethods
-      include MuveError
+      include Muve::Error
       # Configure the adaptor to take care of handling persistence for this
       # model. The adaptor should extend +Muve::Store+.
       #
@@ -193,10 +227,18 @@ module Muve
       # The adaptor currently set to handle persistence for all Muve::Model
       # classes and instances
       def adaptor
-        raise MuveNotConfigured, "the adaptor has not been set" unless (@adaptor || Model.handler)
+        raise NotConfigured, "the adaptor has not been set" unless (@adaptor || Model.handler)
         @adaptor or Model.handler
       end
 
+      def extract(storeable)
+        adaptor.formatter.convert_from_storeable_object(storeable)
+      end
+
+      def convert(resource)
+        adaptor.formatter.convert_to_storeable_object(resource)
+      end
+
       def connection
         Muve::Model.connection
       end
@@ -204,21 +246,27 @@ module Muve
       def database
         Muve::Model.database
       end
+
+      def model_name
+        self.to_s.split("::").last
+      end
   
       # The container (e.g.: collection, tablename or anything that is analogous
       # to this construct) of the resource
       def container
-        raise MuveError::MuveNotConfigured, "container not defined for #{self}"
+        raise Muve::Error::NotConfigured, "container not defined for #{self}"
       end
 
+      # Returns a Hash of the attributes for the given resource
+      # TODO: do we still need this?
       def extract_attributes(resource: self.new, fields: [], invalid_attributes: [], id: nil)
         data = {}
         fields.select{ |k| k != invalid_attributes }.each { |k| 
           # TODO: confirm resource.respond_to? k prior to assigning
-          data[k.to_sym] = resource.public_send(k)
+          data[k.to_sym] = resource.public_send(k) if resource.respond_to? k
         }
         if id
-          data = data.merge(resource.class.adaptor.index_hash id)
+          data = data.merge id: id
         end
         data
       end
@@ -226,7 +274,7 @@ module Muve
       # Finds a resource by id
       def find(id)
         result = self.new()
-        result.send(:populate, self.adaptor.get(self, id))
+        result.send(:populate, extract(self.adaptor.get(self, id)))
         result
       end
 
@@ -237,7 +285,7 @@ module Muve
           (self.adaptor.get(self, nil, params) or []).each do |details|
             details
             result = self.new()
-            result.send(:populate, details)
+            result.send(:populate, extract(details))
             item << result
           end
         end
@@ -255,15 +303,18 @@ module Muve
       # the required private +#fields# method one may specify the known fields
       # of the resource with one line of code.
       def with_fields(*args)
+        fields = self.new.send(:fields) # TODO: Fix this sloppy mess
         attr_accessor *args
-        class_eval "def fields; #{args}; end"
+        class_eval "def fields; #{Set.new(fields + args).to_a}; end"
       end
 
       # Creates a new resource and persists it to the datastore
-      def create(attributes)
-        resource = self.new(attributes)
-        resource.save if resource
-        resource
+      def create(attr)
+        self.new(attr).save
+      end
+
+      def create!(attr)
+        self.new(attr).save!
       end
 
       def destroy_all

data/lib/muve/movement.rb

@@ -3,7 +3,25 @@ module Muve
     include Model
   
     with_fields :traveller, :location, :time
-  
+
+    def latitude
+      location[:latitude] if location
+    end
+
+    def longitude
+      location[:longitude] if location
+    end
+
+    def latitude=(value)
+      location = {} unless location
+      location[:latitude]=(value) 
+    end
+
+    def longitude=(value)
+      location = {} unless location
+      location[:longitude]=(value) 
+    end
+
     def valid?
       assocs.each do |assoc|
         return false unless !assoc.nil? && assoc.valid?
@@ -14,6 +32,15 @@ module Muve
       true
     end
   
+    alias_method :lat,   :latitude
+    alias_method :lat=,  :latitude=
+    alias_method :lon,   :longitude
+    alias_method :lon=,  :longitude=
+    alias_method :lng,   :longitude
+    alias_method :lng=,  :longitude=
+    alias_method :long,  :longitude
+    alias_method :long=, :longitude=
+  
     private
     def assocs
       [

data/lib/muve/place.rb

@@ -0,0 +1,12 @@
+module Muve
+  class Place
+    include Model
+
+    with_fields :name, :location
+
+    def valid?
+      return false unless location.kind_of?(Muve::Location) && location.valid?
+      true
+    end
+  end
+end

data/lib/muve/store.rb

@@ -10,16 +10,29 @@ module Muve
   # Take a look at +Muve::Store::Mongo+ to find an implementation of a store
   # adaptor.
   module Store
-    include MuveError
+    include Muve::Error
     include Muve::Helper
 
+    module Formatter
+      include Muve::Error
+      include Muve::Helper
+
+      def convert_to_storeable_object(resource)
+        raise IncompleteImplementation, "convertor to storeable needed"
+      end
+
+      def convert_from_storeable_object(storeable)
+        raise IncompleteImplementation, "convertor from storeable needed"
+      end
+    end
+
     # gets data from the given container matching the provided details
     #
     # Given a +Place+ resource the following calls may be acceptable
     # - +Adaptor.get(Place, 1232) # find one resource where id = 1232+
     # - +Adaptor.get(Place, nil, { city: 'NYC', rating: 5 })+ # find more
     def get(resource, id=nil, details=nil)
-      raise MuveInvalidQuery unless id || details
+      raise InvalidQuery unless (id || details)
 
       if details
         find(resource, details)
@@ -31,7 +44,7 @@ module Muve
     # creates a resource containing the specified details in the repository.
     # Returns the id of the created object on success, raises an error otherwise
     def create(resource, details)
-      raise MuveIncompleteImplementation, "implement a create handler for #{self}"
+      raise IncompleteImplementation, "implement a create handler for #{self}"
     end
 
     # removes a resource matching the optional +id+ and +details+ 
@@ -39,12 +52,12 @@ module Muve
     # A successful removal operation should returns +true+ while any other 
     # value is considered an error.
     def delete(resource, id, details=nil)
-      raise MuveIncompleteImplementation, "implement a delete handler for #{self}"
+      raise IncompleteImplementation, "implement a delete handler for #{self}"
     end
 
     # update a resource with the identified by +id+ with the given +details+
     def update(resource, id, details)
-      raise MuveIncompleteImplementation, "implement a update handler for #{self}"
+      raise IncompleteImplementation, "implement a update handler for #{self}"
     end
 
     # collect a single resource from the repository that matches the given id 
@@ -54,7 +67,7 @@ module Muve
     #
     #   { id: 12, name: 'Spock', organization: 'The Enterprise' }
     def fetch(resource, id, details={})
-      raise MuveIncompleteImplementation, "implement a fetch handler for #{self}"
+      raise IncompleteImplementation, "implement a fetch handler for #{self}"
     end
 
     # find resources from its repository that match the given id and details
@@ -70,13 +83,13 @@ module Muve
     #     end
     #   end
     def find(resource, details)
-      raise MuveIncompleteImplementation, "implement a find handler for #{self}"
+      raise IncompleteImplementation, "implement a find handler for #{self}"
     end
 
     # counts the resources matching the details, if any.
     # Returns a integer that represents the amount of matching entities found.
     def count(resource, details={})
-      raise MuveIncompleteImplementation, "implement a count handler for #{self}"
+      raise IncompleteImplementation, "implement a count handler for #{self}"
     end
 
     alias_method :destroy, :delete
@@ -86,7 +99,11 @@ module Muve
     # is the +id+ or +_id+ field, while in other cases the index field may be
     # different. The store should take care of index naming.
     def index_hash(index_values)
-      {}
+      raise IncompleteImplementation, "implement the index_hash handler for #{self}"
+    end
+
+    def formatter
+      raise IncompleteImplementation, "specify a formatter"
     end
   end
 end

data/lib/muve/version.rb

@@ -1,3 +1,3 @@
 module Muve
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

data/lib/muve.rb

@@ -18,6 +18,8 @@ module Muve
   require "muve/helpers"
   require "muve/model"
   require "muve/store"
+
+  require "muve/place"
   require "muve/location"
   require "muve/traveller"
   require "muve/movement"
@@ -47,7 +49,7 @@ module Muve
       begin
       @@conn
       rescue => e
-        raise MuveNotConfigured, "the connection has not been defined"
+        raise NotConfigured, "the connection has not been defined"
       end
     end
 
@@ -56,7 +58,7 @@ module Muve
       begin
         @@db
       rescue => e
-        raise MuveNotConfigured, "the database has not been defined"
+        raise NotConfigured, "the database has not been defined"
       end
     end
   end
@@ -69,4 +71,8 @@ module Muve
     Model.connection = connection if connection
     Model.database = database if database
   end
+
+  def self.raise_something
+    raise NotConfigured
+  end
 end

data/spec/factories.rb

@@ -13,4 +13,9 @@ FactoryGirl.define do
     location { build Muve::Location }
     time { Time.now - rand(500000) }
   end
+
+  factory Muve::Place do
+    name { Faker::Venue.name }
+    location { build Muve::Location }
+  end
 end

data/spec/location_spec.rb

@@ -4,12 +4,15 @@ describe Muve::Location do
   let(:latitude) { Faker::Geolocation.lat }
   let(:longitude) { Faker::Geolocation.lng }
 
+  it { expect(Muve::Location.model_name).to eq("Location") }
+
   subject { Muve::Location.new latitude: latitude, longitude: longitude }
   it { expect(subject.latitude).to eq(latitude) }
   it { expect(subject.longitude).to eq(longitude) }
   it { expect(subject.lat).to eq(latitude) }
   it { expect(subject.long).to eq(longitude) }
   it { expect(subject.lng).to eq(longitude) }
+  it { expect{ subject.to_hash }.not_to raise_error }
 
   it { is_expected.to respond_to(:lat) }
   it { is_expected.to respond_to(:latitude) }
@@ -46,7 +49,7 @@ describe Muve::Location do
 
   context "validation" do
     before(:each) { allow_any_instance_of(Muve::Location).to receive(:create_or_update).and_return(true) }
-    it { expect{build(Muve::Location, longitude: -181).save}.to raise_error }
-    it { expect{build(Muve::Location).save}.not_to raise_error }
+    it { expect{build(Muve::Location, longitude: -181).save!}.to raise_error }
+    it { expect{build(Muve::Location).save!}.not_to raise_error }
   end
 end

data/spec/model_spec.rb

@@ -101,7 +101,7 @@ describe 'Model' do
           version: 'failed clone by failed experiment'
         )
       )
-    ).send(:serialized_attributes)).to eq(
+    ).send(:to_hash)).to eq(
       name: 'Stewie Griffin',
       version: 'our instance of the multiverse',
       another: {
@@ -121,7 +121,7 @@ describe 'Model' do
           version: 'failed clone by failed experiment'
         )
       )
-    ).send(:serialized_attributes)).to eq(
+    ).send(:to_hash)).to eq(
       name: 'Stewie Griffin',
       version: 'our instance of the multiverse',
       another: {
@@ -158,6 +158,7 @@ describe 'Model' do
         it { expect(subject.name).to eq('Peter Griffin') }
         it { expect(subject.description).to eq('Surfin-bird lover') }
         it { expect(subject.version).to eq('the fat one') }
+        it { expect(subject.attributes).to eq({ name: 'Peter Griffin', description: 'Surfin-bird lover', version: 'the fat one', age: nil }) }
       end
     end
   end
@@ -168,8 +169,12 @@ describe 'Model' do
     expect(Muve::Model.connection).to be(object)
   end
 
+  it 'allows equality checks between similar-type objects' do
+    expect(Resource.new(name: 'Jones')).to eq(Resource.new(name: 'Jones'))
+  end
+
   it 'raises a not configured exception when connection is not set' do
-    configuration_error = MuveError::MuveNotConfigured
+    configuration_error = Muve::Error::NotConfigured
     Muve::Model.remove_class_variable(:@@conn) if Muve::Model.class_variable_defined?(:@@conn)
 
     expect { Muve::Model.connection }.to raise_error(configuration_error)
@@ -178,7 +183,7 @@ describe 'Model' do
   end
 
   it 'raises a not configured exception when database is not set' do
-    configuration_error = MuveError::MuveNotConfigured
+    configuration_error = Muve::Error::NotConfigured
     Muve::Model.remove_class_variable(:@@db) if Muve::Model.class_variable_defined?(:@@db)
 
     expect { Muve::Model.database }.to raise_error(configuration_error)
@@ -221,6 +226,16 @@ describe 'Model' do
     expect(another.description).to eq('blah')
   end
 
+  it 'allows the modification of attributes' do
+    resource = Resource.new(name: 'muve-resource', version: 0)
+    expect(resource.name).to eq('muve-resource')
+    expect(resource.version).to eq(0)
+
+    resource.attributes = { name: 'french', version: 1 }
+    expect(resource.name).to eq('french')
+    expect(resource.version).to eq(1)
+  end
+
   # TODO: Study if this is desirable perhaps one would rather prefer setting
   # seperate adaptors for different models
   it 'shares the adaptor amongst all its instances' do
@@ -265,6 +280,18 @@ describe 'Model' do
         end
       end
 
+      class GenericFormatter
+        extend Muve::Store::Formatter
+
+        def self.convert_to_storeable_object(resource)
+          resource
+        end
+
+        def self.convert_from_storeable_object(storeable)
+          storeable
+        end
+      end
+
       class GenericAdaptor
         extend Muve::Store
 
@@ -272,6 +299,10 @@ describe 'Model' do
           hash = { :_id => index_value }
           { :_id => index_value }
         end
+
+        def self.formatter
+          GenericFormatter
+        end
       end
 
       Resource.adaptor = GenericAdaptor
@@ -302,12 +333,37 @@ describe 'Model' do
       subject { @res }
       it_behaves_like "an ActiveRecord-like resource"
     end
-
+    
     it 'calls the store create handler upon save' do
       expect(GenericAdaptor).to receive(:create).once
       @res.save
     end
 
+    describe '#reload' do
+      it 'does' do
+        id = SecureRandom.hex
+        initial_attrs = {
+          id: id,
+          name: 'First',
+          version: '0',
+          description: 'before'
+        }
+        final_attrs = {
+          id: id,
+          name: 'Last',
+          version: '99',
+          description: 'after'
+        }
+        allow(GenericAdaptor).to receive(:fetch).and_return(initial_attrs)
+        resource = AnotherResource.find(id)
+        expect(resource.attributes).to include(initial_attrs)
+
+        allow(GenericAdaptor).to receive(:fetch).and_return(final_attrs)
+        expect(resource.attributes).to include(initial_attrs)
+        expect(resource.reload.attributes).to include(final_attrs)
+      end
+    end
+
     describe '#find' do
       before(:each) do
         @id = SecureRandom.hex
@@ -336,6 +392,11 @@ describe 'Model' do
       it 'returns a record that is not a new record' do
         expect(AnotherResource.find(@id).new_record?).to be(false)
       end
+
+      it 'converts stored data to a workable object' do
+        expect(AnotherResource).to receive(:extract).at_least(:once)
+        AnotherResource.find(@id)
+      end
     end
 
     describe '#where' do
@@ -356,6 +417,11 @@ describe 'Model' do
           expect(item).to be_a(AnotherResource)
         end
       end
+
+      it 'converts a stored data to a workable object' do
+        expect(AnotherResource).to receive(:extract).at_least(:once)
+        AnotherResource.where({ name: 'all' }).count
+      end
     end
 
     describe '::create' do
@@ -365,8 +431,8 @@ describe 'Model' do
 
       it 'creates a new instance' do
         attributes = { name: 'Bonobo' }
-        expect(Resource).to receive(:new).with(attributes).once
-        Resource.create(attributes)
+        expect(Resource).to receive(:new).with(attributes).and_return(Resource.new(attributes)).once
+        expect(Resource.create(attributes)).to be_a(Resource)
       end
 
       it 'calls the save handler' do
@@ -383,6 +449,7 @@ describe 'Model' do
     describe '#save' do
       before(:each) do
         @res.name = 'first'
+        # NOTE: returning @id should not work, return should be a boolean
         allow(GenericAdaptor).to receive(:create).and_return(@id = SecureRandom.hex)
       end
 
@@ -402,6 +469,11 @@ describe 'Model' do
         it 'is persisted' do
           expect{ @res.save }.to change{ @res.persisted? }.to(true)
         end
+
+        it 'is converted to a storage-friendly format' do
+          expect(GenericFormatter).to receive(:convert_to_storeable_object).at_least(1)
+          @res.save
+        end
       end
 
       describe 'on a existing record' do
@@ -415,6 +487,12 @@ describe 'Model' do
           @res.name = 'second'
           @res.save
         end
+
+        it 'is converted to a storage-friendly format' do
+          expect(GenericFormatter).to receive(:convert_to_storeable_object).at_least(1)
+          @res.name = 'something'
+          @res.save
+        end
       end
     end
 
@@ -469,7 +547,7 @@ describe 'Model' do
       Resource.find(id)
     end
 
-    it 'converts the attributes to saveable objects' do
+    it 'hashes a flat resource' do
       expect(Resource.new(
         name: 'Peter Griffin',
         version: 'dumbass',
@@ -491,7 +569,7 @@ describe 'Model' do
       )
     end
 
-    it 'converts the attributes to saveable objects' do
+    it 'hashes nested resources' do
       another = AnotherResource.new
       another.send :populate, {
         id: SecureRandom.hex,
@@ -508,7 +586,7 @@ describe 'Model' do
         name: 'Peter Griffin',
         version: 'dumbass',
         another: {
-          _id: another.id,
+          id: another.id,
           name: 'Brian Griffin',
           version: 'the pretentious one',
           description: 'Canine, liberal, writer',
@@ -516,5 +594,62 @@ describe 'Model' do
         }
       )
     end
+
+    it 'returns the attributes on request' do
+      another = AnotherResource.new
+      another.send :populate, {
+        id: SecureRandom.hex,
+        name: 'Brian Griffin',
+        version: 'the pretentious one',
+        description: 'Canine, liberal, writer',
+        age: 8
+      }
+      resource = Resource.new
+      id = SecureRandom.hex
+      resource.send :populate, {
+        id: id,
+        name: 'Peter Griffin', 
+        version: 'dumbass', 
+        another: another
+      }
+
+      expect(resource.attributes).to eq(
+        id: id,
+        name: 'Peter Griffin',
+        version: 'dumbass',
+        another: another
+      )
+    end
+    
+    it 'converts repopulated data to resources' do
+      data = {
+        id: SecureRandom.hex,
+        name: 'Peter Griffin',
+        version: 'dumbass',
+        another: {
+          id: SecureRandom.hex,
+          name: 'Brian Griffin',
+          version: 'the pretentious one',
+          description: 'Canine, liberal, writer',
+          age: 8
+        }
+      }
+      resource = Resource.new
+      resource.send(:populate, data)
+    end
+  end
+
+  describe "#to_param" do
+    it "is equal to the stringified id" do
+      object = AnotherResource.new
+      object.send :populate, {
+        id: SecureRandom.hex,
+        name: 'Brian Griffin',
+        version: 'the pretentious one',
+        description: 'Canine, liberal, writer',
+        age: 8
+      }
+      expect(object.id.to_s).to eq(object.to_param)
+    end
   end
 end

data/spec/place_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Muve::Place do
+  it { expect(Muve::Place.model_name).to eq "Place" }
+
+  it { is_expected.to respond_to :location }
+  it { is_expected.to respond_to :name }
+
+  it 'is invalid when the location is not a Location' do
+    expect(build Muve::Place, location: Object.new).to be_invalid
+    expect(build Muve::Place, location: rand).to be_invalid
+    expect(build Muve::Place, location: build(Muve::Location, latitude: -100)).to be_invalid
+    expect(build Muve::Place, location: build(Muve::Location)).to be_valid
+  end
+end

data/spec/store_spec.rb

@@ -12,6 +12,10 @@ describe Muve::Store do
     class GenericStore
       extend Muve::Store
     end
+
+    class GenericFormatter
+      extend Muve::Store::Formatter
+    end
   end
 
   it 'provides methods to get a resource' do
@@ -33,6 +37,10 @@ describe Muve::Store do
     expect(GenericStore).to respond_to(:update)
   end
 
+  it 'has a formatter' do
+    expect(GenericStore).to respond_to(:formatter)
+  end
+
   it 'raises incomplete implementation errors on non-implemented methods' do
     %w(create delete update fetch find).each do |method|
       expect{
@@ -41,8 +49,10 @@ describe Muve::Store do
         else
           GenericStore.send(method, 'resource', nil)
         end
-      }.to raise_error(MuveError::MuveIncompleteImplementation)
+      }.to raise_error(Muve::Error::IncompleteImplementation)
     end
+
+    expect{GenericStore.send(:formatter)}.to raise_error(Muve::Error::IncompleteImplementation)
   end
 
   it 'attempts to fetch a resource if the id is given' do
@@ -60,4 +70,9 @@ describe Muve::Store do
       GenericStore.get(Resource)
     }.to raise_error
   end
+
+  describe "formatter" do
+    it { expect(GenericFormatter).to respond_to(:convert_to_storeable_object) }
+    it { expect(GenericFormatter).to respond_to(:convert_from_storeable_object) }
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: muve
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - David Asabina
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-26 00:00:00.000000000 Z
+date: 2014-09-12 00:00:00.000000000 Z
 dependencies: []
 description: Basic helpers to be used with Muvement
 email:
@@ -28,6 +28,7 @@ files:
 - lib/muve/location.rb
 - lib/muve/model.rb
 - lib/muve/movement.rb
+- lib/muve/place.rb
 - lib/muve/store.rb
 - lib/muve/traveller.rb
 - lib/muve/version.rb
@@ -38,6 +39,7 @@ files:
 - spec/model_spec.rb
 - spec/movement_spec.rb
 - spec/muve_spec.rb
+- spec/place_spec.rb
 - spec/spec_helper.rb
 - spec/store_spec.rb
 - spec/traveller_spec.rb
@@ -72,6 +74,7 @@ test_files:
 - spec/model_spec.rb
 - spec/movement_spec.rb
 - spec/muve_spec.rb
+- spec/place_spec.rb
 - spec/spec_helper.rb
 - spec/store_spec.rb
 - spec/traveller_spec.rb