Skip to content

Commit 0a64591

Browse files
committed
Add checkstyle rule for toLowerCase/toUpperCase
This commit adds a checkstyle rule that prevents the re-introduction of such calls. See gh-33708
1 parent 5302e7a commit 0a64591

File tree

2 files changed

+18
-0
lines changed

2 files changed

+18
-0
lines changed

src/checkstyle/checkstyle-suppressions.xml

+2
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212

1313
<!-- Global: tests and test fixtures -->
1414
<suppress files="[\\/]src[\\/](test|testFixtures)[\\/](java|java21)[\\/]" checks="AnnotationLocation|AnnotationUseStyle|AtclauseOrder|AvoidNestedBlocks|FinalClass|HideUtilityClassConstructor|InnerTypeLast|JavadocStyle|JavadocType|JavadocVariable|LeftCurly|MultipleVariableDeclarations|NeedBraces|OneTopLevelClass|OuterTypeFilename|RequireThis|SpringCatch|SpringJavadoc|SpringNoThis"/>
15+
<suppress files="[\\/]src[\\/](test|testFixtures)[\\/](java|java21)[\\/]" checks="RegexpSinglelineJava" id="toLowerCaseWithoutLocale"/>
16+
<suppress files="[\\/]src[\\/](test|testFixtures)[\\/](java|java21)[\\/]" checks="RegexpSinglelineJava" id="toUpperCaseWithoutLocale"/>
1517
<suppress files="[\\/]src[\\/](test|testFixtures)[\\/](java|java21)[\\/]org[\\/]springframework[\\/].+(Tests|Suite)" checks="IllegalImport" id="bannedJUnitJupiterImports"/>
1618
<suppress files="[\\/]src[\\/](test|testFixtures)[\\/](java|java21)[\\/]" checks="SpringJUnit5" message="should not be public"/>
1719

src/checkstyle/checkstyle.xml

+16
Original file line numberDiff line numberDiff line change
@@ -221,6 +221,22 @@
221221
<property name="message" value="Please use class literals for primitives and void -- for example, int.class instead of Integer.TYPE."/>
222222
<property name="ignoreComments" value="true"/>
223223
</module>
224+
<module name="com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck">
225+
<property name="id" value="toLowerCaseWithoutLocale"/>
226+
<property name="format" value="\.toLowerCase\(\)"/>
227+
<property name="maximum" value="0"/>
228+
<property name="message"
229+
value="String.toLowerCase() should be String.toLowerCase(Locale.ROOT)"/>
230+
<property name="ignoreComments" value="true"/>
231+
</module>
232+
<module name="com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck">
233+
<property name="id" value="toUpperCaseWithoutLocale"/>
234+
<property name="format" value="\.toUpperCase\(\)"/>
235+
<property name="maximum" value="0"/>
236+
<property name="message"
237+
value="String.toUpperCase() should be String.toUpperCase(Locale.ROOT)"/>
238+
<property name="ignoreComments" value="true"/>
239+
</module>
224240
<module
225241
name="com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck">
226242
<property name="maximum" value="0"/>

0 commit comments

Comments
 (0)