Change exception throwing of the sanitize_excel_sheet_name function

Author: thombashi

pathvalidate/__init__.py

@@ -221,10 +221,14 @@ def sanitize_excel_sheet_name(sheet_name, replacement_text=""):
     :param str replacement_text: Replacement text.
     :return: A replacement string.
     :rtype: str
+    :raises ValueError: If the ``sheet_name`` is a invalid sheet name.
     """
 
-    return __RE_INVALID_EXCEL_SHEET_NAME.sub(
-        replacement_text, sheet_name.strip())
+    try:
+        return __RE_INVALID_EXCEL_SHEET_NAME.sub(
+            replacement_text, sheet_name.strip())
+    except AttributeError as e:
+        raise ValueError(e)
 
 
 def replace_symbol(text, replacement_text=""):

test/test_pathvalidate.py

@@ -345,9 +345,9 @@ def test_normal(self, value, replace_text, expected):
         validate_excel_sheet_name(sanitized_name)
 
     @pytest.mark.parametrize(["value", "expected"], [
-        [None, AttributeError],
-        [1, AttributeError],
-        [True, AttributeError],
+        [None, ValueError],
+        [1, ValueError],
+        [True, ValueError],
     ])
     def test_exception_type(self, value, expected):
         with pytest.raises(expected):