From f27da3bc5dbab4b9d3886ff8584f777abb1b4c73 Mon Sep 17 00:00:00 2001 From: Antonio Aversa Date: Fri, 24 Feb 2023 18:00:29 +0100 Subject: [PATCH] Code review 3 --- ...ouldBePrivateAndReadonly.CSharp9.TopLevelStatements.cs | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/analyzers/tests/SonarAnalyzer.UnitTest/TestCases/LockedFieldShouldBePrivateAndReadonly.CSharp9.TopLevelStatements.cs b/analyzers/tests/SonarAnalyzer.UnitTest/TestCases/LockedFieldShouldBePrivateAndReadonly.CSharp9.TopLevelStatements.cs index 3fcd5a91737..a31ca9aa2f6 100644 --- a/analyzers/tests/SonarAnalyzer.UnitTest/TestCases/LockedFieldShouldBePrivateAndReadonly.CSharp9.TopLevelStatements.cs +++ b/analyzers/tests/SonarAnalyzer.UnitTest/TestCases/LockedFieldShouldBePrivateAndReadonly.CSharp9.TopLevelStatements.cs @@ -2,9 +2,17 @@ // ^^^^^^^^^^ lock (MethodReturningString()) { } // Noncompliant lock (new object()) { } // Noncompliant +lock (staticReadonlyField) { } +lock (staticReadWriteField) {} // Noncompliant string MethodReturningString() => "a string"; +partial class Program +{ + static readonly object staticReadonlyField = new(); + static object staticReadWriteField = new(); +} + class ClassNestedAtTopLevel { readonly object readonlyField = new();