From: Alan T. DeKok <aland@freeradius.org>
Date: Wed, 31 Jul 2019 19:28:21 +0000 (-0400)
Subject: no longer a need for "fail" after break / return
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1d2a329bc99d659902d04f760279e2fdbe79ef8a;p=thirdparty%2Ffreeradius-server.git

no longer a need for "fail" after break / return
---

diff --git a/src/tests/keywords/foreach-break b/src/tests/keywords/foreach-break
index 2b5a3465b72..cbfb19ed9c0 100644
--- a/src/tests/keywords/foreach-break
+++ b/src/tests/keywords/foreach-break
@@ -37,8 +37,6 @@ foreach &request:Reply-Message {
 			&request:Filter-Id += "3"
 		}
 		break
-
-		test_fail
 	}
 }
 
@@ -63,7 +61,5 @@ foreach &request:Reply-Message {
 
 	if ("%{Foreach-Variable-0}" == "3") {
 		break
-
-		test_fail
 	}
 }
diff --git a/src/tests/keywords/foreach-return b/src/tests/keywords/foreach-return
index b5f22d453c9..753c6e98f1d 100644
--- a/src/tests/keywords/foreach-return
+++ b/src/tests/keywords/foreach-return
@@ -33,11 +33,6 @@ foreach &Filter-Id {
 		#  Stop processing "authorize", and go to the next section.
 		#
 		return
-
-		#
-		#  Shouldn't reach this
-		#
-		test_fail
 	}
 
 	if ("%{Foreach-Variable-0}" == "4") {