Ted Kolovos

Replace "AAD functional testing in service principal mode by filtering specific test cases" with "When developing or if you are testing pull requests you may need to execute just a...

"Product functional tests are being run for multiple product-tenant combinations. Development team should check the successful completion of these runs for any regressions. " It is not clear when the...

I don't see a reference to the sample functional test execution script that is reference in this comment. That might make it easier for developers. https://github.com/cisagov/ScubaGear/issues/592#issuecomment-1779479798

In the Adding New Functional Tests section, add a checkbox for "Ensure all Not Applicable test cases for the affected policy are tested." Both Sharepoint and AAD now include N/A...

New Functional tests section. "Ensure that any changes to HTML report output are tested." Not sure what the developer is supposed to do here and how those changes would be...

"compliant" is misspelled in numerous instances as "complaint".

When changing references to Microsoft Entra ID, we need to be sure the related service names such as "Conditional Access" are also updated to Microsoft's new name. I found out...

> @tkol2022 You can check the naming in the table: > > https://learn.microsoft.com/en-us/entra/fundamentals/new-name#glossary-of-updated-terminology > > For example: > > Before: > > * Azure AD multifactor authentication (MFA) > *...

### Items Left to Complete - [x] Remove dependency on Microsoft.Graph.Beta.Identity.Governance from RequiredVersions.ps1 & UninstallModules.ps1 - [x] Modify the functional tests in AAD file aad.g5.testplan.yaml to create JSON compatible with...

### For reviewers: How to test the performance improvement The performance improvement when running AAD is mostly noticeable when the tool executes for the first time in a newly spawned...