dotnet/machinelearning

Tests need their own environment implementation

Open

#925 aberto em 15 de set. de 2018

Ver no GitHub
 (4 comments) (1 reaction) (0 assignees)C# (9.125 stars) (1.901 forks)batch import
P3good first issuetestup-for-grabs

Description

Right now we use TlcEnvironment (or ConsoleEnvironment) which outputs everything to console. And XUnit don't care about console, and if you want to look on output during test execution you need to do something like this: https://github.com/dotnet/machinelearning/blob/4cb7dd9047b4c456e81ba014c664c27efe217351/test/Microsoft.ML.StaticPipelineTesting/StaticPipeTests.cs#L31

We should have XUnitEnvironment or TestEnvironment and redirect all output to ITestOutputHelper.

Guia do colaborador

Pilha de tecnologia
csharp
Domain
testing
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
C#xUnitML.NET codebase basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
The issue requests creating an XUnitEnvironment or TestEnvironment class that redirects output to ITestOutputHelper instead of using TlcEnvironment or ConsoleEnvironment. The existing code in the linked test file shows how output is currently captured using a console output wrapper. The task involves implementing a new environment class that implements the IEnvironment interface and uses ITestOutputHelper to provide test friendly output. This is a localized change, likely in the test infrastructure, and may require modification of existing tests to use the new environment.