Update RSPEC + more test cases

SonarSource · Apr 30, 2024 · 52bf94d · 52bf94d
1 parent e9cf7d8
commit 52bf94d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/analyzers/rspec/cs/S6964.json b/analyzers/rspec/cs/S6964.json
@@ -1,5 +1,5 @@
 {
-  "title": "Value type properties of a model class should be nullable or marked as \"Required\" to avoid under-posting",
+  "title": "Property used as input in a controller action should be nullable or annotated with the Required attribute to avoid under-posting.",
   "type": "CODE_SMELL",
   "status": "ready",
   "remediation": {

diff --git a/analyzers/tests/SonarAnalyzer.Test/TestCases/AspNet/AvoidUnderPosting.cs b/analyzers/tests/SonarAnalyzer.Test/TestCases/AspNet/AvoidUnderPosting.cs
@@ -173,12 +173,13 @@ public class Model<T> where T : Person
 
     public abstract class Person
     {
-        public int Age { get; set; }            // Noncompliant
+        public int Age { get; set; }                    // Noncompliant
     }
 
     public class Developer : Person
     {
         public string ProgramingLanguage { get; set; }
+        public int WorkExperienceInYears { get; set; }  // Noncompliant
     }
 }
 
@@ -194,7 +195,7 @@ public void Create(MyModel model)
     public class Model<T> where T : Model<T>
     {
         public Model<T> SubModel { get; set; }
-        public int ValueProperty { get; set; }  // Noncompliant
+        public int ValueProperty { get; set; }          // Noncompliant
     }
 
     public class MyModel : Model<MyModel>