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)
Fri, Oct 24, 1:16 AM
Unknown Object (File)
Fri, Oct 17, 2:36 AM
Unknown Object (File)
Sat, Sep 27, 7:18 PM
Unknown Object (File)
Sat, Sep 27, 2:45 PM
Unknown Object (File)
Sat, Sep 27, 2:16 PM
Unknown Object (File)
Sat, Sep 27, 10:53 AM
Unknown Object (File)
Sep 23 2025, 1:03 AM
Unknown Object (File)
Sep 21 2025, 9:31 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 Skipped
Unit
Tests Skipped
Build Status
Buildable 54564
Build 51453: arc lint + arc unit

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
94

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
94

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
37

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
154 lines

Diff 130420

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...