Skip to content

Commit 794e49d

Browse files
committed
xdp-filter/tests: Replace FAIL with NOTOK in packet checks
Makes it easier to grep for test failures (using FAIL). Signed-off-by: Toke Høiland-Jørgensen <[email protected]>
1 parent 875843a commit 794e49d

File tree

1 file changed

+22
-22
lines changed

1 file changed

+22
-22
lines changed

xdp-filter/tests/test-xdp-filter.sh

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -102,9 +102,9 @@ test_ports_allow()
102102
check_port tcp $TEST_PORT OK
103103
check_port udp $TEST_PORT OK
104104
check_run $XDP_FILTER port $TEST_PORT -v
105-
check_port tcp $TEST_PORT FAIL
105+
check_port tcp $TEST_PORT NOTOK
106106
check_port tcp $[TEST_PORT+1] OK
107-
check_port udp $TEST_PORT FAIL
107+
check_port udp $TEST_PORT NOTOK
108108
check_port udp $[TEST_PORT+1] OK
109109
check_run $XDP_FILTER port -r $TEST_PORT -v
110110
check_port tcp $TEST_PORT OK
@@ -117,16 +117,16 @@ test_ports_deny()
117117
local TEST_PORT=10000
118118
# default deny mode
119119
check_run $XDP_FILTER load -p deny -f udp,tcp $NS -v
120-
check_port tcp $TEST_PORT FAIL
121-
check_port udp $TEST_PORT FAIL
120+
check_port tcp $TEST_PORT NOTOK
121+
check_port udp $TEST_PORT NOTOK
122122
check_run $XDP_FILTER port $TEST_PORT -v
123123
check_port tcp $TEST_PORT OK
124-
check_port tcp $[TEST_PORT+1] FAIL
124+
check_port tcp $[TEST_PORT+1] NOTOK
125125
check_port udp $TEST_PORT OK
126-
check_port udp $[TEST_PORT+1] FAIL
126+
check_port udp $[TEST_PORT+1] NOTOK
127127
check_run $XDP_FILTER port -r $TEST_PORT -v
128-
check_port tcp $TEST_PORT FAIL
129-
check_port udp $TEST_PORT FAIL
128+
check_port tcp $TEST_PORT NOTOK
129+
check_port udp $TEST_PORT NOTOK
130130
check_run $XDP_FILTER unload $NS -v
131131
}
132132

@@ -140,11 +140,11 @@ test_ipv6_allow()
140140
check_ping6 OK
141141
check_run $XDP_FILTER load -f ipv6 $NS -v
142142
check_run $XDP_FILTER ip $OUTSIDE_IP6
143-
check_ping6 FAIL
143+
check_ping6 NOTOK
144144
check_run $XDP_FILTER ip -r $OUTSIDE_IP6
145145
check_ping6 OK
146146
check_run $XDP_FILTER ip -m src $INSIDE_IP6
147-
check_ping6 FAIL
147+
check_ping6 NOTOK
148148
check_run $XDP_FILTER ip -m src -r $INSIDE_IP6
149149
check_ping6 OK
150150
check_run $XDP_FILTER unload $NS -v
@@ -157,11 +157,11 @@ test_ipv6_deny()
157157
check_run $XDP_FILTER ip $OUTSIDE_IP6
158158
check_ping6 OK
159159
check_run $XDP_FILTER ip -r $OUTSIDE_IP6
160-
check_ping6 FAIL
160+
check_ping6 NOTOK
161161
check_run $XDP_FILTER ip -m src $INSIDE_IP6
162162
check_ping6 OK
163163
check_run $XDP_FILTER ip -m src -r $INSIDE_IP6
164-
check_ping6 FAIL
164+
check_ping6 NOTOK
165165
check_run $XDP_FILTER unload $NS -v
166166
}
167167

@@ -175,11 +175,11 @@ test_ipv4_allow()
175175
check_ping4 OK
176176
check_run $XDP_FILTER load -f ipv4 $NS -v
177177
check_run $XDP_FILTER ip $OUTSIDE_IP4
178-
check_ping4 FAIL
178+
check_ping4 NOTOK
179179
check_run $XDP_FILTER ip -r $OUTSIDE_IP4
180180
check_ping4 OK
181181
check_run $XDP_FILTER ip -m src $INSIDE_IP4
182-
check_ping4 FAIL
182+
check_ping4 NOTOK
183183
check_run $XDP_FILTER ip -m src -r $INSIDE_IP4
184184
check_ping4 OK
185185
check_run $XDP_FILTER unload $NS -v
@@ -192,11 +192,11 @@ test_ipv4_deny()
192192
check_run $XDP_FILTER ip $OUTSIDE_IP4
193193
check_ping4 OK
194194
check_run $XDP_FILTER ip -r $OUTSIDE_IP4
195-
check_ping4 FAIL
195+
check_ping4 NOTOK
196196
check_run $XDP_FILTER ip -m src $INSIDE_IP4
197197
check_ping4 OK
198198
check_run $XDP_FILTER ip -m src -r $INSIDE_IP4
199-
check_ping4 FAIL
199+
check_ping4 NOTOK
200200
check_run $XDP_FILTER unload $NS -v
201201
}
202202

@@ -205,11 +205,11 @@ test_ether_allow()
205205
check_ping6 OK
206206
check_run $XDP_FILTER load -f ethernet $NS -v
207207
check_run $XDP_FILTER ether $OUTSIDE_MAC
208-
check_ping6 FAIL
208+
check_ping6 NOTOK
209209
check_run $XDP_FILTER ether -r $OUTSIDE_MAC
210210
check_ping6 OK
211211
check_run $XDP_FILTER ether -m src $INSIDE_MAC
212-
check_ping6 FAIL
212+
check_ping6 NOTOK
213213
check_run $XDP_FILTER ether -m src -r $INSIDE_MAC
214214
check_ping6 OK
215215
check_run $XDP_FILTER unload $NS -v
@@ -222,11 +222,11 @@ test_ether_deny()
222222
check_run $XDP_FILTER ether $OUTSIDE_MAC
223223
check_ping6 OK
224224
check_run $XDP_FILTER ether -r $OUTSIDE_MAC
225-
check_ping6 FAIL
225+
check_ping6 NOTOK
226226
check_run $XDP_FILTER ether -m src $INSIDE_MAC
227227
check_ping6 OK
228228
check_run $XDP_FILTER ether -m src -r $INSIDE_MAC
229-
check_ping6 FAIL
229+
check_ping6 NOTOK
230230
check_run $XDP_FILTER unload $NS -v
231231
}
232232

@@ -287,7 +287,7 @@ check_port_removal_from_all()
287287

288288
check_run $XDP_FILTER port $TEST_PORT -p tcp,udp -m src,dst
289289
check_status "$TEST_PORT.*src,dst,tcp,udp"
290-
290+
291291
check_run $XDP_FILTER port $TEST_PORT $command_options -r
292292
if [[ -z "$expected_output" ]]; then
293293
check_status_no_match "$TEST_PORT"
@@ -362,7 +362,7 @@ run_python_test()
362362
fi
363363

364364
python="$(get_python)"
365-
if [[ $? -ne 0 ]]; then
365+
if [[ $? -ne 0 ]]; then
366366
return "$SKIPPED_TEST"
367367
fi
368368

0 commit comments

Comments
 (0)