Skip to content

tests/test: Ensure the test fixture files have the expected gid #5455

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 25, 2023

Conversation

n1000
Copy link
Contributor

@n1000 n1000 commented Oct 26, 2023

On some platforms (mostly the BSDs) the test fixture files copied to the /tmp directory will have a different gid than the current egid (due to the sticky bit set on the /tmp directory). Fix this before running the "test" command.

@n1000
Copy link
Contributor Author

n1000 commented Oct 26, 2023

Checking the cspell failure.

@n1000 n1000 force-pushed the freebsd_fixes_push_3 branch from 33e3994 to 1adebef Compare October 26, 2023 06:25
@n1000
Copy link
Contributor Author

n1000 commented Oct 26, 2023

Fixed the cspell failure.

@n1000
Copy link
Contributor Author

n1000 commented Oct 26, 2023

Reviewed the failures for FreeBSD -- it looks like the test runner's primary group is "wheel", that explains why I see a different result on my local FreeBSD system and the pre commit tests.

I am thinking for this specific test_file_not_owned_by_egid test for FreeBSD specifically I could modify the testcase to check the gid of the /sbin/shutdown command, since the group config on that file is "operator", not "wheel". Will try it locally first.

@n1000 n1000 force-pushed the freebsd_fixes_push_3 branch from 1adebef to 3480ade Compare October 26, 2023 19:54
@n1000
Copy link
Contributor Author

n1000 commented Oct 27, 2023

I think the remaining failures might be an test infra issue (unless I missed one).

@tertsdiepraam
Copy link
Member

@sylvestre I don't know much about these things. Does this look good to you?

@n1000 n1000 force-pushed the freebsd_fixes_push_3 branch from 3480ade to 76c7f49 Compare November 30, 2023 08:09
On some platforms (mostly the BSDs) the test fixture files copied to the
/tmp directory will have a different gid than the current egid (due to
the sticky bit set on the /tmp directory). Fix this before running the
"test" command.
@sylvestre sylvestre force-pushed the freebsd_fixes_push_3 branch from 76c7f49 to 1915e3c Compare December 25, 2023 12:10
Copy link

GNU testsuite comparison:

Congrats! The gnu test tests/mv/dir2dir is no longer failing!
Skipping an intermittent issue tests/tail/inotify-dir-recreate

@sylvestre sylvestre merged commit 0fb4d88 into uutils:main Dec 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants