diff options
author | IWASE Yusuke <iwase.yusuke0@gmail.com> | 2016-11-09 14:17:16 +0900 |
---|---|---|
committer | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2016-11-14 18:14:39 +0900 |
commit | a45c180447273ce3e7a0e7bcb16157723621bc63 (patch) | |
tree | 390ff8203dd1528f3f5462451b9e17119cdff869 | |
parent | 9f0461245972f15138bbe074fa836a8eb6662f37 (diff) |
test_rpc: Use numbers.Integral instead of long type
Currently, test_rpc.py uses long type for PyPy interpreter compatibility,
but log type is obsoleted in Python 3.
This patch fixes test_rpc.py to use numbers.Integral and makes its
implemetation more clean.
Signed-off-by: IWASE Yusuke <iwase.yusuke0@gmail.com>
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
-rw-r--r-- | ryu/tests/unit/lib/test_rpc.py | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/ryu/tests/unit/lib/test_rpc.py b/ryu/tests/unit/lib/test_rpc.py index cedab558..b0adf208 100644 --- a/ryu/tests/unit/lib/test_rpc.py +++ b/ryu/tests/unit/lib/test_rpc.py @@ -119,13 +119,7 @@ class Test_rpc(unittest.TestCase): assert isinstance(obj, int) result = c.call(b'resp', [obj]) assert result == obj - import sys - # note: on PyPy, result will be a long type value. - sv = getattr(sys, 'subversion', None) - if sv is not None and sv[0] == 'PyPy': - assert isinstance(result, long) - else: - assert isinstance(result, type(obj)) + assert isinstance(result, numbers.Integral) def test_0_call_int3(self): c = rpc.Client(self._client_sock) @@ -133,16 +127,15 @@ class Test_rpc(unittest.TestCase): assert isinstance(obj, int) result = c.call(b'resp', [obj]) assert result == obj - assert isinstance(result, type(obj)) + assert isinstance(result, numbers.Integral) def test_0_call_long(self): c = rpc.Client(self._client_sock) obj = 0xffffffffffffffff # max value for msgpack - _long = int if six.PY3 else long - assert isinstance(obj, _long) + assert isinstance(obj, numbers.Integral) result = c.call(b'resp', [obj]) assert result == obj - assert isinstance(result, type(obj)) + assert isinstance(result, numbers.Integral) def test_0_call_long2(self): c = rpc.Client(self._client_sock) |