IntelliTect / IntelliTect/TestTools.TestFramework

Consider: Moving gathering test block inputs/outputs to the Add methods

Open
#75 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
1
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Currently, TestBuilder.Build() goes through and finds all inputs (properties, constructor parameters, and Execute parameters,) and outputs (return types) and then cross-checks them all in order to make sure all test blocks have the required dependencies supplied by a prior test block.

It might make an appreciable difference in maintainability and readability in TestBuilder.Build() to instead handle that as part of AddTestBlock, AddAsyncTestBlock, AddFinallyBlock, and AddAsyncFinallyBlock.

Each TestBlock would need an Input and Output property. We don't really care (at that point) how the test block asks for the input; The DI service handles it for constructors, and there's already code to handle properties and execute args.

Contributor guide

Open the contributing guide

Research direction

Start in TestBuilder.Build() and inspect AddTestBlock, AddAsyncTestBlock, AddFinallyBlock, and AddAsyncFinallyBlock. Trace how inputs and outputs are currently gathered and cross-checked, then determine the required TestBlock properties and verify that dependency validation remains correct after moving the work into the Add methods.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
testing
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.