Rename unit test helper class, readd Refresh to CE context menu, don't assign helpfile for protected projects.

Author: Hosch250

RetailCoder.VBE/Rubberduck.csproj

@@ -951,7 +951,7 @@
     <Compile Include="UnitTesting\AssertClass.cs" />
     <Compile Include="UnitTesting\AssertCompletedEventArgs.cs" />
     <Compile Include="UnitTesting\AssertHandler.cs" />
-    <Compile Include="UnitTesting\UnitTestHelpers.cs" />
+    <Compile Include="UnitTesting\UnitTestUtils.cs" />
     <Compile Include="UnitTesting\IAssert.cs" />
     <Compile Include="UnitTesting\ITestEngine.cs" />
     <Compile Include="UnitTesting\ITestRunner.cs" />

RetailCoder.VBE/UI/CodeExplorer/CodeExplorerControl.xaml

@@ -357,6 +357,12 @@
             <Setter Property="ContextMenu">
                 <Setter.Value>
                     <ContextMenu DataContext="{Binding DataContext, Source={x:Reference CodeExplorer}}">
+                        <MenuItem Header="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=Refresh}"
+                                  Command="{Binding RefreshCommand}">
+                            <MenuItem.Icon>
+                                <Image Source="{StaticResource RefreshImage}" />
+                            </MenuItem.Icon>
+                        </MenuItem>
                         <MenuItem Header="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=Navigate}"
                                   FontWeight="Bold"
                                   Command="{Binding NavigateCommand}"

RetailCoder.VBE/UI/UnitTesting/TestExplorerModel.cs

@@ -31,7 +31,7 @@ private void State_StateChanged(object sender, ParserStateEventArgs e)
 
             _dispatcher.Invoke(() =>
             {
-                var tests = UnitTestHelpers.GetAllTests(_vbe, _state).ToList();
+                var tests = UnitTestUtils.GetAllTests(_vbe, _state).ToList();
 
                 var removedTests = Tests.Where(test =>
                              !tests.Any(t =>

RetailCoder.VBE/UnitTesting/UnitTestHelpers.cs renamed to RetailCoder.VBE/UnitTesting/UnitTestUtils.cs

@@ -8,7 +8,7 @@
 
 namespace Rubberduck.UnitTesting
 {
-    public static class UnitTestHelpers
+    public static class UnitTestUtils
     {
         public static IEnumerable<TestMethod> GetAllTests(VBE vbe, RubberduckParserState state)
         {

Rubberduck.Parsing/VBA/RubberduckParserState.cs

@@ -209,6 +209,11 @@ public void RefreshProjects()
                 _projects.Clear();
                 foreach (VBProject project in _vbe.VBProjects)
                 {
+                    if (project.Protection == vbext_ProjectProtection.vbext_pp_locked)
+                    {
+                        continue;
+                    }
+
                     if (string.IsNullOrEmpty(project.HelpFile) || _projects.Keys.Contains(project.HelpFile))
                     {
                         project.AssignProjectId();

RubberduckTests/UnitTesting/DiscoveryTests.cs

@@ -34,7 +34,7 @@ Public Sub TestMethod1()
             parser.Parse(new CancellationTokenSource());
             if (parser.State.Status >= ParserState.Error) { Assert.Inconclusive("Parser Error"); }
 
-            Assert.AreEqual(1, UnitTestHelpers.GetAllTests(vbe.Object, parser.State).Count());
+            Assert.AreEqual(1, UnitTestUtils.GetAllTests(vbe.Object, parser.State).Count());
         }
 
         [TestMethod]
@@ -55,7 +55,7 @@ public void Discovery_IgnoresNonAnnotatedTestMethods()
             parser.Parse(new CancellationTokenSource());
             if (parser.State.Status >= ParserState.Error) { Assert.Inconclusive("Parser Error"); }
 
-            Assert.IsFalse(UnitTestHelpers.GetAllTests(vbe.Object, parser.State).Any());
+            Assert.IsFalse(UnitTestUtils.GetAllTests(vbe.Object, parser.State).Any());
         }
 
         [TestMethod]
@@ -77,7 +77,7 @@ Public Sub TestMethod1()
             parser.Parse(new CancellationTokenSource());
             if (parser.State.Status >= ParserState.Error) { Assert.Inconclusive("Parser Error"); }
 
-            Assert.IsFalse(UnitTestHelpers.GetAllTests(vbe.Object, parser.State).Any());
+            Assert.IsFalse(UnitTestUtils.GetAllTests(vbe.Object, parser.State).Any());
         }
 
         [TestMethod]