tripod 0.10.8 → 0.10.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 65ac9003b41041c38eca5b5a994c5a38fce051a9
-  data.tar.gz: 308116012a566cc07df65597074fa67b8ce371fc
+  metadata.gz: 40ee076fa76f2dd85f1f03f328f38e4e2813612b
+  data.tar.gz: af419e68c83352681b72f40c7d009929f6878ddf
 SHA512:
-  metadata.gz: 51eb6d4cae3e25e1d07585efd3478011f5f772d851c2398f0c06382121bee37074032055d7764248aa243f8a3436236003feec89f2dc5b9802da47482f4ad88f
-  data.tar.gz: 928a70bedfa0b096f93e66cf42a0fd1f302ff1eb0e8b5100412fa132797a15c942392c0524bf2b029151e5f8870e16e6703747464667ffb986cf473638f4d9cb
+  metadata.gz: 7c99e9887bf0e13489e3bcc33aa4d29a83780d82a78b8c1767b308129dba8df70da0e5f3ff966492ac30d962e80fb91d91628481258ae2607814fc826ea6180b
+  data.tar.gz: 65136c91f3e23f23f7061c1b113620176e555eff080f12a104a531edfe24a30022ad6ea64d5c4e4cccda22a320dea8d98db027acc248a13269c9acae7f376d21

data/lib/tripod.rb

@@ -93,9 +93,11 @@ require "tripod/predicates"
 require "tripod/attributes"
 require "tripod/callbacks"
 require "tripod/validations/is_url"
+require "tripod/rdf_type"
 require "tripod/errors"
 require "tripod/repository"
 require "tripod/fields"
+require "tripod/dirty"
 require "tripod/criteria"
 require "tripod/links"
 require "tripod/finders"
@@ -104,6 +106,9 @@ require "tripod/eager_loading"
 require "tripod/serialization"
 require "tripod/state"
 require "tripod/graphs"
+require "tripod/embeds"
+require "tripod/embeds/many"
+require "tripod/embedded_resource"
 require "tripod/version"
 
 # these need to be at the end

data/lib/tripod/attributes.rb

@@ -23,18 +23,18 @@ module Tripod::Attributes
     field ||= self.class.get_field(name)
 
     attr_values = read_predicate(field.predicate)
-   
+
     if field.multivalued
       # If the field is multivalued, return an array of the results
-      # just return the uri or the value of the literal.  
+      # just return the uri or the value of the literal.
       attr_values.map { |v| field.is_uri? ? v :  v.object }
-    else          
+    else
       # If it's not multivalued, return the first (should be only) result.
-      if field.is_uri?    
+      if field.is_uri?
         attr_values.first
       else
         # try to get it in english if it's there. (TODO: make it configurable what the default is)
-        val = attr_values.select{ |v| v.language == :en }.first || attr_values.first 
+        val = attr_values.select{ |v| v.language == :en }.first || attr_values.first
         val.object if val
       end
     end
@@ -70,6 +70,7 @@ module Tripod::Attributes
       new_val = write_value_for_field(value, field)
     end
 
+    attribute_will_change!(name)
     write_predicate(field.predicate, new_val)
   end
   alias :[]= :write_attribute

data/lib/tripod/components.rb

@@ -17,9 +17,12 @@ module Tripod::Components
   include Tripod::Attributes
   include Tripod::Callbacks
   include Tripod::Validations
+  include Tripod::RdfType
   include Tripod::Persistence
   include Tripod::Fields
   include Tripod::Links
+  include Tripod::Embeds
+  include Tripod::Dirty
   include Tripod::Finders
   include Tripod::Repository
   include Tripod::EagerLoading

data/lib/tripod/dirty.rb

