dotnet/machinelearning

Tests need their own environment implementation

Open

#925 geöffnet am 15. Sept. 2018

Auf GitHub ansehen
 (4 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C# (9.125 Stars) (1.901 Forks)batch import
P3good first issuetestup-for-grabs

Beschreibung

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.

Contributor Guide

Tech Stack
csharp
Domain
testing
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
C#xUnitML.NET codebase basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
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.