CVS commit: src/tests/bin/sh

"Robert Elz" <[email protected]>
Newsgroups gmane.os.netbsd.devel.cvs
Message-ID <[email protected]>
Module Name:	src
Committed By:	kre
Date:		Wed Jul  1 10:58:38 UTC 2026

Modified Files:
	src/tests/bin/sh: t_builtins.sh

Log Message:
sh tests: builtins: add more content

Many of the t_builtins sh tests were just placeholders for things to come.
They've been that way for years...

This adds some actual content to some of them - still very primitive in
most cases, but better than was there before.   There are still too many
empty tests (test the builtin exists, but that's it).

Also, add a test for the trap builtin - that wasn't there before as traps
are tested in other of the sh tests ... this new one is therefor not testing
that traps work (or not in any way that's meaningful), rather it is testing
that the various ways the trap command can be used all work.

Currently two of the sub-tests will fail, the new trap test, and the
(was previously empty) jobid test (testing a NetBSD extension builtin).

Adding those failing tests was the inspiration for all of this.  The bug
they expose is exotic, and never likely to be encountered, or do any harm
id it happens to be, as it only occurs in situations where it makes zero
sense to do the commands which fail.

A fix for sh to correct the issue will be coming soon (already ready to
commit, just waiting to allow a test failure to actually occur).

(No pullups).


To generate a diff of this commit:
cvs rdiff -u -r1.8 -r1.9 src/tests/bin/sh/t_builtins.sh

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.