Skip to content

Commit fc0a266

Browse files
jonathanalgarjonathanalgar
committed
tweak comments
1 parent d6e2c67 commit fc0a266

1 file changed

Lines changed: 2 additions & 27 deletions

File tree

‎tests/CustomCode-Analyzer.Tests/AnalyzerTests.cs‎

Lines changed: 2 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ public class AnalyzerTests
88
{
99
public TestContext TestContext { get; set; } = null!;
1010

11+
// https://www.outsystems.com/tk/redirect?g=OS-ELG-MODL-05001 - not implementing
12+
1113
// --------------- NoSingleInterfaceRule (OS-ELG-MODL-05002) ---------------
1214
[TestMethod]
1315
public async Task NoSingleInterfaceRule_InGlobalScope_ReportsWarning()
@@ -38,8 +40,6 @@ public interface ITestInterface
3840
.WithSpan(4, 5, 7, 6);
3941

4042
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
41-
}
42-
// -------------------------------------------------------------------------
4343

4444
// --------------- ManyInterfacesRule (OS-ELG-MODL-05003) ------------------
4545
[TestMethod]
@@ -146,7 +146,6 @@ public void TestMethod() { }
146146

147147
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
148148
}
149-
// -------------------------------------------------------------------------
150149

151150
// --------------- NonPublicInterfaceRule (OS-ELG-MODL-05004) --------------
152151
[TestMethod]
@@ -193,7 +192,6 @@ public void TestMethod() { }
193192

194193
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
195194
}
196-
// -------------------------------------------------------------------------
197195

198196
// --------------- NonInstantiableInterfaceRule (OS-ELG-MODL-05005) --------
199197
[TestMethod]
@@ -248,7 +246,6 @@ public void TestMethod() { }
248246

249247
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
250248
}
251-
// -------------------------------------------------------------------------
252249

253250
// --------------- MissingImplementationRule (OS-ELG-MODL-05006) -----------
254251
[TestMethod]
@@ -288,7 +285,6 @@ public interface ITestInterface
288285
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
289286
}
290287

291-
// -------------------------------------------------------------------------
292288

293289
// --------------- EmptyInterfaceRule (OS-ELG-MODL-05007) ------------------
294290
[TestMethod]
@@ -337,7 +333,6 @@ public interface ITestInterface
337333

338334
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
339335
}
340-
// -------------------------------------------------------------------------
341336

342337
// --------------- ManyImplementationRule (OS-ELG-MODL-05008) --------------
343338
[TestMethod]
@@ -429,7 +424,6 @@ public void TestMethod() { }
429424

430425
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
431426
}
432-
// -------------------------------------------------------------------------
433427

434428
// https://www.outsystems.com/tk/redirect?g=OS-ELG-MODL-05009 - not implementing
435429

@@ -470,7 +464,6 @@ internal struct TestStruct
470464

471465
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
472466
}
473-
// -------------------------------------------------------------------------
474467

475468
// --------------- NonPublicStructureFieldRule (OS-ELG-MODL-05011) ------------
476469
[TestMethod]
@@ -537,7 +530,6 @@ public struct TestStruct
537530

538531
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
539532
}
540-
// -------------------------------------------------------------------------
541533

542534
// --------------- NonPublicIgnoredFieldRule (OS-ELG-MODL-05012) -----------
543535
[TestMethod]
@@ -606,7 +598,6 @@ public struct TestStruct
606598

607599
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
608600
}
609-
// -------------------------------------------------------------------------
610601

611602
// --------------- EmptyStructureRule (OS-ELG-MODL-05013) -----------
612603
[TestMethod]
@@ -649,7 +640,6 @@ public struct TestStruct
649640

650641
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
651642
}
652-
// -------------------------------------------------------------------------
653643

654644
// https://www.outsystems.com/tk/redirect?g=OS-ELG-MODL-05014 - TODO: implement
655645

@@ -814,7 +804,6 @@ public struct UnsupportedType { }
814804
";
815805
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false);
816806
}
817-
// -------------------------------------------------------------------------
818807

