From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tim FitzGeorge To: development@lists.ipfire.org Subject: [PATCH 1/3] network-functions.pl : Enhance testsuite to show cause of error Date: Tue, 24 Dec 2019 12:56:50 +0000 Message-ID: <20191224125652.12232-2-ipfr@tfitzgeorge.me.uk> In-Reply-To: <20191224125652.12232-1-ipfr@tfitzgeorge.me.uk> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============3324433611139396382==" List-Id: --===============3324433611139396382== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Signed-off-by: Tim FitzGeorge --- config/cfgroot/network-functions.pl | 29 +++++++++++++++-------------- 1 file changed, 15 insertions(+), 14 deletions(-) diff --git a/config/cfgroot/network-functions.pl b/config/cfgroot/network-fun= ctions.pl index 8649d0502..56b4bceb7 100644 --- a/config/cfgroot/network-functions.pl +++ b/config/cfgroot/network-functions.pl @@ -449,14 +449,15 @@ sub get_mac_by_name($) { # Remove the next line to enable the testsuite __END__ =20 -sub assert($) { +sub assert($$) { + my $tst =3D shift; my $ret =3D shift; =20 if ($ret) { return; } =20 - print "ASSERTION ERROR"; + print "ASSERTION ERROR - $tst\n"; exit(1); } =20 @@ -464,10 +465,10 @@ sub testsuite() { my $result; =20 my $address1 =3D &ip2bin("8.8.8.8"); - assert($address1 =3D=3D 134744072); + assert('ip2bin("8.8.8.8")', $address1 =3D=3D 134744072); =20 my $address2 =3D &bin2ip($address1); - assert($address2 eq "8.8.8.8"); + assert("bin2ip($address1)", $address2 eq "8.8.8.8"); =20 # Check if valid IP addresses are correctly recognised. foreach my $address ("1.2.3.4", "192.168.180.1", "127.0.0.1") { @@ -486,34 +487,34 @@ sub testsuite() { } =20 $result =3D &check_ip_address_and_netmask("192.168.180.0/255.255.255.0"); - assert($result); + assert('check_ip_address_and_netmask("192.168.180.0/255.255.255.0")', $resu= lt); =20 $result =3D &convert_netmask2prefix("255.255.254.0"); - assert($result =3D=3D 23); + assert('convert_netmask2prefix("255.255.254.0")', $result =3D=3D 23); =20 $result =3D &convert_prefix2netmask(8); - assert($result eq "255.0.0.0"); + assert('convert_prefix2netmask(8)', $result eq "255.0.0.0"); =20 $result =3D &find_next_ip_address("1.2.3.4", 2); - assert($result eq "1.2.3.6"); + assert('find_next_ip_address("1.2.3.4", 2)', $result eq "1.2.3.6"); =20 $result =3D &network_equal("192.168.0.0/24", "192.168.0.0/255.255.255.0"); - assert($result); + assert('network_equal("192.168.0.0/24", "192.168.0.0/255.255.255.0")', $res= ult); =20 $result =3D &network_equal("192.168.0.0/24", "192.168.0.0/25"); - assert(!$result); + assert('network_equal("192.168.0.0/24", "192.168.0.0/25")', !$result); =20 $result =3D &network_equal("192.168.0.0/24", "192.168.0.128/25"); - assert(!$result); + assert('network_equal("192.168.0.0/24", "192.168.0.128/25")', !$result); =20 $result =3D &network_equal("192.168.0.1/24", "192.168.0.XXX/24"); - assert(!$result); + assert('network_equal("192.168.0.1/24", "192.168.0.XXX/24")', !$result); =20 $result =3D &ip_address_in_network("10.0.1.4", "10.0.0.0/8"); - assert($result); + assert('ip_address_in_network("10.0.1.4", "10.0.0.0/8"', $result); =20 $result =3D &ip_address_in_network("192.168.30.11", "192.168.30.0/255.255.2= 55.0"); - assert($result); + assert('ip_address_in_network("192.168.30.11", "192.168.30.0/255.255.255.0"= )', $result); =20 print "Testsuite completed successfully!\n"; =20 --=20 2.16.4 --===============3324433611139396382==--