Timestamp cannot be configured in ExtentSparkReporter Issue#206 (#215)

anshooarora · web-flow · commit f7286189cc29 · 2024-06-14T20:04:29.000+05:30
* Initial: allows setting timestamp format via config #206 * Fix timestamp format * bump version -beta2 * Update SparkTestSPA.cshtml
diff --git a/ExtentReports.Tests/Reporter/SparkReporterConcurrentTest.cs b/ExtentReports.Tests/Reporter/SparkReporterConcurrentTest.cs
@@ -0,0 +1,34 @@
+using System.Linq;
+using System.Threading.Tasks;
+using AventStack.ExtentReports.Model;
+using AventStack.ExtentReports.Reporter;
+using NUnit.Framework;
+using NUnit.Framework.Internal;
+
+namespace AventStack.ExtentReports.Tests.Reporter
+{
+    public class SparkReporterConcurrentTest
+    {
+        private ExtentReports _extent;
+
+        [SetUp]
+        public void Setup()
+        {
+            _extent = new ExtentReports();
+            _extent.AttachReporter(new ExtentSparkReporter("SparkReporterConcurrentTest.html"));
+        }
+
+        [Test]
+        public void ParallelCreateTestsAndFlush()
+        {
+            var rng = Enumerable.Range(0, 100);
+            Parallel.ForEach(rng, new ParallelOptions { MaxDegreeOfParallelism = 4 }, x => {
+                _extent.CreateTest("Test").Info("");
+                if (x % 5 == 0)
+                {
+                    _extent.Flush();
+                }
+            });
+        }
+    }
+}
diff --git a/ExtentReports.Tests/Reporter/SparkReporterTest.cs b/ExtentReports.Tests/Reporter/SparkReporterTest.cs
@@ -151,12 +151,26 @@ public void SparkXMLConfig()
         [Test]
         public void SparkJSONConfig()
         {
-            var xmlConfigPath = Path.Combine(@"../../../../", @"config/spark-config.json");
+            var jsonConfigPath = Path.Combine(@"../../../../", @"config/spark-config.json");
             var spark = new ExtentSparkReporter("Spark.html");
-            spark.LoadJSONConfig(xmlConfigPath);
+            spark.LoadJSONConfig(jsonConfigPath);
             Assert.AreEqual(spark.Config.Theme, Theme.Standard);
             Assert.AreEqual(spark.Config.DocumentTitle, "Extent Framework");
             Assert.AreEqual(spark.Config.Protocol, Protocol.HTTPS);
         }
+
+        [Test]
+        public void SparkReporterTimestamp()
+        {
+            var extent = new ExtentReports();
+            var spark = new ExtentSparkReporter("Index.html");
+            extent.AttachReporter(spark);
+            var jsonConfigPath = Path.Combine(@"../../../../", @"config/spark-config.json");
+            spark.LoadJSONConfig(jsonConfigPath);
+
+            extent.CreateTest("SparkReporterTimestamp")
+                .Pass("Passed");
+            extent.Flush();
+        }
     }
 }
diff --git a/ExtentReports/ExtentReports.csproj b/ExtentReports/ExtentReports.csproj
@@ -4,7 +4,7 @@
     <TargetFramework>netstandard2.0</TargetFramework>
     <PackOnBuild>true</PackOnBuild>
     <RootNamespace>AventStack.ExtentReports</RootNamespace>
-    <PackageVersion>5.0.3-beta</PackageVersion>
+    <PackageVersion>5.0.3-beta2</PackageVersion>
     <Authors>anshooarora</Authors>
     <Copyright>AventStack</Copyright>
     <PackageLicenseUrl>https://github.com/extent-framework/extentreports-csharp/blob/master/LICENSE</PackageLicenseUrl>
