Skip to content

Commit

Permalink
tests had the wrong expectations
Browse files Browse the repository at this point in the history
  • Loading branch information
retailcoder committed Jan 22, 2018
1 parent dc927b7 commit 1a13a0d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
Expand Up @@ -177,7 +177,7 @@ public void ObjectVariableNotSet_GivenVariantVariableAssignedNewObject_ReturnsRe
[Category("Inspections")]
public void ObjectVariableNotSet_GivenVariantVariableAssignedRange_ReturnsNoResult()
{
var expectResultCount = 1;
var expectResultCount = 0;
var input =
@"
Private Sub TestSub(ByRef testParam As Variant)
Expand All @@ -190,7 +190,7 @@ public void ObjectVariableNotSet_GivenVariantVariableAssignedRange_ReturnsNoResu
[Category("Inspections")]
public void ObjectVariableNotSet_GivenVariantVariableAssignedDeclaredRange_ReturnsNoResult()
{
var expectResultCount = 1;
var expectResultCount = 0;
var input =
@"
Private Sub TestSub(ByRef testParam As Variant, target As Range)
Expand Down
12 changes: 6 additions & 6 deletions RubberduckTests/QuickFixes/IgnoreOnceQuickFixTests.cs
Expand Up @@ -586,21 +586,21 @@ public void ObjectVariableNotSet_IgnoreQuickFixWorks()
{
var inputCode =
@"
Private Sub Workbook_Open()
Private Sub DoSomething()
Dim target As Range
target = Range(""A1"")
Dim target As Object
target = New Object
target.Value = ""forgot something?""
End Sub";
var expectedCode =
@"
Private Sub Workbook_Open()
Private Sub DoSomething()
Dim target As Range
Dim target As Object
'@Ignore ObjectVariableNotSet
target = Range(""A1"")
target = New Object
target.Value = ""forgot something?""
Expand Down

0 comments on commit 1a13a0d

Please sign in to comment.