commit feaaa05aa3e18970bb1eebde4a80ffca9ca6c9a2
author kasar <pkasar@vmware.com> Thu Jun 08 03:46:18 2017 -0700
committer tierno <alfonso.tiernosepulveda@telefonica.com> Fri Jun 23 11:13:10 2017 +0200
tree 47e656b1217c0ae424d116032b8c5829b21d2440
parent b7490b5bd885a1b2f5d872d7e2c4cd0c126e94e8

Added validation in new_flavor method and unit tests for new_flavor,delete_flavor,new_image

Change-Id: I5747f07f287ad2b4370b3cd586cf0adcf5b9888a
Signed-off-by: kasar <pkasar@vmware.com>

osm_ro/vimconn_vmware.py

diff --git a/osm_ro/vimconn_vmware.py b/osm_ro/vimconn_vmware.py
index 43ffba8..f9ff2fe 100644
--- a/osm_ro/vimconn_vmware.py
+++ b/osm_ro/vimconn_vmware.py
@@ -754,6 +754,13 @@
         cpu = flavor_data.get(FLAVOR_VCPUS_KEY, 1)
         disk = flavor_data.get(FLAVOR_DISK_KEY, 1)
 
+        if not isinstance(ram, int):
+            raise vimconn.vimconnException("Non-integer value for ram")
+        elif not isinstance(cpu, int):
+            raise vimconn.vimconnException("Non-integer value for cpu")
+        elif not isinstance(disk, int):
+            raise vimconn.vimconnException("Non-integer value for disk")
+
         extended_flv = flavor_data.get("extended")
         if extended_flv:
             numas=extended_flv.get("numas")