Merge pull request #466 from nblumhardt-ro/dashboard-native-rendering

Update `dashboard render` to accept `--native`

Author: KodrAus

src/SeqCli/Cli/Commands/Dashboard/RenderCommand.cs

@@ -58,7 +58,7 @@ public RenderCommand()
         _range = Enable<DateRangeFeature>();
         _signal = Enable<SignalExpressionFeature>();
         _timeout = Enable<TimeoutFeature>();
-        _output = Enable<OutputFormatFeature>();
+        _output = Enable(new OutputFormatFeature(supportNative: true));
         _storagePath = Enable<StoragePathFeature>();
         _connection = Enable<ConnectionFeature>();
     }
@@ -191,7 +191,7 @@ static string BuildSqlQuery(ChartQueryPart query, DateTime rangeStart, DateTime
 
     static SignalExpressionPart? Intersect(params SignalExpressionPart?[] expressions)
     {
-        var result = (SignalExpressionPart?) null;
+        SignalExpressionPart? result = null;
 
         foreach (var s in expressions)
         {

src/SeqCli/Output/NativeFormatter.cs

@@ -255,7 +255,6 @@ public static void WriteQueryResult(TextWriter output, QueryResultPart result)
                         output.Write(' ');
                     output.Write(heading);
                 }
-                output.WriteLine();
             }
             else
             {

test/SeqCli.EndToEnd/Dashboard/RenderTestCase.cs

@@ -18,7 +18,7 @@ public Task ExecuteAsync(
         var exit = runner.Exec("dashboard list");
         Assert.Equal(0, exit);
 
-        var id = runner.LastRunProcess.Output.Split(' ')[0];
+        var id = runner.LastRunProcess!.Output.Split(' ')[0];
 
         exit = runner.Exec("dashboard render", $"-i {id} -c \"All Events\" --last 1d --by 1h --no-color");
         Assert.Equal(0, exit);