@@ -47,11 +47,19 @@
 
     it "should save the edited node data" do
       pansy = Chef::Node.new
+      @node.name("new_node_name")
       @knife.should_receive(:edit_data).with(@node).and_return(pansy)
       pansy.should_receive(:save)
       @knife.run
     end
 
+    it "should not save the unedited node data" do
+      pansy = Chef::Node.new
+      @knife.should_receive(:edit_data).with(@node).and_return(pansy)
+      pansy.should_not_receive(:save)
+      @knife.run
+    end
+
     it "should not print the node" do
       @knife.should_not_receive(:output).with("poop")
       @knife.run
 
@@ -39,25 +39,36 @@
       @knife.run
     end
 
-    it "should edit the node data" do
+    it "should edit the role data" do
       @knife.should_receive(:edit_data).with(@role)
       @knife.run
     end
 
-    it "should save the edited node data" do
+    it "should save the edited role data" do
       pansy = Chef::Role.new
+
+      @role.name("new_role_name")
       @knife.should_receive(:edit_data).with(@role).and_return(pansy)
       pansy.should_receive(:save)
       @knife.run
     end
 
-    it "should not print the node" do
+    it "should not save the unedited role data" do
+      pansy = Chef::Role.new
+
+      @knife.should_receive(:edit_data).with(@role).and_return(pansy)
+      pansy.should_not_receive(:save)
+      @knife.run
+
+    end
+
+    it "should not print the role" do
       @knife.should_not_receive(:output)
       @knife.run
     end
 
     describe "with -p or --print-after" do
-      it "should pretty print the node, formatted for display" do
+      it "should pretty print the role, formatted for display" do
         @knife.config[:print_after] = true
         @knife.should_receive(:output).with(@role)
         @knife.run