active-triples 0.11.0 → 1.0.0.rc1

data/spec/active_triples/node_config_spec.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+describe ActiveTriples::NodeConfig do
+  subject(:config) { described_class.new(term, predicate) }
+  let(:predicate)  { RDF::URI(nil) }
+  let(:term)       { :moomin }
+
+  it { is_expected.to have_attributes(predicate: predicate, term: term) }
+
+  describe '#[]' do
+    subject(:config) { described_class.new(term, predicate, some_opt: :moomin) }
+
+    it 'accesses arbitrary opts' do
+      expect(config[:some_opt]).to eq :moomin
+    end
+  end
+
+  describe '#cast' do
+    it 'defaults to true' do
+      expect { config.cast = false }
+        .to change { config.cast }
+        .from(true)
+        .to(false)
+    end
+
+    it 'is set with the initializer' do
+      expect(described_class.new(term, predicate, cast: false).cast).to be false
+    end
+  end
+
+  describe '#with_index' do
+    it 'yields an index configuration object' do
+      expect { |b| config.with_index(&b) }
+        .to yield_with_args(an_instance_of(described_class::IndexObject))
+    end
+
+    it 'accepts behavior settings' do
+      config.with_index do |index|
+        index.as :moomin, :snork
+      end
+
+      expect(config.behaviors).to contain_exactly :moomin, :snork
+    end
+
+    it 'accepts type settings' do
+      config.with_index do |index|
+        index.type :moomin
+      end
+
+      expect(config.type).to eq :moomin
+    end
+  end
+end

data/spec/active_triples/persistence_strategies/parent_strategy_spec.rb

@@ -4,7 +4,7 @@ require 'spec_helper'
 describe ActiveTriples::ParentStrategy do
   subject { described_class.new(rdf_source) }
   let(:rdf_source) { BasicPersistable.new }
-  
+
   shared_context 'with a parent' do
     subject      { rdf_source.persistence_strategy }
     let(:parent) { BasicPersistable.new }
@@ -50,7 +50,7 @@ describe ActiveTriples::ParentStrategy do
       rdf_source.insert(*statements)
       subject.persist!
     end
-    
+
     let(:statements) do
       [RDF::Statement(subject.source.rdf_subject, RDF::Vocab::DC.title, 'moomin'),
        RDF::Statement(subject.parent, RDF::Vocab::DC.relation, subject.source.rdf_subject)]
@@ -58,7 +58,7 @@ describe ActiveTriples::ParentStrategy do
 
     it 'removes graph from the parent' do
       subject.destroy
-      
+
       statements.each do |statement|
         expect(subject.parent.statements).not_to have_statement statement
       end
@@ -83,18 +83,18 @@ describe ActiveTriples::ParentStrategy do
 
     context 'with parent' do
       include_context 'with a parent'
-      
+
       it 'gives the parent' do
         expect(subject.ancestors).to contain_exactly(parent)
       end
 
       context 'and nested parents' do
         let(:parents) do
