MS.TEAMS.5.xv1 - update warning message to reflect the correct reason for the policy fail (public reported issue) #1601
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🗣 Description
Teams policies MS.TEAMS.5.1v1 - MS.TEAMS.5.3v1 deals with application permission policies - in case of a failure ScubaGear generates a warning message (since this is a SHOULD policy). Currently the warning message correctly identifies the offending policies but incorrectly indicates them as meeting policies. This PR is to change warning message to correctly identify them as app permission policies.
💭 Motivation and context
Closes #1495
🧪 Testing
Run ScubaGear against a tenant. Configure the tenant to fail the TEAMS.5.1v1. Make sure that the scubagear results report correctly identifies the result for 5.1 and the warning message correctly identifies the offending 'app permission policy'.
Warning message before the fix:

Warning message after the fix:

✅ Pre-approval checklist
✅ Pre-merge checklist
PR passed smoke test check.
Feature branch has been rebased against changes from parent branch, as needed
Use
Rebase branch
button below or use this reference to rebase from the command line.Resolved all merge conflicts on branch
Notified merge coordinator that PR is ready for merge via comment mention
Demonstrate changes to the team for questions and comments.
(Note: Only required for issues of size
Medium
or larger)✅ Post-merge checklist