Allow a trailing CancellationToken parameter on MSTest test methods
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 22/100
Línea de trabajo
The proposal touches MSTest test discovery, execution (both reflection and source-generated paths), argument binding for DataRow and DynamicData, and the MSTest analyzers, so start by tracing how TestContext.CancellationToken is currently made available to a test body. Done means a test method with a trailing CancellationToken parameter receives the same token as TestContext, data-driven methods bind their data arguments unchanged, and invalid signatures are rejected consistently across discovery, execution and analyzers. The issue has no assignee or pull request yet, but the design spans several subsystems, so confirm the approach with maintainers before writing code.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Allow MSTest test methods to declare one additional trailing CancellationToken parameter. MSTest supplies TestContext.CancellationToken automatically, so a non-parameterized test can take one parameter and a parameterized test with X data arguments can take X + 1 parameters.
Background and Motivation
Tests currently need access to TestContext just to forward its cancellation token to asynchronous operations. A method parameter makes cooperative cancellation explicit and avoids that boilerplate.
Proposed Feature
[TestMethod]
public async Task TestAsync(CancellationToken cancellationToken)
{
await RunAsync(cancellationToken);
}
[TestMethod]
[DataRow(42)]
public async Task ParameterizedTestAsync(int value, CancellationToken cancellationToken)
{
await RunAsync(value, cancellationToken);
}
- Inject the exact current
TestContext.CancellationToken, preserving its existing cancellation and timeout lifetime. - Support ordinary methods and data-driven methods, including
DataRowandDynamicData. - Keep discovery, reflection/source-generated execution, argument validation, and MSTest analyzers consistent.
- Preserve existing test signatures and data binding; do not inject arbitrary parameters or change lifecycle method signatures.
- Add regression coverage for token identity, cancellation, data arguments, invalid signatures, and existing behavior.
Alternative Designs
Continue using TestContext.CancellationToken inside the test body. No new attribute or general dependency-injection mechanism is needed for the proposed convenience.
- Lenguaje dominante
- C#
- Estrellas
- 1.1k
- Forks
- 312
- Merge medio
- 10 h 39 min
- PR fusionados (30 d)
- 349
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de microsoft/testfx
-
type/automation type/tech-debt
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
Los mantenedores suelen responder en 1 día
-
needs/triage
Dificultad 3/5 1-2 días Aptitud para principiantes 57/100
Los mantenedores suelen responder en 1 día
-
[aw] Detection RunsAbiertoagentic-workflows
Dificultad 1/5 Menos de una hora Aptitud para principiantes 1/100
microsoft/testfx#12224 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
needs/triage
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
microsoft/testfx#12205 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
type/automation type/test-gap
Dificultad 3/5 Medio día Aptitud para principiantes 15/100
Los mantenedores suelen responder en 1 día
Todos los issues de microsoft/testfx
Issues similares
-
t/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
dotnet/maui#39950 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
CommunityToolkit/Datasync#600 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
NimblePros/deviq-hugo#121 ·
Los mantenedores suelen responder en 1 día
-
RANDOM_REPLACE does not keep the date format when the month abbreviation has a trailing periodAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
philterd/phileas-dotnet#188 ·
-
Add All ContributorsAbiertoenhancement ready-for-human
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
fboucher/2d6-dungeon-app#288 · 1 comentario ·
Los mantenedores suelen responder en 1 día