Added an inspection, quickfix and tests. Added separate Step statement to the parser in order to use it in the inspection.

Author: AnnaVel

Rubberduck.Inspections/Concrete/StepOneIsRedundantInspection.cs

@@ -0,0 +1,72 @@
+using Rubberduck.Inspections.Abstract;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using Rubberduck.Parsing.Inspections.Abstract;
+using Rubberduck.Parsing.Inspections.Resources;
+using Rubberduck.Parsing.VBA;
+using Rubberduck.Parsing.Grammar;
+using Antlr4.Runtime.Misc;
+using Antlr4.Runtime;
+using Rubberduck.Parsing;
+using Rubberduck.VBEditor;
+using Rubberduck.Inspections.Results;
+using static Rubberduck.Parsing.Grammar.VBAParser;
+
+namespace Rubberduck.Inspections.Concrete
+{
+    public sealed class StepOneIsRedundantInspection : ParseTreeInspectionBase
+    {
+        public StepOneIsRedundantInspection(RubberduckParserState state) : base(state) { }
+
+        public override Type Type => typeof(StepOneIsRedundantInspection);
+
+        public override CodeInspectionType InspectionType => CodeInspectionType.LanguageOpportunities;
+
+        public override IEnumerable<IInspectionResult> GetInspectionResults()
+        {
+            return Listener.Contexts
+                .Where(result => !IsIgnoringInspectionResultFor(result.ModuleName, result.Context.Start.Line))
+                .Select(result => new QualifiedContextInspectionResult(this,
+                                                        InspectionsUI.StepOneIsRedundantInspectionResultFormat,
+                                                        result));
+        }
+
+        public override IInspectionListener Listener { get; } =
+            new StepOneIsRedundantListener();
+    }
+
+    public class StepOneIsRedundantListener : VBAParserBaseListener, IInspectionListener
+    {
+        private readonly List<QualifiedContext<ParserRuleContext>> _contexts = new List<QualifiedContext<ParserRuleContext>>();
+        public IReadOnlyList<QualifiedContext<ParserRuleContext>> Contexts => _contexts;
+
+        public QualifiedModuleName CurrentModuleName
+        {
+            get;
+            set;
+        }
+
+        public void ClearContexts()
+        {
+            _contexts.Clear();
+        }
+
+        public override void EnterForNextStmt([NotNull] VBAParser.ForNextStmtContext context)
+        {
+            StepStmtContext stepStatement = context.stepStmt();
+
+            if (stepStatement == null)
+            {
+                return;
+            }
+
+            string stepText = stepStatement.expression().GetText();
+
+            if(stepText == "1")
+            {
+                _contexts.Add(new QualifiedContext<ParserRuleContext>(CurrentModuleName, stepStatement));
+            }
+        }
+    }
+}

Rubberduck.Inspections/QuickFixes/RemoveStepOneQuickFix.cs

@@ -0,0 +1,38 @@
+using Rubberduck.Inspections.Abstract;
+using Rubberduck.Inspections.Concrete;
+using Rubberduck.Parsing.Inspections.Abstract;
+using Rubberduck.Parsing.Inspections.Resources;
+using Rubberduck.Parsing.Rewriter;
+using Rubberduck.Parsing.VBA;
+
+namespace Rubberduck.Inspections.QuickFixes
+{
+    public class RemoveStepOneQuickFix : QuickFixBase
+    {
+        private readonly RubberduckParserState _state;
+
+        public RemoveStepOneQuickFix(RubberduckParserState state)
+            : base(typeof(StepOneIsRedundantInspection))
+        {
+            _state = state;
+        }
+
+        public override bool CanFixInProcedure => false;
+
+        public override bool CanFixInModule => false;
+
+        public override bool CanFixInProject => false;
+
+        public override string Description(IInspectionResult result)
+        {
+            return InspectionsUI.RemoveStepOneQuickFix;
+        }
+
+        public override void Fix(IInspectionResult result)
+        {
+            IModuleRewriter rewriter = _state.GetRewriter(result.QualifiedSelection.QualifiedName);
+            var context = result.Context;
+            rewriter.Remove(context);
+        }
+    }
+}

Rubberduck.Inspections/Rubberduck.Inspections.csproj

@@ -68,6 +68,7 @@
     <Compile Include="Concrete\EmptyForEachBlockInspection.cs" />
     <Compile Include="Concrete\EmptyForLoopBlockInspection.cs" />
     <Compile Include="Concrete\EmptyWhileWendBlockInspection.cs" />
+    <Compile Include="Concrete\StepOneIsRedundantInspection.cs" />
     <Compile Include="Concrete\StopKeywordInspection.cs" />
     <Compile Include="Concrete\LineLabelNotUsedInspection.cs" />
     <Compile Include="Concrete\IntegerDataTypeInspection.cs" />
@@ -121,6 +122,7 @@
     <Compile Include="QuickFixes\ChangeDimToPrivateQuickFix.cs" />
     <Compile Include="QuickFixes\ChangeIntegerToLongQuickFix.cs" />
     <Compile Include="Abstract\QuickFixBase.cs" />
+    <Compile Include="QuickFixes\RemoveStepOneQuickFix.cs" />
     <Compile Include="QuickFixes\RemoveStopKeywordQuickFix.cs" />
     <Compile Include="QuickFixes\SpecifyExplicitByRefModifierQuickFix.cs" />
     <Compile Include="QuickFixes\ChangeProcedureToFunctionQuickFix.cs" />