Remove explicit casts areou extract interface commands

Author: MDoerner

Rubberduck.Core/UI/CodeExplorer/Commands/CodeExplorerExtractInterfaceCommand.cs

@@ -16,13 +16,13 @@ public class CodeExplorerExtractInterfaceCommand : CodeExplorerCommandBase
             typeof(CodeExplorerComponentViewModel)
         };
 
-        private readonly IParserStatusProvider _state;
+        private readonly RubberduckParserState _state;
         private readonly ExtractInterfaceRefactoring _refactoring;
         private readonly ExtractInterfaceFailedNotifier _failureNotifier;
 
         public CodeExplorerExtractInterfaceCommand(
             ExtractInterfaceRefactoring refactoring,
-            IParserStatusProvider state,
+            RubberduckParserState state,
             ExtractInterfaceFailedNotifier failureNotifier,
             IVbeEvents vbeEvents) 
             : base(vbeEvents)
@@ -38,16 +38,16 @@ public CodeExplorerExtractInterfaceCommand(
 
         private bool SpecialEvaluateCanExecute(object parameter)
         {
-            return _state.Status == ParserState.Ready &&
-                   parameter is CodeExplorerComponentViewModel node &&
-                   _refactoring.CanExecute((RubberduckParserState)_state, node.QualifiedSelection.Value.QualifiedName);
+            return _state.Status == ParserState.Ready 
+                   && parameter is CodeExplorerComponentViewModel node 
+                   && _refactoring.CanExecute(_state, node.QualifiedSelection.Value.QualifiedName);
         }
 
         private bool FurtherCanExecuteEvaluation(object parameter)
         {
-            return _state.Status == ParserState.Ready &&
-                parameter is CodeExplorerItemViewModel node &&
-                node.Declaration != null;
+            return _state.Status == ParserState.Ready 
+                   && parameter is CodeExplorerItemViewModel node 
+                   && node.Declaration != null;
         }
 
         protected override void OnExecute(object parameter)

Rubberduck.Core/UI/Command/Refactorings/RefactorCodePaneCommandBase.cs

@@ -10,7 +10,11 @@ public abstract class RefactorCodePaneCommandBase : RefactorCommandBase
     {
         protected readonly ISelectionProvider SelectionProvider;
 
-        protected RefactorCodePaneCommandBase(IRefactoring refactoring, IRefactoringFailureNotifier failureNotifier, ISelectionProvider selectionProvider, IParserStatusProvider parserStatusProvider)
+        protected RefactorCodePaneCommandBase(
+            IRefactoring refactoring, 
+            IRefactoringFailureNotifier failureNotifier, 
+            ISelectionProvider selectionProvider, 
+            IParserStatusProvider parserStatusProvider)
             : base (refactoring, failureNotifier, parserStatusProvider)
         {
             SelectionProvider = selectionProvider;

Rubberduck.Core/UI/Command/Refactorings/RefactorExtractInterfaceCommand.cs

@@ -1,10 +1,4 @@
-using System.Collections.Generic;
-using System.Linq;
 using System.Runtime.InteropServices;
-using Antlr4.Runtime;
-using Rubberduck.Parsing;
-using Rubberduck.Parsing.Grammar;
-using Rubberduck.Parsing.Symbols;
 using Rubberduck.Parsing.VBA;
 using Rubberduck.Refactorings.ExtractInterface;
 using Rubberduck.UI.Command.Refactorings.Notifiers;
@@ -16,8 +10,8 @@ namespace Rubberduck.UI.Command.Refactorings
     public class RefactorExtractInterfaceCommand : RefactorCodePaneCommandBase
     {
         private readonly RubberduckParserState _state;
+        private readonly ExtractInterfaceRefactoring _extractInterfaceRefactoring;
 
-        
         public RefactorExtractInterfaceCommand(
             ExtractInterfaceRefactoring refactoring, 
             ExtractInterfaceFailedNotifier extractInterfaceFailedNotifier, 
@@ -26,6 +20,7 @@ public RefactorExtractInterfaceCommand(
             :base(refactoring, extractInterfaceFailedNotifier, selectionProvider, state)
         {
             _state = state;
+            _extractInterfaceRefactoring = refactoring;
 
             AddToCanExecuteEvaluation(SpecializedEvaluateCanExecute);
         }
@@ -37,7 +32,7 @@ private bool SpecializedEvaluateCanExecute(object parameter)
             {
                 return false;
             }
-            return ((ExtractInterfaceRefactoring)Refactoring).CanExecute(_state, activeSelection.Value.QualifiedName);
+            return _extractInterfaceRefactoring.CanExecute(_state, activeSelection.Value.QualifiedName);
         }
     }
 }

Rubberduck.Refactorings/ExtractInterface/ExtractInterfaceRefactoring.cs

@@ -142,6 +142,7 @@ private string GetInterfaceModuleBody(ExtractInterfaceModel model)
             DeclarationType.UserForm
         };
 
+        //TODO: Redesign how refactoring commands are wired up to make this a responsibility of the command again. 
         public bool CanExecute(RubberduckParserState state, QualifiedModuleName qualifiedName)
         {
             var interfaceClass = state.AllUserDeclarations.SingleOrDefault(item =>