From 0d087ae4cd2c41806639d52a6129fb7684170cef Mon Sep 17 00:00:00 2001 From: Dev Date: Thu, 9 Apr 2026 14:44:07 +0200 Subject: [PATCH] docs(phase-19): add research and validation strategy Co-Authored-By: Claude Opus 4.6 (1M context) --- .../19-VALIDATION.md | 83 +++++++++++++++++++ 1 file changed, 83 insertions(+) create mode 100644 .planning/phases/19-app-registration-removal/19-VALIDATION.md diff --git a/.planning/phases/19-app-registration-removal/19-VALIDATION.md b/.planning/phases/19-app-registration-removal/19-VALIDATION.md new file mode 100644 index 0000000..26cfd2a --- /dev/null +++ b/.planning/phases/19-app-registration-removal/19-VALIDATION.md @@ -0,0 +1,83 @@ +--- +phase: 19 +slug: app-registration-removal +status: draft +nyquist_compliant: false +wave_0_complete: false +created: 2026-04-09 +--- + +# Phase 19 — Validation Strategy + +> Per-phase validation contract for feedback sampling during execution. + +--- + +## Test Infrastructure + +| Property | Value | +|----------|-------| +| **Framework** | xUnit 2.9.3 | +| **Config file** | `SharepointToolbox.Tests/SharepointToolbox.Tests.csproj` | +| **Quick run command** | `dotnet test --filter "Category=Unit" --no-build` | +| **Full suite command** | `dotnet test --no-build` | +| **Estimated runtime** | ~15 seconds | + +--- + +## Sampling Rate + +- **After every task commit:** Run `dotnet test --filter "Category=Unit" --no-build` +- **After every plan wave:** Run `dotnet test --no-build` +- **Before `/gsd:verify-work`:** Full suite must be green +- **Max feedback latency:** 15 seconds + +--- + +## Per-Task Verification Map + +| Task ID | Plan | Wave | Requirement | Test Type | Automated Command | File Exists | Status | +|---------|------|------|-------------|-----------|-------------------|-------------|--------| +| 19-01-01 | 01 | 1 | APPREG-01 | unit | `dotnet test --filter "FullyQualifiedName~AppRegistrationServiceTests" --no-build` | :x: W0 | :white_large_square: pending | +| 19-01-02 | 01 | 1 | APPREG-02 | unit | `dotnet test --filter "FullyQualifiedName~AppRegistrationServiceTests" --no-build` | :x: W0 | :white_large_square: pending | +| 19-01-03 | 01 | 1 | APPREG-03 | unit (mock) | `dotnet test --filter "FullyQualifiedName~AppRegistrationServiceTests" --no-build` | :x: W0 | :white_large_square: pending | +| 19-01-04 | 01 | 1 | APPREG-04 | unit | `dotnet test --filter "FullyQualifiedName~AppRegistrationServiceTests" --no-build` | :x: W0 | :white_large_square: pending | +| 19-02-01 | 02 | 1 | APPREG-05 | unit (mock) | `dotnet test --filter "FullyQualifiedName~ProfileManagementViewModelRegistrationTests" --no-build` | :x: W0 | :white_large_square: pending | +| 19-02-02 | 02 | 1 | APPREG-06 | unit (mock) | `dotnet test --filter "FullyQualifiedName~AppRegistrationServiceTests" --no-build` | :x: W0 | :white_large_square: pending | + +*Status: :white_large_square: pending · :white_check_mark: green · :x: red · :warning: flaky* + +--- + +## Wave 0 Requirements + +- [ ] `SharepointToolbox.Tests/Services/AppRegistrationServiceTests.cs` — stubs for APPREG-02, APPREG-03, APPREG-04, APPREG-06 +- [ ] `SharepointToolbox.Tests/ViewModels/ProfileManagementViewModelRegistrationTests.cs` — stubs for APPREG-01, APPREG-05 +- [ ] `SharepointToolbox/Services/IAppRegistrationService.cs` — interface stub +- [ ] `SharepointToolbox/Core/Models/AppRegistrationResult.cs` — result model stub + +*Existing xUnit infrastructure covers test framework — no new packages needed.* + +--- + +## Manual-Only Verifications + +| Behavior | Requirement | Why Manual | Test Instructions | +|----------|-------------|------------|-------------------| +| Register App button visible in profile dialog | APPREG-01 | WPF UI rendering | Open profile create dialog, verify "Register App" button is present and enabled | +| Fallback instructions panel renders correctly | APPREG-04 | WPF UI rendering | Trigger non-admin path, verify step-by-step instructions display | +| Remove App button visible and functional | APPREG-05 | WPF UI rendering + live Entra | Open profile edit dialog for registered tenant, verify "Remove App" button | +| Re-authentication required after removal | APPREG-06 | Live MSAL session state | Remove app, attempt operation, verify login prompt appears | + +--- + +## Validation Sign-Off + +- [ ] All tasks have `` verify or Wave 0 dependencies +- [ ] Sampling continuity: no 3 consecutive tasks without automated verify +- [ ] Wave 0 covers all MISSING references +- [ ] No watch-mode flags +- [ ] Feedback latency < 15s +- [ ] `nyquist_compliant: true` set in frontmatter + +**Approval:** pending