No-export remove test.

Author: Hosch250

RetailCoder.VBE/Navigation/CodeExplorer/CodeExplorerViewModel.cs

@@ -191,8 +191,7 @@ private void ParserState_StateChanged(object sender, EventArgs e)
                 .Where(grouping => grouping.Key != null)
                 .ToList();
 
-            if (
-                userDeclarations.Any(
+            if (userDeclarations.Any(
                     grouping => grouping.All(declaration => declaration.DeclarationType != DeclarationType.Project)))
             {
                 return;

RubberduckTests/CodeExplorer/CodeExplorerTests.cs

@@ -1,12 +1,14 @@
 using System.Collections.Generic;
 using System.Linq;
+using System.Windows.Forms;
 using System.Windows.Input;
 using Microsoft.Vbe.Interop;
 using Microsoft.VisualStudio.TestTools.UnitTesting;
 using Moq;
 using Rubberduck.Navigation.CodeExplorer;
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Settings;
+using Rubberduck.UI;
 using Rubberduck.UI.CodeExplorer.Commands;
 using Rubberduck.UnitTesting;
 using Rubberduck.VBEditor.VBEHost;
@@ -101,6 +103,39 @@ public void AddTestModule()
             Assert.IsTrue(vbComponents.Count(c => c.Type == vbext_ComponentType.vbext_ct_StdModule) == 2);
         }
 
+        [TestMethod]
+        public void RemoveModule_Cancel()
+        {
+            var builder = new MockVbeBuilder();
+            var project = builder.ProjectBuilder("TestProject1", vbext_ProjectProtection.vbext_pp_none)
+                .AddComponent("Class1", vbext_ComponentType.vbext_ct_ClassModule, "")
+                .Build();
+            var vbe = builder.AddProject(project).Build();
+
+            var messageBox = new Mock<IMessageBox>();
+            messageBox.Setup(m =>
+                    m.Show(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<MessageBoxButtons>(),
+                        It.IsAny<MessageBoxIcon>())).Returns(DialogResult.No);
+
+            var commands = new List<ICommand>
+            {
+                new CodeExplorer_RemoveCommand(null, messageBox.Object)
+            };
+
+            var state = new RubberduckParserState();
+            var vm = new CodeExplorerViewModel(state, commands);
+
+            var parser = MockParser.Create(vbe.Object, state);
+            parser.Parse();
+            if (parser.State.Status == ParserState.Error) { Assert.Inconclusive("Parser Error"); }
+
+            vm.SelectedItem = vm.Projects.First().Items.First().Items.First();
+            vm.RemoveCommand.Execute(vm.SelectedItem);
+
+            var vbComponents = vbe.Object.VBProjects.Item(0).VBComponents.Cast<VBComponent>();
+            Assert.IsTrue(vbComponents.Count(c => c.Type == vbext_ComponentType.vbext_ct_StdModule) == 0);
+        }
+
         #region
         private Configuration GetDefaultUnitTestConfig()
         {

RubberduckTests/Mocks/MockProjectBuilder.cs

@@ -143,6 +143,11 @@ private Mock<VBComponents> CreateComponentsMock()
                 _components.Add(CreateComponentMock("test", c, "", new Selection()).Object);
             });
 
+            result.Setup(m => m.Remove(It.IsAny<VBComponent>())).Callback((VBComponent c) =>
+            {
+                _components.Remove(c);
+            });
+
             return result;
         }