Remove FFmpegFileExists

Author: jonghwanhyeon

ffmpeg/__init__.py

@@ -1,11 +1,4 @@
-from .errors import (
-    FFmpegAlreadyExecuted,
-    FFmpegError,
-    FFmpegFileExists,
-    FFmpegFileNotFound,
-    FFmpegInvalidCommand,
-    FFmpegUnsupportedCodec,
-)
+from .errors import FFmpegAlreadyExecuted, FFmpegError, FFmpegFileNotFound, FFmpegInvalidCommand, FFmpegUnsupportedCodec
 from .ffmpeg import FFmpeg
 from .progress import Progress
 

ffmpeg/errors.py

@@ -39,13 +39,6 @@ class FFmpegAlreadyExecuted(FFmpegError):
     "Represents FFmpeg is being executed"
 
 
-class FFmpegFileExists(FFmpegError):
-    "Represents the output file already exists. You can overwrite it using the `-y` option"
-    _patterns = [
-        r"already exists",
-    ]
-
-
 class FFmpegFileNotFound(FFmpegError):
     "Represents an input file was not found"
     _patterns = [

tests/test_asyncio_errors.py

@@ -3,13 +3,7 @@
 
 import pytest
 
-from ffmpeg import (
-    FFmpegAlreadyExecuted,
-    FFmpegFileExists,
-    FFmpegFileNotFound,
-    FFmpegInvalidCommand,
-    FFmpegUnsupportedCodec,
-)
+from ffmpeg import FFmpegAlreadyExecuted, FFmpegFileNotFound, FFmpegInvalidCommand, FFmpegUnsupportedCodec
 from ffmpeg.asyncio import FFmpeg
 
 
@@ -36,38 +30,6 @@ async def test_asyncio_raises_already_executed(
         await ffmpeg.execute()
 
 
-@pytest.mark.asyncio
-async def test_asyncio_raises_file_exists(
-    assets_path: Path,
-    tmp_path: Path,
-):
-    source_path = assets_path / "pier-39.ts"
-    target_path = tmp_path / "pier-39.mp4"
-
-    (
-        await FFmpeg()
-        .option("y")
-        .input(source_path)
-        .output(
-            target_path,
-            codec="copy",
-        )
-        .execute()
-    )
-
-    with pytest.raises(FFmpegFileExists):
-        # Note: pytest sets stdin to a null object, so "Overwrite? [y/N]" prompt will be ignored
-        (
-            await FFmpeg()
-            .input(source_path)
-            .output(
-                target_path,
-                codec="copy",
-            )
-            .execute()
-        )
-
-
 @pytest.mark.asyncio
 async def test_asyncio_raises_file_not_found(
     assets_path: Path,

tests/test_errors.py

@@ -2,14 +2,7 @@
 
 import pytest
 
-from ffmpeg import (
-    FFmpeg,
-    FFmpegAlreadyExecuted,
-    FFmpegFileExists,
-    FFmpegFileNotFound,
-    FFmpegInvalidCommand,
-    FFmpegUnsupportedCodec,
-)
+from ffmpeg import FFmpeg, FFmpegAlreadyExecuted, FFmpegFileNotFound, FFmpegInvalidCommand, FFmpegUnsupportedCodec
 
 
 def test_raises_already_executed(
@@ -34,37 +27,6 @@ def test_raises_already_executed(
         ffmpeg.execute()
 
 
-def test_raises_file_exists(
-    assets_path: Path,
-    tmp_path: Path,
-):
-    source_path = assets_path / "pier-39.ts"
-    target_path = tmp_path / "pier-39.mp4"
-
-    (
-        FFmpeg()
-        .option("y")
-        .input(source_path)
-        .output(
-            target_path,
-            codec="copy",
-        )
-        .execute()
-    )
-
-    with pytest.raises(FFmpegFileExists):
-        # Note: pytest sets stdin to a null object, so "Overwrite? [y/N]" prompt will be ignored
-        (
-            FFmpeg()
-            .input(source_path)
-            .output(
-                target_path,
-                codec="copy",
-            )
-            .execute()
-        )
-
-
 def test_raises_file_not_found(
     assets_path: Path,
     tmp_path: Path,