Differential D25220

Add tests for "add", "change" and "delete" functionality of /sbin/route.
ClosedPublic
Actions

Authored by ahsanb on Jun 10 2020, 10:18 PM.

Tags

None

Referenced Files

	F81594433: D25220.id73448.diff
	Thu, Apr 18, 4:31 PM

	Unknown Object (File)
	Tue, Apr 16, 1:45 PM

	Unknown Object (File)
	Mar 11 2024, 7:40 PM

	Unknown Object (File)
	Mar 11 2024, 7:40 PM

	Unknown Object (File)
	Mar 11 2024, 7:37 PM

	Unknown Object (File)
	Mar 11 2024, 7:37 PM

	Unknown Object (File)
	Mar 11 2024, 7:36 PM

	Unknown Object (File)
	Mar 8 2024, 3:22 AM

View All 45 Files

Subscribers

Details

Reviewers

kp
thj

Commits

rS363417: Add tests for "add", "change" and "delete" functionality of /sbin/route.

Summary

This revision includes the tests for the route utility. The tests cover the add, change and delete functionality of a route.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ahsanb created this revision.Jun 10 2020, 10:18 PM

Herald added a subscriber: imp. · View Herald TranscriptJun 10 2020, 10:18 PM

ahsanb requested review of this revision.Jun 10 2020, 10:18 PM

kp accepted this revision.Jun 17 2020, 9:26 AM

This revision is now accepted and ready to land.Jun 17 2020, 9:26 AM

thj accepted this revision.Jun 21 2020, 10:33 AM

Added the v6 tests.

This revision now requires review to proceed.Jun 21 2020, 9:15 PM

Replace ipv6 ULA addresses with the documentation/example ipv6 addresses

thj added inline comments.Jul 13 2020, 7:58 PM

sbin/route/tests/basic.sh
90 ↗	(On Diff #73668)	I think we need to use IPv6 Unique Local Addresses for tests and not documentation addresses. The pass_block.sh test from last year has an example of these. https://github.com/freebsd/freebsd/blob/master/tests/sys/netpfil/common/pass_block.sh#L88 This applies to all v6 addresses used in this test

thj accepted this revision.Jul 22 2020, 6:48 AM

thj added inline comments.

sbin/route/tests/basic.sh
90 ↗	(On Diff #73668)	We have decided to use documentation addresses for all network tests so this comment can be struck

This revision is now accepted and ready to land.Jul 22 2020, 6:48 AM

Closed by commit rS363417: Add tests for "add", "change" and "delete" functionality of /sbin/route. (authored by thj). · Explain WhyJul 22 2020, 1:50 PM

This revision was automatically updated to reflect the committed changes.

thj added a commit: rS363417: Add tests for "add", "change" and "delete" functionality of /sbin/route..

Revision Contents
Changeset List

Path

Size

head/

etc/

mtree/

2 lines

sbin/

route/

3 lines

tests/

11 lines

125 lines

47 lines

Diff 74785

head/etc/mtree/BSD.tests.dist

Loading...

head/sbin/route/Makefile

Loading...

head/sbin/route/tests/Makefile

Loading...

head/sbin/route/tests/basic.sh

Loading...

head/sbin/route/tests/utils.subr

Loading...