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

Resolved windows filesystem bug

Browse source
This commit is contained in:
Kieran Eglin 2024-02-24 15:40:07 -08:00
parent 52c0a1742a
commit bf677f1c6d
No known key found for this signature in database
GPG key ID: 193984967FCF432D
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}