commit 1921fd585cb0314bb7e6e165727664c52052dd97
author Janos Follath <janos.follath@arm.com> Fri Nov 18 17:51:02 2022 +0000
committer Janos Follath <janos.follath@arm.com> Mon Nov 21 08:56:14 2022 +0000
tree 6b069ccf37a4f805ce79375ce4dc0108ab10755e
parent a36a3d36b5c749011f8b94f88d37a3f3523ff8a8

Bignum tests: use arity in bignum_mod_raw

This makes a couple of properties redundant which are cleaned up.

Signed-off-by: Janos Follath <janos.follath@arm.com>

scripts/mbedtls_dev/bignum_mod_raw.py

diff --git a/scripts/mbedtls_dev/bignum_mod_raw.py b/scripts/mbedtls_dev/bignum_mod_raw.py
index e2d8cd6..6c217c2 100644
--- a/scripts/mbedtls_dev/bignum_mod_raw.py
+++ b/scripts/mbedtls_dev/bignum_mod_raw.py
@@ -57,6 +57,7 @@
     test_function = "mpi_mod_raw_to_mont_rep"
     test_name = "Convert into Mont: "
     input_style = "arch_split"
+    arity = 1
 
     test_data_moduli = ["b",
                         "fd",
@@ -111,12 +112,8 @@
     descr_tpl = '{} #{} N: \"{}\" A: \"{}\".'
 
     def result(self) -> List[str]:
-        return [self.hex_x]
-
-    def arguments(self) -> List[str]:
-        return [bignum_common.quote_str(n) for n in [self.arg_n,
-                                                     self.arg_a,
-                                                     self.hex_x]]
+        result = (self.int_a * self.r) % self.int_n
+        return [self.format_result(result)]
 
     def description(self) -> str:
         return self.descr_tpl.format(self.test_name,
@@ -134,13 +131,6 @@
                         continue
                     yield cls(n, i, bits_in_limb=bil).create_test_case()
 
-    @property
-    def x(self) -> int: # pylint: disable=invalid-name
-        return (self.int_a * self.r) % self.int_n
-
-    @property
-    def hex_x(self) -> str:
-        return "{:x}".format(self.x).zfill(self.hex_digits)
 
 class BignumModRawConvertFromMont(BignumModRawConvertToMont):
     """ Test cases for mpi_mod_raw_from_mont_rep(). """
@@ -169,9 +159,11 @@
                           "138a7e6bfbc319ebd1725dacb9a359cbf693f2ecb785efb9d627"
                          ]
 
-    @property
-    def x(self): # pylint: disable=invalid-name
-        return (self.int_a * self.r_inv) % self.int_n
+    def result(self) -> List[str]:
+        result = (self.int_a * self.r_inv) % self.int_n
+        return [self.format_result(result)]
+
+
 # END MERGE SLOT 7
 
 # BEGIN MERGE SLOT 8