-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NyanCat v1.0.0 #46
NyanCat v1.0.0 #46
Conversation
To test out the reporter you need to register the reporter in public static class Tasty
{
static Tasty()
{
DefaultScope = new TastyScope();
// ConsoleReporter.Register();
NyanCatReporter.Register();
DefaultScope.RegisterTransport(NamedPipeRemoteHook.CreateNamedPipeTransportStream);
}
dotnet run -f netcoreapp3.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably we have more luck with https://github.com/alexrp/system-terminal cause it has a virtual scroll mode.
When running build locally it blows up on the integration tests with a For now we should be fine by logging out the exception to stdout in the public static class ReportTestMiddleware
{
public static TestExecutor UseTestReporters(this TestExecutor executor)
=> executor.Use(async (context, next) =>
{
try
{
await next();
}
finally
{
try
{
await context.CurrentScope.Report(context.CurrentCase);
}
catch (Exception ex)
{
Console.WriteLine($"Reporter failed: {ex}");
}
}
});
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides the lacking summary reporter this looks awesome! Thanks @hazard999 ♥ |
ReportSummary
First draft of a NyanCatReporter
Fixes #39