Address review comments, add easter egg.

Author: comintern

Rubberduck.Core/UI/GridViewSort.cs

@@ -436,6 +436,7 @@
                     <converters:NonZeroToVisibilityConverter x:Key="NonZeroToVisibility"/>
                     <BitmapImage x:Key="IgnoredTestImage" UriSource="pack://application:,,,/Rubberduck.Resources;component/Icons/Fugue/minus-white.png" />
                     <BitmapImage x:Key="InconclusiveTestImage" UriSource="pack://application:,,,/Rubberduck.Resources;component/Icons/Fugue/exclamation.png" />
+                    <BitmapImage x:Key="EasterEggTestImage" UriSource="pack://application:,,,/Rubberduck.Resources;component/Icons/Fugue/skull-mad.png" />
                     <Style x:Key="ResultsTestOutcomeStyle" TargetType="TextBlock" >
                         <Setter Property="Margin" Value="4,0,12,0"/>
                         <Setter Property="TextWrapping" Value="WrapWithOverflow"/>
@@ -451,6 +452,13 @@
                 <TextBlock Margin="2" Padding="4,2,4,2" FontWeight="Bold" VerticalAlignment="Center"
                            Text="{Resx ResxName=Rubberduck.Resources.UnitTesting.TestExplorer, Key=TestOutcome_SummaryCaption}"/>
                 <TextBlock Style="{StaticResource ResultsTestOutcomeStyle}" Text="{Binding Model.LastTestRunSummary, Mode=OneWay}"/>
+                <StackPanel Orientation="Horizontal" Visibility="{Binding Model.LastTestSpectacularFailCount, Mode=OneWay, Converter={StaticResource NonZeroToVisibility}}" >
+                    <Image Style="{StaticResource TestOutcomeIconStyle}" Source="{StaticResource EasterEggTestImage}" Margin="4,0,4,0"/>
+                    <TextBlock Style="{StaticResource ResultsTestOutcomeStyle}">
+                        <Run Text="{Binding Model.LastTestSpectacularFailCount, Mode=OneWay}"/>
+                        <Run Text="{Resx ResxName=Rubberduck.Resources.UnitTesting.TestExplorer, Key=TestOutcome_SpectacularFail}"/>
+                    </TextBlock>
+                </StackPanel>
                 <StackPanel Orientation="Horizontal" Visibility="{Binding Model.LastTestFailedCount, Mode=OneWay, Converter={StaticResource NonZeroToVisibility}}" >
                     <Image Style="{StaticResource TestOutcomeIconStyle}" Source="{StaticResource RunFailedTestsImage}" Margin="4,0,4,0"/>
                     <TextBlock Style="{StaticResource ResultsTestOutcomeStyle}">

Rubberduck.Core/UI/UnitTesting/TestExplorerControl.xaml

@@ -100,6 +100,8 @@ public bool IsRefreshing
         public string LastTestRunSummary =>
             string.Format(Resources.UnitTesting.TestExplorer.TestOutcome_RunSummaryFormat, CurrentRunTestCount, Tests.Count, TimeSpan.FromMilliseconds(TotalDuration));
 
+        public int LastTestSpectacularFailCount => Tests.Count(test => test.Result.Outcome == TestOutcome.SpectacularFail && _testEngine.LastRunTests.Contains(test.Method));
+
         public int LastTestFailedCount => Tests.Count(test => test.Result.Outcome == TestOutcome.Failed && _testEngine.LastRunTests.Contains(test.Method));
 
         public int LastTestInconclusiveCount => Tests.Count(test => test.Result.Outcome == TestOutcome.Inconclusive && _testEngine.LastRunTests.Contains(test.Method));
@@ -179,6 +181,7 @@ private void HandleRunCompletion(object sender, TestRunCompletedEventArgs e)
             IsBusy = false;
 
             OnPropertyChanged(nameof(LastTestRunSummary));
+            OnPropertyChanged(nameof(LastTestSpectacularFailCount));
             OnPropertyChanged(nameof(LastTestFailedCount));
             OnPropertyChanged(nameof(LastTestIgnoredCount));
             OnPropertyChanged(nameof(LastTestInconclusiveCount));
