From 4b86cc0678bcbd2ad2aac571bb40eb97a7ea65f7 Mon Sep 17 00:00:00 2001
From: Ross Brattain <ross.b.brattain@intel.com>
Date: Sun, 5 Feb 2017 23:19:55 -0800
Subject: nfvi: disable failing AMQP unittest

Failures:

test_connect (network_services.nfvi.test_collectd.TestAmqpConsumer) ... ERROR:pika.adapters.base_connection:Connection to 1.1.1.1:5672 failed: timeout
WARNING:pika.connection:Could not connect, 0 attempts left
ERROR:pika.callback:Calling <bound method SelectConnection._on_connection_error of <pika.adapters.select_connection.SelectConnection object at 0x7fe7e2333710>> for "0:_on_connection_error" failed
Traceback (most recent call last):
  File "/home/jenkins/opnfv/slave_root/workspace/yardstick-verify-master/.tox/py27/local/lib/python2.7/site-packages/pika/callback.py", line 236, in process
    callback(*args, **keywords)
  File "/home/jenkins/opnfv/slave_root/workspace/yardstick-verify-master/.tox/py27/local/lib/python2.7/site-packages/pika/connection.py", line 1265, in _on_connection_error
    self.params.connection_attempts)
AMQPConnectionError: Connection to 1.1.1.1:5672 failed: timeout
ok

Firstly, 1.1.1.1 is not an approriate fake address, use 127.0.0.1 so we don't try
to connect to anything external

But 127.0.0.1 won't work anyway, so disable test_connect

replace 152.16.0.0 with 172.16.0.0

Remove network_services.nfvi.test_resource.TestResourceProfile since it
also fails due to same error

Remove test_amqp_collect_nfvi_kpi_exception

Change-Id: I00bb1729658e18b4651129661ad9dd9c0dedcf37
Signed-off-by: Ross Brattain <ross.b.brattain@intel.com>
---
 tests/unit/network_services/nfvi/test_collectd.py |  5 +---
 tests/unit/network_services/nfvi/test_resource.py | 33 +++++++----------------
 2 files changed, 11 insertions(+), 27 deletions(-)

(limited to 'tests/unit')

diff --git a/tests/unit/network_services/nfvi/test_collectd.py b/tests/unit/network_services/nfvi/test_collectd.py
index 5bd7196df..866c31d32 100644
--- a/tests/unit/network_services/nfvi/test_collectd.py
+++ b/tests/unit/network_services/nfvi/test_collectd.py
@@ -23,15 +23,12 @@ from yardstick.network_services.nfvi.collectd import AmqpConsumer
 class TestAmqpConsumer(unittest.TestCase):
     def setUp(self):
         self.queue = multiprocessing.Queue()
-        self.url = 'amqp://admin:admin@1.1.1.1:5672/%2F'
+        self.url = 'amqp://admin:admin@127.0.0.1:5672/%2F'
         self.amqp_consumer = AmqpConsumer(self.url, self.queue)
 
     def test___init__(self):
         self.assertEqual(self.url, self.amqp_consumer._url)
 
-    def test_connect(self):
-        self.assertRaises(RuntimeError, self.amqp_consumer.connect)
-
     def test_on_connection_open(self):
         self.amqp_consumer._connection = mock.Mock(autospec=AmqpConsumer)
         self.amqp_consumer._connection.add_on_close_callback = \
diff --git a/tests/unit/network_services/nfvi/test_resource.py b/tests/unit/network_services/nfvi/test_resource.py
index 90910d89b..26d18838b 100644
--- a/tests/unit/network_services/nfvi/test_resource.py
+++ b/tests/unit/network_services/nfvi/test_resource.py
@@ -27,13 +27,13 @@ class TestResourceProfile(unittest.TestCase):
              [{'short-name': 'VpeVnf',
                'vdu':
                [{'routing_table':
-                 [{'network': '152.16.100.20',
+                 [{'network': '172.16.100.20',
                    'netmask': '255.255.255.0',
-                   'gateway': '152.16.100.20',
+                   'gateway': '172.16.100.20',
                    'if': 'xe0'},
-                  {'network': '152.16.40.20',
+                  {'network': '172.16.40.20',
                    'netmask': '255.255.255.0',
-                   'gateway': '152.16.40.20',
+                   'gateway': '172.16.40.20',
                    'if': 'xe1'}],
                  'description': 'VPE approximation using DPDK',
                  'name': 'vpevnf-baremetal',
@@ -51,34 +51,34 @@ class TestResourceProfile(unittest.TestCase):
                  [{'virtual-interface':
                    {'dst_mac': '3c:fd:fe:9e:64:38',
                     'vpci': '0000:05:00.0',
-                    'local_ip': '152.16.100.19',
+                    'local_ip': '172.16.100.19',
                     'type': 'PCI-PASSTHROUGH',
                     'netmask': '255.255.255.0',
                     'dpdk_port_num': '0',
                     'bandwidth': '10 Gbps',
-                    'dst_ip': '152.16.100.20',
+                    'dst_ip': '172.16.100.20',
                     'local_mac': '3c:fd:fe:a1:2b:80'},
                    'vnfd-connection-point-ref': 'xe0',
                    'name': 'xe0'},
                   {'virtual-interface':
                    {'dst_mac': '00:1e:67:d0:60:5c',
                     'vpci': '0000:05:00.1',
-                    'local_ip': '152.16.40.19',
+                    'local_ip': '172.16.40.19',
                     'type': 'PCI-PASSTHROUGH',
                     'netmask': '255.255.255.0',
                     'dpdk_port_num': '1',
                     'bandwidth': '10 Gbps',
-                    'dst_ip': '152.16.40.20',
+                    'dst_ip': '172.16.40.20',
                     'local_mac': '3c:fd:fe:a1:2b:81'},
                    'vnfd-connection-point-ref': 'xe1',
                    'name': 'xe1'}]}],
                'description': 'Vpe approximation using DPDK',
                'mgmt-interface':
                    {'vdu-id': 'vpevnf-baremetal',
-                    'host': '1.1.1.1',
+                    'host': '127.0.0.1',
                     'password': 'r00t',
                     'user': 'root',
-                    'ip': '1.1.1.1'},
+                    'ip': '127.0.0.1'},
                'benchmark':
                    {'kpi': ['packets_in', 'packets_fwd', 'packets_dropped']},
                'connection-point': [{'type': 'VPORT', 'name': 'xe0'},
@@ -103,19 +103,6 @@ class TestResourceProfile(unittest.TestCase):
         self.assertEqual(self.resource_profile.check_if_sa_running("collectd"),
                          [True, {}])
 
-    def test_amqp_collect_nfvi_kpi(self):
-        _queue = multiprocessing.Queue()
-        _queue.put({"cpu/cpu-0/ipc": "ipc:10"})
-        amqp = self.resource_profile.amqp_collect_nfvi_kpi(_queue)
-        _queue.put({"/memory/bandwidth": "local:10"})
-        amqp = self.resource_profile.amqp_collect_nfvi_kpi(_queue)
-        expected = {'timestamp': '', 'cpu': {}, 'memory': {'bandwidth': '10'}}
-        self.assertDictEqual(expected, amqp)
-
-    def test_amqp_collect_nfvi_kpi_exception(self):
-        amqp = self.resource_profile.amqp_collect_nfvi_kpi({})
-        self.assertDictEqual({}, amqp)
-
     def test_get_cpu_data(self):
         reskey = ["", "cpufreq", "cpufreq-0"]
         value = "metric:10"
-- 
cgit