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.

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

New unit test to check for wrongfully assigned ReadWrite-permissions. #4320

Closed
wants to merge 68 commits into from

Conversation

andikrueger
Copy link
Collaborator

Pull Request (PR) description

New unit test to check for wrongfully assigned ReadWrite-permissions.

NikCharlebois and others added 17 commits February 14, 2024 08:57
IntuneDeviceConfigurationPolicyAndroidWorkProfile: Fix typo that makes it export incorrectly and not reporting the correct state of resource
IntuneDeviceConfigurationNetworkBoundaryPolicyWindows10: Fix Test-TargetResource and use correct filter to retrieve policy
Fix nested change detection in CIMInstance and resource comparison
IntuneDeviceAndAppManagementAssignmentFilter: Fix Test-TargetResource to ensure that resource reports its correct state
IntuneAppConfigurationPolicy: Fixed tests so that resource reports its correct state
DLPCompliance Rules additional parameters
@andikrueger
Copy link
Collaborator Author

Just to add some more context to this PR

Right now, there a few resources that would request write Permission even though only an export of the tenant is needed. This PR tries to add a quality gate to the settings.json files to only allow read permission for Microsoft graph in the “read” case.

@andikrueger
Copy link
Collaborator Author

Will create a new PR as the last rebase messed this one up.

@andikrueger andikrueger deleted the UnitTestReadPermissions branch February 14, 2024 19:40
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.

5 participants