819808
// --------------- ParameterByReferenceRule (OS-ELG-MODL-05016) ------------
820809
[TestMethod]
@@ -902,7 +891,6 @@ public void ReadValue(in double number) { }
902891

903892
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
904893
}
905-
// -------------------------------------------------------------------------
906894

907895
// --------------- UnsupportedTypeMappingRule (OS-ELG-MODL-05017) ----------
908896
[TestMethod]
@@ -945,7 +933,6 @@ public struct TestStruct
945933

946934
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
947935
}
948-
// -------------------------------------------------------------------------
949936

950937
// --------------- MissingPublicImplementationRule (OS-ELG-MODL-05018) --------
951938
[TestMethod]
@@ -999,7 +986,6 @@ public void TestMethod() { }
999986

1000987
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
1001988
}
1002-
// -------------------------------------------------------------------------
1003989

1004990
// --------------- NameMaxLengthExceededRule (OS-ELG-MODL-05019) -----------
1005991
[TestMethod]
@@ -1075,7 +1061,6 @@ public void Method() { }
10751061

10761062
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
10771063
}
1078-
// -------------------------------------------------------------------------
10791064

10801065
// --------------- NameBeginsWithNumbersRule (OS-ELG-MODL-05020) -----------
10811066
[TestMethod]
@@ -1099,8 +1084,6 @@ public void Method() { }
10991084

11001085
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
11011086
}
1102-
// -------------------------------------------------------------------------
1103-
11041087

11051088
// --------------- UnsupportedCharactersInNameRule (OS-ELG-MODL-05021) -----
11061089
[TestMethod]
@@ -1124,7 +1107,6 @@ public void Method() { }
11241107

11251108
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
11261109
}
1127-
// -------------------------------------------------------------------------
11281110

11291111
// --------------- NameBeginsWithUnderscoresRule (OS-ELG-MODL-05022) -------
11301112
[TestMethod]
@@ -1189,7 +1171,6 @@ public void _TestMethod() { }
11891171

11901172
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
11911173
}
1192-
// -------------------------------------------------------------------------
11931174

11941175
// https://www.outsystems.com/tk/redirect?g=OS-ELG-MODL-05023 - not implementing
11951176

@@ -1227,7 +1208,6 @@ public struct MyStruct { }
12271208

12281209
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
12291210
}
1230-
// -------------------------------------------------------------------------
12311211

12321212
// --------------- DuplicateNameRule (OS-ELG-MODL-05025) -------------------
12331213
[TestMethod]
@@ -1261,7 +1241,6 @@ public struct Structure // Duplicate name in different namespace
12611241

12621242
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
12631243
}
1264-
// -------------------------------------------------------------------------
12651244

12661245
// --------------- UnsupportedDefaultValueRule (OS-ELG-MODL-05026) ---------
12671246
[TestMethod]
@@ -1330,7 +1309,6 @@ public void TestMethod(string text1 = Constants.DefaultText,
13301309

13311310
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
13321311
}
1333-
// -------------------------------------------------------------------------
13341312

13351313
// https://www.outsystems.com/tk/redirect?g=OS-ELG-MODL-05027 - not implementing
13361314

@@ -1410,7 +1388,6 @@ public void ProcessMetadata(string metadata) { }
14101388

14111389
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: false, expected);
14121390
}
1413-
// -------------------------------------------------------------------------
14141391

14151392
// ----------------------------------------------- MIXED TESTS!
14161393
[TestMethod]
@@ -1609,7 +1586,6 @@ public List<ResultStruct> ProcessData(InputStruct data)
16091586
}
16101587

16111588
// ----------------------------------------------- OTHER TESTS!
1612-
16131589
[TestMethod]
16141590
public async Task AnalyzerRules_Disabled_IfSDKNotAvailable()
16151591
{
@@ -1630,7 +1606,6 @@ public int Add(int a, int b)
16301606
await CSharpAnalyzerVerifier<Analyzer>.VerifyAnalyzerAsync(test, TestContext, skipSDKreference: true);
16311607
}
16321608

1633-
16341609
[TestMethod]
16351610
public async Task ValidImplementation_DeeperNamespace_NoWarning()
16361611
{

0 commit comments

Comments
 (0)