Add filters to test results list in VisualAssertion.Dashboard #8

- add filter by status

Author: cezarypiatek

Src/VisualAssertions.Dashboard/Content/sb-admin/css/sb-admin.css

@@ -224,4 +224,8 @@ ul.alert-dropdown {
 
 .panel-yellow > a:hover {
     color: #df8a13;
+}
+
+.top-offset {
+    margin-top: 20px;
 }

Src/VisualAssertions.Dashboard/Controllers/HomeController.cs

@@ -33,6 +33,12 @@ public ActionResult GetTestsFromSessionSession(long sessionId, string browserNam
             return View("TestResultList", tests);
         }
 
+        public ActionResult GetTestsFromSessionInStatus(long sessionId, string browserName, TestResultStatus status)
+        {
+            var tests = this.testResultService.GetTestsFromSessionInStatus(sessionId, browserName, status);
+            return View("TestResultsInStatus", tests);
+        }
+
         public ActionResult GetTestResult(long testId)
         {
             var test = this.testResultService.GetTestResult(testId);

Src/VisualAssertions.Dashboard/Services/TestResults/Queries/FindTestResultsFromSession.cs

@@ -0,0 +1,40 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using Tellurium.VisualAssertions.Infrastructure;
+using Tellurium.VisualAssertions.Screenshots.Domain;
+
+namespace Tellurium.VisualAssertions.Dashboard.Services.TestResults.Queries
+{
+    public class FindTestResultsFromSession : IQueryAll<TestResult>
+    {
+        private readonly long sessionId;
+        private readonly string browserName;
+        private readonly TestResultStatus resultStatus;
+
+        public FindTestResultsFromSession(long sessionId, string browserName, TestResultStatus resultStatus)
+        {
+            this.sessionId = sessionId;
+            this.browserName = browserName;
+            this.resultStatus = resultStatus;
+        }
+
+        public List<TestResult> GetQuery(IQueryable<TestResult> query)
+        {
+            var testFromSession = query.Where(x=>x.TestSession.Id == sessionId)
+                .Where(x=>x.BrowserName == browserName);
+
+            switch (resultStatus)
+            {
+                case TestResultStatus.All:
+                    return testFromSession.ToList();
+                case TestResultStatus.Passed:
+                    return testFromSession.Where(x=>x.TestPassed).ToList();
+                case TestResultStatus.Failed:
+                    return testFromSession.Where(x => x.TestPassed == false).ToList();
+                default:
+                    throw new ArgumentOutOfRangeException();
+            }
+        }
+    }
+}

Src/VisualAssertions.Dashboard/Services/TestResults/TestResultService.cs

@@ -87,11 +87,15 @@ public Bitmap GetScreenshot(long testId, ScreenshotType screenshotType)
 
         public TestResultListViewModel GetTestsFromSession(long sessionId, string browserName)
         {
-            var testSession = this.testSessionRepository.Get(sessionId);
-            var testResults = testSession.TestResults.Where(x => x.BrowserName == browserName).ToList();
+            var query = new FindTestResultsFromSession(sessionId, browserName, TestResultStatus.All);
+            var testResults = this.testRepository.FindAll(query);
+            var failedCount = testResults.Count(x => x.TestPassed == false);
             return new TestResultListViewModel()
             {
-                TestSessionId = testSession.Id,
+                AllCount = testResults.Count,
+                FailedCount = failedCount,
+                PassedCount = testResults.Count-failedCount,
+                TestSessionId = sessionId,
                 BrowserName = browserName,
                 TestResults = testResults.ConvertAll(MapToTestResultListItemDTO)
             };
@@ -154,6 +158,16 @@ public TestResultListItemDTO GetTestResultPreview(long testSessionId, long patte
             var testResult = this.testRepository.FindOne(query);
             return MapToTestResultListItemDTO(testResult);
         }
+
+        public TestResultsInStatusViewModel GetTestsFromSessionInStatus(long sessionId, string browserName, TestResultStatus status)
+        {
+            var query = new FindTestResultsFromSession(sessionId, browserName, status);
+            var testResults = this.testRepository.FindAll(query);
+            return new TestResultsInStatusViewModel()
+            {
+                TestResults = testResults.ConvertAll(MapToTestResultListItemDTO)
+            };
+        }
     }
 
     public interface ITestResultService
@@ -167,6 +181,7 @@ public interface ITestResultService
         ProjectListViewModel GetProjectsList();
         TestResultDetailsViewModel GetTestResultDetails(long testResultId);
         TestResultListItemDTO GetTestResultPreview(long testSessionId, long patternId);
+        TestResultsInStatusViewModel GetTestsFromSessionInStatus(long sessionId, string browserName, TestResultStatus status);
     }
 
     public enum ScreenshotType
@@ -182,11 +197,26 @@ public class TestResultDetailsViewModel
         public bool TestPassed { get; set; }
     }
 
+    public enum TestResultStatus
+    {
+        All,
+        Passed,
+        Failed
+    }
+
     public class TestResultListViewModel
     {
         public List<TestResultListItemDTO> TestResults { get; set; }
         public long TestSessionId { get; set; }
         public string BrowserName { get; set; }
+        public int AllCount { get; set; }
+        public int PassedCount { get; set; }
+        public int FailedCount { get; set; }
+    }
+
+    public class TestResultsInStatusViewModel
+    {
+        public List<TestResultListItemDTO> TestResults { get; set; }
     }
 
     public class TestResultListItemDTO

