ngircd/src/testsuite, branch master

Test suite: Correctly test for LOGNAME and USER

2024-04-13T14:04:29+00:00

Test suite: Don't use "pgrep -u" when LOGNAME and USER are not set

2024-04-13T13:54:06+00:00

Thanks for reporting this on IRC, luca!

Test suite: Use $USER in getpid.sh when $LOGNAME is not set

2024-04-13T10:22:35+00:00

The LOGNAME environment variable is not set in GitHub "actions", for example ...

Test suite: wait for ERROR message on QUIT

2024-04-05T21:16:07+00:00

Wait for the "ERROR :Closing connection" message sent by ngIRCd when handling the QUIT command, do not wait for "Connection closed" which is actually output by the telnet(1) command and is implementation dependant! For example, on Haiku OS, this is not always(!) echoed (the command seems to hang sometimes?) which results in unpredictable failures in the test suite ...

Test suite: simplify and enhance getpid.sh

2024-04-05T21:16:07+00:00

- Try to only search for processes of the current user. - Prefer using pgrep (in addition to pidof) when available. - Streamline system overrides. - Get rid of HEAD_FLAGS: all systems so far support "-1". - Use "ps -o pid,comm" as the default, which is POSIX.1. - Use "sort -r" to hopefully get the newest (=highest) PID, which is the case on older systems not using randomized PIDs at least.

Test suite: clean up scripts

2024-04-05T21:16:07+00:00

- Reformat code. - Cleanup some glitches, streamline scripts ... - Enable "set -u": Error on unset variables. - Detect "$srcdir" in prep-server3 script, too.

Revert "testsuite: Pass -nameopt to openssl s_client."

2024-04-02T20:06:38+00:00

Although this is a nice looking solution, it is not that portable: for example, the "openssl s_client" command of LibreSSL 3.8.2 on OpenBSD 7.4 does not support it. So let's revert back to the "uglier" but working variant ... This reverts commit bdb55fb4b322b2c84530855a3b5148a0e387f5b4.

S2S-TLS: Fix "make check" in separate build directory

2024-03-23T19:23:32+00:00

testsuite: Pass -nameopt to openssl s_client.

2024-02-05T12:39:15+00:00

The default value for the -nameopt option changed in OpenSSL 3.2 from `oneline' to `utf8'. The `oneline' option also included a space around the fields which is not the case for `utf8'. This means that CN = my.first.domain.tld changed to CN=my.first.domain.tld and is now longer recognized, leading to test failure. This can be fixed by either going back to `oneline' or keeping `utf8' and adding additionally `space_eq'. Anoter way would be to teach the expect that the space is optional. Add explicit -nameopt option with `utf8,space_eq' which is understood by by OpenSSL 3.2 and earlier to make explicit. Remove the wildcard. Signed-off-by: Sebastian Andrzej Siewior

Testsuite: Test for the openssl command before using it

2024-01-23T13:43:06+00:00

And skip the tests calling it instead of failing!