git: 8934c3dc780d - main - pf tests: Set require.kmods in divert-to tests

From: Mark Johnston <markj_at_FreeBSD.org>
Date: Mon, 26 Jan 2026 17:24:55 UTC
The branch main has been updated by markj:

URL: https://cgit.FreeBSD.org/src/commit/?id=8934c3dc780d6631cb23e4d98c7b988da9a5b703

commit 8934c3dc780d6631cb23e4d98c7b988da9a5b703
Author:     Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2026-01-26 17:24:25 +0000
Commit:     Mark Johnston <markj@FreeBSD.org>
CommitDate: 2026-01-26 17:24:25 +0000

    pf tests: Set require.kmods in divert-to tests
    
    Tests should declare their dependencies rather than testing for them at
    runtime.
    
    Reviewed by:    igoro, kp, glebius
    MFC after:      2 weeks
    Sponsored by:   OPNsense
    Sponsored by:   Klara, Inc.
    Differential Revision:  https://reviews.freebsd.org/D54846
---
 tests/sys/netpfil/pf/divert-to.sh | 31 ++++++++-----------------------
 1 file changed, 8 insertions(+), 23 deletions(-)

diff --git a/tests/sys/netpfil/pf/divert-to.sh b/tests/sys/netpfil/pf/divert-to.sh
index 3028c9e75afd..2e0f6920db27 100644
--- a/tests/sys/netpfil/pf/divert-to.sh
+++ b/tests/sys/netpfil/pf/divert-to.sh
@@ -60,30 +60,16 @@
 
 . $(atf_get_srcdir)/utils.subr
 
-divert_init()
-{
-	if ! kldstat -q -m ipdivert; then
-		atf_skip "This test requires ipdivert"
-	fi
-}
-
-dummynet_init()
-{
-	if ! kldstat -q -m dummynet; then
-		atf_skip "This test requires dummynet"
-	fi
-}
-
 atf_test_case "in_div" "cleanup"
 in_div_head()
 {
 	atf_set descr 'Test inbound > diverted | divapp terminated'
 	atf_set require.user root
+	atf_set require.kmods ipdivert
 }
 in_div_body()
 {
 	pft_init
-	divert_init
 
 	epair=$(vnet_mkepair)
 	vnet_mkjail div ${epair}b
@@ -118,11 +104,11 @@ in_div_in_head()
 {
 	atf_set descr 'Test inbound > diverted > inbound | host terminated'
 	atf_set require.user root
+	atf_set require.kmods ipdivert
 }
 in_div_in_body()
 {
 	pft_init
-	divert_init
 
 	epair=$(vnet_mkepair)
 	vnet_mkjail div ${epair}b
@@ -157,11 +143,11 @@ out_div_head()
 {
 	atf_set descr 'Test outbound > diverted | divapp terminated'
 	atf_set require.user root
+	atf_set require.kmods ipdivert
 }
 out_div_body()
 {
 	pft_init
-	divert_init
 
 	epair=$(vnet_mkepair)
 	vnet_mkjail div ${epair}b
@@ -197,11 +183,11 @@ out_div_out_head()
 {
 	atf_set descr 'Test outbound > diverted > outbound | network terminated'
 	atf_set require.user root
+	atf_set require.kmods ipdivert
 }
 out_div_out_body()
 {
 	pft_init
-	divert_init
 
 	epair=$(vnet_mkepair)
 	vnet_mkjail div ${epair}b
@@ -237,11 +223,11 @@ in_div_in_fwd_out_div_out_head()
 {
 	atf_set descr 'Test inbound > diverted > inbound > forwarded > outbound > diverted > outbound | network terminated'
 	atf_set require.user root
+	atf_set require.kmods ipdivert
 }
 in_div_in_fwd_out_div_out_body()
 {
 	pft_init
-	divert_init
 
 	# host <a--epair0--b> router <a--epair1--b> site
 	epair0=$(vnet_mkepair)
@@ -293,12 +279,11 @@ in_dn_in_div_in_out_div_out_dn_out_head()
 {
 	atf_set descr 'Test inbound > delayed+diverted > outbound > diverted+delayed > outbound | network terminated'
 	atf_set require.user root
+	atf_set require.kmods dummynet ipdivert
 }
 in_dn_in_div_in_out_div_out_dn_out_body()
 {
 	pft_init
-	divert_init
-	dummynet_init
 
 	epair=$(vnet_mkepair)
 	vnet_mkjail alcatraz ${epair}b
@@ -377,12 +362,12 @@ pr260867_head()
 {
 	atf_set descr 'Test for the loop reported in PR260867'
 	atf_set require.user root
+	atf_set require.kmods ipdivert
 }
 
 pr260867_body()
 {
 	pft_init
-	divert_init
 
 	epair=$(vnet_mkepair)
 
@@ -417,12 +402,12 @@ pr260867_icmp_head()
 {
 	atf_set descr 'Variant of the PR260867 test'
 	atf_set require.user root
+	atf_set require.kmods ipdivert
 }
 
 pr260867_icmp_body()
 {
 	pft_init
-	divert_init
 
 	epair=$(vnet_mkepair)