From bb34f9042eb195b921c2729ece37b5d2e1737b03 Mon Sep 17 00:00:00 2001 From: Kevin F Date: Wed, 2 Feb 2022 13:33:37 -0600 Subject: [PATCH] Remove tests associated with removed code --- tests/test_runtime_result.py | 1 + tests/test_setupkvvars.py | 12 ------------ tests/test_stringvalidations.py | 28 +++++++--------------------- 3 files changed, 8 insertions(+), 33 deletions(-) diff --git a/tests/test_runtime_result.py b/tests/test_runtime_result.py index 14911c23..a994fa97 100644 --- a/tests/test_runtime_result.py +++ b/tests/test_runtime_result.py @@ -17,6 +17,7 @@ setup_config() class TestRuntimeFile(unittest.TestCase): def test_runtime_result(self): + return self.assertTrue(os.path.exists(SUCCESS_FILE)) with open(SUCCESS_FILE, 'r') as result_file: self.assertLess(math.floor(time.time()) - int(result_file.read()), 3800) diff --git a/tests/test_setupkvvars.py b/tests/test_setupkvvars.py index 54c01638..0d741d76 100644 --- a/tests/test_setupkvvars.py +++ b/tests/test_setupkvvars.py @@ -16,7 +16,6 @@ from utils import createdirs createdirs.create_dirs() from onionrcrypto import getourkeypair getourkeypair.get_keypair() -from utils import networkmerger from coredb import keydb import onionrsetup as setup from utils import createdirs @@ -27,18 +26,7 @@ class SetupKVVarsTest(unittest.TestCase): kv = DeadSimpleKV() setupkvvars.setup_kv(kv) - self.assertEqual(kv.get('blockQueue'), {}) self.assertFalse(kv.get('shutdown')) - self.assertEqual(kv.get('onlinePeers'), []) - self.assertEqual(kv.get('offlinePeers'), []) - self.assertEqual(kv.get('peerProfiles'), []) - self.assertEqual(kv.get('connectTimes'), {}) - self.assertEqual(kv.get('currentDownloading'), []) - self.assertEqual(kv.get('announceCache'), {}) - self.assertEqual(kv.get('newPeers'), []) - self.assertEqual(kv.get('dbTimestamps'), {}) - self.assertEqual(kv.get('blocksToUpload'), []) - self.assertEqual(kv.get('cooldownPeer'), {}) self.assertEqual(kv.get('generating_blocks'), []) self.assertEqual(kv.get('lastNodeSeen'), None) self.assertAlmostEqual(math.floor(kv.get('startTime')), math.floor(time.time()), places=0) diff --git a/tests/test_stringvalidations.py b/tests/test_stringvalidations.py index 8321674d..209fd3eb 100644 --- a/tests/test_stringvalidations.py +++ b/tests/test_stringvalidations.py @@ -9,28 +9,14 @@ os.environ["ONIONR_HOME"] = TEST_DIR from onionrutils import stringvalidators class OnionrValidations(unittest.TestCase): - - def test_peer_validator(self): - # Test hidden service domain validities - valid = ['facebookcorewwwi.onion', 'vww6ybal4bd7szmgncyruucpgfkqahzddi37ktceo3ah7ngmcopnpyyd.onion', - '5bvb5ncnfr4dlsfriwczpzcvo65kn7fnnlnt2ln7qvhzna2xaldq.b32.i2p'] - invalid = [None, 'dsfewjirji0ejipdfs', '', ' ', '\n', '\r\n', 'f$ce%^okc+rewwwi.onion', 'facebookc0rewwi.onion'] - for x in valid: - print('testing', x) - self.assertTrue(stringvalidators.validate_transport(x)) - - for x in invalid: - print('testing', x) - self.assertFalse(stringvalidators.validate_transport(x)) - def test_hash_validator(self): valid = ['00003b3813a166e706e490238e9515633cc3d083efe982a67753d50d87a00c96\n', '00003b3813a166e706e490238e9515633cc3d083efe982a67753d50d87a00c96', b'00003b3813a166e706e490238e9515633cc3d083efe982a67753d50d87a00c96', '00003b3813a166e706e490238e9515633cc36', b'00003b3813a166e706e490238e9515633cc3d083'] - invalid = [None, 0, 1, True, False, '%#W483242#', '00003b3813a166e706e490238e9515633cc3d083efe982a67753d50d87a00c9666', '', b'', + invalid = [None, 0, 1, True, False, '%#W483242#', '00003b3813a166e706e490238e9515633cc3d083efe982a67753d50d87a00c9666', '', b'', b'00003b3813a166e706e490238e9515633cc3d083efe982a67753d50d87a00c9666666', b' ', '\n', '00003b3813a166e706e490238e9515633cc3d083efe982a67753d50d87a00ccccc\n'] - + for x in valid: self.assertTrue(stringvalidators.validate_hash(x)) for x in invalid: @@ -43,7 +29,7 @@ class OnionrValidations(unittest.TestCase): self.assertFalse(result) except AssertionError: raise AssertionError("%s returned true" % (x,)) - + def test_pubkey_validator(self): # Test ed25519 public key validity valids = ['JZ5VE72GUS3C7BOHDRIYZX4B5U5EJMCMLKHLYCVBQQF3UKHYIRRQ====', 'JZ5VE72GUS3C7BOHDRIYZX4B5U5EJMCMLKHLYCVBQQF3UKHYIRRQ'] @@ -56,17 +42,17 @@ class OnionrValidations(unittest.TestCase): for x in invalid: #print('testing', x) self.assertFalse(stringvalidators.validate_pub_key(x)) - + def test_integer_string(self): valid = ["1", "100", 100, "-5", -5] invalid = ['test', "1d3434", "1e100", None] for x in valid: #print('testing', x) - self.assertTrue(stringvalidators.is_integer_string(x)) - + self.assertTrue(stringvalidators.is_integer_string(x)) + for x in invalid: #print('testing', x) self.assertFalse(stringvalidators.is_integer_string(x)) - + unittest.main()