active-fedora 6.6.0.rc5 → 6.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c25ddcf281f3eb10b558ee44dc144880d3551c31
-  data.tar.gz: f2f13e587a927cfe015eec879a023c8407eb2fac
+  metadata.gz: ef5496bb9123668b10de796159b394a50a1ec1da
+  data.tar.gz: 70b9bf4b2d64f1f0a6ffd3b5eb81dca852f47cd6
 SHA512:
-  metadata.gz: 317a0a139aa98883265545996a6587a8f26020a38d3aefbd001770f30c756a35c60079b49b9cd9deabf6ac357f7d204b2f36191b1b222af165e43a186bddd88d
-  data.tar.gz: 5777d15192af9d28f43f59c6f27640b55419cb74fc50df642981d038825a259504e54d086a9694eedec07ae700890cae55125fa8a70a8f147529a2e62606c59e
+  metadata.gz: c87b4be4a4d26c6424cdfa4d7ca73981760fa36d9f91b3e62ecf8396153ee581914f594cdb9a4edaaca1c73f52332ffba2b78af3f0dd57bee176add2e0381d1c
+  data.tar.gz: 15f245fddca1d9fb480cca04e41708e4e2fc1ec8cecd23409346101295bedecd286517e55b19d1420ba4511d076842f141d1171f9d16198b5ff6b3d9d3a11bc6

data/History.txt

@@ -1,4 +1,8 @@
-v6.6.0 (rc5)
+v6.6.0
+Add _destroy method for building nested forms [Justin Coyne]
+Removed the pid placeholder `__DO_NOT_USE__` [Justin Coyne]
+Allow relationships to accept instances of RDF::URI as their predicates [Justin Coyne]
+Remove internal deprecation [Justin Coyne]
 Gracefully handle an ObjectNotFound in find_all [Justin Coyne]
 Revert "Removed unused code that breaks certain use cases (Fixes #234)." [Jeremy Friesen]
 Removed unset opts parameter from call to reify_solr_results, introduced in PR #236. [David Chandek-Stark]
@@ -7,7 +11,18 @@ Fix deprecation warning when using the ids setter of a has_many collection relat
 Fix deprecation warning when loading a has_many association [Justin Coyne]
 Explicitly set cast to false on reload to avoid deprecation warnings [Justin Coyne]
 Cast relationship results if class is ActiveFedora::Base [Justin Coyne]
