Skip to content

Commit cf766e3

Browse files
Vogel612Vogel612
committed
Add Categories to 120 new tests
1 parent 9229fba commit cf766e3

File tree

9 files changed

+207
-94
lines changed

9 files changed

+207
-94
lines changed

‎RubberduckTests/Grammar/AnnotationTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,8 @@ public void InterfaceAnnotation_TypeIsInterface()
106106
Assert.AreEqual(AnnotationType.Interface, annotation.AnnotationType);
107107
}
108108

109+
[Category("Grammar")]
110+
[Category("Annotations")]
109111
[Test]
110112
public void DescriptionAnnotation_TypeIsDescription()
111113
{

‎RubberduckTests/IoCContainer/IoCRegistrationTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ namespace RubberduckTests.IoCContainer
1414
public class IoCRegistrationTests
1515
{
1616
[Test]
17+
[Category("IoC_Registration")]
1718
public void RegistrationOfRubberduckIoCContainerWithSC_NoException()
1819
{
1920
var vbeBuilder = new MockVbeBuilder();
@@ -36,6 +37,7 @@ public void RegistrationOfRubberduckIoCContainerWithSC_NoException()
3637
}
3738

3839
[Test]
40+
[Category("IoC_Registration")]
3941
public void RegistrationOfRubberduckIoCContainerWithoutSC_NoException()
4042
{
4143
var vbeBuilder = new MockVbeBuilder();

‎RubberduckTests/IoCContainer/IoCResolvingTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ namespace RubberduckTests.IoCContainer
1717
public class IoCResolvingTests
1818
{
1919
[Test]
20+
[Category("IoC_Registration")]
2021
public void ResolveInspections_NoException()
2122
{
2223
var vbeBuilder = new MockVbeBuilder();
@@ -53,6 +54,7 @@ public void ResolveInspections_NoException()
5354
}
5455

5556
[Test]
57+
[Category("IoC_Registration")]
5658
public void ResolveRubberduckParserState_NoException()
5759
{
5860
var vbeBuilder = new MockVbeBuilder();

‎RubberduckTests/Refactoring/RenameTests.cs

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -416,6 +416,8 @@ End Sub"
416416
//below can only be tested if implemented (and tested) within Excel.
417417

418418
[Test, Ignore("")]
419+
[Category("Refactorings")]
420+
[Category("Rename")]
419421
public void RenameRefactoring_RenameControlFromEventHandler()
420422
{
421423
var tdo = new RenameTestsDataObject(selection: "cmdBtn1", newName: "cmdBigButton");
@@ -449,6 +451,8 @@ End Sub"
449451
}
450452

451453
[Test, Ignore("")]
454+
[Category("Refactorings")]
455+
[Category("Rename")]
452456
public void RenameRefactoring_RenameControlFromEventHandlerNameCollision()
453457
{
454458
var tdo = new RenameTestsDataObject(selection: "cmdBtn1", newName: "cmdBigButton");
@@ -484,6 +488,8 @@ End Sub"
484488
}
485489

486490
[Test, Ignore("")]
491+
[Category("Refactorings")]
492+
[Category("Rename")]
487493
public void RenameRefactoring_RenameControlRenameInReference()
488494
{
489495
var tdo = new RenameTestsDataObject(selection: "cmdBtn1", newName: "cmdBigButton");
@@ -519,6 +525,8 @@ End Sub"
519525
}
520526

521527
[Test, Ignore("")]
528+
[Category("Refactorings")]
529+
[Category("Rename")]
522530
public void RenameRefactoring_RenameControlFromEventHandlerReference()
523531
{
524532
var tdo = new RenameTestsDataObject(selection: "cmdBtn1", newName: "cmdBigButton");
@@ -554,6 +562,8 @@ End Sub"
554562
}
555563

556564
[Test, Ignore("")]
565+
[Category("Refactorings")]
566+
[Category("Rename")]
557567
public void RenameRefactoring_RenameControlHandlesUnderscoresInNewName()
558568
{
559569
var tdo = new RenameTestsDataObject(selection: "bigButton_ClickAgain", newName: "bigButton_ClickAgain_AndAgain");
@@ -571,6 +581,8 @@ End Sub"
571581
}
572582

573583
[Test, Ignore("")]
584+
[Category("Refactorings")]
585+
[Category("Rename")]
574586
public void RenameRefactoring_RenameControlSimilarNames()
575587
{
576588
var tdo = new RenameTestsDataObject(selection: "bigButton", newName: "smallButton");

‎RubberduckTests/Settings/HotkeyFactoryTests.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ namespace RubberduckTests.Settings
1111
public class HotkeyFactoryTests
1212
{
1313
[Test]
14+
[Category("Hotkeys")]
1415
public void CreatingHotkeyReturnsNullWhenNoSettingProvided()
1516
{
1617
var factory = new HotkeyFactory(null);
@@ -21,6 +22,7 @@ public void CreatingHotkeyReturnsNullWhenNoSettingProvided()
2122
}
2223

2324
[Test]
25+
[Category("Hotkeys")]
2426
public void CreatingHotkeyReturnsNullWhenNoMatchingCommandExists()
2527
{
2628
var mockCommand = new Mock<CommandBase>(null).Object;
@@ -33,6 +35,7 @@ public void CreatingHotkeyReturnsNullWhenNoMatchingCommandExists()
3335
}
3436

3537
[Test]
38+
[Category("Hotkeys")]
3639
public void CreatingHotkeyReturnsCorrectResult()
3740
{
3841
var mockCommand = new Mock<CommandBase>(null).Object;

‎RubberduckTests/VBEditor/ComSafeManagerTests.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@ namespace RubberduckTests.VBEditor
66
[TestFixture()]
77
public class ComSafeManagerTests
88
{
9-
[Test()]
9+
[Test]
10+
[Category("COM")]
1011
public void ComSafeReturnedOnSecondIvokationOfGetCurrentComSafeIsTheSame()
1112
{
1213
ComSafeManager.DisposeAndResetComSafe(); //Resetting to get a claen start.
@@ -17,7 +18,8 @@ public void ComSafeReturnedOnSecondIvokationOfGetCurrentComSafeIsTheSame()
1718
Assert.AreSame(comSafe1, comSafe2);
1819
}
1920

20-
[Test()]
21+
[Test]
22+
[Category("COM")]
2123
public void AfterCallingResetComSafeGetCurrentComSafeReturnsDifferentSafe()
2224
{
2325
ComSafeManager.DisposeAndResetComSafe(); //Resetting to get a claen start.

‎RubberduckTests/VBEditor/ComSafeTests.cs

Lines changed: 26 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@ public abstract class ComSafeTestBase
1010
{
1111
protected abstract IComSafe TestComSafe();
1212

13-
[Test()]
13+
[Test]
14+
[Category("COM")]
1415
public void TryRemoveOnNewComSafe_ReturnsFalse()
1516
{
1617
var comSafe = TestComSafe();
@@ -20,7 +21,8 @@ public void TryRemoveOnNewComSafe_ReturnsFalse()
2021
Assert.IsFalse(result);
2122
}
2223

23-
[Test()]
24+
[Test]
25+
[Category("COM")]
2426
public void TryRemoveWithItemPreviouslyAddedToComSafe_ReturnsTrue()
2527
{
2628
var comSafe = TestComSafe();
@@ -31,7 +33,8 @@ public void TryRemoveWithItemPreviouslyAddedToComSafe_ReturnsTrue()
3133
Assert.IsTrue(result);
3234
}
3335

34-
[Test()]
36+
[Test]
37+
[Category("COM")]
3538
public void TryRemoveWithOtherItemPreviouslyAddedToComSafe_ReturnsFalse()
3639
{
3740
var comSafe = TestComSafe();
@@ -43,7 +46,8 @@ public void TryRemoveWithOtherItemPreviouslyAddedToComSafe_ReturnsFalse()
4346
Assert.IsFalse(result);
4447
}
4548

46-
[Test()]
49+
[Test]
50+
[Category("COM")]
4751
public void TryRemoveWithItemAndOtherItemPreviouslyAddedToComSafe_ReturnsTrue()
4852
{
4953
var comSafe = TestComSafe();
@@ -56,7 +60,8 @@ public void TryRemoveWithItemAndOtherItemPreviouslyAddedToComSafe_ReturnsTrue()
5660
Assert.IsTrue(result);
5761
}
5862

59-
[Test()]
63+
[Test]
64+
[Category("COM")]
6065
public void SecondTryRemoveWithItemPreviouslyAddedToComSafe_ReturnsFalse()
6166
{
6267
var comSafe = TestComSafe();
@@ -68,7 +73,8 @@ public void SecondTryRemoveWithItemPreviouslyAddedToComSafe_ReturnsFalse()
6873
Assert.IsFalse(result);
6974
}
7075

71-
[Test()]
76+
[Test]
77+
[Category("COM")]
7278
public void SecondTryRemoveWithItemPreviouslyAddedToComSafeTwice_ReturnsFalse()
7379
{
7480
var comSafe = TestComSafe();
@@ -81,7 +87,8 @@ public void SecondTryRemoveWithItemPreviouslyAddedToComSafeTwice_ReturnsFalse()
8187
Assert.IsFalse(result);
8288
}
8389

84-
[Test()]
90+
[Test]
91+
[Category("COM")]
8592
public void AddedSafeComWrapperGetsDisposedOnDisposalOfComSafe()
8693
{
8794
var comSafe = TestComSafe();
@@ -95,7 +102,8 @@ public void AddedSafeComWrapperGetsDisposedOnDisposalOfComSafe()
95102
mock.Verify(wrapper => wrapper.Dispose(), Times.Once);
96103
}
97104

98-
[Test()]
105+
[Test]
106+
[Category("COM")]
99107
public void SafeComWrapperAddedTwiceGetsDisposedOnceOnDisposalOfComSafe()
100108
{
101109
var comSafe = TestComSafe();
@@ -110,7 +118,8 @@ public void SafeComWrapperAddedTwiceGetsDisposedOnceOnDisposalOfComSafe()
110118
mock.Verify(wrapper => wrapper.Dispose(), Times.Once);
111119
}
112120

113-
[Test()]
121+
[Test]
122+
[Category("COM")]
114123
public void RemovedSafeComWrapperDoesNotGetDisposedOnDisposalOfComSafe()
115124
{
116125
var comSafe = TestComSafe();
@@ -125,7 +134,8 @@ public void RemovedSafeComWrapperDoesNotGetDisposedOnDisposalOfComSafe()
125134
mock.Verify(wrapper => wrapper.Dispose(), Times.Never);
126135
}
127136

128-
[Test()]
137+
[Test]
138+
[Category("COM")]
129139
public void SafeComWrapperRemovedAfterHavingBeenAddedTwiceDoesNotGetDisposedOnDisposalOfComSafe()
130140
{
131141
var comSafe = TestComSafe();
@@ -141,7 +151,8 @@ public void SafeComWrapperRemovedAfterHavingBeenAddedTwiceDoesNotGetDisposedOnDi
141151
mock.Verify(wrapper => wrapper.Dispose(), Times.Never);
142152
}
143153

144-
[Test()]
154+
[Test]
155+
[Category("COM")]
145156
public void AddedSafeComWrapperGetsDisposedOnDisposalOfAfterOtherItemGotRemovedComSafe()
146157
{
147158
var comSafe = TestComSafe();
@@ -158,7 +169,8 @@ public void AddedSafeComWrapperGetsDisposedOnDisposalOfAfterOtherItemGotRemovedC
158169
mock.Verify(wrapper => wrapper.Dispose(), Times.Once);
159170
}
160171

161-
[Test()]
172+
[Test]
173+
[Category("COM")]
162174
public void AddedSafeComWrapperDoesNotGetDisposedAgainOnSecondDisposalOfComSafe()
163175
{
164176
var comSafe = TestComSafe();
@@ -173,7 +185,8 @@ public void AddedSafeComWrapperDoesNotGetDisposedAgainOnSecondDisposalOfComSafe(
173185
mock.Verify(wrapper => wrapper.Dispose(), Times.Once);
174186
}
175187

176-
[Test()]
188+
[Test]
189+
[Category("COM")]
177190
public void AfterDisposalTryRemoveReturnsFalseForAddedItem()
178191
{
179192
var comSafe = TestComSafe();

0 commit comments

Comments
 (0)