User perms are not checked on file access

[toddr]

Currently we don't check perms with file access. Given we're providing modes, we might as well check current UID/GID and do the right fail if necessary.

[FGasper]

It might be cleaner yet to implement it as some sort of state variable in Test::MockFile. That way the process doesn’t need to change its credentials, and the test credentials are abstracted from the process’s.

[toddr]

Example:

my $file = Test::MockFile->file( $file, $contents, { uid => 0, mode => 0600 } );
$> = 6;
open(my $fh, '<', $file) or die; # Should die because user doesn't have perms.

[toddr]

@Koan-Bot is this resolved by merged changes?

[Koan-Bot]

❌ Unknown command is. Here are the commands I support:

• @Koan-Bot fix — "Queue a fix mission for a GitHub issue — understand, plan, test, implement, and submit a PR"
• @Koan-Bot implement — "Queue an implementation mission for a GitHub issue"
• @Koan-Bot rebase — "Queue a PR rebase (ex: /rebase https://github.com/owner/repo/pull/42)"
• @Koan-Bot recreate — "Recreate a diverged PR from scratch on current upstream (ex: /recreate https://github.com/owner/repo/pull/42)"
• @Koan-Bot refactor — "Queue a refactoring mission for a PR, issue, or file"
• @Koan-Bot review — "Queue a code review for a PR or issue"

Usage: @Koan-Bot <command> in any PR or issue comment.

[atoomic]

@Koan-Bot fix