rubberduck-vba
diff --git a/‎Rubberduck.Inspections/Abstract/InspectionBase.cs
Lines changed: 3 additions & 1 deletion b/‎Rubberduck.Inspections/Abstract/InspectionBase.cs
Lines changed: 3 additions & 1 deletion
diff --git a/‎Rubberduck.Inspections/Inspector.cs
Lines changed: 1 addition & 1 deletion b/‎Rubberduck.Inspections/Inspector.cs
Lines changed: 1 addition & 1 deletion
diff --git a/‎Rubberduck.Parsing/Inspections/Abstract/IInspection.cs
Lines changed: 3 additions & 1 deletion b/‎Rubberduck.Parsing/Inspections/Abstract/IInspection.cs
Lines changed: 3 additions & 1 deletion
diff --git a/‎RubberduckTests/Inspections/ApplicationWorksheetFunctionInspectionTests.cs
Lines changed: 8 additions & 8 deletions b/‎RubberduckTests/Inspections/ApplicationWorksheetFunctionInspectionTests.cs
Lines changed: 8 additions & 8 deletions
diff --git a/‎RubberduckTests/Inspections/AssignedByValParameterInspectionTests.cs
Lines changed: 8 additions & 7 deletions b/‎RubberduckTests/Inspections/AssignedByValParameterInspectionTests.cs
Lines changed: 8 additions & 7 deletions
diff --git a/‎RubberduckTests/Inspections/ConstantNotUsedInspectionTests.cs
Lines changed: 9 additions & 8 deletions b/‎RubberduckTests/Inspections/ConstantNotUsedInspectionTests.cs
Lines changed: 9 additions & 8 deletions
diff --git a/‎RubberduckTests/Inspections/DefaultProjectNameInspectionTests.cs
Lines changed: 3 additions & 2 deletions b/‎RubberduckTests/Inspections/DefaultProjectNameInspectionTests.cs
Lines changed: 3 additions & 2 deletions
@@ -9,6 +9,7 @@
 using Rubberduck.Parsing.VBA;
 using Rubberduck.VBEditor;
 using System.Diagnostics;
+using System.Threading;
 using NLog;
 
 namespace Rubberduck.Inspections.Abstract
@@ -156,8 +157,9 @@ public int CompareTo(object obj)
         /// <summary>
         /// A method that inspects the parser state and returns all issues it can find.
         /// </summary>
+        /// <param name="token"></param>
         /// <returns></returns>
