Merge pull request #141 from douglasgabriel/blank_disk

Leaving the disk capacity blank if the LXCA doesn't provide this information
ManageIQ · Mar 21, 2018 · df476a0 · df476a0
2 parents d5f3d3e + 55c4830
commit df476a0
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...geiq/providers/lenovo/physical_infra_manager/parsers/components/physical_server_parser.rb b/...geiq/providers/lenovo/physical_infra_manager/parsers/components/physical_server_parser.rb
@@ -57,7 +57,7 @@ def get_disk_capacity(node)
               total_disk_cap += disk['capacity'] unless disk['capacity'].nil?
             end
           end
-          total_disk_cap
+          total_disk_cap.positive? ? total_disk_cap : nil
         end
 
         def get_memory_info(node)

diff --git a/spec/models/manageiq/providers/lenovo/physical_infra_manager/refresh_parser_spec.rb b/spec/models/manageiq/providers/lenovo/physical_infra_manager/refresh_parser_spec.rb
@@ -86,7 +86,7 @@
       computer_system = physical_server[:computer_system]
       hardware = computer_system[:hardware]
 
-      expect(hardware[:disk_capacity]).to eq(0)
+      expect(hardware[:disk_capacity]).to be_nil
     end
   end