spice 0.4.2 → 0.5.0.beta.1

data/spec/spice/cookbook_spec.rb

@@ -3,105 +3,66 @@ require 'spec_helper'
 module Spice
   describe Cookbook do
     describe ".all" do
+      before { stub_cookbook_list }
+      subject { Cookbook.all }
 
-      it "returns a list of cookbooks" do
-        stub_cookbook_list
-        Cookbook.all.length.should == 1
-      end
+      it { should have_body(cookbook_list_response) }
+      it { should respond_with(200) }
     end
-
+    
     describe ".show" do
-      
-      context "valid" do
-        it "returns a valid cookbook" do
-          stub_cookbook_show("monkeypants")
-          cookbook = Cookbook.show(:name => "monkeypants")
-          cookbook["name"].should == "monkeypants"
-          cookbook["admin"].should == true
-        end
+      context "if the cookbook is found" do
+        before { stub_cookbook_show }
+        subject { Cookbook.show(:name => "testcookbook") }
+    
+        it { should have_body(cookbook_show_response) }
+        it { should respond_with(200) } 
       end
       
-      context "errors" do
-        it "return a 404 when a cookbook is not found" do
-          stub_cookbook_show("applesauce", 404)
-          lambda { Cookbook.show(:name => "applesauce") }.
-            should raise_error(RestCookbook::ResourceNotFound)
-        end
+      context "if the cookbook is not found" do
+        before { stub_cookbook_show(404) }
+        subject { Cookbook.show(:name => "testcookbook") }
         
-        it "raises ArgumentError if option :name not present" do
-          stub_cookbook_show("pizza")
-          lambda {Cookbook.show() }.should raise_error ArgumentError
-        end
+        it { should_not have_body(cookbook_show_response) }
+        it { should respond_with(404) }
       end
     end
-
+    
     describe ".create" do
-
-      context "valid" do
-        it "creates a valid non-admin cookbook" do
-          stub_cookbook_create("spork")
-          cookbook = Cookbook.create(:name => "spork", :admin => false)
-          cookbook["private_key"].should == "RSA PRIVATE KEY"
-          cookbook["uri"].should == "http://http://localhost:4000/cookbooks/spork"
-        end
+      context "if the cookbook can be created" do
+        before { stub_cookbook_create }
+        subject { Cookbook.create(:name => "testcookbook") }
         
-        it "creates a valid admin cookbook" do
-          stub_cookbook_create("pants", true)
-          response = Cookbook.create(:name => "pants", :admin => true)
-          response["private_key"].should == "RSA PRIVATE KEY"
-          response["uri"].should == "http://http://localhost:4000/cookbooks/pants"
-
-          stub_cookbook_show("pants")
-          cookbook = Cookbook.show(:name => "pants")
-          cookbook["admin"].should == true
-        end
+        it { should have_body(cookbook_create_response) }
+        it { should respond_with(201) }
       end
       
-      context "errors" do
-        it "does not create a cookbook that already exists" do
-          stub_cookbook_create("pants", false, 409)
-          lambda { Cookbook.create(:name => "pants", :admin => false) }.
-            should raise_error(RestCookbook::Conflict)
-        end
-      end
-    end
-
-    describe ".update" do
-
-      context "valid" do
-        it "makes a cookbook an admin" do
-          stub_cookbook_update("awesome", true, false, 200)
-          Cookbook.update(:name => "awesome", :admin => true, :private_key => false)
-        end
+      context "if the cookbook already exists" do
+        before { stub_cookbook_create(409) }
+        subject { Cookbook.create(:name => "testcookbook") }
         
-        it "regenerates the cookbook private key" do
-          stub_cookbook_update("awesome", false, true, 200)
-          Cookbook.update(:name => "awesome", :admin => false, :private_key => true)
-        end
-      end
+        it { should have_body(cookbook_conflict) }
+        it { should respond_with(409) }
+      end      
     end
-
+    
     describe ".delete" do
-
-      context "valid" do
-        it "deletes a cookbook" do
-          stub_cookbook_delete("spork")
-          Cookbook.delete(:name => "spork")
-        end
+      context "if the cookbook can be deleted" do
+        before { stub_cookbook_delete }
+        subject { Cookbook.delete(:name => "testcookbook") }
+        
+        it { should have_body(cookbook_delete_response) }
+        it { should respond_with(200) }
       end
-      
-      context "errors" do
-        it "raises ArgumentError if option :name not present" do
-          stub_cookbook_delete("pizza")
-          lambda {Cookbook.delete }.should raise_error ArgumentError
-        end
+    
+      context "if the cookbook cannot be deleted" do
+        before { stub_cookbook_delete(404) }
+        subject { Cookbook.delete(:name => "testcookbook") }
         
-        it "does not delete a non-existent cookbook" do
-          stub_cookbook_delete("spork", 404)
-          lambda { Cookbook.delete(:name => "spork") }.
-            should raise_error(RestCookbook::ResourceNotFound)
-        end
+        it { should have_body(cookbook_not_found) }
+        it { should respond_with(404) }
       end
     end