-          [double('second', persistence_strategy: double('strategy2')), 
+          [double('second', persistence_strategy: double('strategy2')),
            double('third', persistence_strategy: double('strategy3'))]
         end
         let(:last) { double('last', persistence_strategy: double('last_strategy')) }
-        
+
         it 'gives all ancestors' do
           allow(parent.persistence_strategy)
             .to receive(:parent).and_return(parents.first)
@@ -102,7 +102,7 @@ describe ActiveTriples::ParentStrategy do
             .to receive(:parent).and_return(parents[1])
           allow(parents[1].persistence_strategy)
             .to receive(:parent).and_return(last)
-          
+
           expect(subject.ancestors)
             .to contain_exactly(*(parents << parent << last))
         end
@@ -148,7 +148,19 @@ describe ActiveTriples::ParentStrategy do
   describe '#parent' do
     it { is_expected.to have_attributes(:parent => nil) }
 
-    it 'requires its parent to be RDF::Mutable' do
+    context 'with a parent' do
+      include_context 'with a parent'
+      it { is_expected.to have_attributes(:parent => parent) }
+    end
+  end
+
+  describe '#parent=' do
+    it 'requires a non-nil value' do
+      expect { subject.parent = nil }
+        .to raise_error described_class::NilParentError
+    end
+
+    it 'requires the value to be RDF::Mutable' do
       expect { subject.parent = Object.new }
         .to raise_error described_class::UnmutableParentError
     end
@@ -159,11 +171,6 @@ describe ActiveTriples::ParentStrategy do
       expect { subject.parent = immutable }
         .to raise_error described_class::UnmutableParentError
     end
-
-    context 'with a parent' do
-      include_context 'with a parent'
-      it { is_expected.to have_attributes(:parent => parent) }
-    end
   end
 
   describe '#persist!' do
@@ -205,7 +212,7 @@ describe ActiveTriples::ParentStrategy do
           parent.persistence_strategy.parent = last
           rdf_source.reload
         end
-        
+
         it 'writes to #parent graph when parent changes while child is live' do
           parent.insert(parent_st)
           parent.persist!
@@ -248,16 +255,16 @@ describe ActiveTriples::ParentStrategy::Ancestors do
     context 'with parents' do
       let(:parent) { BasicPersistable.new }
       let(:last)   { BasicPersistable.new }
-      
+
       before do
         parent.set_persistence_strategy(ActiveTriples::ParentStrategy)
         parent.persistence_strategy.parent = last
         rdf_source.set_persistence_strategy(ActiveTriples::ParentStrategy)
         rdf_source.persistence_strategy.parent = parent
       end
-      
+
       it { expect(subject.each).to be_a Enumerator }
-      
+
       it 'enumerates ancestors' do
         expect(subject.each).to contain_exactly(parent, last)
       end

data/spec/active_triples/property_spec.rb

@@ -17,6 +17,16 @@ describe ActiveTriples::Property do
     expect(subject.class_name).to eq "Test"
   end
 
+  it 'should hold a block' do
+    fake = Object.new
+    property = described_class.new(options) do
+      fake.configure
+    end
+
+    expect(fake).to receive(:configure)
+    property.config.call
+  end
+
   describe "#to_h" do
     it "should not return the property's name" do
       expect(subject.to_h).to eq (

data/spec/active_triples/rdf_source_spec.rb

@@ -18,8 +18,9 @@ describe ActiveTriples::RDFSource do
   end
 
   before { @enumerable = subject }
+
   let(:source_class) { Class.new { include ActiveTriples::RDFSource } }
-  let(:uri) { RDF::URI('http://example.org/moomin') }
+  let(:uri)          { RDF::URI('http://example.org/moomin') }
 
   subject { source_class.new }
 
@@ -107,6 +108,45 @@ describe ActiveTriples::RDFSource do
     end
   end
 
+  describe 'observers' do
+    let(:observer) { double('observer') }
+
+    before { subject.add_observer(observer) }
+
+    include_context 'with properties'
+
+    it 'notifies an observer of changes' do
+      expect(observer)
+        .to receive(:notify)
+        .with(:creator, a_collection_containing_exactly('moomin'))
+
+      subject.creator = 'moomin'
+    end
+
+    it 'notifies muliple observers of changes' do
+      other_observer = double('second observer')
+      values         = ['moomin', 'snork']
+
+      expect(observer)
+        .to receive(:notify)
+        .with(:creator, a_collection_containing_exactly(*values))
+      expect(other_observer)
+        .to receive(:notify)
+        .with(:creator, a_collection_containing_exactly(*values))
+
+      subject.add_observer(other_observer)
+      subject.creator = values
+    end
+
+    it 'does not notify removed observers' do
+      expect(observer).not_to receive(:notify)
+
+      subject.delete_observer(observer)
+
+      subject.creator = 'moomin'
+    end
+  end
+
   describe '#==' do
     shared_examples 'Term equality' do
       it 'equals itself' do
@@ -172,9 +212,9 @@ describe ActiveTriples::RDFSource do
 
     context 'with statements for other subjects' do
       before do
-        subject << 
+        subject <<
           RDF::Statement(RDF::URI('http://example.org/OTHER_SUBJECT'),
-                         RDF::URI('http://example.org/ontology/OTHER_PRED'), 
+                         RDF::URI('http://example.org/ontology/OTHER_PRED'),
                          'OTHER_OBJECT')
       end
 
@@ -191,6 +231,16 @@ describe ActiveTriples::RDFSource do
     end
   end
 
+  describe '#default_labels' do
+    it 'prefers skos:prefLabel' do
+      expect(subject.default_labels.first).to eq RDF::Vocab::SKOS.prefLabel
+    end
+
+    it 'values are all valid predicates' do
+      subject.default_labels.each { |term| expect(term).to be_uri }
+    end
+  end
+
   describe '#fetch' do
     it 'raises an error when it is a node' do
       expect { subject.fetch }
@@ -271,6 +321,18 @@ describe ActiveTriples::RDFSource do
     end
   end
 
+  describe '#inspect' do
+    it 'includes bnode id' do
+      expect(subject.inspect).to include subject.to_base
+    end
+
+    it 'includes uri' do
+      subject.set_subject!('http://example.org/moomin')
+
+      expect(subject.inspect).to include subject.to_base
+    end
+  end
+
   describe '#rdf_subject' do
     its(:rdf_subject) { is_expected.to be_a_node }
 
@@ -373,6 +435,7 @@ describe ActiveTriples::RDFSource do
 
     shared_examples 'setting values' do
       include_context 'with properties'
+
       after do
         Object.send(:remove_const, 'SourceWithCreator') if
           defined? SourceWithCreator
@@ -473,6 +536,39 @@ describe ActiveTriples::RDFSource do
       end
     end
 
+    describe 'on child nodes' do
+      let(:parent)  { source_class.new }
+      let(:subject) { source_class.new(uri, parent) }
+
+      include_examples 'setting values' do
+        let(:value) do
+          ['moomin',
+           Date.today,
+           RDF::Node.new,
+           source_class.new,
+           source_class.new(uri / 'new'),
+           subject]
+        end
+      end
+
+      it 'does not change parent' do
+        property = RDF::Vocab::DC.title
+
+        expect { subject.set_value(property, 'Comet in Moominland') }
+          .not_to change { parent.to_a }
+      end
+
+      it 'persists to parent' do
+        property = RDF::Vocab::DC.title
+
+        subject.set_value(property, 'Comet in Moominland')
+
+        expect { subject.persist! }
+          .to change { parent.to_a }
+          .to include RDF::Statement(subject, property, 'Comet in Moominland')
+      end
+    end
+
     context 'with reciprocal relations' do
       let(:document) { source_class.new }
       let(:person) { source_class.new }
@@ -550,15 +646,15 @@ describe ActiveTriples::RDFSource do
         expect { subject.set_value(predicate, child_other) }
           .not_to change { child_other.persistence_strategy.parent }
       end
-      
+
       context 'when setting to a relation' do
         it 'adds child node data to graph' do
           other << RDF::Statement(other, RDF::URI('p'), 'o')
-          
-          relation_source = source_class.new 
+
+          relation_source = source_class.new
           relation_source.set_value(predicate, other)
           relation = relation_source.get_values(predicate)
-          
+
           expect { subject.set_value(predicate, relation) }
             .to change { subject.statements.to_a }
             .to include(*other.statements.to_a)
@@ -673,119 +769,4 @@ describe ActiveTriples::RDFSource do
       expect { dummy_source.new.test_title }.not_to raise_error
     end
   end
-
-  describe 'sources with properties with class_name defined' do
-    before(:context) do
-      class DummyChapter < ActiveTriples::Resource
-        ontology = RDF::URI('http://www.example.com/ontology')
-        type     = RDF::URI('http://www.example.com/type')
-
-        configure repository: :default, type: type / 'Chapter'
-
-        property :title,    predicate: ontology / 'title'
-        property :subtitle, predicate: ontology / 'subtitle'
-      end
-
-      class DummyBook < ActiveTriples::Resource
-        ontology = RDF::URI('http://www.example.com/ontology')
-        type     = RDF::URI('http://www.example.com/type')
-
-        configure repository: :default, type: type / 'Book'
-
-        property :title,       predicate: ontology / 'title'
-        property :has_chapter, predicate: ontology / 'hasChapter',
-                               class_name: DummyChapter
-      end
-    end
-
-    context 'when loading models from graph' do
-      before(:context) do
-        r = RDF::Repository.new
-        ActiveTriples::Repositories.repositories[:default] = r
-
-        @book_url = 'http://www.example.com/BOOK_URI'
-        @book_title = 'Example Book.'
-        @chapter_title = 'Chapter 1'
-        ttl = "<#{@book_url}> a <http://www.example.com/type/Book>;
-                 <http://www.example.com/ontology/hasChapter> [
-                   a <http://www.example.com/type/Chapter>;
-                   <http://www.example.com/ontology/title> \"#{@chapter_title}\"
-                 ];
-               <http://www.example.com/ontology/title> \"#{@book_title}\" ."
-        book_graph = ::RDF::Graph.new.from_ttl ttl
-        r = ActiveTriples::Repositories.repositories[:default]
-        r << book_graph
-
-        @book = DummyBook.new(RDF::URI.new(@book_url))
-      end
-
-      it 'populates DummyBook properly' do
-        expect(@book.rdf_subject.to_s).to eq @book_url
-        expect(@book).to be_a DummyBook
-        expect(@book.type)
-          .to include(RDF::URI.new('http://www.example.com/type/Book'))
-        expect(@book.title.first).to eq @book_title
-      end
-
-      it 'populates DummyChapter properly' do
-        chapter = @book.has_chapter.first
-        expect(chapter).to be_a DummyChapter
-        expect(chapter.title.first).to eq @chapter_title
-        expect(chapter.type)
-          .to include(RDF::URI.new('http://www.example.com/type/Chapter'))
-      end
-    end
-
-    context 'when loading models through properties' do
-      before(:context) do
-        r = RDF::Repository.new
-        ActiveTriples::Repositories.repositories[:default] = r
-
-        bk1 = DummyBook.new('http://www.example.com/book1')
-        bk1.title = 'Learning about Explicit Links in ActiveTriples'
-
-        ch1 = DummyChapter.new('http://www.example.com/book1/chapter1')
-        ch1.title = 'Defining a source with an Explicit Link'
-        bk1.has_chapter = ch1
-        ch1.persist!
-        bk1.persist!
-
-        @bk1 = DummyBook.new('http://www.example.com/book1')
-        @ch1 = DummyChapter.new('http://www.example.com/book1/chapter1')
-      end
-
-      it 'populates DummyBook (resumed resource) properly' do
-        expect(@bk1.type.first)
-          .to eq RDF::URI('http://www.example.com/type/Book')
-        expect(@bk1.title.first)
-          .to eq 'Learning about Explicit Links in ActiveTriples'
-      end
-
-      it 'populates DummyChapter (property resource) properly' do
-        ch1 = @bk1.has_chapter.first
-        expect(ch1.type.first)
-          .to eq RDF::URI('http://www.example.com/type/Chapter')
-        expect(ch1.title.first)
-          .to eq 'Defining a source with an Explicit Link'
-      end
-
-      it 'populates DummyChapter (directly from repository) properly' do
-        expect(@ch1.type.first)
-          .to eq RDF::URI('http://www.example.com/type/Chapter')
-        expect(@ch1.title.first)
-          .to eq 'Defining a source with an Explicit Link'
-      end
-
-      it 'does not reload from repository twice' do
-        ch1 = @bk1.has_chapter.first
-        expect(ch1.title.first).to eq 'Defining a source with an Explicit Link'
-
-        @ch1.subtitle = 'Changed after original load'
-        @ch1.persist!
-        expect(@ch1.subtitle).to eq ['Changed after original load']
-
-        expect(ch1.subtitle).to eq []
-      end
-    end
-  end
 end