Issue checkstyle#15456: Missing Ctor check

romani · Jan 26, 2025 · 838bc73 · 838bc73
1 parent 9d8aa3d
commit 838bc73
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/bdd/InlineConfigParser.java b/src/test/java/com/puppycrawl/tools/checkstyle/bdd/InlineConfigParser.java
@@ -217,7 +217,6 @@ public final class InlineConfigParser {
             "com.puppycrawl.tools.checkstyle.checks.coding.IllegalTypeCheck",
             "com.puppycrawl.tools.checkstyle.checks.coding.MagicNumberCheck",
             "com.puppycrawl.tools.checkstyle.checks.coding.MatchXpathCheck",
-            "com.puppycrawl.tools.checkstyle.checks.coding.MissingCtorCheck",
             "com.puppycrawl.tools.checkstyle.checks.coding.MissingSwitchDefaultCheck",
             "com.puppycrawl.tools.checkstyle.checks.coding.ModifiedControlVariableCheck",
             "com.puppycrawl.tools.checkstyle.checks.coding.MultipleStringLiteralsCheck",

diff --git a/...com/puppycrawl/tools/checkstyle/checks/coding/missingctor/InputMissingCtorEmptyClass.java b/...com/puppycrawl/tools/checkstyle/checks/coding/missingctor/InputMissingCtorEmptyClass.java
@@ -6,7 +6,7 @@
 
 package com.puppycrawl.tools.checkstyle.checks.coding.missingctor;
 
-public class InputMissingCtorEmptyClass // violation
+public class InputMissingCtorEmptyClass // violation, 'Class should define a constructor'
 {
 }
 // we shouldn't flag abstract classes

diff --git a/.../puppycrawl/tools/checkstyle/checks/coding/missingctor/InputMissingCtorNestedClasses.java b/.../puppycrawl/tools/checkstyle/checks/coding/missingctor/InputMissingCtorNestedClasses.java
@@ -6,9 +6,9 @@
 
 package com.puppycrawl.tools.checkstyle.checks.coding.missingctor;
 
-public class InputMissingCtorNestedClasses { // violation
-    class Inner1 { // violation
-        class Inner2 { // violation
+public class InputMissingCtorNestedClasses { // violation, 'Class should define a constructor'
+    class Inner1 { // violation, 'Class should define a constructor'
+        class Inner2 { // violation, 'Class should define a constructor'
 
         }
     }