Files
2026-02-19 22:07:11 +02:00

1.2 KiB

Audit - NotifySmokeCheck

Project

  • Path: src/Tools/NotifySmokeCheck/NotifySmokeCheck.csproj
  • Module: Tools
  • Kind: Tool
  • SDK: Microsoft.NET.Sdk
  • TargetFramework: net10.0
  • Audit date (UTC): 2026-01-30

Coding Standards Findings

  • Status: FAIL
  • Nullable: enable
  • TreatWarningsAsErrors: explicit true
  • Deterministic: inherited true
  • 100-line rule violations: 1
  • Service locator usage (BuildServiceProvider/GetService): 0
  • Analyzer enforcement: missing repo-wide (see summary).

Details

  • 100-line files:
    • src/Tools/NotifySmokeCheck/NotifySmokeCheckRunner.cs (515 lines)
  • Service locator matches:
    • none

Fix Guidance

  • Split files over 100 lines into smaller types or partials.

Testing Fullness Findings

  • Status: FAIL
  • Expected layers: Unit, CLI, Offline
  • Detected test projects: src/Tools/__Tests/NotifySmokeCheck.Tests/NotifySmokeCheck.Tests.csproj [Unit]
  • Missing layers: CLI, Offline

Manual checks required

  • Observability contract tests for WebService/Worker.
  • Offline execution (tests must run without network access).

Fix Guidance

  • Add offline/airgap coverage with fixtures only.
  • Add CLI tests for exit codes and golden output.