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

User temp_account instead of Administrator for testing

Browse source
This commit is contained in:
Twangboy 2025-01-15 11:47:23 -07:00 • committed by Daniel Wozniak
parent 4e03997e27
commit 5ff2c7e2a2
1 changed files with 21 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
tests/pytests/functional/modules/win_file/test_check_perms.py
View file

@ -3,8 +3,11 @@ Tests for win_file execution module
"""
import pytest
from saltfactories.utils import random_string
import salt.modules.cmdmod as cmd
import salt.modules.win_file as win_file
import salt.modules.win_useradd as win_useradd
import salt.utils.win_dacl as win_dacl
from salt.exceptions import CommandExecutionError
from tests.support.mock import patch
@ -25,8 +28,20 @@ def configure_loader_modules():
},
"__opts__": {"test": False},
},
win_useradd: {
"__salt__": {
"cmd.run_all": cmd.run_all,
},
}
}
@pytest.fixture
def temp_account():
user_name = random_string("test-account-", uppercase=False)
with pytest.helpers.create_account(username=user_name) as account:
win_useradd.addgroup(account.username, "Users")
yield account.username
@pytest.fixture(scope="function")
def test_file():
@ -184,7 +199,7 @@ def test_check_perms_inheritance_true(test_file):
assert result == expected
def test_check_perms_reset_test_true(test_file):
def test_check_perms_reset_test_true(test_file, temp_account):
"""
Test resetting perms with test=True. This shows minimal changes
"""
@ -193,7 +208,7 @@ def test_check_perms_reset_test_true(test_file):
# Set some permissions
win_dacl.set_permissions(
obj_name=str(test_file),
principal="Administrator",
principal=temp_account,
permissions="full_control",
)
expected = {
@ -204,7 +219,7 @@ def test_check_perms_reset_test_true(test_file):
"Users": {"permissions": "read_execute"},
},
"remove_perms": {
"Administrator": {
f"{temp_account}": {
"grant": {
"applies to": "This folder only",
"permissions": "Full control",
@ -228,7 +243,7 @@ def test_check_perms_reset_test_true(test_file):
assert result == expected
def test_check_perms_reset(test_file):
def test_check_perms_reset(test_file, temp_account):
"""
Test resetting perms on a File
"""
@ -237,7 +252,7 @@ def test_check_perms_reset(test_file):
# Set some permissions
win_dacl.set_permissions(
obj_name=str(test_file),
principal="Administrator",
principal=temp_account,
permissions="full_control",
)
expected = {
@ -248,7 +263,7 @@ def test_check_perms_reset(test_file):
"Users": {"permissions": "read_execute"},
},
"remove_perms": {
"Administrator": {
f"{temp_account}": {
"grant": {
"applies to": "This folder only",
"permissions": "Full control",