Skip to content
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.

Sign up for GitHub

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

Fix test: tests/pytests/functional/states/file/test__check_directory_win.py #67163

Closed
wants to merge 1 commit into from
Closed

Fix test: tests/pytests/functional/states/file/test__check_directory_win.py #67163

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions tests/pytests/functional/states/file/test__check_directory_win.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@

import salt.states.file as file
import salt.utils.win_dacl as win_dacl
import salt.utils.win_functions as win_functions

pytestmark = [
pytest.mark.windows_whitelisted,
Expand Down Expand Up @@ -52,8 +51,7 @@ def temp_path(tmp_path):
# Now we create a directory for testing that does inherit those permissions from the above, new parent directory
test_dir = tmp_path / "test_dir"
test_dir.mkdir()
current_user = win_functions.get_current_user(with_domain=False)
assert win_dacl.get_owner(obj_name=str(test_dir)) == current_user
assert win_dacl.get_owner(obj_name=str(test_dir)) == "Administrators"
# We do want the test directory to inherit permissions from the parent directory
assert win_dacl.get_inheritance(obj_name=str(test_dir))
# Make sure the permissions are inherited from the parent
Expand Down
Loading