Add ignore/unignore group command

Respective command displays when mouse is over group header, when applicable.

Author: IvenBach

Rubberduck.Core/UI/UnitTesting/TestExplorerControl.xaml

@@ -454,14 +454,16 @@
                                     <Image Source="{StaticResource UnignoreTestImage}" />
                                 </MenuItem.Icon>
                             </MenuItem>
-                            <MenuItem Command="{Binding IgnoreSelectedGroupCommand}"
-                                      Header="{Resx ResxName=Rubberduck.Resources.UnitTesting.TestExplorer, Key=TestExplorer_ContextMenuIgnoreGroup}">
-                                <MenuItem.Icon>
+							<MenuItem Command="{Binding IgnoreSelectedGroupCommand}"
+                                      Header="{Resx ResxName=Rubberduck.Resources.UnitTesting.TestExplorer, Key=TestExplorer_ContextMenuIgnoreGroup}"
+									  Visibility="{Binding DisplayIgnoreGroupLabel, Converter={StaticResource BoolToVisibility}}">
+								<MenuItem.Icon>
                                     <Image Source="{StaticResource IgnoreTestGroupImage}" />
                                 </MenuItem.Icon>
                             </MenuItem>
                             <MenuItem Command="{Binding UnignoreSelectedGroupCommand}"
-                                      Header="{Resx ResxName=Rubberduck.Resources.UnitTesting.TestExplorer, Key=TestExplorer_ContextMenuUnignoreGroup}">
+                                      Header="{Resx ResxName=Rubberduck.Resources.UnitTesting.TestExplorer, Key=TestExplorer_ContextMenuUnignoreGroup}"
+									  Visibility="{Binding DisplayUnignoreGroupLabel, Converter={StaticResource BoolToVisibility}}">
                                 <MenuItem.Icon>
                                     <Image Source="{StaticResource UnignoreTestGroupImage}" />
                                 </MenuItem.Icon>

Rubberduck.Core/UI/UnitTesting/TestExplorerViewModel.cs

@@ -140,6 +140,8 @@ private void RefreshContextMenu()
         {
             OnPropertyChanged(nameof(DisplayUnignoreTestLabel));
             OnPropertyChanged(nameof(DisplayIgnoreTestLabel));
+            OnPropertyChanged(nameof(DisplayUnignoreGroupLabel));
+            OnPropertyChanged(nameof(DisplayIgnoreGroupLabel));
         }
 
         private static readonly Dictionary<TestExplorerGrouping, PropertyGroupDescription> GroupDescriptions = new Dictionary<TestExplorerGrouping, PropertyGroupDescription>
@@ -245,6 +247,36 @@ private void HandleTestCompletion(object sender, TestCompletedEventArgs e)
         private TestMethod _mousedOverTestMethod => ((TestMethodViewModel)SelectedItem).Method;
         public bool DisplayUnignoreTestLabel => SelectedItem != null && _mousedOverTestMethod.IsIgnored;
         public bool DisplayIgnoreTestLabel => SelectedItem != null && !_mousedOverTestMethod.IsIgnored;
+
+        public bool DisplayUnignoreGroupLabel
+        {
+            get
+            {
+                if (MouseOverGroup?.Items == null)
+                {
+                    return false;
+                }
+
+                var testsInGroup = MouseOverGroup.Items.Cast<TestMethodViewModel>();
+
+                return testsInGroup.Where(test => test.Method.IsIgnored).Any(); ;
+            }
+        }
+
+        public bool DisplayIgnoreGroupLabel
+        {
+            get
+            {
+                if (MouseOverGroup?.Items == null)
+                {
+                    return false;
+                }
+
+                var testsInGroup = MouseOverGroup.Items.Cast<TestMethodViewModel>();
+
+                return testsInGroup.Where(test => test.Method.IsIgnored).Count() != MouseOverGroup.Items.Count;
+            }
+        }
 
         #region Commands
 
@@ -428,10 +460,19 @@ private void ExecuteIgnoreGroupCommand (object parameter)
 
             foreach (TestMethodViewModel test in testGroup.Items)
             {
-                //    var testMethod = parameter == null
-                //        ? _mousedOverTestMethod
-                //        : (parameter as TestMethodViewModel).Method;                
-                AnnotationUpdater.AddAnnotation(rewriteSession, test.Method.Declaration, ignoreTestAnnotation);
+                var needsIgnoreAnnotationAdded = true;
+                foreach (var annotation in test.Method.Declaration.Annotations)
+                {
+                    if (annotation.Annotation is IgnoreTestAnnotation)
+                    {
+                        needsIgnoreAnnotationAdded = false;
+                    };
+                }
+
+                if (needsIgnoreAnnotationAdded)
+                {
+                    AnnotationUpdater.AddAnnotation(rewriteSession, test.Method.Declaration, ignoreTestAnnotation);
+                }
             }
 
             rewriteSession.TryRewrite();
@@ -441,15 +482,11 @@ private void ExecuteUnignoreGroupCommand(object parameter)
         {
             var rewriteSession = RewritingManager.CheckOutCodePaneSession();
             var testGroup = GroupContainingSelectedTest(MouseOverTest);
+
             foreach (TestMethodViewModel test in testGroup.Items)
             {
-                //ExecuteUnignoreTestCommand(test);
-                //var ignoreTestAnnotations = test.Method.Declaration.Annotations
-                //    .Where(iannotations => iannotations.AnnotationType == Parsing.Annotations.AnnotationType.IgnoreTest);
-
                 var ignoreTestAnnotations = test.Method.Declaration.Annotations
-                    .Where(pta => pta.Annotation.Target == AnnotationTarget.Member
-                        && pta.AnnotationArguments.Contains("'@IgnoreTest"));
+                    .Where(pta => pta.Annotation is IgnoreTestAnnotation);
 
                 foreach (var ignoreTestAnnotation in ignoreTestAnnotations)
                 {

Rubberduck.Resources/UnitTesting/TestExplorer.Designer.cs

@@ -352,9 +352,9 @@
     <value>Unignore test</value>
   </data>
   <data name="TestExplorer_ContextMenuIgnoreGroup" xml:space="preserve">
-    <value>Ignore all tests in group</value>
+    <value>Ignore group tests</value>
   </data>
   <data name="TestExplorer_ContextMenuUnignoreGroup" xml:space="preserve">
-    <value>Unignore all tests in group</value>
+    <value>Unignore group tests</value>
   </data>
 </root>