Merge pull request #173 from agrare/fix_infra_manager_spec_tests

Fix broken specs from EMS hostname validation and vCloud Refresh (cherry picked from commit e017119)
ManageIQ · Jan 15, 2018 · f6792d7 · f6792d7
1 parent 0f1be92
commit f6792d7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/manageiq-providers-vmware.gemspec b/manageiq-providers-vmware.gemspec
@@ -14,6 +14,7 @@ Gem::Specification.new do |s|
   s.files = Dir["{app,config,lib}/**/*"]
 
   s.add_dependency("fog-vcloud-director", ["~> 0.1.8"])
+  s.add_dependency "fog-core",                "~>1.40"
   s.add_dependency "vmware_web_service",      "~>0.2.4"
   s.add_dependency "rbvmomi",                 "~>1.11.3"
 

diff --git a/spec/models/manageiq/providers/vmware/infra_manager_spec.rb b/spec/models/manageiq/providers/vmware/infra_manager_spec.rb
@@ -50,11 +50,6 @@
       expect(@ems.remote_console_vmrc_support_known?).to be_truthy
     end
 
-    it "false for missing hostname" do
-      @ems.update_attributes(:hostname => nil, :api_version => "5.0", :uid_ems => "2E1C1E82-BD83-4E54-9271-630C6DFAD4D1")
-      expect(@ems.remote_console_vmrc_support_known?).not_to be_truthy
-    end
-
     it "false for blank hostname" do
       @ems.update_attributes(:hostname => "", :api_version => "5.0", :uid_ems => "2E1C1E82-BD83-4E54-9271-630C6DFAD4D1")
       expect(@ems.remote_console_vmrc_support_known?).not_to be_truthy
@@ -135,7 +130,7 @@
     end
 
     it "will restart EventCatcher when hostname changes" do
-      @ems.update_attributes(:hostname => "something else")
+      @ems.update_attributes(:hostname => "something-else")
       assert_event_catcher_restart_queued
     end