From 499b8ed37528379433661ac3c8fc94f60cc36384 Mon Sep 17 00:00:00 2001 From: Sander van Grieken Date: Thu, 8 Feb 2024 13:45:46 +0100 Subject: [PATCH] tests: payment_identifier: add some more whitespace situations, fix script spk --- electrum/tests/test_payment_identifier.py | 29 +++++++++++++++-------- 1 file changed, 19 insertions(+), 10 deletions(-) diff --git a/electrum/tests/test_payment_identifier.py b/electrum/tests/test_payment_identifier.py index 472aec03c..a06ea34d0 100644 --- a/electrum/tests/test_payment_identifier.py +++ b/electrum/tests/test_payment_identifier.py @@ -161,7 +161,6 @@ class TestPaymentIdentifier(ElectrumTestCase): 'bc1qj3zx2zc4rpv3npzmznxhdxzn0wm7pzqp8p2293,0.01', 'bc1q66ex4c3vek4cdmrfjxtssmtguvs3r30pf42jpj,0.01', 'bc1qy7ps80x5csdqpfcekn97qfljxtg2lrya8826ds,!', - 'script(OP_RETURN deadbeef),0' ]) pi = PaymentIdentifier(self.wallet, pi_str) self.assertTrue(pi.is_valid()) @@ -177,16 +176,26 @@ class TestPaymentIdentifier(ElectrumTestCase): self.assertTrue(pi.is_multiline()) def test_spk(self): - pi_str = 'bc1qj3zx2zc4rpv3npzmznxhdxzn0wm7pzqp8p2293' - pi = PaymentIdentifier(None, pi_str) - self.assertTrue(pi.is_valid()) - self.assertTrue(pi.is_available()) + address = 'bc1qj3zx2zc4rpv3npzmznxhdxzn0wm7pzqp8p2293' + for pi_str in [ + f'{address}', + f' {address}', + f'{address} ', + f'{address}'.upper(), + ]: + pi = PaymentIdentifier(None, pi_str) + self.assertTrue(pi.is_valid()) + self.assertTrue(pi.is_available()) - # TODO: script should be valid? - # pi_str = 'script(OP_RETURN baddc0ffee),0' - # pi = PaymentIdentifier(None, pi_str) - # self.assertTrue(pi.is_valid()) - # self.assertTrue(pi.is_available()) + spk = 'script(OP_RETURN baddc0ffee)' + for pi_str in [ + f'{spk}', + f' {spk}', + f'{spk} ', + ]: + pi = PaymentIdentifier(None, pi_str) + self.assertTrue(pi.is_valid()) + self.assertTrue(pi.is_available()) def test_email_and_domain(self): pi_str = 'some.domain'