+     
   end
 end

data/spec/spice/data_bag_spec.rb

@@ -9,12 +9,12 @@ module Spice
       it { should have_body(data_bag_list_response) }
       it { should respond_with(200) }
     end
-
+    
     describe ".show" do
       context "if the data bag is found" do
         before { stub_data_bag_show }
         subject { DataBag.show(:name => "users") }
-
+    
         it { should have_body(data_bag_show_response) }
         it { should respond_with(200) } 
       end
@@ -45,7 +45,7 @@ module Spice
         it { should respond_with(409) }
       end      
     end
-
+    
     describe ".delete" do
       context "if the data bag can be deleted" do
         before { stub_data_bag_delete }
@@ -82,87 +82,86 @@ module Spice
       end
     end
     
-    describe ".create_item" do
-      context "if the data bag item can be created" do
-        before { stub_data_bag_item_create }
-        subject do
-          Spice::DataBag.create_item(
-            :name => "users", :id => "adam", :real_name => "Adam Jacob"
-          )
-        end
-        
-        it { should have_body(data_bag_item_create_response) }
-        it { should respond_with(201) }
-      end
-      
-      context "if the data bag item cannot be created" do
-        before { stub_data_bag_item_create(409) }
-        subject do
-          Spice::DataBag.create_item(
-            :name => "users", :id => "adam", :real_name => "Adam Jacob"
-          )
-        end
-
-        it { should have_body(data_bag_item_conflict) }
-        it { should respond_with(409) }
-      end
-      
-      context "if the data bag does not exist" do
-        before { stub_data_bag_item_create(404) }
-        subject do
-          Spice::DataBag.create_item(
-            :name => "users", :id => "adam", :real_name => "Adam Jacob"
-          )
-        end
-
-        it { should have_body(data_bag_not_found) }
-        it { should respond_with(404) }
-      end
-    end
-    
-    describe ".update_item" do
-      context "if the data bag item can be updated" do
-        before { stub_data_bag_item_update }
-        subject do
-          Spice::DataBag.update_item(
-            :name => "users", :id => "adam", :title => "Supreme Awesomer"
-          )
-        end
-        
-        it { should have_body(data_bag_item_update_response) }
-        it { should respond_with(200) }
-      end
-      
-      context "if the data bag item cannot be updated" do
-        before { stub_data_bag_item_update(404) }
-        subject do 
-          Spice::DataBag.update_item(
-            :name => "users", :id => "adam", :title => "Supreme Awesomer"
-          )
-        end
-        it { should have_body(data_bag_item_not_found) }
-        it { should respond_with(404) }
-      end
-    end
-    
-    describe ".delete_item" do
-      context "if the data bag item can be deleted" do
-        before { stub_data_bag_item_delete }
-        subject{ Spice::DataBag.delete_item(:name => "users", :id => "adam") }
-        
-        it { should have_body(data_bag_item_delete_response) }
-        it { should respond_with(200) }
-      end
-      
-      context "if the data bag item cannot be deleted" do
-        before { stub_data_bag_item_delete(404) }
-        subject{ Spice::DataBag.delete_item(:name => "users", :id => "adam") }
-        
-        it { should have_body(data_bag_item_not_found) }
-        it { should respond_with(404) }
-        
-        
-      end
-    end
+     describe ".create_item" do
+       context "if the data bag item can be created" do
+         before { stub_data_bag_item_create }
+         subject do
+           Spice::DataBag.create_item(
+             :name => "users", :id => "adam", :real_name => "Adam Jacob"
+           )
+         end
+         
+         it { should have_body(data_bag_item_create_response) }
+         it { should respond_with(201) }
+       end
+       
+       context "if the data bag item cannot be created" do
+         before { stub_data_bag_item_create(409) }
+         subject do
+           Spice::DataBag.create_item(
+             :name => "users", :id => "adam", :real_name => "Adam Jacob"
+           )
+         end
+     
+         it { should have_body(data_bag_item_conflict) }
+         it { should respond_with(409) }
+       end
+       
+       context "if the data bag does not exist" do
+         before { stub_data_bag_item_create(404) }
+         subject do
+           Spice::DataBag.create_item(
+             :name => "users", :id => "adam", :real_name => "Adam Jacob"
+           )
+         end
+     
+         it { should have_body(data_bag_not_found) }
+         it { should respond_with(404) }
+       end
+     end
+     
+     describe ".update_item" do
+       context "if the data bag item can be updated" do
+         before { stub_data_bag_item_update }
+         subject do
+           Spice::DataBag.update_item(
+             :name => "users", :id => "adam", :title => "Supreme Awesomer"
+           )
+         end
+         
+         it { should have_body(data_bag_item_update_response) }
+         it { should respond_with(200) }
+       end
+       
+       context "if the data bag item cannot be updated" do
+         before { stub_data_bag_item_update(404) }
+         subject do 
+           Spice::DataBag.update_item(
+             :name => "users", :id => "adam", :title => "Supreme Awesomer"
+           )
+         end
+         it { should have_body(data_bag_item_not_found) }
+         it { should respond_with(404) }
+       end
+     end
+     
+     describe ".delete_item" do
+       context "if the data bag item can be deleted" do
+         before { stub_data_bag_item_delete }
+         subject{ Spice::DataBag.delete_item(:name => "users", :id => "adam") }
+         
+         it { should have_body(data_bag_item_delete_response) }
+         it { should respond_with(200) }
+       end
+       
+       context "if the data bag item cannot be deleted" do
+         before { stub_data_bag_item_delete(404) }
+         subject{ Spice::DataBag.delete_item(:name => "users", :id => "adam") }
+         
+         it { should have_body(data_bag_item_not_found) }
+         it { should respond_with(404) }
+       end
+     end
+     
   end
 end