diff --git a/ExtentReports/Reporter/Config/ExtentSparkReporterConfig.cs b/ExtentReports/Reporter/Config/ExtentSparkReporterConfig.cs
@@ -10,7 +10,9 @@ public class ExtentSparkReporterConfig : InteractiveReporterConfig
     {
         private const string OfflinePackage = "extent";
         private const string Package = "AventStack.ExtentReports.Views.Spark.Offline";
+        private const string DefaultTimeStampFormat = "MMM dd, yyyy HH:mm:ss";
         private bool _offline = false;
+        private string _timestampFormat = DefaultTimeStampFormat;
 
         internal ExtentSparkReporter Reporter { get; set; }
 
@@ -32,6 +34,19 @@ public bool OfflineMode
             }
         }
 
+        [XmlElement("timeStampFormat")]
+        public string TimeStampFormat
+        {
+            get
+            {
+                return string.IsNullOrEmpty(_timestampFormat) ? DefaultTimeStampFormat : _timestampFormat;
+            }
+            set
+            {
+                _timestampFormat = value;
+            }
+        }
+
         public ExtentSparkReporterConfig() { }
 
         [JsonConstructor]
diff --git a/ExtentReports/Reporter/Model/SparkReporterTest.cs b/ExtentReports/Reporter/Model/SparkReporterTest.cs
@@ -0,0 +1,16 @@
+using AventStack.ExtentReports.Model;
+
+namespace AventStack.ExtentReports.Reporter.Model
+{
+    public class SparkReporterTest
+    {
+        public SparkReporterTest(ExtentSparkReporter reporter, Test test)
+        {
+            Reporter = reporter;
+            Test = test;
+        }
+
+        public ExtentSparkReporter Reporter;
+        public Test Test;
+    }
+}
diff --git a/ExtentReports/Views/Spark/Partials/Log.cshtml b/ExtentReports/Views/Spark/Partials/Log.cshtml
@@ -5,7 +5,7 @@
 @using RazorEngine.Templating
 
 @{
- var logs = new List<Log>(Model.Logs);
+ var logs = Model.Test.Logs;
 }
 
 <table class="table table-sm">
@@ -16,7 +16,7 @@
       var status = log.Status.ToString().ToLower();
       <tr class="event-row">
         <td><span class="badge log @status-bg">@log.Status</span></td>
-        <td>@log.Timestamp.ToShortTimeString()</td>
+        <td>@log.Timestamp.ToString(Model.Reporter.Config.TimeStampFormat)</td>
         <td>
           @if (log.HasException)
           { 
diff --git a/ExtentReports/Views/Spark/Partials/RecurseNodes.cshtml b/ExtentReports/Views/Spark/Partials/RecurseNodes.cshtml
@@ -1,12 +1,13 @@
 ﻿@using AventStack.ExtentReports
 @using AventStack.ExtentReports.Model
+@using AventStack.ExtentReports.Reporter.Model
 @using System
 @using RazorEngine
 @using RazorEngine.Templating
 
-@if (Model.HasChildren)
+@if (Model.Test.HasChildren)
 {
-  var children = new List<Test>(Model.Children);
+  var children = Model.Test.Children;
   
   <div class="accordion">
     @foreach (var child in children)
@@ -48,7 +49,7 @@
             <div class="detail-body mt-4">
               @if (child.HasGeneratedLog)
               {
-               var genLog = new List<Log>(child.GeneratedLog);
+               var genLog = child.GeneratedLog;
                @foreach (var l in genLog)
                {
                <div>
@@ -57,7 +58,7 @@
                </div>
                }
               }
-              @if (child.HasLog) { @Include("Log", child) }
+              @if (child.HasLog) { @Include("Log", new SparkReporterTest(Model.Reporter, child)) }
             </div>
             }
             @foreach (var m in child.Media)
@@ -69,7 +70,8 @@
         }
         @if (child.HasChildren)
         {
-          @Include("RecurseNodes", child)
+          var reporterTest = new SparkReporterTest(Model.Reporter, child);
+          @Include("RecurseNodes", reporterTest)
         }
       </div>
     }
diff --git a/ExtentReports/Views/Spark/Partials/SparkBDD.cshtml b/ExtentReports/Views/Spark/Partials/SparkBDD.cshtml
@@ -4,10 +4,10 @@
 @using RazorEngine
 @using RazorEngine.Templating
 
