diff options
author | Paul Vinciguerra <pvinci@vinciconsulting.com> | 2018-11-26 09:57:21 -0800 |
---|---|---|
committer | Ole Trøan <otroan@employees.org> | 2018-11-27 09:00:51 +0000 |
commit | 6c746172ef2a8ab7b6a267a889fedd1336f00371 (patch) | |
tree | adc72a32626288b7a8e1aa058496786c623996bd | |
parent | f970d2f8c6a291cdd0bc559afc966b5d32d85f2c (diff) |
VPP-1508 Fix for bug introduced in tests on use of repr.
This fixes the change from https://gerrit.fd.io/r/#/c/16175/
...
======
Totals
======
Ran: 856 tests in 2123.0000 sec.
- Passed: 667
- Skipped: 157
- Expected Fail: 0
- Unexpected Success: 0
- Failed: 32
Sum of execute time for each test: 1641.0832 sec.
==============
Worker Balance
==============
- Worker 0 (416 tests) => 0:35:06.788054
- Worker 1 (440 tests) => 0:31:13.814911
Test id Runtime (s)
-------------------------------------------------------------------------------------- -----------
test.test_vcl.VCLCutThruTestCase.test_ldp_cut_thru_bi_dir_nsock 120.382
test.test_vcl.VCLThruHostStackGroupBTestCase.test_ldp_thru_host_stack_bi_dir_nsock 60.368
test.test_vcl.VCLIpv6ThruHostStackGroupBTestCase.test_ldp_thru_host_stack_bi_dir_nsock 60.356
test.test_igmp.TestIgmp.test_igmp_host 47.411
test.test_reassembly.TestFIFReassembly.test_fif6 38.561
test.test_reassembly.TestFIFReassembly.test_fif4 37.045
test.test_gbp.TestGBP.test_gbp 30.041
test.test_dhcp.TestDHCP.test_dhcp_proxy 29.339
test.test_vcl.VCLCutThruTestCase.test_ldp_cut_thru_iperf3 20.489
test.test_neighbor.ARPTestCase.test_arp 19.004
Change-Id: Ic1565f14962f157d5041230de3aeeab0b85f21e1
Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
-rw-r--r-- | test/remote_test.py | 6 | ||||
-rw-r--r-- | test/test_bfd.py | 4 | ||||
-rw-r--r-- | test/vpp_interface.py | 4 | ||||
-rw-r--r-- | test/vpp_object.py | 4 | ||||
-rw-r--r-- | test/vpp_papi_provider.py | 6 |
5 files changed, 12 insertions, 12 deletions
diff --git a/test/remote_test.py b/test/remote_test.py index 18db39ccb63..40e798444b1 100644 --- a/test/remote_test.py +++ b/test/remote_test.py @@ -6,7 +6,7 @@ import unittest from multiprocessing import Process, Pipe from pickle import dumps -import six +from six import moves from framework import VppTestCase @@ -102,7 +102,7 @@ class RemoteClass(Process): self._pipe = Pipe() # pipe for input/output arguments def __repr__(self): - return six.reprlib(RemoteClassAttr(self, None)) + return moves.reprlib.repr(RemoteClassAttr(self, None)) def __str__(self): return str(RemoteClassAttr(self, None)) @@ -194,7 +194,7 @@ class RemoteClass(Process): def _get_local_repr(self, path): try: obj = self._get_local_object(path) - return six.reprlib(obj) + return moves.reprlib.repr(obj) except AttributeError: return None diff --git a/test/test_bfd.py b/test/test_bfd.py index 9bae4fe5933..635e7e12a76 100644 --- a/test/test_bfd.py +++ b/test/test_bfd.py @@ -11,7 +11,7 @@ from random import randint, shuffle, getrandbits from socket import AF_INET, AF_INET6, inet_ntop from struct import pack, unpack -import six +from six import moves from scapy.layers.inet import UDP, IP from scapy.layers.inet6 import IPv6 from scapy.layers.l2 import Ether @@ -606,7 +606,7 @@ def verify_udp(test, packet): def verify_event(test, event, expected_state): """ Verify correctness of event values. """ e = event - test.logger.debug("BFD: Event: %s" % six.reprlib(e)) + test.logger.debug("BFD: Event: %s" % moves.reprlib.repr(e)) test.assert_equal(e.sw_if_index, test.vpp_session.interface.sw_if_index, "BFD interface index") diff --git a/test/vpp_interface.py b/test/vpp_interface.py index 0f798cb63e6..8639f8745c5 100644 --- a/test/vpp_interface.py +++ b/test/vpp_interface.py @@ -1,7 +1,7 @@ import socket from abc import abstractmethod, ABCMeta -import six +from six import moves from util import Host, mk_ll_addr, mactobinary @@ -240,7 +240,7 @@ class VppInterface(object): raise Exception( "Could not find interface with sw_if_index %d " "in interface dump %s" % - (self.sw_if_index, six.reprlib(r))) + (self.sw_if_index, moves.reprlib.repr(r))) self._local_ip6_ll = mk_ll_addr(self.local_mac) self._local_ip6n_ll = socket.inet_pton(socket.AF_INET6, self.local_ip6_ll) diff --git a/test/vpp_object.py b/test/vpp_object.py index 8fe549e74ff..1b0fadae569 100644 --- a/test/vpp_object.py +++ b/test/vpp_object.py @@ -2,7 +2,7 @@ from abc import ABCMeta, abstractmethod -import six +from six import moves class VppObject(object): @@ -83,6 +83,6 @@ class VppObjectRegistry(object): if failed: logger.error("REG: Couldn't remove configuration for object(s):") for obj in failed: - logger.error(six.reprlib(obj)) + logger.error(moves.reprlib.repr(obj)) raise Exception("Couldn't remove configuration for object(s): %s" % (", ".join(str(x) for x in failed))) diff --git a/test/vpp_papi_provider.py b/test/vpp_papi_provider.py index eecf6e257e8..91e46df1bf3 100644 --- a/test/vpp_papi_provider.py +++ b/test/vpp_papi_provider.py @@ -3,7 +3,7 @@ import os import time from collections import deque -import six +from six import moves from hook import Hook @@ -187,14 +187,14 @@ class VppPapiProvider(object): if hasattr(reply, 'retval') and reply.retval >= 0: msg = "API call passed unexpectedly: expected negative "\ "return value instead of %d in %s" % \ - (reply.retval, six.reprlib(reply)) + (reply.retval, moves.reprlib.repr(reply)) self.test_class.logger.info(msg) raise UnexpectedApiReturnValueError(msg) elif self._expect_api_retval == self._zero: if hasattr(reply, 'retval') and reply.retval != expected_retval: msg = "API call failed, expected %d return value instead "\ "of %d in %s" % (expected_retval, reply.retval, - six.reprlib(reply)) + moves.reprlib.repr(reply)) self.test_class.logger.info(msg) raise UnexpectedApiReturnValueError(msg) else: |