chore: include file extension in max filename len

syrupy-project · Nov 13, 2020 · 5d2b9ca · 5d2b9ca
1 parent 6b47681
commit 5d2b9ca
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/syrupy/extensions/single_file.py b/src/syrupy/extensions/single_file.py
@@ -69,6 +69,7 @@ def _write_snapshot_fossil(self, *, snapshot_fossil: "SnapshotFossil") -> None:
 
     def __clean_filename(self, filename: str) -> str:
         filename = str(filename).strip().replace(" ", "_")
+        max_filename_length = 255 - len(self._file_extension or "")
         exclude_chars = '\\/?%*:|"<>,;='
         exclude_categ = ("C",)
         cleaned_filename = "".join(
@@ -77,4 +78,4 @@ def __clean_filename(self, filename: str) -> str:
             if c not in exclude_chars
             and not any(categ in category(c) for categ in exclude_categ)
         )
-        return cleaned_filename[:255]  # maximum filename length
+        return cleaned_filename[:max_filename_length]