Individual product report.json files are not covered by smoke test

[mitchelbaker-cisa]

🐛 Summary

#970 added support for a ScubaResults.json which houses metadata, a json version of the html reports, and more. The PR adds test coverage for the MergeJson parameter across:

• Invoke-RunCached.Tests.ps1
• Invoke-Scuba.Tests.ps1
• Invoke-ScubaConfig.Tests.ps1
• Merge-JsonOutput.Tests.ps1

However, if MergeJson is not specified, individual product report.json (AADReport.json, DefenderReport.json, etc.) files are created by default.

These are not included in SmokeTest001.Tests.ps1:

To reproduce

Run Invoke-SCuBA without -MergeJson parameter from main branch/v1.3.0

Expected behavior

1. When reviewing Create Json Output #970 Crutch mentioned not needing the individual json reports (comment). If these files are not necessary, then the tool should not create them and coverage is not needed in the smoke test.
2. If we deem these files necessary, then we should update the smoke test.

[schrolla]

@adhilto Please add reference to MergeJson flag removal/change issue to this for discussion.

[adhilto]

Note that the individual json files would go away completely if/when we make MergeJson the default behavior (#1205), dependent on #1023. If we do make that change, then I think this issue would be OBE.

[buidav]

Superseded by #1317