Skip to content

Commit 68eb7ef

Browse files
Update test to proof leading trivia is maintained
Add documentation to solution
1 parent 9231d99 commit 68eb7ef

File tree

2 files changed

+12
-11
lines changed

2 files changed

+12
-11
lines changed

src/nunit.analyzers.sln

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "documentation", "documentat
4343
..\documentation\NUnit1031.md = ..\documentation\NUnit1031.md
4444
..\documentation\NUnit1032.md = ..\documentation\NUnit1032.md
4545
..\documentation\NUnit1033.md = ..\documentation\NUnit1033.md
46+
..\documentation\NUnit1034.md = ..\documentation\NUnit1034.md
4647
..\documentation\NUnit2001.md = ..\documentation\NUnit2001.md
4748
..\documentation\NUnit2002.md = ..\documentation\NUnit2002.md
4849
..\documentation\NUnit2003.md = ..\documentation\NUnit2003.md

src/nunit.analyzers.tests/TestFixtureShouldBeAbstract/TestFixtureShouldBeAbstractCodeFixTests.cs

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -15,41 +15,41 @@ public class TestFixtureShouldBeAbstractCodeFixTests
1515
private static readonly ExpectedDiagnostic expectedDiagnostic =
1616
ExpectedDiagnostic.Create(AnalyzerIdentifiers.BaseTestFixtureIsNotAbstract);
1717

18-
private static readonly IEnumerable<string> testMethodRelatedAttributes =
18+
private static readonly IEnumerable<string[]> testMethodRelatedAttributes =
1919
[
20-
"OneTimeSetUp",
21-
"OneTimeTearDown",
22-
"SetUp",
23-
"TearDown",
24-
"Test",
20+
["", "OneTimeSetUp"],
21+
["internal ", "OneTimeTearDown"],
22+
["public ", "SetUp"],
23+
["partial ", "TearDown"],
24+
["internal partial ", "Test"],
2525
];
2626

2727
[TestCaseSource(nameof(testMethodRelatedAttributes))]
28-
public void FixWhenBaseFixtureIsNotAbstract(string attribute)
28+
public void FixWhenBaseFixtureIsNotAbstract(string modifiers, string attribute)
2929
{
3030
var testCode = TestUtility.WrapClassInNamespaceAndAddUsing($@"
3131
// Base Fixture
32-
public class ↓BaseFixture
32+
{modifiers}class ↓BaseFixture
3333
{{
3434
[{attribute}]
3535
public void BaseFixtureMethod() {{ }}
3636
}}
3737
38-
public class DerivedFixture : BaseFixture
38+
{modifiers}class DerivedFixture : BaseFixture
3939
{{
4040
[Test]
4141
public void DerivedFixtureMethod() {{ }}
4242
}}");
4343

4444
var fixedCode = TestUtility.WrapClassInNamespaceAndAddUsing($@"
4545
// Base Fixture
46-
public abstract class BaseFixture
46+
{modifiers}abstract class BaseFixture
4747
{{
4848
[{attribute}]
4949
public void BaseFixtureMethod() {{ }}
5050
}}
5151
52-
public class DerivedFixture : BaseFixture
52+
{modifiers}class DerivedFixture : BaseFixture
5353
{{
5454
[Test]
5555
public void DerivedFixtureMethod() {{ }}

0 commit comments

Comments
 (0)