Complete the stubbed plumbing for Consolidate action by DaveTryon · Pull Request #1106 · microsoft/sbom-tool (original) (raw)

PR #1104 compiled and passed all E2E tests, but was unable to get code into the SbomConsolidationWorkflow class. This adds more plumbing so that when we execute sbomtool.exe consolidate, we get into the correct workflow. I'll add the config mapping in a subsequent PR.

I discovered with this PR that we have separate implementations of *Service.cs in the Microsoft.Sbom.Tool and DotNetTool projects. Most of the existing duplicated classes are mostly in sync, with some larger differences in ValidationService.cs. This feels like a significant violation of the DRY principle, but untangling it right is out of scope. The same Program.cs file gets used in both projects, so maybe that's a potential future pattern to sort things out.