test(developer): increase test coverage with InternalsVisibleTo pattern#1
Merged
johnazariah merged 11 commits intomainfrom Jan 19, 2026
Conversation
…nerate-tests pattern
…ializer, RoslynTestGenerator, DeveloperAgentProvider - CodeGraphIndexerTests: 4 tests for IndexAsync/ReindexAsync - GuardianRegistryInitializerTests: 4 tests for StartAsync/StopAsync - RoslynTestGeneratorTests: 2 tests for GenerateTestsAsync - DeveloperAgentProviderTests: 1 test for GetAgents - GuardianExecutorTests: 2 additional tests for overloads Generated using Aura test generation with manual fixes for IOptions<T> mocking. Developer module: 176 -> 189 tests
…tionTools (189->196)
- LanguageToolFactoryTests: +10 tests covering tool registration, handler execution, fallback behavior, schema generation - DeveloperAgentProviderTests: +8 tests covering agent types, priority ordering, metadata, capabilities - WorkflowServiceTests: +3 tests for RejectStep, ReassignStep, UpdateStepDescription Total Developer module tests: 199 -> 217
- Add InternalsVisibleTo for Aura.Module.Developer.Tests - Change ParseBuildErrors and BuildError to internal - Change SanitizeForMermaid to internal
- Add BuildFixLoopToolsTests: tool registration, ParseBuildErrors - Add VisualizationToolsTests: tool registration, SanitizeForMermaid - Add GuardianRegistryTests: watcher setup, disposal patterns - Create TestableFileSystem helper for mocking FileSystemWatcher Tests: 558 → 597 (+39 tests)
johnazariah
deleted the
workflow/increase-test-coverage-using-aura-test-generation-aae5
branch
johnazariah
pushed a commit
that referenced
this pull request
Feb 20, 2026
…espace resolution StoryService (P0 #1): - Extract GetStoryOrThrowAsync, GetStoryWithStepsOrThrowAsync, GetStepOrThrow - Replace 12 duplicated fetch-and-throw patterns across 13 methods - Unify exception type to InvalidOperationException Debug logging (P0 #2): - Demote 8 [REACT-DEBUG] LogWarning to LogDebug in ReActExecutor - Demote 2 [STEP-DEBUG] LogWarning to LogDebug in StoryService.Execution - Keep LogError for genuine timeout errors Namespace resolution (P0 #3): - Create RoslynSyntaxHelpers.GetContainingNamespace shared helper - Remove duplicate GetNamespace from CSharpIngesterAgent and RoslynCodeIngestor - Simplify inline namespace resolution in RoslynRefactoringService.Interface Build: 0 warnings, 0 errors. Tests: 829 passing (449+278+102).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds 39 new tests (558 → 597) using the InternalsVisibleTo pattern to test internal methods.
Changes
Infrastructure
InternalsVisibleTotoAura.Module.Developer.csprojParseBuildErrorsandBuildErroras internal inBuildFixLoopToolsSanitizeForMermaidas internal inVisualizationToolsNew Tests
ParseBuildErrorsfor MSBuild/Rust/TypeScript formatsSanitizeForMermaidpure function testsTestableFileSystemhelper for mockingFileSystemWatcherNotes
GuardianDefinition.Triggerstyped asIReadOnlyList<T>cannot be deserialized by YamlDotNet (separate fix needed)