Src/VisualAssertions.Dashboard/Views/Home/TestResultInfo.cshtml

@@ -10,16 +10,18 @@
 <div class="panel @rowClass test-row" id="@rowId">
     <div class="panel-heading ">
         <div class="row">
-            <div class="col-md-10 test-row-expander" @widget.GetAttributeString()>
+            <div class="col-md-9 test-row-expander" @widget.GetAttributeString()>
                 <span class="glyphicon glyphicon-menu-right"></span>
                 @Model.ScreenshotName
             </div>
-            <div class="col-md-2">
-                @(Html.ActionLink<TestCaseController>(c=>c.GetTestCase(Model.TestCaseId),"Edit pattern", new { @class = "btn btn-primary" }))
-                @if (Model.CanShowMarkAsPattern)
-                {
-                    @(Ajax.ChangeActionLink<HomeController>(c => c.MarkAsPattern(Model.TestResultId), "Mark as pattern", "Do you really want to mark this as pattern?", rowId, new {@class = "btn btn-primary"}))
-                }
+            <div class="col-md-3">
+                <div class="pull-right">
+                    @(Html.ActionLink<TestCaseController>(c => c.GetTestCase(Model.TestCaseId), "Edit pattern", new { @class = "btn btn-primary" }))
+                    @if (Model.CanShowMarkAsPattern)
+                    {
+                        @(Ajax.ChangeActionLink<HomeController>(c => c.MarkAsPattern(Model.TestResultId), "Mark as pattern", "Do you really want to mark this as pattern?", rowId, new { @class = "btn btn-primary" }))
+                    }    
+                </div>
             </div>
         </div>
     </div>

Src/VisualAssertions.Dashboard/Views/Home/TestResultList.cshtml

@@ -6,15 +6,20 @@
     ViewBag.Title = "Tests Results";
     Layout = "~/Views/_Layout.cshtml";
 }
-
-<div class="row">
-    <div class="col-md-12">
+<div class="container-fluid">
+    <div class="row">
+        @(Ajax.ActionLink<HomeController>(c => c.GetTestsFromSessionInStatus(Model.TestSessionId, Model.BrowserName, TestResultStatus.All), "All tests: " +Model.AllCount, "TestResultsList", new { @class = "btn btn-primary" }))
+        @(Ajax.ActionLink<HomeController>(c => c.GetTestsFromSessionInStatus(Model.TestSessionId, Model.BrowserName, TestResultStatus.Passed), "Passed tests: "+Model.PassedCount, "TestResultsList", new { @class = "btn btn-success" }))
+        @(Ajax.ActionLink<HomeController>(c => c.GetTestsFromSessionInStatus(Model.TestSessionId, Model.BrowserName, TestResultStatus.Failed), "Failed tests: "+Model.FailedCount, "TestResultsList", new { @class = "btn btn-danger" }))
+    </div>
+    <div class="row">
         @(Ajax.ChangeActionLink<HomeController>(c => c.MarkAllAsPattern(Model.TestSessionId, Model.BrowserName), "Mark all as pattern", "Do you really want to replace patter for all cases?", htmlAttributes: new { @class = "btn btn-primary pull-right" }))
     </div>
+    <div id="TestResultsList" class="row top-offset">
+        @foreach (var testResultInfo in Model.TestResults)
+        {
+            Html.RenderPartial("TestResultInfo", testResultInfo);
+        }
+    </div>
+    
 </div>
-@foreach (var testResultInfo in Model.TestResults)
-{
-    Html.RenderPartial("TestResultInfo", testResultInfo);
-}
-
-

Src/VisualAssertions.Dashboard/Views/Home/TestResultsInStatus.cshtml

@@ -0,0 +1,6 @@
+@model TestResultsInStatusViewModel
+
+@foreach (var testResultInfo in Model.TestResults)
+{
+    Html.RenderPartial("TestResultInfo", testResultInfo);
+}

Src/VisualAssertions.Dashboard/Views/_Layout.cshtml

@@ -296,11 +296,6 @@
                     <h1 class="page-header">
                         Dashboard <small>@ViewBag.Title</small>
                     </h1>
-                    <ol class="breadcrumb">
-                        <li class="active">
-                            <i class="fa fa-dashboard"></i> Dashboard
-                        </li>
-                    </ol>
                 </div>
             </div>
             <!-- /.row -->

Src/VisualAssertions.Dashboard/VisualAssertions.Dashboard.csproj

@@ -196,6 +196,7 @@
     <Compile Include="Models\Home\BrowserPatternDTO.cs" />
     <Compile Include="Services\TestCase\TestCaseService.cs" />
     <Compile Include="Services\TestResults\Queries\FindTestResultForPatternInSession.cs" />
+    <Compile Include="Services\TestResults\Queries\FindTestResultsFromSession.cs" />
     <Compile Include="Services\TestResults\TestCaseCategoriesListViewModel.cs" />
     <Compile Include="Services\TestResults\TestResultService.cs" />
     <Compile Include="Global.asax.cs">
@@ -264,6 +265,7 @@
     <Content Include="Views\TestCase\CategoriesList.cshtml" />
     <Content Include="Views\Home\TestResultInfoPreloaded.cshtml" />
     <Content Include="Views\TestCase\TestCase.cshtml" />
+    <Content Include="Views\Home\TestResultsInStatus.cshtml" />
     <None Include="Web.Debug.config">
       <DependentUpon>Web.config</DependentUpon>
     </None>