@@ -242,15 +245,16 @@ private void OnTestCompleted(TestCompletedEventArgs args)
             { TestOutcome.Succeeded, Colors.LimeGreen },
             { TestOutcome.Inconclusive, Colors.Gold },
             { TestOutcome.Ignored, Colors.Orange },
-            { TestOutcome.Failed, Colors.Red }
+            { TestOutcome.Failed, Colors.Red },
+            { TestOutcome.SpectacularFail, Colors.Black }
         };
 
-        private TestOutcome _bestOutcome = TestOutcome.Unknown;
+        private TestOutcome _worstOutcome = TestOutcome.Unknown;
         private void UpdateProgressBar(TestOutcome output, bool reset = false)
         {
             if (reset)
             {
-                _bestOutcome = TestOutcome.Unknown;
+                _worstOutcome = TestOutcome.Unknown;
                 ExecutedCount = 0;
                 CurrentRunTestCount = 0;
                 ProgressBarColor = OutcomeColors[TestOutcome.Unknown];
@@ -259,12 +263,12 @@ private void UpdateProgressBar(TestOutcome output, bool reset = false)
 
             ExecutedCount++;
 
-            if (_bestOutcome != TestOutcome.Unknown && output >= _bestOutcome)
+            if (_worstOutcome != TestOutcome.Unknown && output >= _worstOutcome)
             {
                 return;
             }
 
-            _bestOutcome = output;
+            _worstOutcome = output;
             ProgressBarColor = OutcomeColors[output];
         }
 

Rubberduck.Core/UI/UnitTesting/TestExplorerModel.cs

@@ -22,7 +22,8 @@ public class TestOutcomeImageSourceConverter : ImageSourceConverter, IMultiValue
                 { TestOutcome.Succeeded, ToImageSource(RubberduckUI.tick_circle) },
                 { TestOutcome.Failed, ToImageSource(RubberduckUI.cross_circle) },
                 { TestOutcome.Inconclusive, ToImageSource(RubberduckUI.exclamation) },
-                { TestOutcome.Ignored, ToImageSource(RubberduckUI.minus_white) }
+                { TestOutcome.Ignored, ToImageSource(RubberduckUI.minus_white) },
+                { TestOutcome.SpectacularFail, ToImageSource(RubberduckUI.skull_mad) }
             };
 
         public override object Convert(object value, Type targetType, object parameter, CultureInfo culture)

Rubberduck.Core/UI/UnitTesting/TestOutcomeImageSourceConverter.cs

@@ -1523,4 +1523,7 @@ NOTE: Restart is required for the setting to take effect.</value>
   <data name="hourglass" type="System.Resources.ResXFileRef, System.Windows.Forms">
     <value>Icons\Fugue\hourglass.png;System.Drawing.Bitmap, System.Drawing, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a</value>
   </data>
+  <data name="skull_mad" type="System.Resources.ResXFileRef, System.Windows.Forms">
+    <value>Icons\Fugue\skull-mad.png;System.Drawing.Bitmap, System.Drawing, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a</value>
+  </data>
 </root>

Rubberduck.Resources/Icons/Fugue/skull-mad.png

@@ -339,4 +339,8 @@
   <data name="TestOutcome_SummaryCaption" xml:space="preserve">
     <value>Summary:</value>
   </data>
+  <data name="TestOutcome_SpectacularFail" xml:space="preserve">
+    <value>Spectactular Failure</value>
+    <comment>Easter egg status.</comment>
+  </data>
 </root>

Rubberduck.Resources/RubberduckUI.Designer.cs

@@ -362,6 +362,11 @@ private TestResult EvaluateResults(IEnumerable<AssertCompletedEventArgs> assertR
             var result = new AssertCompletedEventArgs(TestOutcome.Succeeded);
             var asserted = assertResults.ToList();
 
+            if (asserted.Count(assert => assert.Outcome == TestOutcome.Failed) >= 10)
+            {
+                return new TestResult(TestOutcome.SpectacularFail, result.Message, duration);
+            }
+
             if (asserted.Any(assertion => assertion.Outcome != TestOutcome.Succeeded))
             {
                 result = asserted.First(assertion => assertion.Outcome != TestOutcome.Succeeded);