@@ -0,0 +1,68 @@
+module Tripod::Dirty
+  extend ActiveSupport::Concern
+
+  def changed_attributes
+    @changed_attributes ||= {}
+  end
+
+  def changed
+    changed_attributes.keys
+  end
+
+  def changes
+    changed.reduce({}) do |memo, attr|
+      change = attribute_change(attr)
+      memo[attr] = change if change
+      memo
+    end
+  end
+
+  def attribute_will_change!(attr)
+    changed_attributes[attr] = read_attribute(attr) unless changed_attributes.has_key?(attr)
+  end
+
+  def attribute_change(attr)
+    [ changed_attributes[attr], read_attribute(attr) ] if attribute_changed?(attr)
+  end
+
+  def attribute_changed?(attr)
+    return false unless changed_attributes.has_key?(attr)
+    (changed_attributes[attr] != read_attribute(attr))
+  end
+
+  def post_persist
+    changed_attributes.clear
+  end
+
+  module ClassMethods
+    def create_dirty_methods(name, meth)
+      create_dirty_change_check(name, meth)
+      create_dirty_change_accessor(name, meth)
+      create_dirty_was_accessor(name, meth)
+    end
+
+    def create_dirty_change_accessor(name, meth)
+      generated_methods.module_eval do
+        re_define_method("#{meth}_change") do
+          attribute_change(name)
+        end
+      end
+    end
+
+    def create_dirty_change_check(name, meth)
+      generated_methods.module_eval do
+        re_define_method("#{meth}_changed?") do
+          attribute_changed?(name)
+        end
+      end
+    end
+
+    def create_dirty_was_accessor(name, meth)
+      generated_methods.module_eval do
+        re_define_method("#{meth}_was") do
+          changed_attributes[name]
+        end
+      end
+    end
+  end
+end

data/lib/tripod/embedded_resource.rb

@@ -7,6 +7,7 @@ module Tripod::EmbeddedResource
   include Tripod::Attributes
   include Tripod::Validations
   include Tripod::Fields
+  include Tripod::Dirty
   include Tripod::RdfType
 
   attr_reader :uri

data/lib/tripod/embeds.rb

@@ -26,6 +26,10 @@ module Tripod::Embeds
       # use this as a way to get to all the embedded properties for validation
       @_EMBEDDED ||= []
       @_EMBEDDED << name
+
+      # add statements to our hydrate query so the repository is populated appropriately
+      append_to_hydrate_construct ->(u) { "#{ u } <#{ predicate.to_s }> ?es . ?es ?ep ?eo ." }
+      append_to_hydrate_where ->(u) { "OPTIONAL { #{ u } <#{ predicate.to_s }> ?es . ?es ?ep ?eo . }" }
     end
 
     def get_embedded

data/lib/tripod/fields.rb

@@ -58,7 +58,7 @@ module Tripod::Fields
     # Define a field attribute for the +Resource+.
     #
     # @example Set the field.
-    #   Person.add_field(:name, :predicate => 'http://myfield')
+    #   Person.add_field(:name, 'http://myfield')
     #
     # @param [ Symbol ] name The name of the field.
     # @param [ String, RDF::URI ] predicate The predicate for the field.
@@ -95,6 +95,9 @@ module Tripod::Fields
       create_field_getter(name, meth, field)
       create_field_setter(name, meth, field)
       create_field_check(name, meth, field)
+
+      # from dirty.rb
+      create_dirty_methods(name, meth)
     end
 
     # Create the getter method for the provided field.

data/lib/tripod/finders.rb

@@ -172,15 +172,16 @@ module Tripod::Finders
     def _describe_query_for_select(select_sparql, opts={})
       uri_variable = opts[:uri_variable] || "uri"
       "