-* Need to put changes for rc1 here* 
+Adding a loop for getting more than solr_page_size (default 200) results in an has and belongs to many association [Carolyn Cole]
+Fixed syntax error on hash [Justin Coyne]
+Set cast parameter to avoid a deprecation warning [Justin Coyne]
+Only deprecate the cast parameter to find when on ActiveFedora::Base [Justin Coyne]
+Deprecate .find's cast default option [Jeremy Friesen]
+Remove outdated fixture loading line from README. [Steven Anderson]
+Support for extended cmodels (read-only) [Steven Anderson]
+Fix tense on deprecation warning [Steven Anderson]
+Fix tense on deprecation warning [Justin Coyne]
+delegate and delegate_to should use 'multiple' rather than 'unique'.  Ref #147 [Justin Coyne]
+Deprecated find(:all), find(:first), find(:last) [Justin Coyne]
+Prevent infinite loop when Fedora and Solr are out of sync (fixes #199) [Michael B. Klein]
 
 v6.5.1 (2013-09-10)
 
@@ -74,7 +89,6 @@ An rdf node should be able to set rdf:about [Justin Coyne]
 
 Remove RdfObject#get_values, it's inherited [Justin Coyne]
 
-
 v6.4.4 (2013-07-29)
 
 Bumping to version 6.4.4 [Jeremy Friesen]

data/lib/active_fedora/associations.rb

@@ -227,7 +227,8 @@ module ActiveFedora
           end
 
           redefine_method("#{reflection.name}_id=") do |new_value|
-            obj =  new_value.blank? ? nil : reflection.klass.find(new_value)
+            options = reflection.klass == ActiveFedora::Base ? {cast: true} : {}
+            obj =  new_value.blank? ? nil : reflection.klass.find(new_value, options)
             send("#{reflection.name}=", obj)
           end
           redefine_method("#{reflection.name}_id") do 

data/lib/active_fedora/base.rb

@@ -71,11 +71,6 @@ module ActiveFedora
       @marked_for_destruction
     end
 
-    def reload(options = nil)
-      @marked_for_destruction = false
-      super
-    end
-
     # Constructor.  You may supply a custom +:pid+, or we call the Fedora Rest API for the
     # next available Fedora pid, and mark as new object.
     # Also, if +attrs+ does not contain +:pid+ but does contain +:namespace+ it will pass the
@@ -96,6 +91,7 @@ module ActiveFedora
 
     # Reloads the object from Fedora.
     def reload
+      raise ActiveFedora::ObjectNotFoundError, "Can't reload an object that hasn't been saved" unless persisted?
       clear_association_cache
       init_with(self.class.find(self.pid, cast: false).inner_object)
     end

data/lib/active_fedora/indexing.rb

@@ -49,7 +49,7 @@ module ActiveFedora
     # @param [Hash] solr_doc @deafult an empty Hash
     def solrize_relationships(solr_doc = Hash.new)
       relationships.each_statement do |statement|
-        predicate = RelsExtDatastream.short_predicate(statement.predicate)
+        predicate = Predicates.short_predicate(statement.predicate)
         literal = statement.object.kind_of?(RDF::Literal)
         val = literal ? statement.object.value : statement.object.to_str
         ::Solrizer::Extractor.insert_solr_field_value(solr_doc, solr_name(predicate, :symbol), val )

data/lib/active_fedora/nested_attributes.rb

@@ -92,6 +92,15 @@ module ActiveFedora
       end
     end
 
+    # Returns ActiveFedora::Base#marked_for_destruction? It's
+    # used in conjunction with fields_for to build a form element for the
+    # destruction of this association.
+    #
+    # See ActionView::Helpers::FormHelper::fields_for for more info.
+    def _destroy
+      marked_for_destruction?
+    end
+
     private 
 
     # Attribute hash keys that should not be assigned as normal attributes.

data/lib/active_fedora/predicates.rb

@@ -1,5 +1,18 @@
 module ActiveFedora
   module Predicates
+    def self.short_predicate(predicate)
+      # for this regex to short-circuit correctly, namespaces must be sorted into descending order by length
+      if match = /^(#{Predicates.predicate_mappings.keys.sort.reverse.join('|')})(.+)$/.match(predicate.to_str)
+        namespace = match[1]
+        predicate = match[2]
+        Predicates.predicate_mappings[namespace].invert[predicate]
+      elsif predicate.kind_of? RDF::URI
+        predicate.to_s.split('/', 4).last.gsub(/(\/|#)/, '_').underscore
+      else
+        raise "Unable to parse predicate: #{predicate}"
+      end
+    end
+    
     def self.find_graph_predicate(predicate)
       #TODO, these could be cached
       case predicate

data/lib/active_fedora/relationship_graph.rb

@@ -10,44 +10,47 @@ module ActiveFedora
     end
 
     def has_predicate?(predicate)
-      relationships.has_key? predicate
+      relationships.has_key? uri_predicate(predicate)
     end
     
     def add(predicate, object, literal=false)
-      unless relationships.has_key? predicate
-        relationships[predicate] = []
+      uri = uri_predicate(predicate)
+      unless has_predicate? uri
+        relationships[uri] = []
       end
       object = RDF::Literal.new(object) if literal
-      unless relationships[predicate].include?(object)
+      unless relationships[uri].include?(object)
         @dirty = true
-        relationships[predicate] << object
+        relationships[uri] << object
       end
     end
 
     # Remove the statement matching the predicate and object
-    # [predicate] the predicate to delete
-    # [object] the object to delete, if nil, all statements with this predicate are deleted.
+    # @param predicate [Symbol, URI] the predicate to delete
+    # @param object the object to delete, if nil, all statements with this predicate are deleted.
     def delete(predicate, object = nil)
-      return unless relationships.has_key? predicate
+      uri = uri_predicate(predicate)
+      return unless has_predicate? uri 
       if object.nil?
         @dirty = true
-        relationships.delete(predicate)
+        relationships.delete(uri)
         return
       end
-      if relationships[predicate].include?(object)
+      if relationships[uri].include?(object)
         @dirty = true
-        relationships[predicate].delete(object)
+        relationships[uri].delete(object)
       end
-      if object.respond_to?(:internal_uri) && relationships[predicate].include?(object.internal_uri)
+      if object.respond_to?(:internal_uri) && relationships[uri].include?(object.internal_uri)
         @dirty = true
-        relationships[predicate].delete(object.internal_uri)
+        relationships[uri].delete(object.internal_uri)
       elsif object.is_a? String 
-        relationships[predicate].delete_if{|obj| obj.respond_to?(:internal_uri) && obj.internal_uri == object}
+        relationships[uri].delete_if{|obj| obj.respond_to?(:internal_uri) && obj.internal_uri == object}
       end
     end
 
     def [](predicate)
-      relationships[predicate]
+      uri = uri_predicate(predicate)
+      relationships[uri]
     end
     
     def to_graph(subject_uri)
@@ -65,7 +68,7 @@ module ActiveFedora
 
     # Create an RDF statement
     # @param uri a string represending the subject
-    # @param predicate a predicate symbol
+    # @param predicate [Symbol, URI] a predicate
     # @param target an object to store
     def build_statement(uri, predicate, target)
       raise "Not allowed anymore" if uri == :self
@@ -87,10 +90,14 @@ module ActiveFedora
         end
         object = RDF::URI.new(target)
       end
-      predicate = ActiveFedora::Predicates.find_graph_predicate(predicate) unless predicate.kind_of? RDF::URI
-      RDF::Statement.new(subject, predicate, object)
+      RDF::Statement.new(subject, uri_predicate(predicate), object)
     
     end
 
+    def uri_predicate(predicate)
+      return predicate if predicate.kind_of? RDF::URI
+      ActiveFedora::Predicates.find_graph_predicate(predicate)
+    end
+
   end
 end

data/lib/active_fedora/rels_ext_datastream.rb

@@ -52,9 +52,8 @@ module ActiveFedora
         RDF::RDFXML::Reader.new(xml) do |reader|
           reader.each_statement do |statement|
             literal = statement.object.kind_of?(RDF::Literal)
-            predicate = self.short_predicate(statement.predicate)
             object = literal ? statement.object.value : statement.object.to_str
-            tmpl.model.add_relationship(predicate, object, literal)
+            tmpl.model.add_relationship(statement.predicate, object, literal)
           end
         end
         tmpl.relationships_are_not_dirty!
@@ -91,19 +90,6 @@ module ActiveFedora
       end
     end
 
-    def self.short_predicate(predicate)
-      # for this regex to short-circuit correctly, namespaces must be sorted into descending order by length
-      if match = /^(#{Predicates.predicate_mappings.keys.sort.reverse.join('|')})(.+)$/.match(predicate.to_str)
-        namespace = match[1]
-        predicate = match[2]
-        pred = Predicates.predicate_mappings[namespace].invert[predicate]
-        pred
-      else
-        raise "Unable to parse predicate: #{predicate}"
-      end
-    end
-    
-    
     # ** EXPERIMENTAL **
     # 
     # This is utilized by ActiveFedora::Base.load_instance_from_solr to load 

data/lib/active_fedora/semantic_node.rb

@@ -23,7 +23,7 @@ module ActiveFedora
     end
 
     # Add a relationship to the Object.
-    # @param predicate
+    # @param predicate [Symbol]  The short version of the predicate
     # @param target Either a string URI or an object that is a kind of ActiveFedora::Base 
     def add_relationship(predicate, target, literal=false)
       object_relations.add(predicate, target, literal)

data/lib/active_fedora/unsaved_digital_object.rb

@@ -4,7 +4,7 @@ module ActiveFedora
     include DigitalObject::DatastreamBootstrap
     attr_accessor :original_class, :ownerId, :state, :datastreams, :label, :namespace
 
-    PLACEHOLDER = '__DO_NOT_USE__'
+    PLACEHOLDER = nil
     
     def initialize(original_class, namespace, pid=nil)
       @pid = pid

data/lib/active_fedora/version.rb

@@ -1,3 +1,3 @@
 module ActiveFedora
-  VERSION = "6.6.0.rc5"
+  VERSION = "6.6.0"
 end

data/spec/integration/associations_spec.rb

@@ -1,6 +1,30 @@
 require 'spec_helper'
 
 describe ActiveFedora::Base do
+  describe "use a URI as the property" do
+    before do
+      class Book < ActiveFedora::Base 
+        belongs_to :author, :property=>RDF::DC.creator, :class_name=>'Person'
+      end
+
+      class Person < ActiveFedora::Base
+      end
+    end
+
+    after do
+      Object.send(:remove_const, :Book)
+      Object.send(:remove_const, :Person)
+    end
+
+    let(:person) { Person.create}
+    let(:book) { Book.new(author: person) }
+
+    it "should go" do
+      book.save
+    end
+
+  end
+
   describe "complex example" do
     before do
       class Library < ActiveFedora::Base 
@@ -473,6 +497,32 @@ describe ActiveFedora::Base do
     end
   end
 
+  describe "belongs_to when class_name is ActiveFedora::Base" do
+    before :all do
+      class Textbook < ActiveFedora::Base
+        belongs_to :container, :property=>:is_part_of, :class_name=>'ActiveFedora::Base'
+      end
+      class Shelf < ActiveFedora::Base; end
+    end
+
+    after :all do
+      Object.send(:remove_const, :Textbook)
+      Object.send(:remove_const, :Shelf)
+    end
+
+    after do
+      shelf.destroy
+    end
+
+    let(:shelf) { Shelf.create}
+    subject { Textbook.new }
+
+    it "Should not raise a deprecation message" do
+      Deprecation.should_not_receive(:warn) # a deprecation in 6.6.0 that's going away in 7.0.0
+      subject.container_id =  shelf.id
+    end
+  end
+
   describe "single direction habtm" do
     before :all do
       class Course < ActiveFedora::Base
@@ -801,10 +851,6 @@ describe ActiveFedora::Base do
           @simple_collection = SimpleCollection.create
           @complex_collection = ComplexCollection.create
 
-          #Need to add the simpler cmodel here as currently inheritance support is read-only.
-          #See ActiveFedora pull request 207 on how to do this programmatically.
-          @complex_collection.add_relationship(:has_model, @complex_object.class.superclass.to_class_uri)
-
           @simple_object = SimpleObject.create
           @simple_object_second = SimpleObject.create
           @complex_object = ComplexObject.create
@@ -860,4 +906,4 @@ describe ActiveFedora::Base do
       end
     end
   end
-end
+end

data/spec/integration/nested_attribute_spec.rb

@@ -43,6 +43,10 @@ describe "NestedAttribute behavior" do
     end
   end
 
+  it "should have _destroy" do
+    Bar.new._destroy.should be_false
+  end
+
   it "should update the child objects" do
     @car, @bar1, @bar2 = create_car_with_bars
 

data/spec/integration/ntriples_datastream_spec.rb

@@ -147,8 +147,8 @@ describe ActiveFedora::NtriplesRDFDatastream do
     end
 
     it "should write rdf with proper subjects" do
-      @subject.rdf.type = "Frog"
       @subject.inner_object.pid = 'test:99'
+      @subject.rdf.type = "Frog"
       @subject.save!
       @subject.reload
       @subject.rdf.graph.dump(:ntriples).should == "<http://oregondigital.org/ns/99> <http://purl.org/dc/terms/type> \"Frog\" .\n"

data/spec/unit/base_extra_spec.rb

@@ -60,7 +60,7 @@ describe ActiveFedora::Base do
     it "should delete object from repository and index" do
       @test_object.inner_object.stub(:delete)
       mock_conn = double("SolrConnection")
-      mock_conn.should_receive(:delete_by_id).with("__DO_NOT_USE__") 
+      mock_conn.should_receive(:delete_by_id).with(nil) 
       mock_conn.should_receive(:commit)
       mock_ss = double("SolrService")
       mock_ss.stub(:conn).and_return(mock_conn)

data/spec/unit/base_spec.rb

@@ -202,10 +202,9 @@ describe ActiveFedora::Base do
 
     ### Methods for ActiveModel::Conversions
     it "should have to_param once it's saved" do
-
       @test_object.to_param.should be_nil
-      @test_object.inner_object.stub(:new? => false)
-      @test_object.to_param.should == @test_object.pid
+      @test_object.inner_object.stub(:new? => false, :pid => 'foo:123')
+      @test_object.to_param.should == 'foo:123'
     end
 
     it "should have to_key once it's saved" do
@@ -448,12 +447,13 @@ describe ActiveFedora::Base do
       it "should add pid, system_create_date, system_modified_date and object_state from object attributes" do
         @test_object.should_receive(:create_date).and_return("2012-03-04T03:12:02Z")
         @test_object.should_receive(:modified_date).and_return("2012-03-07T03:12:02Z")
+        @test_object.stub(pid: 'changeme:123')
         @test_object.state = "D"
         solr_doc = @test_object.to_solr
         solr_doc[ActiveFedora::SolrService.solr_name("system_create", :stored_sortable, type: :date)].should eql("2012-03-04T03:12:02Z")
         solr_doc[ActiveFedora::SolrService.solr_name("system_modified", :stored_sortable, type: :date)].should eql("2012-03-07T03:12:02Z")
         solr_doc[ActiveFedora::SolrService.solr_name("object_state", :stored_sortable)].should eql("D")
-        solr_doc[:id].should eql("#{@test_object.pid}")
+        solr_doc[:id].should eql("changeme:123")
       end
 
       it "should omit base metadata and RELS-EXT if :model_only==true" do

data/spec/unit/datastream_spec.rb

@@ -19,7 +19,7 @@ describe ActiveFedora::Datastream do
   end
   
   it "should be inspectable" do
-    @test_datastream.inspect.should match /#<ActiveFedora::Datastream @pid=\"__DO_NOT_USE__\" @dsid=\"abcd\" @controlGroup=\"M\" changed=\"true\" @mimeType=\"\" >/
+    @test_datastream.inspect.should match /#<ActiveFedora::Datastream @pid=\"\" @dsid=\"abcd\" @controlGroup=\"M\" changed=\"true\" @mimeType=\"\" >/
   end
 
   describe '#validate_content_present' do

data/spec/unit/predicates_spec.rb

@@ -1,7 +1,32 @@
 require 'spec_helper'
 
-
 describe ActiveFedora::Predicates do
+  describe "#short_predicate" do
+    it 'should parse strings' do
+      ActiveFedora::Predicates.short_predicate('http://www.openarchives.org/OAI/2.0/itemID').should == :oai_item_id
+    end
+    it 'should parse uris' do
+      ActiveFedora::Predicates.short_predicate(RDF::DC.creator).should == 'dc_terms_creator'
+      ActiveFedora::Predicates.short_predicate(RDF::SKOS.hasTopConcept).should == '2004_02_skos_core_has_top_concept'
+    end
+    before(:all) do
+      @original_mapping = ActiveFedora::Predicates.predicate_config[:predicate_mapping]
+    end
+    after(:all) do
+      ActiveFedora::Predicates.predicate_config[:predicate_mapping] = @original_mapping
+    end
+    it "should find predicates regardless of order loaded or shared namespace prefixes" do
+      ActiveFedora::Predicates.predicate_config[:predicate_mapping] = {
+        "http://example.org/"=>{:ceo => 'Manager'},
+        "http://example.org/zoo/wolves/"=>{:alpha => 'Manager'},
+        "http://example.org/zoo/"=>{:keeper => 'Manager'}
+        }
+      ActiveFedora::Predicates.short_predicate("http://example.org/zoo/Manager").should == :keeper
+      ActiveFedora::Predicates.short_predicate("http://example.org/zoo/wolves/Manager").should == :alpha
+      ActiveFedora::Predicates.short_predicate("http://example.org/Manager").should == :ceo
+    end
+  end
+  
   it 'should provide .default_predicate_namespace' do
     ActiveFedora::Predicates.default_predicate_namespace.should == 'info:fedora/fedora-system:def/relations-external#'
   end

data/spec/unit/relationship_graph_spec.rb

@@ -13,8 +13,8 @@ describe ActiveFedora::RelationshipGraph do
     end
 
     it "should initialize new relation keys" do
-      @graph[:fictional_key].should be_empty
-      @graph[:fictional_key].should respond_to(:<<)
+      @graph[:has_description].should be_empty
+      @graph[:has_description].should respond_to(:<<)
     end
 
   end
@@ -22,11 +22,11 @@ describe ActiveFedora::RelationshipGraph do
   it "should add relationships" do
     @n2 = ActiveFedora::Base.new
     @graph.add(:has_part, @n1)
-    @graph.relationships[:has_part].should == [@n1]
+    @graph[:has_part].should == [@n1]
     @graph.add(:has_part, @n2)
-    @graph.relationships[:has_part].should == [@n1, @n2]
+    @graph[:has_part].should == [@n1, @n2]
     @graph.add(:has_part, @n2)
-    @graph.relationships[:has_part].should == [@n1, @n2]
+    @graph[:has_part].should == [@n1, @n2]
     @graph.dirty.should be_true
   end
 

data/spec/unit/rels_ext_datastream_spec.rb

@@ -1,12 +1,6 @@
 require 'spec_helper'
 
 describe ActiveFedora::RelsExtDatastream do
-  describe "short_predicate" do
-    it 'should parse' do
-      ActiveFedora::RelsExtDatastream.short_predicate('http://www.openarchives.org/OAI/2.0/itemID').should == :oai_item_id
-    end
-  end
-  
   before(:all) do
     @pid = "test:sample_pid"
   
@@ -172,22 +166,4 @@ describe ActiveFedora::RelsExtDatastream do
     end
   end
   
-  describe "#short_predicate" do
-    before(:all) do
-      @original_mapping = ActiveFedora::Predicates.predicate_config[:predicate_mapping]
-    end
-    after(:all) do
-      ActiveFedora::Predicates.predicate_config[:predicate_mapping] = @original_mapping
-    end
-    it "should find predicates regardless of order loaded or shared namespace prefixes" do
-      ActiveFedora::Predicates.predicate_config[:predicate_mapping] = {
-        "http://example.org/"=>{:ceo => 'Manager'},
-        "http://example.org/zoo/wolves/"=>{:alpha => 'Manager'},
-        "http://example.org/zoo/"=>{:keeper => 'Manager'}
-        }
-      ActiveFedora::RelsExtDatastream.short_predicate("http://example.org/zoo/Manager").should == :keeper
-      ActiveFedora::RelsExtDatastream.short_predicate("http://example.org/zoo/wolves/Manager").should == :alpha
-      ActiveFedora::RelsExtDatastream.short_predicate("http://example.org/Manager").should == :ceo
-    end
-  end
 end

data/spec/unit/semantic_node_spec.rb

@@ -90,7 +90,7 @@ describe ActiveFedora::SemanticNode do
       it "should not be written into the graph until it is saved" do
         @n1 = ActiveFedora::Base.new
         @node.add_relationship(:has_part, @n1)
-        @node.relationships.statements.to_a.first.object.to_s.should == 'info:fedora/__DO_NOT_USE__' 
+        @node.relationships.statements.to_a.first.object.to_s.should == 'info:fedora/' 
         @n1.save
         @node.relationships.statements.to_a.first.object.to_s.should == @n1.internal_uri
       end
@@ -117,12 +117,12 @@ describe ActiveFedora::SemanticNode do
       before do
         @node.add_relationship(:is_member_of, 'demo:9')
         @node.add_relationship(:is_member_of, 'demo:7')
-        @node.add_relationship(:is_brother_of, 'demo:9')
+        @node.add_relationship(:has_description, 'demo:9')
       end
       it "should clear the specified relationship" do
         @node.clear_relationship(:is_member_of)
         @node.relationships(:is_member_of).should == []
-        @node.relationships(:is_brother_of).should == ['demo:9']
+        @node.relationships(:has_description).should == ['demo:9']
       end
       
     end

data/spec/unit/solr_config_options_spec.rb

@@ -27,8 +27,9 @@ describe ActiveFedora do
       SOLR_DOCUMENT_ID = "id"
     end
     it "should be used by ActiveFedora::Base.to_solr" do
+      @test_object.stub(pid: 'changeme:123')
       SOLR_DOCUMENT_ID = "MY_SAMPLE_ID"
-      @test_object.to_solr[SOLR_DOCUMENT_ID.to_sym].should_not be_nil
+      @test_object.to_solr[SOLR_DOCUMENT_ID.to_sym].should == 'changeme:123'
       @test_object.to_solr[:id].should be_nil
     end
 

data/spec/unit/unsaved_digital_object_spec.rb

@@ -16,8 +16,8 @@ describe ActiveFedora::UnsavedDigitalObject do
       @obj.ownerId.should == 'D'
     end
 
-    it "should have a default pid" do
-      @obj.pid.should == "__DO_NOT_USE__"
+    it "should not have a default pid" do
+      @obj.pid.should be_nil
     end
     it "should be able to set the pid" do
       @obj.pid = "my:new_object"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active-fedora
 version: !ruby/object:Gem::Version
-  version: 6.6.0.rc5
+  version: 6.6.0
 platform: ruby
 authors:
 - Matt Zumwalt
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-04 00:00:00.000000000 Z
+date: 2013-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rsolr
@@ -550,9 +550,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>'
+  - - '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.0.5