data/spec/spice/node_spec.rb

@@ -2,5 +2,80 @@ require 'spec_helper'
 
 module Spice
   describe Node do
+    describe ".all" do
+      before { stub_node_list }
+      subject { Node.all }
+
+      it { should have_body(node_list_response) }
+      it { should respond_with(200) }
+    end
+    
+    describe ".show" do
+      context "if the node is found" do
+        before { stub_node_show }
+        subject { Node.show(:name => "testnode") }
+    
+        it { should have_body(node_show_response) }
+        it { should respond_with(200) } 
+      end
+      
+      context "if the node is not found" do
+        before { stub_node_show(404) }
+        subject { Node.show(:name => "testnode") }
+        
+        it { should_not have_body(node_show_response) }
+        it { should respond_with(404) }
+      end
+    end
+    
+    describe ".create" do
+      context "if the node can be created" do
+        before { stub_node_create }
+        subject { Node.create(:name => "testnode") }
+        
+        it { should have_body(node_create_response) }
+        it { should respond_with(201) }
+      end
+      
+      context "if the node has a run list" do
+        before { stub_node_create_with_run_list }
+        subject do
+          Node.create(
+            :name => "testnode", 
+            :run_list => ["recipe[unicorn]" ]
+          )
+        end
+        
+        it { should have_body(node_create_response_with_run_list) }
+        it { should respond_with(201) }
+      end
+      
+      context "if the node already exists" do
+        before { stub_node_create(409) }
+        subject { Node.create(:name => "testnode") }
+        
+        it { should have_body(node_conflict) }
+        it { should respond_with(409) }
+      end      
+    end
+    
+    describe ".delete" do
+      context "if the node can be deleted" do
+        before { stub_node_delete }
+        subject { Node.delete(:name => "testnode") }
+        
+        it { should have_body(node_delete_response) }
+        it { should respond_with(200) }
+      end
+    
+      context "if the node cannot be deleted" do
+        before { stub_node_delete(404) }
+        subject { Node.delete(:name => "testnode") }
+        
+        it { should have_body(node_not_found) }
+        it { should respond_with(404) }
+      end
+    end
+     
   end
 end

data/spec/spice/role_spec.rb

@@ -2,6 +2,67 @@ require 'spec_helper'
 
 module Spice
   describe Role do
+    describe ".all" do
+      before { stub_role_list }
+      subject { Role.all }
 
+      it { should have_body(role_list_response) }
+      it { should respond_with(200) }
+    end
+    
+    describe ".show" do
+      context "if the role is found" do
+        before { stub_role_show }
+        subject { Role.show(:name => "testrole") }
+    
+        it { should have_body(role_show_response) }
+        it { should respond_with(200) } 
+      end
+      
+      context "if the role is not found" do
+        before { stub_role_show(404) }
+        subject { Role.show(:name => "testrole") }
+        
+        it { should_not have_body(role_show_response) }
+        it { should respond_with(404) }
+      end
+    end
+    
+    describe ".create" do
+      context "if the role can be created" do
+        before { stub_role_create }
+        subject { Role.create(:name => "testrole") }
+        
+        it { should have_body(role_create_response) }
+        it { should respond_with(201) }
+      end
+      
+      context "if the role already exists" do
+        before { stub_role_create(409) }
+        subject { Role.create(:name => "testrole") }
+        
+        it { should have_body(role_conflict) }
+        it { should respond_with(409) }
+      end      
+    end
+    
+    describe ".delete" do
+      context "if the role can be deleted" do
+        before { stub_role_delete }
+        subject { Role.delete(:name => "testrole") }
+        
+        it { should have_body(role_delete_response) }
+        it { should respond_with(200) }
+      end
+    
+      context "if the role cannot be deleted" do
+        before { stub_role_delete(404) }
+        subject { Role.delete(:name => "testrole") }
+        
+        it { should have_body(role_not_found) }
+        it { should respond_with(404) }
+      end
+    end
+     
   end
 end