-        CONSTRUCT { ?tripod_construct_s ?tripod_construct_p ?tripod_construct_o } 
-        WHERE {           
-          {
-            SELECT (?#{uri_variable} as ?tripod_construct_s)
-            { 
-              #{select_sparql} 
-            } 
-          }
-          ?tripod_construct_s ?tripod_construct_p ?tripod_construct_o . 
+        CONSTRUCT {
+          ?tripod_construct_s ?tripod_construct_p ?tripod_construct_o .
+          #{ all_triples_construct('?tripod_construct_s') }
+        }
+        WHERE {
+          { SELECT (?#{uri_variable} as ?tripod_construct_s) {
+            #{select_sparql}
+          } }
+          ?tripod_construct_s ?tripod_construct_p ?tripod_construct_o .
+          #{ all_triples_where('?tripod_construct_s') }
         }
       "
     end
@@ -216,6 +217,10 @@ module Tripod::Finders
         data_graph = RDF::Graph.new
         repo.query( [RDF::URI.new(u), :predicate, :object] ) do |statement|
           data_graph << statement
+
+          if statement.object.is_a? RDF::Node
+            repo.query( [statement.object, :predicate, :object] ) {|s| data_graph << s}
+          end
         end
 
         # use it to hydrate this resource

data/lib/tripod/links.rb

@@ -124,13 +124,16 @@ module Tripod::Links
 
             uris = read_attribute(link.field_name)
 
-            filter_str = "FILTER("
+            filter_str = ""
+            
             if uris.any?
-              filter_str += uris.map {|u| "?uri = <#{u.to_s}>" }.join(" || ")
+              filter_str += " VALUES ?uri { <"
+              filter_str += uris.join("> <")
+              filter_str += "> } "
             else
-              filter_str += "1 = 0"
+              filter_str += "FILTER (1 = 0)"
             end
-            filter_str += ")"
+            
 
             criteria.where(filter_str).resources
           else

data/lib/tripod/persistence.rb

@@ -91,6 +91,7 @@ module Tripod::Persistence
         end
 
         @new_record = false # if running in a trans, just assume it worked. If the query is dodgy, it will throw an exception later.
+        post_persist
         true
       else
         false

data/lib/tripod/repository.rb

@@ -56,11 +56,7 @@ module Tripod::Repository
   end
 
   def retrieve_triples_from_database(accept_header="application/n-triples")
-    graph_selector = self.graph_uri.present? ? "<#{graph_uri.to_s}>" : "?g"
-    Tripod::SparqlClient::Query.query(
-      "CONSTRUCT {<#{uri}> ?p ?o} WHERE { GRAPH #{graph_selector} { <#{uri}> ?p ?o } }",
-      accept_header
-    )
+    Tripod::SparqlClient::Query.query(self.class.all_triples_query(uri, graph_uri: self.graph_uri), accept_header)
   end
 
   # returns a graph of triples from the underlying repository where this resource's uri is the subject.
@@ -87,6 +83,33 @@ module Tripod::Repository
       repo
     end
 
+    def append_to_hydrate_construct(statement)
+      @construct_statements ||= []
+      @construct_statements << statement
+    end
+
+    def append_to_hydrate_where(statement)
+      @where_statements ||= []
+      @where_statements << statement
+    end
+
+    def all_triples_query(uri, opts={})
+      graph_uri = opts.fetch(:graph_uri, nil)
+      graph_selector = graph_uri.present? ? "<#{graph_uri.to_s}>" : "?g"
+      uri_selector = "<#{uri}>"
+      "CONSTRUCT { #{uri_selector} ?p ?o . #{ all_triples_construct(uri_selector) } } WHERE { GRAPH #{graph_selector} { #{uri_selector} ?p ?o . #{ all_triples_where(uri_selector) } } }"
+    end
+
+    def all_triples_construct(uri)
+      extra_construct = @construct_statements.map{|s| s.call(uri) }.join if @construct_statements.present?
+      extra_construct || ''
+    end
+
+    def all_triples_where(uri)
+      extra_where = @where_statements.map{|s| s.call(uri) }.join if @where_statements.present?
+      extra_where || ''
+    end
+
   end
 
-end
+end

data/lib/tripod/resource.rb

@@ -13,8 +13,6 @@ module Tripod::Resource
     validates_presence_of :graph_uri
     # uri is a valid linked data url
     validates :uri, is_url: true
-    # every instance of a resource has an rdf type field, which is set at the class level
-    class_attribute :_RDF_TYPE
     # the Graph URI is set at the class level by default also, although this can be overridden in the constructor
     class_attribute :_GRAPH_URI
   end
@@ -52,7 +50,7 @@ module Tripod::Resource
     run_callbacks :initialize do
       graph_uri ||= self.class.get_graph_uri unless ignore_graph
       @graph_uri = RDF::URI(graph_uri) if graph_uri
-      self.rdf_type = self.class.get_rdf_type if respond_to?(:rdf_type=) && self.class.get_rdf_type
+      set_rdf_type
     end
   end
 
@@ -107,17 +105,6 @@ module Tripod::Resource
       other.class == Class ? self <= other : other.is_a?(self)
     end
 
-    # makes a "field" on this model called rdf_type
-    # and sets a class level _RDF_TYPE variable with the rdf_type passed in.
-    def rdf_type(new_rdf_type)
-      self._RDF_TYPE = RDF::URI.new(new_rdf_type.to_s)
-      field :rdf_type, RDF.type, :multivalued => true, :is_uri => true # things can have more than 1 type and often do
-    end
-
-    def get_rdf_type
-      self._RDF_TYPE
-    end
-
     def graph_uri(new_graph_uri)
       self._GRAPH_URI = new_graph_uri
     end
@@ -130,4 +117,4 @@ module Tripod::Resource
 end
 
 # causes any hooks to be fired, if they've been setup on_load of :tripod.
-ActiveSupport.run_load_hooks(:triploid, Tripod::Resource)
+ActiveSupport.run_load_hooks(:triploid, Tripod::Resource)

data/lib/tripod/version.rb

@@ -1,3 +1,3 @@
 module Tripod
-  VERSION = "0.10.8"
+  VERSION = "0.10.9"
 end

data/spec/app/models/dog.rb

@@ -14,4 +14,6 @@ class Dog
 
   linked_to :arch_enemy, 'http://example.com/archenemy', class_name: 'Dog'
   linked_to :enemies, 'http://example.com/enemy', class_name: 'Dog'
-end
+
+  embeds :fleas, 'http://example.com/fleas'
+end

data/spec/spec_helper.rb

@@ -31,7 +31,7 @@ Tripod.configure do |config|
   # config.update_endpoint = 'http://127.0.0.1:3030/tripod-test/update'
   # config.query_endpoint = 'http://127.0.0.1:3030/tripod-test/sparql'
   # config.data_endpoint = 'http://127.0.0.1:3030/tripod-test/data'
-  
+
   config.update_endpoint = 'http://127.0.0.1:3002/sparql/raw/update'
   config.query_endpoint = 'http://127.0.0.1:3002/sparql/raw'
   #config.data_endpoint = 'http://127.0.0.1:3030/tripod-test/data'
@@ -41,4 +41,4 @@ end
 Dir[ File.join(MODELS, "*.rb") ].sort.each do |file|
   name = File.basename(file, ".rb")
   autoload name.camelize.to_sym, name
-end
+end

data/spec/tripod/attributes_spec.rb

@@ -112,7 +112,7 @@ describe Tripod::Attributes do
     end
 
     context "where field is given" do
-      let(:field) { Person.send(:field_for, :hat_type, 'http://example.com/hat', {}) }
+      let(:field) { Person.send(:add_field, :hat_type, 'http://example.com/hat') }
 
       it "should derive the predicate name from the given field" do
         person.write_attribute(:hat_type, 'http://example.com/bowlerhat', field)
@@ -121,7 +121,7 @@ describe Tripod::Attributes do
     end
 
     context "where a field of a particular datatype is given" do
-      let(:field) { Person.send(:field_for, :hat_size, 'http://example.com/hatsize', {datatype: RDF::XSD.integer}) }
+      let(:field) { Person.send(:add_field, :hat_size, 'http://example.com/hatsize', {datatype: RDF::XSD.integer}) }
 
       it "should derive the datatype from the given field" do
         person.write_attribute(:hat_size, 10, field)
@@ -130,7 +130,7 @@ describe Tripod::Attributes do
     end
 
     context "where a multi-valued field of a given datatype is given" do
-      let(:field) { Person.send(:field_for, :hat_heights, 'http://example.com/hatheight', {datatype: RDF::XSD.integer, multivalued: true}) }
+      let(:field) { Person.send(:add_field, :hat_heights, 'http://example.com/hatheight', {datatype: RDF::XSD.integer, multivalued: true}) }
 
       it "should co-erce the values passed" do
         person.write_attribute(:hat_heights, [5, 10, 15], field)
@@ -144,4 +144,4 @@ describe Tripod::Attributes do
       end
     end
   end
-end
+end

data/spec/tripod/dirty_spec.rb

@@ -0,0 +1,59 @@
+require 'spec_helper'
+
+describe Tripod::Dirty do
+  let(:dog) { Dog.new('http://example.com/dog/spot') }
+
+  describe '#changes' do
+    before do
+      dog.name = 'Spot'
+    end
+
+    it 'should report the original and current values for a changed field' do
+      expect(dog.changes[:name]).to eq([nil, 'Spot'])
+    end
+
+    context 'when the field is set more than once' do
+      before do
+        dog.name = 'Zit'
+      end
+
+      it 'should still report the original value correctly' do
+        expect(dog.changes[:name]).to eq([nil, 'Zit'])
+      end
+    end
+
+    context 'when the field is set back to its original value' do
+      before do
+        dog.name = nil
+      end
+
+      it 'should no longer report a change to the field' do
+        expect(dog.changes.keys).to_not include(:name)
+      end
+    end
+
+    context 'on save' do
+      before { dog.save }
+
+      it 'should reset changes' do
+        expect(dog.changes).to be_empty
+      end
+    end
+  end
+
+  context 'field methods' do
+    before { dog.name = 'Wrex' }
+
+    it 'should create a <field>_change method' do
+      expect(dog.name_change).to eq([nil, 'Wrex'])
+    end
+
+    it 'should create a <field>_changed? method' do
+      expect(dog.name_changed?).to eq(true)
+    end
+
+    it 'should create a <field>_was method' do
+      expect(dog.name_was).to eq(nil)
+    end
+  end
+end

data/spec/tripod/embedded_resource_spec.rb

@@ -1,14 +1,14 @@
-# require 'spec_helper'
-# 
-# module Tripod
-#   describe EmbeddedResource do
-#     describe 'an instance' do
-#       let(:flea) { Flea.new }
-# 
-#       it 'should have getters & setters on fields' do
-#         flea.name = 'Bob'
-#         expect(flea.name).to eq('Bob')
-#       end
-#     end
-#   end
-# end
+require 'spec_helper'
+
+module Tripod
+  describe EmbeddedResource do
+    describe 'an instance' do
+      let(:flea) { Flea.new }
+
+      it 'should have getters & setters on fields' do
+        flea.name = 'Bob'
+        expect(flea.name).to eq('Bob')
+      end
+    end
+  end
+end

data/spec/tripod/embeds_spec.rb

@@ -1,63 +1,62 @@
-# require 'spec_helper'
-# 
-# describe Tripod::Embeds do
-#   let(:uri) { 'http://example.com/id/spot' }
-#   let(:dog) {
-#     d = Dog.new(uri)
-#     d.name = "Spot"
-#     d
-#   }
-#   let(:flea) {
-#     f = Flea.new
-#     f.name = 'Starsky'
-#     f
-#   }
-# 
-#   it 'should set and get embedded resources through the proxy' do
-#     dog.fleas << flea
-#     expect(dog.fleas.include?(flea)).to eq(true)
-#   end
-# 
-#   it 'should validate embedded resources' do
-#     dog.fleas << Flea.new
-#     expect(dog.valid?).to eq(false)
-#   end
-# 
-#   context 'given a saved instance' do
-#     before do
-#       dog.fleas << flea
-#       dog.save
-#     end
-# 
-#     context 'retrieved by uri' do
-#       let(:dogg) { Dog.find(uri) }
-# 
-#       it 'should hydrate embedded resources from the triple store' do
-#         f = dogg.fleas.first
-#         expect(f.name).to eq(flea.name)
-#       end
-#     end
-# 
-#     context 'retrieved as part of a resource collection' do
-#       let(:dogg) { Dog.all.resources.first }
-# 
-#       # RR: this is commented out as I backed out the (slow) update to the hydration CONSTRUCT
-#       it 'should hydrate embedded resources from the triple store' do
-#         f = dogg.fleas.first
-#         expect(f.name).to eq(flea.name)
-#       end
-#     end
-#   end
-# 
-# 
-#   describe 'delete' do
-#     before do
-#       dog.fleas << flea
-#     end
-# 
-#     it 'should remove all trace of the resource' do
-#       dog.fleas.delete(flea)
-#       expect(dog.fleas.include?(flea)).to eq(false)
-#     end
-#   end
-# end
+require 'spec_helper'
+
+describe Tripod::Embeds do
+  let(:uri) { 'http://example.com/id/spot' }
+  let(:dog) {
+    d = Dog.new(uri)
+    d.name = "Spot"
+    d
+  }
+  let(:flea) {
+    f = Flea.new
+    f.name = 'Starsky'
+    f
+  }
+
+  it 'should set and get embedded resources through the proxy' do
+    dog.fleas << flea
+    expect(dog.fleas.include?(flea)).to eq(true)
+  end
+
+  it 'should validate embedded resources' do
+    dog.fleas << Flea.new
+    expect(dog.valid?).to eq(false)
+  end
+
+  context 'given a saved instance' do
+    before do
+      dog.fleas << flea
+      dog.save
+    end
+
+    context 'retrieved by uri' do
+      let(:dogg) { Dog.find(uri) }
+
+      it 'should hydrate embedded resources from the triple store' do
+        f = dogg.fleas.first
+        expect(f.name).to eq(flea.name)
+      end
+    end
+
+    context 'retrieved as part of a resource collection' do
+      let(:dogg) { Dog.all.resources.first }
+
+      it 'should hydrate embedded resources from the triple store' do
+        f = dogg.fleas.first
+        expect(f.name).to eq(flea.name)
+      end
+    end
+  end
+
+
+  describe 'delete' do
+    before do
+      dog.fleas << flea
+    end
+
+    it 'should remove all trace of the resource' do
+      dog.fleas.delete(flea)
+      expect(dog.fleas.include?(flea)).to eq(false)
+    end
+  end
+end

data/spec/tripod/repository_spec.rb

@@ -30,9 +30,7 @@ describe Tripod::Repository do
 
         context 'graph_uri set on object' do
           it 'populates the object with triples, restricted to the graph_uri' do
-            Tripod::SparqlClient::Query.should_receive(:query).with(
-              "CONSTRUCT {<#{person.uri}> ?p ?o} WHERE { GRAPH <#{person.graph_uri}> { <#{person.uri}> ?p ?o } }",
-              "application/n-triples").and_call_original
+            Tripod::SparqlClient::Query.should_receive(:query).with(Person.all_triples_query(person.uri, graph_uri: person.graph_uri), 'application/n-triples').and_call_original
             person.hydrate!
             person.repository.should_not be_empty
           end
@@ -40,9 +38,7 @@ describe Tripod::Repository do
 
         context 'graph_uri not set on object' do
           it 'populates the object with triples, not to a graph' do
-            Tripod::SparqlClient::Query.should_receive(:query).with(
-              "CONSTRUCT {<#{graphless_resource.uri}> ?p ?o} WHERE { GRAPH ?g { <#{graphless_resource.uri}> ?p ?o } }",
-              "application/n-triples").and_call_original
+            Tripod::SparqlClient::Query.should_receive(:query).with(Person.all_triples_query(person.uri), 'application/n-triples').and_call_original
             graphless_resource.hydrate!
             graphless_resource.repository.should_not be_empty
           end
@@ -73,4 +69,4 @@ describe Tripod::Repository do
   end
 
 
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tripod
 version: !ruby/object:Gem::Version
-  version: 0.10.8
+  version: 0.10.9
 platform: ruby
 authors:
 - Ric Roberts
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-26 00:00:00.000000000 Z
+date: 2015-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -213,6 +213,7 @@ files:
 - lib/tripod/components.rb
 - lib/tripod/criteria.rb
 - lib/tripod/criteria/execution.rb
+- lib/tripod/dirty.rb
 - lib/tripod/eager_loading.rb
 - lib/tripod/embedded_resource.rb
 - lib/tripod/embeds.rb
@@ -261,6 +262,7 @@ files:
 - spec/tripod/attributes_spec.rb
 - spec/tripod/criteria_execution_spec.rb
 - spec/tripod/criteria_spec.rb
+- spec/tripod/dirty_spec.rb
 - spec/tripod/eager_loading_spec.rb
 - spec/tripod/embedded_resource_spec.rb
 - spec/tripod/embeds_spec.rb
@@ -311,6 +313,7 @@ test_files:
 - spec/tripod/attributes_spec.rb
 - spec/tripod/criteria_execution_spec.rb
 - spec/tripod/criteria_spec.rb
+- spec/tripod/dirty_spec.rb
 - spec/tripod/eager_loading_spec.rb
 - spec/tripod/embedded_resource_spec.rb
 - spec/tripod/embeds_spec.rb