From: Alan T. DeKok <aland@freeradius.org>
Date: Tue, 20 Apr 2021 13:32:54 +0000 (-0400)
Subject: don't check the results of delete, there may be no data to delete
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9166ac0aec2a6c7b0793592cce63f06b149b7fce;p=thirdparty%2Ffreeradius-server.git

don't check the results of delete, there may be no data to delete
---

diff --git a/src/tests/modules/sql/auth.unlang b/src/tests/modules/sql/auth.unlang
index 5c973d078e0..947a1eb3a23 100644
--- a/src/tests/modules/sql/auth.unlang
+++ b/src/tests/modules/sql/auth.unlang
@@ -2,6 +2,7 @@
 #  Clear out old data
 #
 "%{sql:${delete_from_radcheck} 'user_auth'}"
+"%{sql:${delete_from_radreply} 'user_auth'}"
 
 if ("%{sql:${insert_into_radcheck} ('user_auth', 'NAS-IP-Address', '==', '1.2.3.4')}" != "1") {
 	test_fail
@@ -11,10 +12,6 @@ if ("%{sql:${insert_into_radcheck} ('user_auth', 'Password.Cleartext', ':=', 'pa
 	test_fail
 }
 
-if ("%{sql:${delete_from_radreply} 'user_auth'}" != "1") {
-	test_fail
-}
-
 if ("%{sql:${insert_into_radreply} ('user_auth', 'Idle-Timeout', ':=', '3600')}" != "1") {
 	test_fail
 }
diff --git a/src/tests/modules/sql/reject.unlang b/src/tests/modules/sql/reject.unlang
index 1c68fd724a4..880198a4760 100644
--- a/src/tests/modules/sql/reject.unlang
+++ b/src/tests/modules/sql/reject.unlang
@@ -2,6 +2,7 @@
 #  Clear out old data
 #
 "%{sql:${delete_from_radcheck} 'user_reject'}"
+"%{sql:${delete_from_radreply} 'user_reject'}"
 
 #
 #  Add in the check items.
@@ -14,10 +15,6 @@ if ("%{sql:${insert_into_radcheck} ('user_reject', 'Password.Cleartext', ':=', '
 	test_fail
 }
 
-if ("%{sql:${delete_from_radreply} 'user_reject'}" != "1") {
-	test_fail
-}
-
 if ("%{sql:${insert_into_radreply} ('user_reject', 'Reply-Message', ':=', 'Authentication failed')}" != "1") {
 	test_fail
 }