RenameTestCases capitalize each word should now work for words in brackets (#498)

Author: bhirsz

docs/releasenotes/4.0.0.rst

@@ -301,6 +301,7 @@ Bugs
 
 - Keyword name will no longer be prefixed with continuation marks (``...``) if name was longer than line length limit (#494)
 - It is now safe to use Tasks sections with ``MergeAndOrderSections`` transformer with Robot Framework 6.0+ (#490)
+- Test case titles are now capitalized with ``RenameTestCases:capitalize_each_word=True`` even if the word is inside brackets (#485)
 - It is now possible to use disabler in the first line and enable it later (previously the whole file was always ignored) (#479)::
 
     # robotidy: off

robotidy/transformers/RenameTestCases.py

@@ -1,4 +1,5 @@
 import re
+import string
 from typing import Optional
 
 from robot.api.parsing import Token
@@ -8,6 +9,36 @@
 from robotidy.transformers import Transformer
 
 
+def cap_string_until_succeed(word: str):
+    """
+    Yield characters from the word and capitalize character until we are able to make char uppercase.
+    """
+    capitalize = True
+    for char in word:
+        if capitalize:
+            char = char.upper()
+            if char.isupper():
+                capitalize = False
+        yield char
+
+
+def cap_word(word: str):
+    """
+    Capitalize the word. The word can start with ( or contain ':
+
+        word -> Word
+        (word -> (Word
+        word's -> Word's
+
+    """
+    if not word or any(c.isupper() for c in word):  # ignore JIRA or sOme
+        return word
+    new_word = word.capitalize()
+    if new_word != word:
+        return new_word
+    return "".join(cap_string_until_succeed(word))
+
+
 class RenameTestCases(Transformer):
     r"""
     Enforce test case naming.
@@ -88,8 +119,8 @@ def visit_TestCaseName(self, node):  # noqa
         token = node.get_token(Token.TESTCASE_NAME)
         if token.value:
             if self.capitalize_each_word:
-                token_value = " ".join(f"{word[0].upper()}{word[1:]}" for word in token.value.split(" "))
-                token.value = token_value.lstrip()
+                value = token.value.strip()
+                token.value = " ".join(cap_word(word) for word in value.split(" "))
             else:
                 token.value = token.value[0].upper() + token.value[1:]
             if self.replace_pattern is not None:

tests/atest/transformers/RenameTestCases/expected/replace_pattern_empty.robot

@@ -14,3 +14,6 @@ Containing replace pattern
 
 Remove special chars: ?$@
     No Operation
+
+Capitalize words (after !special that's
+    No operation

tests/atest/transformers/RenameTestCases/expected/replace_pattern_placeholder.robot

@@ -14,3 +14,6 @@ Containing replace pattern PLACEHOLDER
 
 Remove special chars: ?$@
     No Operation
+
+Capitalize words (after !special that's
+    No operation

tests/atest/transformers/RenameTestCases/expected/replace_pattern_special_chars.robot

@@ -14,3 +14,6 @@ Containing replace pattern JIRA-1234
 
 Remove special chars
     No Operation
+
+Capitalize words (after !special that's
+    No operation

tests/atest/transformers/RenameTestCases/expected/selected.robot

@@ -14,3 +14,6 @@ Containing replace pattern JIRA-1234
 
 Remove special chars: ?$@
     No Operation
+
+Capitalize words (after !special that's
+    No operation

tests/atest/transformers/RenameTestCases/expected/test.robot

@@ -14,3 +14,6 @@ Containing replace pattern JIRA-1234
 
 Remove special chars: ?$@
     No Operation
+
+Capitalize words (after !special that's
+    No operation

tests/atest/transformers/RenameTestCases/expected/upper_case.robot

@@ -14,3 +14,6 @@ Containing Replace Pattern JIRA-1234
 
 Remove Special Chars: ?$@
     No Operation
+
+Capitalize Words (After !Special That's
+    No operation

tests/atest/transformers/RenameTestCases/source/test.robot

@@ -14,3 +14,6 @@ Containing replace pattern JIRA-1234
 
 Remove special chars: ?$@
     No Operation
+
+Capitalize words (after !special that's
+    No operation