change behaviour of extension in file_path

faker/providers/file/__init__.py

@@ -1,7 +1,8 @@
+import re
 import string
 
 from collections import OrderedDict
-from typing import Dict, Optional
+from typing import Dict, List, Optional, Union
 
 from .. import BaseProvider, ElementsType
 
@@ -257,7 +258,7 @@ def file_path(
         self,
         depth: int = 1,
         category: Optional[str] = None,
-        extension: Optional[str] = None,
+        extension: Optional[Union[str, List[str]]] = None,
         absolute: Optional[bool] = True,
     ) -> str:
         """Generate an pathname to a file.
@@ -269,12 +270,24 @@ def file_path(
         If ``absolute`` is ``True`` (default), the generated path starts with
         ``/`` and is absolute. Otherwise, the generated path is relative.
 
+        If used, ``extension`` can be either a string, forcing that extension, a
+        list of strings (one will be picked at random), or an empty list (the
+        path will have no extension). Default behaviour is the same as |file_name|
+
         :sample: size=10
         :sample: depth=3
         :sample: depth=5, category='video'
         :sample: depth=5, category='video', extension='abcdef'
+        :sample: extension=[]
+        :sample: extension=["a", "bc", "def"]
         """
+        if isinstance(extension, list):
+            if extension:
+                extension = self.random_element(extension)
+            else:
+                extension = "..."
         file: str = self.file_name(category, extension)
+        file = re.sub(r"\.{4}$", "", file)
         path: str = f"/{file}"
         for _ in range(0, depth):
             path = f"/{self.generator.word()}{path}"

tests/providers/test_file.py

@@ -20,9 +20,13 @@ def test_file_path(self):
             file_path = self.fake.file_path(depth=3)
             assert re.search(r"\/\w+\/\w+\/\w+\.\w+", file_path)
             file_path = self.fake.file_path(extension="pdf")
-            assert re.search(r"\/\w+\/\w+\.pdf", file_path)
+            assert re.search(r"\/\w+\/\w+\.pdf$", file_path)
+            file_path = self.fake.file_path(extension=["a", "bc", "def", "ghij", "klmno"])
+            assert re.search(r"\/\w+\/\w+\.(a|bc|def|ghij|klmno)$", file_path)
+            file_path = self.fake.file_path(extension=[])
+            assert re.search(r"\/\w+\/\w+$", file_path)
             file_path = self.fake.file_path(category="image")
-            assert re.search(r"\/\w+\/\w+\.(bmp|gif|jpeg|jpg|png|tiff)", file_path)
+            assert re.search(r"\/\w+\/\w+\.(bmp|gif|jpeg|jpg|png|tiff)$", file_path)
 
     def test_unix_device(self):
         reg_device = re.compile(r"^/dev/(vd|sd|xvd)[a-z]$")