Page MenuHomeFreeBSD

Add test for regression fixed in r329743
Needs ReviewPublic

Authored by rstone on Mar 1 2018, 9:37 PM.
Tags
None
Referenced Files
F106229420: D14559.diff
Fri, Dec 27, 1:53 PM
Unknown Object (File)
Nov 24 2024, 9:09 PM
Unknown Object (File)
Oct 7 2024, 12:51 PM
Unknown Object (File)
Oct 7 2024, 11:24 AM
Unknown Object (File)
Oct 4 2024, 11:59 AM
Unknown Object (File)
Oct 4 2024, 10:01 AM
Unknown Object (File)
Oct 3 2024, 11:50 PM
Unknown Object (File)
Oct 3 2024, 7:13 AM
Subscribers
None

Details

Reviewers
asomers
Summary

In r264986, the behaviour of route(8) regresssed and it became
no longer possible to modify a route via "route change" unless
you explicitly specified the route's gateway. Add an explicit
test for this behaviour to ensure that this doesn't happen again.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15334
Build 15390: arc lint + arc unit

Event Timeline

asomers requested changes to this revision.Mar 1 2018, 10:01 PM

Most of the other tests in this file relate specifically to multifib functionality. I think your test should go into a new file. You can pull the helper routines out into a .subr file.

This revision now requires changes to proceed.Mar 1 2018, 10:01 PM

Move non-fib test into separate test file

Where is route.subr? And where did ipv4_move_subnet_route come from? I think you used the wrong base revision to generate the review.

Where is route.subr? And where did ipv4_move_subnet_route come from? I think you used the wrong base revision to generate the review.

This based off of the change in https://reviews.freebsd.org/D14547

Why don't you just combine this review with that one? It would be easier to review that way.