Merge remote-tracking branch 'origin/develop' into release/1.99.0

Author: gkellogg

lib/rdf/spec/enumerable.rb

@@ -40,7 +40,7 @@
 
     it "returns false if any statement is invalid" do
       if subject.respond_to?(:<<) && (subject.writable? rescue true)
-        s = RDF::Statement.from([nil, nil, nil])
+        s = RDF::Statement(RDF::URI("http://rubygems.org/gems/rdf"), RDF::Literal("literal"), RDF::URI("http://ar.to/#self"))
         if subject.supports?(:validity)
           expect(s).not_to  be_valid
           subject << s

lib/rdf/spec/inspects.rb

@@ -22,7 +22,7 @@ def inspect
 
 class RDF::Graph
   def inspect
-    "\n" + dump(:n3) + "\n"
+    "\n" + dump(RDF.const_defined?(:Turtle) ? :ttl : :ntriples, standard_prefixes: true) + "\n"
   end
 end
 

lib/rdf/spec/writable.rb

@@ -66,22 +66,22 @@
   end
 
   context "when inserting statements" do
-    it "is_expected.to support #insert" do
+    it "should support #insert" do
       is_expected.to respond_to(:insert) if subject.writable?
     end
 
-    it "is_expected.to not raise errors" do
+    it "should not raise errors" do
       expect { subject.insert(statement) }.not_to raise_error if subject.writable?
     end
 
-    it "is_expected.to support inserting one statement at a time" do
+    it "should support inserting one statement at a time" do
       if subject.writable?
         subject.insert(statement)
         is_expected.to have_statement(statement)
       end
     end
 
-    it "is_expected.to support inserting multiple statements at a time" do
+    it "should support inserting multiple statements at a time" do
       if subject.writable?
         subject.insert(*statements)
         statements.each do |statement|
@@ -90,22 +90,31 @@
       end
     end
 
-    it "is_expected.to insert statements successfully" do
+    it "should insert statements successfully" do
       if subject.writable?
         subject.insert(*statements)
         expect(subject.count).to eq count
       end
     end
 
-    it "is_expected.to not insert a statement twice" do
+    it "should not insert a statement twice" do
       if subject.writable?
         subject.insert(statement)
         subject.insert(statement)
         expect(subject.count).to eq 1
       end
     end
 
-    it "is_expected.to treat statements with a different context as distinct", unless: RDF::VERSION.to_s >= "1.99" do
+    it "should not insert an incomplete statement" do
+      if subject.writable?
+        expect {subject.insert(RDF::Statement.from(statement.to_hash.merge(subject: nil)))}.to raise_error(ArgumentError)
+        expect {subject.insert(RDF::Statement.from(statement.to_hash.merge(predicate: nil)))}.to raise_error(ArgumentError)
+        expect {subject.insert(RDF::Statement.from(statement.to_hash.merge(object: nil)))}.to raise_error(ArgumentError)
+        expect(subject.count).to eql 0
+      end
+    end
+
+    it "should treat statements with a different context as distinct", unless: RDF::VERSION.to_s >= "1.99" do
       if subject.writable?
         s1 = statement.dup
         s1.context = nil
@@ -121,7 +130,7 @@
       end
     end
 
-    it "is_expected.to treat statements with a different graph_name as distinct", if: RDF::VERSION.to_s >= "1.99" do
+    it "should treat statements with a different graph_name as distinct", if: RDF::VERSION.to_s >= "1.99" do
       if subject.writable?
         s1 = statement.dup
         s1.graph_name = nil