danvics/pinchflat
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Resolved windows filesystem bug (#38)

Browse source
This commit is contained in:
Kieran 2024-02-28 08:56:25 -08:00 committed by GitHub
parent 52c0a1742a
commit 3bb7edba0b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/pinchflat/profiles/options/yt_dlp/download_option_builder.ex
View file

@ -34,7 +34,7 @@ defmodule Pinchflat.Profiles.Options.YtDlp.DownloadOptionBuilder do
# This will be updated a lot as I add new options to profiles # This will be updated a lot as I add new options to profiles
defp default_options do defp default_options do
[:no_progress] [:no_progress, :windows_filenames]
end end
defp subtitle_options(media_profile) do defp subtitle_options(media_profile) do

9
test/pinchflat/profiles/options/yt_dlp/download_option_builder_test.exs
View file

@ -16,6 +16,15 @@ defmodule Pinchflat.Profiles.Options.YtDlp.DownloadOptionBuilderTest do
end end
end end
describe "build/1 when testing default options" do
test "it includes default options" do
assert {:ok, res} = DownloadOptionBuilder.build(@media_profile)
assert :no_progress in res
assert :windows_filenames in res
end
end
describe "build/1 when testing subtitle options" do describe "build/1 when testing subtitle options" do
test "includes :write_subs option when specified" do test "includes :write_subs option when specified" do
media_profile = %MediaProfile{@media_profile | download_subs: true} media_profile = %MediaProfile{@media_profile | download_subs: true}