Page MenuHomeFreeBSD
Differential D55153

pwd: Add tests
ClosedPublic
Actions

Authored by des on Feb 7 2026, 10:53 AM.
Tags
None
Referenced Files
F153364469: D55153.id.diff
Mon, Apr 20, 5:47 PM
Unknown Object (File)
Sat, Apr 18, 9:45 PM
Unknown Object (File)
Sun, Apr 12, 7:13 PM
Unknown Object (File)
Sun, Apr 12, 5:28 PM
Unknown Object (File)
Sat, Apr 11, 8:55 PM
Unknown Object (File)
Sun, Apr 5, 7:19 PM
Unknown Object (File)
Wed, Apr 1, 5:43 AM
Unknown Object (File)
Mar 11 2026, 12:03 PM
View All 27 Files
Subscribers
asomers
imp
ivy

Details

Reviewers
imp
Group Reviewers
tests
Commits
rG11a22ea28ecf: pwd: Add tests
rG36fa67513b33: pwd: Add tests
rG36ec2b1c104f: pwd: Add tests
Summary

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des created this revision.Feb 7 2026, 10:53 AM
Herald added subscribers: ivy, imp. · View Herald TranscriptFeb 7 2026, 10:53 AM
des requested review of this revision.Feb 7 2026, 10:53 AM
Harbormaster completed remote builds in B70488: Diff 171397.Feb 7 2026, 10:54 AM
des added a parent revision: D55146: pwd: Clean up and adopt POSIX semantics.Feb 7 2026, 10:54 AM
asomers added a subscriber: asomers.Feb 7 2026, 3:43 PM
asomers added inline comments.
bin/pwd/tests/pwd_test.sh
27

This line is redundant.

49–57

Shouldn't these two paragraphs belong in the logical test?

des marked 2 inline comments as done.Feb 7 2026, 4:18 PM
des added inline comments.
bin/pwd/tests/pwd_test.sh
27

It is also harmless.

49–57

No, the test does exactly what its description says.

des marked 2 inline comments as done.Feb 7 2026, 4:19 PM
imp accepted this revision.Feb 7 2026, 6:09 PM
This revision is now accepted and ready to land.Feb 7 2026, 6:09 PM
des updated this revision to Diff 171559.Feb 9 2026, 5:40 PM

rename directory and symlink so tests are more self-explanatory
add more physical cases

This revision now requires review to proceed.Feb 9 2026, 5:40 PM
Harbormaster completed remote builds in B70555: Diff 171559.Feb 9 2026, 5:40 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 10 2026, 2:29 PM
Closed by commit rG36ec2b1c104f: pwd: Add tests (authored by des). · Explain Why
This revision was automatically updated to reflect the committed changes.
des added a commit: rG36ec2b1c104f: pwd: Add tests.
des added a commit: rG36fa67513b33: pwd: Add tests.Feb 15 2026, 8:58 AM
des added a commit: rG11a22ea28ecf: pwd: Add tests.
ngie mentioned this in D40611: Add `pwd` test.Mon, Apr 20, 9:03 PM

Revision Contents
Changeset List

PathSize
bin/
pwd/
8 lines
tests/
4 lines
73 lines
etc/
mtree/
2 lines

Diff 171648

View Options

bin/pwd/Makefile

Loading...
View Options

bin/pwd/tests/Makefile

Loading...
View Options

bin/pwd/tests/pwd_test.sh

Loading...
View Options

etc/mtree/BSD.tests.dist

Loading...