standardize_test_format/__init__.py

@@ -6,8 +6,7 @@ Tool for standardizing Java tests.
 import re
 
 PATTERN_JAVADOC = r'/\*\*(?P<javadoc>.*(\n\s*\*.*)*?)\s*\*/'
-PATTERN_DISPLAY_NAME_EXPRESSION = r'\s*(?:"[^"]*?")(?:\s*\+\s*"[^"]*?")*\s*'
-PATTERN_DISPLAY_NAME = r'@DisplayName\((?P<display>'+PATTERN_DISPLAY_NAME_EXPRESSION+r')\)'
+PATTERN_DISPLAY_NAME = r'@DisplayName\(\s*"(?P<display>[^"]*?)"\s*\)'
 
 TEST_PATTERN: re.Pattern = re.compile(
         r''
@@ -24,7 +23,7 @@ IGNORABLE_WORDS_IN_NAME = frozenset({
     'a',
     'the',
     'is',
-    'are',
+    'are'
 })
 
 def format_test_prefix(name: str, description: str, annotation:
@@ -53,7 +52,7 @@ def to_camel_case_word(word: str, first: bool = False) -> str | None:
     return (word[0].lower() if first else word[0].upper()) + word[1:]
 
 def to_camel_case(description: str) -> str:
-    description = description.replace(',',' ').replace('.',' ').replace('"', ' ').replace('+', ' ').strip(' \t.')
+    description = description.strip(' \t.').replace(',',' ')
     words = description.split(' ')
 
     words[0] = to_camel_case_word(words[0], first=True)
@@ -76,18 +75,6 @@ def parse_javadoc_to_description(text: str| None):
     text = re.sub(r'\{@link\s+(\w+)\}', r'\1', text)
     return text
 
-
-def parse_display_name_to_description(text: str|None):
-    if text is None:
-        return ''
-
-    text = text.strip()
-    text = re.sub(r'"\s*\+\s*"', '', text)
-    text = text.strip('"')
-    text = text.strip()
-    return text
-
-
 def replace_test_pattern(match: re.Match, with_javadoc: bool, with_display_name: bool) -> str:
     javadoc = parse_javadoc_to_description(match.group('javadoc'))
 
@@ -95,7 +82,7 @@ def replace_test_pattern(match: re.Match, with_javadoc: bool, with_display_name:
     annotation = match.group('annotation').strip()
     visibility = (match.group('visibility') or '').strip()
     name = match.group('name').strip()
-    display = parse_display_name_to_description(match.group('display'))
+    display = (match.group('display') or '').strip()
 
 
     description = display or javadoc or from_camel_case(name)

test/test_standardize.py

@@ -92,18 +92,6 @@ package test;
 public void zkBinderContextImplPropagatesLotOfFieldsDirectlyFromTheUnderlyingZkBinderContext()
 """
 
-INPUT_5C = """
-package test;
-
-@Test
-@DisplayName(
-  "ZkBinderContextImpl propagates a lot of "
-      + "fields directly from "
-      + "the underlying ZkBinderContext"
-)
-public void zkBinderContextImplPropagatesLotOfFieldsDirectlyFromTheUnderlyingZkBinderContext()
-"""
-
 OUTPUT_5 = """
 package test;
 import org.junit.jupiter.api.DisplayName;
@@ -131,6 +119,3 @@ def test_5():
 
 def test_5b():
     assert standardize_java_text(INPUT_5B.strip(),False,True) == OUTPUT_5.strip()
-
-def test_5c():
-    assert standardize_java_text(INPUT_5C.strip(),False,True) == OUTPUT_5.strip()