spec/vcloud/terremark/ecloud/models/internet_service_spec.rb in fog-0.5.2 vs spec/vcloud/terremark/ecloud/models/internet_service_spec.rb in fog-0.5.3

@@ -5,12 +5,12 @@
     subject { @vcloud.vdcs[0].public_ips[0].internet_services[0] }
 
     describe :class do
       subject { Fog::Vcloud::Terremark::Ecloud::InternetService }
 
-      it { should have_identity :href }
-      it { should have_only_these_attributes [:href, :name, :id, :protocol, :port, :enabled, :description, :public_ip, :timeout, :redirect_url, :monitor, :backup_service_data] }
+      it { should have_identity(:href) }
+      it { should have_only_these_attributes([:href, :name, :id, :protocol, :port, :enabled, :description, :public_ip, :timeout, :redirect_url, :monitor, :backup_service_data]) }
     end
 
     context "with no uri" do
 
       subject { Fog::Vcloud::Terremark::Ecloud::InternetService.new() }
@@ -27,11 +27,11 @@
         pip
       }
       let(:composed_public_ip_data) { @vcloud.vdcs[0].public_ips[0].internet_services[0].send(:_compose_ip_data) }
       let(:composed_service_data) { @vcloud.vdcs[0].public_ips[0].internet_services[0].send(:_compose_service_data) }
 
-      it { should be_an_instance_of Fog::Vcloud::Terremark::Ecloud::InternetService }
+      it { should be_an_instance_of(Fog::Vcloud::Terremark::Ecloud::InternetService) }
 
       its(:href)               { should == @mock_service.href }
       its(:identity)           { should == @mock_service.href }
       its(:name)               { should == @mock_service.name }
       its(:id)                 { should == @mock_service.object_id.to_s }
@@ -71,9 +71,32 @@
         end
 
         specify do
           expect { subject.backup_service_uri = @mock_backup_service.href }.
             to change { subject.send(:_compose_service_data)[:backup_service_uri] }.from(nil).to(@mock_backup_service.href)
+        end
+      end
+
+      describe "disable monitoring via #monitor=" do
+        specify do
+          expect { subject.monitor = {:type => "Disabled", :is_enabled => "true" }; subject.save }.to change {subject.monitor}.from(nil).to(ecloud_disabled_default_monitor)
+        end
+
+      end
+
+      describe "disable monitoring via #disable_monitor" do
+        specify do
+          expect { subject.disable_monitor }.to change {subject.monitor}.from(nil).to(ecloud_disabled_default_monitor)
+        end
+      end
+
+      context "with a disabled monitor" do
+        before { subject.disable_monitor }
+
+        describe "enable ping monitoring via #enable_ping_monitor" do
+          specify do
+            expect { subject.enable_ping_monitor }.to change {subject.monitor}.from(ecloud_disabled_default_monitor).to(nil)
+          end
         end
       end
     end
   end
 else