spec/unit/identifier_spec.rb in ezid-client-1.0.1 vs spec/unit/identifier_spec.rb in ezid-client-1.1.0

@@ -76,10 +76,17 @@
         expect(subject).to receive(:save) { double }
         subject.update(metadata)
       end
     end
 
+    describe "#update_metadata" do
+      it "should update the metadata" do
+        subject.update_metadata(:status => "public", _target: "localhost", "dc.creator" => "Me")
+        expect(subject.metadata.to_h).to eq({"_status"=>"public", "_target"=>"localhost", "dc.creator"=>"Me"})
+      end
+    end
+
     describe "#reload" do
       let(:metadata) { "_profile: erc" }
       before { allow(subject).to receive(:id) { "id" } }
       it "should reinitialize the metadata from EZID" do
         expect(subject.client).to receive(:get_identifier_metadata).with("id") { double(id: "id", metadata: metadata) }
@@ -87,33 +94,33 @@
         subject.reload
       end
     end
 
     describe "#reset" do
+      before { subject.metadata = Metadata.new(status: "public") }
       it "should clear the local metadata" do
-        expect(subject.metadata).to receive(:clear)
-        subject.reset
+        expect { subject.reset }.to change { subject.metadata.empty? }.from(false).to(true)
       end
     end
 
     describe "#persisted?" do
-      it "should be false if id is nil" do
-        expect(subject).not_to be_persisted
+      describe "after initialization" do
+        it { is_expected.not_to be_persisted }
       end
-      context "when `created' is nil" do
-        before { allow(subject).to receive(:id) { "ark:/99999/fk4fn19h88" } }
-        it "should be false" do
-          expect(subject).not_to be_persisted
+      describe "when saving an unpersisted object" do
+        before { allow(subject).to receive(:create_or_mint) { nil } }
+        it "should mark it as persisted" do
+          expect { subject.save }.to change(subject, :persisted?).from(false).to(true)
         end
       end
-      context "when id and `created' are present" do
+      describe "when saving a persisted object" do
         before do
-          allow(subject).to receive(:id) { "ark:/99999/fk4fn19h88" }
-          subject.metadata["_created"] = "1416507086"
+          allow(subject).to receive(:persisted?) { true }
+          allow(subject).to receive(:modify) { nil } 
         end
-        it "should be true" do
-          expect(subject).to be_persisted
+        it "should not change the persisted status" do
+          expect { subject.save }.not_to change(subject, :persisted?)
         end
       end
     end
 
     describe "#delete" do
@@ -141,17 +148,18 @@
         end
       end
     end
 
     describe "#save" do
-      before { allow(subject).to receive(:reload) { double } }
       context "when the identifier is persisted" do
+        let(:metadata) { Metadata.new }
         before do
           allow(subject).to receive(:id) { "id" }
           allow(subject).to receive(:persisted?) { true }
+          allow(subject).to receive(:metadata) { metadata }
         end
         it "should modify the identifier" do
-          expect(subject.client).to receive(:modify_identifier).with("id", subject.metadata) { double(id: "id") }
+          expect(subject.client).to receive(:modify_identifier).with("id", metadata) { double(id: "id") }
           subject.save
         end
       end
       context "when the identifier is not persisted" do
         before do