Page MenuHomeFreeBSD
Differential D42714

lockf: add some tests
ClosedPublic
Actions

Authored by kevans on Nov 22 2023, 5:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 11, 2:21 AM
Unknown Object (File)
Sat, Oct 10, 11:09 PM
Unknown Object (File)
Fri, Oct 9, 4:00 PM
Unknown Object (File)
Fri, Oct 9, 3:18 PM
Unknown Object (File)
Tue, Sep 29, 10:54 AM
Unknown Object (File)
Fri, Sep 25, 11:17 PM
Unknown Object (File)
Wed, Sep 23, 2:03 PM
Unknown Object (File)
Wed, Sep 23, 1:08 PM
View All Files
Subscribers
0mp
allanjude
des
imp

Details

Reviewers
allanjude
Group Reviewers
Klara
Commits
rG296a5a4db1fc: lockf: add some tests
Summary

Provide basic coverage for the existing options, nothing deeper (e.g.,
pipe closing behavior) is tested in this set.

Sponsored by: Klara, Inc.

Diff Detail

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

Event Timeline

kevans created this revision.Nov 22 2023, 5:12 AM
Herald added a subscriber: imp. · View Herald TranscriptNov 22 2023, 5:12 AM
kevans requested review of this revision.Nov 22 2023, 5:12 AM
Harbormaster completed remote builds in B54562: Diff 130418.Nov 22 2023, 5:12 AM
kevans added a parent revision: D42713: lockf: don't hold stdin/stdout/stderr open.Nov 22 2023, 5:12 AM
kevans updated this revision to Diff 130420.Nov 22 2023, 7:02 AM

Stop waiting on some of the sleeps, improve reliability

Harbormaster completed remote builds in B54564: Diff 130420.Nov 22 2023, 7:02 AM
kevans added a child revision: D42718: lockf: allow locking file descriptors.Nov 22 2023, 8:02 AM
0mp added a subscriber: 0mp.Nov 22 2023, 9:39 AM
0mp added inline comments.
usr.bin/lockf/tests/lockf_test.sh
95

What's the reason for that? Why is failing under 10 seconds considered a success?

allanjude accepted this revision.Nov 22 2023, 1:49 PM
allanjude added a subscriber: allanjude.

reviewed-by: allanjude

usr.bin/lockf/tests/lockf_test.sh
95

the -n flag means: exit immediately if testlock does not already exist

This revision is now accepted and ready to land.Nov 22 2023, 1:49 PM
des added a subscriber: des.Nov 22 2023, 3:12 PM
des added inline comments.
usr.bin/lockf/tests/lockf_test.sh
38

style(9) no longer requires a blank line at the top of a function

Closed by commit rG296a5a4db1fc: lockf: add some tests (authored by kevans). · Explain WhyNov 26 2023, 4:13 AM
This revision was automatically updated to reflect the committed changes.
kevans marked an inline comment as done.
kevans added a commit: rG296a5a4db1fc: lockf: add some tests.

Revision Contents
Changeset List

PathSize
etc/
mtree/
2 lines
usr.bin/
lockf/
5 lines
tests/
6 lines
148 lines

Diff 130591

View Options

etc/mtree/BSD.tests.dist

Loading...
View Options

usr.bin/lockf/Makefile

Loading...
View Options

usr.bin/lockf/tests/Makefile

Loading...
View Options

usr.bin/lockf/tests/lockf_test.sh

Loading...