-@if (Model.HasChildren)
+@if (Model.Test.HasChildren)
 {
-  var children = new List<Test>(Model.Children);
-  @Include("SparkMediaTest", Model)
+  var children = Model.Test.Children;
+  @Include("SparkMediaTest", Model.Test)
   <div class="accordion mt-4">
     @foreach (var node in children)
     {
@@ -26,7 +26,7 @@
           if (node.BddType.Name == "ScenarioOutline")
           {
             var css = node.Status == Status.Pass ? "collapse" : "";
-            var scenarios = new List<Test>(node.Children);
+            var scenarios = node.Children;
             <div class="scenario_outline @css">
               @foreach (var child in scenarios)
               {
@@ -43,7 +43,7 @@
                   </div>
                   @{
                     css = child.Status == Status.Pass ? "collapse" : "";
-                    var steps = new List<Test>(child.Children);
+                    var steps = child.Children;
                   }
                   @foreach (var step in steps)
                   {
@@ -60,7 +60,7 @@
           else
           {
             var css = node.Status == Status.Pass ? "collapse" : "";
-            var steps = new List<Test>(node.Children);
+            var steps = node.Children;
             <div class="@css">
               <div class="card-body">
                 @Include("SparkMediaTest", node)
diff --git a/ExtentReports/Views/Spark/Partials/SparkStandard.cshtml b/ExtentReports/Views/Spark/Partials/SparkStandard.cshtml
@@ -4,17 +4,17 @@
 @using RazorEngine
 @using RazorEngine.Templating
 
-@foreach (var m in Model.Media)
+@foreach (var m in Model.Test.Media)
 {
   @Include("SparkMedia", m)
 }
 
-@if (Model.HasAnyLog)
+@if (Model.Test.HasAnyLog)
 {
 <div class="detail-body mt-4">
-  @if (Model.HasGeneratedLog)
+  @if (Model.Test.HasGeneratedLog)
   {
-   var genLog = new List<Log>(Model.GeneratedLog);
+   var genLog = new List<Log>(Model.Test.GeneratedLog);
    @foreach (var l in genLog)
    {
    <div>
@@ -23,10 +23,10 @@
    </div>
    }
   }
-  @if (Model.HasLog) { @Include("Log", Model) }
+  @if (Model.Test.HasLog) { @Include("Log", Model) }
 </div>
 }
-@if (Model.HasChildren) 
+@if (Model.Test.HasChildren) 
 { 
   <div class="mt-4">@Include("RecurseNodes", Model)</div> 
 }
diff --git a/ExtentReports/Views/Spark/Partials/SparkTestSPA.cshtml b/ExtentReports/Views/Spark/Partials/SparkTestSPA.cshtml
@@ -1,5 +1,6 @@
 ﻿@using AventStack.ExtentReports
 @using AventStack.ExtentReports.Model
+@using AventStack.ExtentReports.Reporter.Model
 @using System
 @using RazorEngine
 @using RazorEngine.Templating
@@ -60,6 +61,7 @@
       <ul class="test-list-item">
         @foreach (var test in testList)
         {
+        var reporterTest = new SparkReporterTest(Model, test);
         var timeTaken = Math.Round(test.TimeTaken, 2);
         var status = test.Status.ToString().ToLower();
         var authors = ""; var tags = ""; var devices = "";
@@ -82,8 +84,8 @@
               <div class="p-v-10">
                 <div class="info">
                   <h5 class="test-status text-@status">@test.Name</h5>
-                  <span class='badge badge-success'>@test.StartTime</span>
-                  <span class='badge badge-danger'>@test.EndTime</span>
+                  <span class='badge badge-success'>@test.StartTime.ToString(Model.Config.TimeStampFormat)</span>
+                  <span class='badge badge-danger'>@test.EndTime.ToString(Model.Config.TimeStampFormat)</span>
                   <span class='badge badge-default'>@timeTaken ms</span>
                   &middot; <span class='uri-anchor badge badge-default'>#test-id=@test.Id</span>
                   <span title='Skip to the next failed step' class='badge badge-danger pointer float-right ne ml-1'><i class="fa fa-fast-forward"></i></span>
@@ -94,7 +96,7 @@
                 @if (!string.IsNullOrEmpty(test.Description)) { <div class="m-t-10 m-l-5">@test.Description</div> }
               </div>
             </div>
-            @if (isbdd) { @Include("SparkBDD", test) } else { @Include("SparkStandard", test) }
+            @if (isbdd) { @Include("SparkBDD", reporterTest) } else { @Include("SparkStandard", reporterTest) }
           </div>
         </li>
         }
diff --git a/ExtentReports/Views/Spark/Partials/StepDetails.cshtml b/ExtentReports/Views/Spark/Partials/StepDetails.cshtml
@@ -6,7 +6,7 @@
 
 @if (Model.HasLog)
 {
-  var logs = new List<Log>(Model.Logs);
+  var logs = Model.Logs;
   foreach (var log in logs)
   {
     <div>
diff --git a/config/spark-config.json b/config/spark-config.json
@@ -7,7 +7,7 @@
   "thumbnailForBase64": false,
   "documentTitle": "Extent Framework",
   "reportName": "ExtentReports",
-  "timeStampFormat": "MMM dd, yyyy HH:mm:ss a",
+  "timeStampFormat": "MMM dd, yyyy HH:mm:ss",
   "js": "",
   "css": ""
 }

Original file line number	Diff line number	Diff line change
`@@ -1,12 +1,13 @@`
`1`	`1`	`@using AventStack.ExtentReports`
`2`	`2`	`@using AventStack.ExtentReports.Model`
	`3`	`+@using AventStack.ExtentReports.Reporter.Model`
`3`	`4`	`@using System`
`4`	`5`	`@using RazorEngine`
`5`	`6`	`@using RazorEngine.Templating`
`6`	`7`
`7`		`-@if (Model.HasChildren)`
	`8`	`+@if (Model.Test.HasChildren)`
`8`	`9`	`{`
`9`		`- var children = new List<Test>(Model.Children);`
	`10`	`+ var children = Model.Test.Children;`
`10`	`11`
`11`	`12`	`<div class="accordion">`
`12`	`13`	`@foreach (var child in children)`
`@@ -48,7 +49,7 @@`
`48`	`49`	`<div class="detail-body mt-4">`
`49`	`50`	`@if (child.HasGeneratedLog)`
`50`	`51`	`{`
`51`		`- var genLog = new List<Log>(child.GeneratedLog);`
	`52`	`+ var genLog = child.GeneratedLog;`
`52`	`53`	`@foreach (var l in genLog)`
`53`	`54`	`{`
`54`	`55`	`<div>`
`@@ -57,7 +58,7 @@`
`57`	`58`	`</div>`
`58`	`59`	`}`
`59`	`60`	`}`
`60`		`- @if (child.HasLog) { @Include("Log", child) }`
	`61`	`+ @if (child.HasLog) { @Include("Log", new SparkReporterTest(Model.Reporter, child)) }`
`61`	`62`	`</div>`
`62`	`63`	`}`
`63`	`64`	`@foreach (var m in child.Media)`
`@@ -69,7 +70,8 @@`
`69`	`70`	`}`
`70`	`71`	`@if (child.HasChildren)`
`71`	`72`	`{`
`72`		`- @Include("RecurseNodes", child)`
	`73`	`+ var reporterTest = new SparkReporterTest(Model.Reporter, child);`
	`74`	`+ @Include("RecurseNodes", reporterTest)`
`73`	`75`	`}`
`74`	`76`	`</div>`
`75`	`77`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,10 +4,10 @@`
`4`	`4`	`@using RazorEngine`
`5`	`5`	`@using RazorEngine.Templating`
`6`	`6`
`7`		`-@if (Model.HasChildren)`
	`7`	`+@if (Model.Test.HasChildren)`
`8`	`8`	`{`
`9`		`- var children = new List<Test>(Model.Children);`
`10`		`- @Include("SparkMediaTest", Model)`
	`9`	`+ var children = Model.Test.Children;`
	`10`	`+ @Include("SparkMediaTest", Model.Test)`
`11`	`11`	`<div class="accordion mt-4">`
`12`	`12`	`@foreach (var node in children)`
`13`	`13`	`{`
`@@ -26,7 +26,7 @@`
`26`	`26`	`if (node.BddType.Name == "ScenarioOutline")`
`27`	`27`	`{`
`28`	`28`	`var css = node.Status == Status.Pass ? "collapse" : "";`
`29`		`- var scenarios = new List<Test>(node.Children);`
	`29`	`+ var scenarios = node.Children;`
`30`	`30`	`<div class="scenario_outline @css">`
`31`	`31`	`@foreach (var child in scenarios)`
`32`	`32`	`{`
`@@ -43,7 +43,7 @@`
`43`	`43`	`</div>`
`44`	`44`	`@{`
`45`	`45`	`css = child.Status == Status.Pass ? "collapse" : "";`
`46`		`- var steps = new List<Test>(child.Children);`
	`46`	`+ var steps = child.Children;`
`47`	`47`	`}`
`48`	`48`	`@foreach (var step in steps)`
`49`	`49`	`{`
`@@ -60,7 +60,7 @@`
`60`	`60`	`else`
`61`	`61`	`{`
`62`	`62`	`var css = node.Status == Status.Pass ? "collapse" : "";`
`63`		`- var steps = new List<Test>(node.Children);`
	`63`	`+ var steps = node.Children;`
`64`	`64`	`<div class="@css">`
`65`	`65`	`<div class="card-body">`
`66`	`66`	`@Include("SparkMediaTest", node)`
Original file line number	Diff line number	Diff line change
`@@ -4,17 +4,17 @@`
`4`	`4`	`@using RazorEngine`
`5`	`5`	`@using RazorEngine.Templating`
`6`	`6`
`7`		`-@foreach (var m in Model.Media)`
	`7`	`+@foreach (var m in Model.Test.Media)`
`8`	`8`	`{`
`9`	`9`	`@Include("SparkMedia", m)`
`10`	`10`	`}`
`11`	`11`
`12`		`-@if (Model.HasAnyLog)`
	`12`	`+@if (Model.Test.HasAnyLog)`
`13`	`13`	`{`
`14`	`14`	`<div class="detail-body mt-4">`
`15`		`- @if (Model.HasGeneratedLog)`
	`15`	`+ @if (Model.Test.HasGeneratedLog)`
`16`	`16`	`{`
`17`		`- var genLog = new List<Log>(Model.GeneratedLog);`
	`17`	`+ var genLog = new List<Log>(Model.Test.GeneratedLog);`
`18`	`18`	`@foreach (var l in genLog)`
`19`	`19`	`{`
`20`	`20`	`<div>`
`@@ -23,10 +23,10 @@`
`23`	`23`	`</div>`
`24`	`24`	`}`
`25`	`25`	`}`
`26`		`- @if (Model.HasLog) { @Include("Log", Model) }`
	`26`	`+ @if (Model.Test.HasLog) { @Include("Log", Model) }`
`27`	`27`	`</div>`
`28`	`28`	`}`
`29`		`-@if (Model.HasChildren)`
	`29`	`+@if (Model.Test.HasChildren)`
`30`	`30`	`{`
`31`	`31`	`<div class="mt-4">@Include("RecurseNodes", Model)</div>`
`32`	`32`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@`
`6`	`6`
`7`	`7`	`@if (Model.HasLog)`
`8`	`8`	`{`
`9`		`- var logs = new List<Log>(Model.Logs);`
	`9`	`+ var logs = Model.Logs;`
`10`	`10`	`foreach (var log in logs)`
`11`	`11`	`{`
`12`	`12`	`<div>`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`"thumbnailForBase64": false,`
`8`	`8`	`"documentTitle": "Extent Framework",`
`9`	`9`	`"reportName": "ExtentReports",`
`10`		`- "timeStampFormat": "MMM dd, yyyy HH:mm:ss a",`
	`10`	`+ "timeStampFormat": "MMM dd, yyyy HH:mm:ss",`
`11`	`11`	`"js": "",`
`12`	`12`	`"css": ""`
`13`	`13`	`}`