saltstack/salt
1
0
Fork 0
mirror of https://github.com/saltstack/salt.git synced 2025-04-17 10:10:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Updated selinux commands and tests after reviewer comments, need the extra space

Browse source
This commit is contained in:
David Murphy 2024-10-24 09:33:34 -06:00 • committed by Daniel Wozniak
parent d19258590b
commit ed89424059
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
salt/modules/selinux.py
View file

@ -616,7 +616,7 @@ def _fcontext_add_or_delete_policy(
if "add" == action:
# need to use --modify if context for name file exists, otherwise ValueError
filespec = re.escape(name)
cmd = f"semanage fcontext -l | grep -E '{filespec}'"
cmd = f"semanage fcontext -l | grep -E '{filespec} '"
current_entry_text = __salt__["cmd.shell"](cmd, ignore_retcode=True)
if current_entry_text != "":
action = "modify"

4
tests/pytests/unit/modules/test_selinux.py
View file

@ -401,7 +401,7 @@ def test_selinux_add_policy_regex(name, sel_type):
):
selinux.fcontext_add_policy(name, sel_type=sel_type)
filespec = re.escape(name)
expected_cmd_shell = f"semanage fcontext -l | grep -E '{filespec}'"
expected_cmd_shell = f"semanage fcontext -l | grep -E '{filespec} '"
mock_cmd_shell.assert_called_once_with(
expected_cmd_shell,
ignore_retcode=True,
@ -433,7 +433,7 @@ def test_selinux_add_policy_shorter_path(name, sel_type):
):
selinux.fcontext_add_policy(name, sel_type=sel_type)
filespec = re.escape(name)
expected_cmd_shell = f"semanage fcontext -l | grep -E '{filespec}'"
expected_cmd_shell = f"semanage fcontext -l | grep -E '{filespec} '"
mock_cmd_shell.assert_called_once_with(
expected_cmd_shell,
ignore_retcode=True,