aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authormoneromooo-monero <moneromooo-monero@users.noreply.github.com>2019-03-22 17:59:27 +0000
committermoneromooo-monero <moneromooo-monero@users.noreply.github.com>2019-04-11 11:07:55 +0000
commit21b1ac1dd2d3358686cf3a8bee82fd56a4893581 (patch)
tree650e553c4713eb219f3c6823503559ff76c9f2ad
parentMerge pull request #5382 (diff)
downloadmonero-21b1ac1dd2d3358686cf3a8bee82fd56a4893581.tar.xz
functional_tests: add bans tests
-rwxr-xr-xtests/functional_tests/bans.py117
-rw-r--r--utils/python-rpc/framework/daemon.py21
2 files changed, 138 insertions, 0 deletions
diff --git a/tests/functional_tests/bans.py b/tests/functional_tests/bans.py
new file mode 100755
index 000000000..bb3051a6a
--- /dev/null
+++ b/tests/functional_tests/bans.py
@@ -0,0 +1,117 @@
+#!/usr/bin/env python3
+
+# Copyright (c) 2019 The Monero Project
+#
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without modification, are
+# permitted provided that the following conditions are met:
+#
+# 1. Redistributions of source code must retain the above copyright notice, this list of
+# conditions and the following disclaimer.
+#
+# 2. Redistributions in binary form must reproduce the above copyright notice, this list
+# of conditions and the following disclaimer in the documentation and/or other
+# materials provided with the distribution.
+#
+# 3. Neither the name of the copyright holder nor the names of its contributors may be
+# used to endorse or promote products derived from this software without specific
+# prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND ANY
+# EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+# MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL
+# THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+# PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+# INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+# STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF
+# THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+import time
+
+"""Test peer baning RPC calls
+
+Test the following RPCs:
+ - set_bans
+ - get_bans
+
+"""
+
+from framework.daemon import Daemon
+
+class BanTest():
+ def run_test(self):
+ print 'Testing bans'
+
+ daemon = Daemon()
+ res = daemon.get_bans()
+ assert 'bans' not in res or len(res.bans) == 0
+
+ daemon.set_bans([{'host': '1.2.3.4', 'ban': True, 'seconds': 100}])
+ res = daemon.get_bans()
+ assert len(res.bans) == 1
+ assert res.bans[0].host == '1.2.3.4'
+ assert res.bans[0].seconds >= 98 and res.bans[0].seconds <= 100 # allow for slow RPC
+
+ daemon.set_bans([{'host': '5.6.7.8', 'ban': True, 'seconds': 100}])
+ res = daemon.get_bans()
+ assert len(res.bans) == 2
+ for i in range(2):
+ assert res.bans[i].host == '1.2.3.4' or res.bans[i].host == '5.6.7.8'
+ assert res.bans[i].seconds >= 7 and res.bans[0].seconds <= 100 # allow for slow RPC
+
+ daemon.set_bans([{'host': '1.2.3.4', 'ban': False}])
+ res = daemon.get_bans()
+ assert len(res.bans) == 1
+ assert res.bans[0].host == '5.6.7.8'
+ assert res.bans[0].seconds >= 98 and res.bans[0].seconds <= 100 # allow for slow RPC
+
+ time.sleep(2)
+
+ res = daemon.get_bans()
+ assert len(res.bans) == 1
+ assert res.bans[0].host == '5.6.7.8'
+ assert res.bans[0].seconds >= 96 and res.bans[0].seconds <= 98 # allow for slow RPC
+
+ daemon.set_bans([{'host': '3.4.5.6', 'ban': False}])
+ res = daemon.get_bans()
+ assert len(res.bans) == 1
+ assert res.bans[0].host == '5.6.7.8'
+ assert res.bans[0].seconds >= 96 and res.bans[0].seconds <= 98 # allow for slow RPC
+
+ daemon.set_bans([{'host': '3.4.5.6', 'ban': True, 'seconds': 2}])
+ res = daemon.get_bans()
+ assert len(res.bans) == 2
+ for i in range(2):
+ assert res.bans[i].host == '5.6.7.8' or res.bans[i].host == '3.4.5.6'
+ if res.bans[i].host == '5.6.7.8':
+ assert res.bans[i].seconds >= 96 and res.bans[0].seconds <= 98 # allow for slow RPC
+ else:
+ assert res.bans[i].seconds >= 1 and res.bans[0].seconds <= 2 # allow for slow RPC
+
+ time.sleep(2)
+ res = daemon.get_bans()
+ assert len(res.bans) == 1
+ assert res.bans[0].host == '5.6.7.8'
+ assert res.bans[0].seconds >= 94 and res.bans[0].seconds <= 96 # allow for slow RPC
+
+ daemon.set_bans([{'host': '5.6.7.8', 'ban': True, 'seconds': 20}])
+ res = daemon.get_bans()
+ assert len(res.bans) == 1
+ assert res.bans[0].host == '5.6.7.8'
+ assert res.bans[0].seconds >= 18 and res.bans[0].seconds <= 20 # allow for slow RPC
+
+ daemon.set_bans([{'host': '5.6.7.8', 'ban': True, 'seconds': 200}])
+ res = daemon.get_bans()
+ assert len(res.bans) == 1
+ assert res.bans[0].host == '5.6.7.8'
+ assert res.bans[0].seconds >= 198 and res.bans[0].seconds <= 200 # allow for slow RPC
+
+ daemon.set_bans([{'host': '5.6.7.8', 'ban': False}])
+ res = daemon.get_bans()
+ assert 'bans' not in res or len(res.bans) == 0
+
+
+if __name__ == '__main__':
+ BanTest().run_test()
diff --git a/utils/python-rpc/framework/daemon.py b/utils/python-rpc/framework/daemon.py
index f60fe62db..714018b39 100644
--- a/utils/python-rpc/framework/daemon.py
+++ b/utils/python-rpc/framework/daemon.py
@@ -217,3 +217,24 @@ class Daemon(object):
'id': '0'
}
return self.rpc.send_json_rpc_request(get_version)
+
+ def get_bans(self):
+ get_bans = {
+ 'method': 'get_bans',
+ 'params': {
+ },
+ 'jsonrpc': '2.0',
+ 'id': '0'
+ }
+ return self.rpc.send_json_rpc_request(get_bans)
+
+ def set_bans(self, bans = []):
+ set_bans = {
+ 'method': 'set_bans',
+ 'params': {
+ 'bans': bans
+ },
+ 'jsonrpc': '2.0',
+ 'id': '0'
+ }
+ return self.rpc.send_json_rpc_request(set_bans)