git: c9c2452a2535 - main - unix tests: Skip random_eor_and_waitall unconditionally

From: Mark Johnston <markj_at_FreeBSD.org>
Date: Wed, 25 Sep 2024 07:13:47 UTC
The branch main has been updated by markj:

URL: https://cgit.FreeBSD.org/src/commit/?id=c9c2452a25355db7d89bfc93fd9d50f46690949c

commit c9c2452a25355db7d89bfc93fd9d50f46690949c
Author:     Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2024-09-21 12:41:06 +0000
Commit:     Mark Johnston <markj@FreeBSD.org>
CommitDate: 2024-09-25 11:44:14 +0000

    unix tests: Skip random_eor_and_waitall unconditionally
    
    This test always fails, I don't see any reason to make it conditional on
    the "CI" test parameter.
    
    There is at least one test bug here, we're using the wrong sysctl to
    obtain the receive buffer size, but fixing that is not sufficient.
    
    PR:             279354
    Reviewed by:    glebius
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D46726
---
 tests/sys/kern/unix_seqpacket_test.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/tests/sys/kern/unix_seqpacket_test.c b/tests/sys/kern/unix_seqpacket_test.c
index ba3de0adb1e9..d142e228b036 100644
--- a/tests/sys/kern/unix_seqpacket_test.c
+++ b/tests/sys/kern/unix_seqpacket_test.c
@@ -1197,8 +1197,7 @@ ATF_TC_BODY(random_eor_and_waitall, tc)
 	size_t off;
 	int fd[2], eor;
 
-	if (atf_tc_get_config_var_as_bool_wd(tc, "ci", false))
-		atf_tc_skip("https://bugs.freebsd.org/279354");
+	atf_tc_skip("https://bugs.freebsd.org/279354");
 
 	arc4random_buf(params.seed, sizeof(params.seed));
 	printf("Using seed:");
@@ -1210,7 +1209,7 @@ ATF_TC_BODY(random_eor_and_waitall, tc)
 	for (u_int i = 0; i < RANDOM_TESTSIZE / (u_int )sizeof(long); i++)
 		((long *)params.sendbuf)[i] = nrand48(&params.seed[0]);
 
-	ATF_REQUIRE(sysctlbyname("net.local.stream.recvspace",
+	ATF_REQUIRE(sysctlbyname("net.local.seqpacket.recvspace",
 	    &params.recvspace, &(size_t){sizeof(u_long)}, NULL, 0) != -1);
 	ATF_REQUIRE((recvbuf =
 	    malloc(RANDOM_RECVSIZE * params.recvspace)) != NULL);