dotnet/machinelearning

Tests need their own environment implementation

Offen

#925 geöffnet am 15.09.2018

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

Repository-Metriken

Stars
 (9.125 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

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

Research-Richtung
Untersuche die vorhandenen Klassen TlcEnvironment und ConsoleEnvironment, verstehe, wie sie in die Konsole ausgeben, und erstelle eine neue TestEnvironment Klasse, die ITestEnvironment (oder ähnlich) implementiert und in xUnit Tests ITestOutputHelper für die Ausgabe verwendet.
Tech Stack
csharp
Domain
testing
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#xUnit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65