public void VariableNotUsed_ReturnsResult_MultipleVariables_SomeAssigned()
{
const string inputCode =
@"Sub Foo()
Dim var1 as Integer
var1 = 8
Dim var2 as String
Goo var1
End Sub
Sub Goo(ByVal arg1 As Integer)
End Sub";
//Arrange
var builder = new MockVbeBuilder();
VBComponent component;
var vbe = builder.BuildFromSingleStandardModule(inputCode, out component);
var mockHost = new Mock <IHostApplication>();
mockHost.SetupAllProperties();
var parser = MockParser.Create(vbe.Object, new RubberduckParserState(new Mock <ISinks>().Object));
parser.Parse(new CancellationTokenSource());
if (parser.State.Status >= ParserState.Error)
{
Assert.Inconclusive("Parser Error");
}
var inspection = new VariableNotUsedInspection(parser.State);
var inspectionResults = inspection.GetInspectionResults();
Assert.AreEqual(1, inspectionResults.Count());
}
public void UnassignedVariable_DoesNotReturnResult()
{
const string inputCode =
@"Function Foo() As Boolean
Dim var1 as String
var1 = ""test""
Goo var1
End Function
Sub Goo(ByVal arg1 As String)
End Sub";
//Arrange
var builder = new MockVbeBuilder();
VBComponent component;
var vbe = builder.BuildFromSingleStandardModule(inputCode, out component);
var mockHost = new Mock <IHostApplication>();
mockHost.SetupAllProperties();
var parser = MockParser.Create(vbe.Object, new RubberduckParserState());
parser.Parse();
if (parser.State.Status == ParserState.Error)
{
Assert.Inconclusive("Parser Error");
}
var inspection = new VariableNotUsedInspection(parser.State);
var inspectionResults = inspection.GetInspectionResults();
Assert.AreEqual(0, inspectionResults.Count());
}
public void UnassignedVariable_InMultideclarationByStmtSeparators_WithFollowingCommentLine_DoesNotRemoveCommentLineOrOtherDeclarations()
{
const string inputCode =
@"Function Foo() As String
Dim var1 As String:Dim var2 As String
' Comment
var2 = ""Something""
Foo = var2
End Function";
const string expectedCode =
@"Function Foo() As String
Dim var2 As String
' Comment
var2 = ""Something""
Foo = var2
End Function";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out var component);
using (var state = MockParser.CreateAndParse(vbe.Object))
{
var inspection = new VariableNotUsedInspection(state);
new RemoveUnusedDeclarationQuickFix(state).Fix(inspection.GetInspectionResults(CancellationToken.None).First());
var rewriter = state.GetRewriter(component);
Assert.AreEqual(expectedCode, rewriter.GetText());
}
}
public void VariableNotUsed_ReturnsResult_MultipleVariables_SomeAssigned()
{
const string inputCode =
@"Sub Foo()
Dim var1 as Integer
var1 = 8
Dim var2 as String
Goo var1
End Sub
Sub Goo(ByVal arg1 As Integer)
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
using (var state = MockParser.CreateAndParse(vbe.Object))
{
var inspection = new VariableNotUsedInspection(state);
var inspectionResults = inspection.GetInspectionResults();
Assert.AreEqual(1, inspectionResults.Count());
}
}
public void UnassignedVariable_WithCommentOnSameLineAndFollowingStuff_DoesNotRemoveComment()
{
const string inputCode =
@"Function Foo() As String
Dim var1 As String ' Comment
Dim var2 As String
var2 = ""Something""
Foo = var2
End Function";
const string expectedCode =
@"Function Foo() As String
' Comment
Dim var2 As String
var2 = ""Something""
Foo = var2
End Function";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out var component);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state);
new RemoveUnusedDeclarationQuickFix(state).Fix(inspection.GetInspectionResults().First());
var rewriter = state.GetRewriter(component);
Assert.AreEqual(expectedCode, rewriter.GetText());
}
public void UnassignedVariable_ReturnsResult_QuickFixWorks()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
End Sub";
const string expectedCode =
@"Sub Foo()
End Sub";
//Arrange
var builder = new MockVbeBuilder();
VBComponent component;
var vbe = builder.BuildFromSingleStandardModule(inputCode, out component);
var project = vbe.Object.VBProjects.Item(0);
var module = project.VBComponents.Item(0).CodeModule;
var mockHost = new Mock <IHostApplication>();
mockHost.SetupAllProperties();
var parser = MockParser.Create(vbe.Object, new RubberduckParserState());
parser.Parse();
if (parser.State.Status == ParserState.Error)
{
Assert.Inconclusive("Parser Error");
}
var inspection = new VariableNotUsedInspection(parser.State);
inspection.GetInspectionResults().First().QuickFixes.First().Fix();
Assert.AreEqual(expectedCode, module.Lines());
}
public void InspectionName()
{
const string inspectionName = "VariableNotUsedInspection";
var inspection = new VariableNotUsedInspection(null, new Mock <IMessageBox>().Object);
Assert.AreEqual(inspectionName, inspection.Name);
}
public void InspectionName()
{
const string inspectionName = "VariableNotUsedInspection";
var inspection = new VariableNotUsedInspection(null);
Assert.AreEqual(inspectionName, inspection.Name);
}
public void UnassignedVariable_ReturnsResult_MultipleVariables()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
Dim var2 As Date
End Sub";
//Arrange
var builder = new MockVbeBuilder();
VBComponent component;
var vbe = builder.BuildFromSingleStandardModule(inputCode, out component);
var mockHost = new Mock <IHostApplication>();
mockHost.SetupAllProperties();
var parser = MockParser.Create(vbe.Object, new RubberduckParserState());
parser.Parse();
if (parser.State.Status == ParserState.Error)
{
Assert.Inconclusive("Parser Error");
}
var inspection = new VariableNotUsedInspection(parser.State);
var inspectionResults = inspection.GetInspectionResults();
Assert.AreEqual(2, inspectionResults.Count());
}
public void VariableNotUsed_Ignored_DoesNotReturnResult()
{
const string inputCode =
@"Sub Foo()
'@Ignore VariableNotUsed
Dim var1 As String
End Sub";
//Arrange
var builder = new MockVbeBuilder();
VBComponent component;
var vbe = builder.BuildFromSingleStandardModule(inputCode, out component);
var mockHost = new Mock <IHostApplication>();
mockHost.SetupAllProperties();
var parser = MockParser.Create(vbe.Object, new RubberduckParserState(new Mock <ISinks>().Object));
parser.Parse(new CancellationTokenSource());
if (parser.State.Status >= ParserState.Error)
{
Assert.Inconclusive("Parser Error");
}
var inspection = new VariableNotUsedInspection(parser.State);
var inspectionResults = inspection.GetInspectionResults();
Assert.IsFalse(inspectionResults.Any());
}
public void VariableNotUsed_ReturnsResult()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state);
var inspectionResults = inspection.GetInspectionResults();
Assert.AreEqual(1, inspectionResults.Count());
}
public void VariableNotUsed_DoesNotReturnsResult_UsedInNameStatement()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
Name ""foo"" As var1
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state);
var inspectionResults = inspection.GetInspectionResults();
Assert.IsFalse(inspectionResults.Any());
}
public void VariableNotUsed_ReturnsResult_MultipleVariables()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
Dim var2 As Date
End Sub";
IVBComponent component;
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out component);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state, new Mock <IMessageBox>().Object);
var inspectionResults = inspection.GetInspectionResults();
Assert.AreEqual(2, inspectionResults.Count());
}
public void VariableNotUsed_Ignored_DoesNotReturnResult()
{
const string inputCode =
@"Sub Foo()
'@Ignore VariableNotUsed
Dim var1 As String
End Sub";
IVBComponent component;
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out component);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state, new Mock <IMessageBox>().Object);
var inspectionResults = inspection.GetInspectionResults();
Assert.IsFalse(inspectionResults.Any());
}
public void VariableNotUsed_ReturnsResult_MultipleVariables()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
Dim var2 As Date
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
using (var state = MockParser.CreateAndParse(vbe.Object))
{
var inspection = new VariableNotUsedInspection(state);
var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
Assert.AreEqual(2, inspectionResults.Count());
}
}
public void VariableNotUsed_Ignored_DoesNotReturnResult()
{
const string inputCode =
@"Sub Foo()
'@Ignore VariableNotUsed
Dim var1 As String
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
using (var state = MockParser.CreateAndParse(vbe.Object))
{
var inspection = new VariableNotUsedInspection(state);
var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
Assert.IsFalse(inspectionResults.Any());
}
}
public void VariableUsed_DoesNotReturnResultIfAssigned()
{
const string inputCode =
@"Function Foo() As Boolean
Dim var1 as String
var1 = ""test""
End Function";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
using (var state = MockParser.CreateAndParse(vbe.Object))
{
var inspection = new VariableNotUsedInspection(state);
var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
Assert.AreEqual(0, inspectionResults.Count());
}
}
public void UnassignedVariable_QuickFixWorks()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
End Sub";
const string expectedCode =
@"Sub Foo()
End Sub";
IVBComponent component;
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out component);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state, new Mock <IMessageBox>().Object);
inspection.GetInspectionResults().First().QuickFixes.First().Fix();
Assert.AreEqual(expectedCode, component.CodeModule.Content());
}
public void VariableUsed_DoesNotReturnResult()
{
const string inputCode =
@"Function Foo() As Boolean
Dim var1 as String
var1 = ""test""
Goo var1
End Function
Sub Goo(ByVal arg1 As String)
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out _);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state);
var inspectionResults = inspection.GetInspectionResults();
Assert.AreEqual(0, inspectionResults.Count());
}
public void UnusedVariable_IgnoreQuickFixWorks()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
End Sub";
const string expectedCode =
@"Sub Foo()
'@Ignore VariableNotUsed
Dim var1 As String
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out var component);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state);
new IgnoreOnceQuickFix(state, new[] { inspection }).Fix(inspection.GetInspectionResults().First());
Assert.AreEqual(expectedCode, state.GetRewriter(component).GetText());
}
public void UnassignedVariable_QuickFixWorks()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
End Sub";
const string expectedCode =
@"Sub Foo()
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out var component);
using (var state = MockParser.CreateAndParse(vbe.Object))
{
var inspection = new VariableNotUsedInspection(state);
new RemoveUnusedDeclarationQuickFix(state).Fix(inspection.GetInspectionResults(CancellationToken.None).First());
var rewriter = state.GetRewriter(component);
Assert.AreEqual(expectedCode, rewriter.GetText());
}
}
public void UnassignedVariable_WithFollowingCommentLine_DoesNotRemoveCommentLine()
{
const string inputCode =
@"Sub Foo()
Dim var1 As String
' Comment
End Sub";
const string expectedCode =
@"Sub Foo()
' Comment
End Sub";
var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out var component);
var state = MockParser.CreateAndParse(vbe.Object);
var inspection = new VariableNotUsedInspection(state);
new RemoveUnusedDeclarationQuickFix(state).Fix(inspection.GetInspectionResults().First());
var rewriter = state.GetRewriter(component);
Assert.AreEqual(expectedCode, rewriter.GetText());
}
public void InspectionType()
{
var inspection = new VariableNotUsedInspection(null, new Mock <IMessageBox>().Object);
Assert.AreEqual(CodeInspectionType.CodeQualityIssues, inspection.InspectionType);
}
public void InspectionType()
{
var inspection = new VariableNotUsedInspection(null);
Assert.AreEqual(CodeInspectionType.CodeQualityIssues, inspection.InspectionType);
}
