Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

CVS commit: src/tests/bin/sh



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.




Home | Main Index | Thread Index | Old Index