-        public IEnumerable<IInspectionResult> GetInspectionResults()
+        public IEnumerable<IInspectionResult> GetInspectionResults(CancellationToken token)
         {
             var _stopwatch = new Stopwatch();
             _stopwatch.Start();
 
@@ -156,7 +156,7 @@ private static void RunInspection(IInspection inspection, ConcurrentBag<IInspect
             {
                 try
                 {
-                    var inspectionResults = inspection.GetInspectionResults();
+                    var inspectionResults = inspection.GetInspectionResults(token);
 
                     token.ThrowIfCancellationRequested();
 
 
@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using System.Threading;
 
 namespace Rubberduck.Parsing.Inspections.Abstract
 {
@@ -11,7 +12,8 @@ public interface IInspection : IInspectionModel, IComparable<IInspection>, IComp
         /// <summary>
         /// Runs code inspection and returns inspection results.
         /// </summary>
+        /// <param name="token"></param>
         /// <returns>Returns inspection results, if any.</returns>
-        IEnumerable<IInspectionResult> GetInspectionResults();
+        IEnumerable<IInspectionResult> GetInspectionResults(CancellationToken token);
     }
 }
@@ -51,7 +51,7 @@ End Sub
             {
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -75,7 +75,7 @@ End Sub
             {
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -99,7 +99,7 @@ End Sub
             {
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -125,7 +125,7 @@ End Sub
             {
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -147,7 +147,7 @@ End Sub
             {
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
@@ -171,7 +171,7 @@ End Sub
             {
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
@@ -202,7 +202,7 @@ End Sub
                 }
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
@@ -225,7 +225,7 @@ End Sub
             {
 
                 var inspection = new ApplicationWorksheetFunctionInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
 
@@ -1,4 +1,5 @@
 using System.Linq;
+using System.Threading;
 using NUnit.Framework;
 using RubberduckTests.Mocks;
 using Rubberduck.Inspections.Concrete;
@@ -23,7 +24,7 @@ public void AssignedByValParameter_ReturnsResult_Sub()
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new AssignedByValParameterInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -43,7 +44,7 @@ public void AssignedByValParameter_ReturnsResult_Function()
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new AssignedByValParameterInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -64,7 +65,7 @@ public void AssignedByValParameter_ReturnsResult_MultipleParams()
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new AssignedByValParameterInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(2, inspectionResults.Count());
             }
@@ -83,7 +84,7 @@ public void AssignedByValParameter_DoesNotReturnResult()
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new AssignedByValParameterInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
@@ -104,7 +105,7 @@ Public Sub Foo(ByVal arg1 As String)
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new AssignedByValParameterInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
@@ -127,7 +128,7 @@ Dim var1 As Integer
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new AssignedByValParameterInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -163,7 +164,7 @@ End Sub
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new AssignedByValParameterInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
 
@@ -1,4 +1,5 @@
 using System.Linq;
+using System.Threading;
 using NUnit.Framework;
 using Rubberduck.Inspections.Concrete;
 using RubberduckTests.Mocks;
@@ -22,7 +23,7 @@ public void ConstantNotUsed_ReturnsResult()
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -43,7 +44,7 @@ public void ConstantNotUsed_ReturnsResult_MultipleConsts()
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(2, inspectionResults.Count());
             }
@@ -69,7 +70,7 @@ Public Sub Goo(ByVal arg1 As Integer)
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -93,7 +94,7 @@ Public Sub Goo(ByVal arg1 As Integer)
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(0, inspectionResults.Count());
             }
@@ -115,7 +116,7 @@ Public Sub Foo()
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
@@ -137,7 +138,7 @@ Public Sub Foo()
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
@@ -159,7 +160,7 @@ Public Sub Foo()
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsTrue(inspectionResults.Any());
             }
@@ -180,7 +181,7 @@ public void ConstantNotUsed_Ignored_DoesNotReturnResult()
             {
 
                 var inspection = new ConstantNotUsedInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.IsFalse(inspectionResults.Any());
             }
 
@@ -1,4 +1,5 @@
 using System.Linq;
+using System.Threading;
 using NUnit.Framework;
 using Rubberduck.Inspections.Concrete;
 using Rubberduck.VBEditor.SafeComWrappers;
@@ -22,7 +23,7 @@ public void DefaultProjectName_ReturnsResult()
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new DefaultProjectNameInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(1, inspectionResults.Count());
             }
@@ -41,7 +42,7 @@ public void DefaultProjectName_DoesNotReturnResult()
             using (var state = MockParser.CreateAndParse(vbe.Object))
             {
                 var inspection = new DefaultProjectNameInspection(state);
-                var inspectionResults = inspection.GetInspectionResults();
+                var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);
 
                 Assert.AreEqual(0, inspectionResults.Count());
             }
Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ private static void RunInspection(IInspection inspection, ConcurrentBag<IInspect`
`156`	`156`	`{`
`157`	`157`	`try`
`158`	`158`	`{`
`159`		`- var inspectionResults = inspection.GetInspectionResults();`
	`159`	`+ var inspectionResults = inspection.GetInspectionResults(token);`
`160`	`160`
`161`	`161`	`token.ThrowIfCancellationRequested();`
`162`	`162`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`using System;`
`2`	`2`	`using System.Collections.Generic;`
	`3`	`+using System.Threading;`
`3`	`4`
`4`	`5`	`namespace Rubberduck.Parsing.Inspections.Abstract`
`5`	`6`	`{`
`@@ -11,7 +12,8 @@ public interface IInspection : IInspectionModel, IComparable<IInspection>, IComp`
`11`	`12`	`/// <summary>`
`12`	`13`	`/// Runs code inspection and returns inspection results.`
`13`	`14`	`/// </summary>`
	`15`	`+ /// <param name="token"></param>`
`14`	`16`	`/// <returns>Returns inspection results, if any.</returns>`
`15`		`- IEnumerable<IInspectionResult> GetInspectionResults();`
	`17`	`+ IEnumerable<IInspectionResult> GetInspectionResults(CancellationToken token);`
`16`	`18`	`}`
`17`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ End Sub`
`51`	`51`	`{`
`52`	`52`
`53`	`53`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`54`		`- var inspectionResults = inspection.GetInspectionResults();`
	`54`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`55`	`55`
`56`	`56`	`Assert.AreEqual(1, inspectionResults.Count());`
`57`	`57`	`}`
`@@ -75,7 +75,7 @@ End Sub`
`75`	`75`	`{`
`76`	`76`
`77`	`77`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`78`		`- var inspectionResults = inspection.GetInspectionResults();`
	`78`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`79`	`79`
`80`	`80`	`Assert.AreEqual(1, inspectionResults.Count());`
`81`	`81`	`}`
`@@ -99,7 +99,7 @@ End Sub`
`99`	`99`	`{`
`100`	`100`
`101`	`101`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`102`		`- var inspectionResults = inspection.GetInspectionResults();`
	`102`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`103`	`103`
`104`	`104`	`Assert.AreEqual(1, inspectionResults.Count());`
`105`	`105`	`}`
`@@ -125,7 +125,7 @@ End Sub`
`125`	`125`	`{`
`126`	`126`
`127`	`127`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`128`		`- var inspectionResults = inspection.GetInspectionResults();`
	`128`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`129`	`129`
`130`	`130`	`Assert.AreEqual(1, inspectionResults.Count());`
`131`	`131`	`}`
`@@ -147,7 +147,7 @@ End Sub`
`147`	`147`	`{`
`148`	`148`
`149`	`149`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`150`		`- var inspectionResults = inspection.GetInspectionResults();`
	`150`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`151`	`151`
`152`	`152`	`Assert.IsFalse(inspectionResults.Any());`
`153`	`153`	`}`
`@@ -171,7 +171,7 @@ End Sub`
`171`	`171`	`{`
`172`	`172`
`173`	`173`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`174`		`- var inspectionResults = inspection.GetInspectionResults();`
	`174`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`175`	`175`
`176`	`176`	`Assert.IsFalse(inspectionResults.Any());`
`177`	`177`	`}`
`@@ -202,7 +202,7 @@ End Sub`
`202`	`202`	`}`
`203`	`203`
`204`	`204`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`205`		`- var inspectionResults = inspection.GetInspectionResults();`
	`205`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`206`	`206`
`207`	`207`	`Assert.IsFalse(inspectionResults.Any());`
`208`	`208`	`}`
`@@ -225,7 +225,7 @@ End Sub`
`225`	`225`	`{`
`226`	`226`
`227`	`227`	`var inspection = new ApplicationWorksheetFunctionInspection(state);`
`228`		`- var inspectionResults = inspection.GetInspectionResults();`
	`228`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`229`	`229`
`230`	`230`	`Assert.IsFalse(inspectionResults.Any());`
`231`	`231`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`using System.Linq;`
	`2`	`+using System.Threading;`
`2`	`3`	`using NUnit.Framework;`
`3`	`4`	`using RubberduckTests.Mocks;`
`4`	`5`	`using Rubberduck.Inspections.Concrete;`
`@@ -23,7 +24,7 @@ public void AssignedByValParameter_ReturnsResult_Sub()`
`23`	`24`	`using (var state = MockParser.CreateAndParse(vbe.Object))`
`24`	`25`	`{`
`25`	`26`	`var inspection = new AssignedByValParameterInspection(state);`
`26`		`- var inspectionResults = inspection.GetInspectionResults();`
	`27`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`27`	`28`
`28`	`29`	`Assert.AreEqual(1, inspectionResults.Count());`
`29`	`30`	`}`
`@@ -43,7 +44,7 @@ public void AssignedByValParameter_ReturnsResult_Function()`
`43`	`44`	`using (var state = MockParser.CreateAndParse(vbe.Object))`
`44`	`45`	`{`
`45`	`46`	`var inspection = new AssignedByValParameterInspection(state);`
`46`		`- var inspectionResults = inspection.GetInspectionResults();`
	`47`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`47`	`48`
`48`	`49`	`Assert.AreEqual(1, inspectionResults.Count());`
`49`	`50`	`}`
`@@ -64,7 +65,7 @@ public void AssignedByValParameter_ReturnsResult_MultipleParams()`
`64`	`65`	`using (var state = MockParser.CreateAndParse(vbe.Object))`
`65`	`66`	`{`
`66`	`67`	`var inspection = new AssignedByValParameterInspection(state);`
`67`		`- var inspectionResults = inspection.GetInspectionResults();`
	`68`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`68`	`69`
`69`	`70`	`Assert.AreEqual(2, inspectionResults.Count());`
`70`	`71`	`}`
`@@ -83,7 +84,7 @@ public void AssignedByValParameter_DoesNotReturnResult()`
`83`	`84`	`using (var state = MockParser.CreateAndParse(vbe.Object))`
`84`	`85`	`{`
`85`	`86`	`var inspection = new AssignedByValParameterInspection(state);`
`86`		`- var inspectionResults = inspection.GetInspectionResults();`
	`87`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`87`	`88`
`88`	`89`	`Assert.IsFalse(inspectionResults.Any());`
`89`	`90`	`}`
`@@ -104,7 +105,7 @@ Public Sub Foo(ByVal arg1 As String)`
`104`	`105`	`using (var state = MockParser.CreateAndParse(vbe.Object))`
`105`	`106`	`{`
`106`	`107`	`var inspection = new AssignedByValParameterInspection(state);`
`107`		`- var inspectionResults = inspection.GetInspectionResults();`
	`108`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`108`	`109`
`109`	`110`	`Assert.IsFalse(inspectionResults.Any());`
`110`	`111`	`}`
`@@ -127,7 +128,7 @@ Dim var1 As Integer`
`127`	`128`	`using (var state = MockParser.CreateAndParse(vbe.Object))`
`128`	`129`	`{`
`129`	`130`	`var inspection = new AssignedByValParameterInspection(state);`
`130`		`- var inspectionResults = inspection.GetInspectionResults();`
	`131`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`131`	`132`
`132`	`133`	`Assert.AreEqual(1, inspectionResults.Count());`
`133`	`134`	`}`
`@@ -163,7 +164,7 @@ End Sub`
`163`	`164`	`using (var state = MockParser.CreateAndParse(vbe.Object))`
`164`	`165`	`{`
`165`	`166`	`var inspection = new AssignedByValParameterInspection(state);`
`166`		`- var inspectionResults = inspection.GetInspectionResults();`
	`167`	`+ var inspectionResults = inspection.GetInspectionResults(CancellationToken.None);`
`167`	`168`
`168`	`169`	`Assert.IsFalse(inspectionResults.Any());`
`169`	`170`	`}`