forked from fcode/delarte
🚸 remove flags from script prototype
🩹 naming: not "languages", "version"
This commit is contained in:
parent
3b220c6346
commit
442df05ea4
|
@ -50,10 +50,10 @@ or
|
|||
$ delarte --help
|
||||
ArteTV dowloader.
|
||||
|
||||
usage: delarte [-h|--help] - print this message
|
||||
or: delarte program_page_url - show available versions
|
||||
or: delarte program_page_url -l version - show available resolutions
|
||||
or: delarte program_page_url -l version -r resolution - download the given video
|
||||
usage: delarte [-h|--help] - print this message
|
||||
or: delarte program_page_url - show available versions
|
||||
or: delarte program_page_url version - show available resolutions
|
||||
or: delarte program_page_url version resolution - download the given video
|
||||
```
|
||||
|
||||
🔧 How it works
|
||||
|
|
|
@ -9,34 +9,24 @@ class Parser(argparse.ArgumentParser):
|
|||
"""Generate a parser."""
|
||||
super().__init__(
|
||||
description="downloads Arte's videos with subtitles",
|
||||
epilog="examples:\n"
|
||||
"\tdelarte -h\t\t\t\t\t-> shows this help\n"
|
||||
"\tdelarte [url]\t\t\t\t\t-> shows available languages\n"
|
||||
"\tdelarte [url] -l [language]\t\t\t-> shows available resolutions\n"
|
||||
"\tdelarte [url] -l [language] -r [resolution]\t-> "
|
||||
"downloads Arte's video\n",
|
||||
formatter_class=argparse.RawTextHelpFormatter,
|
||||
)
|
||||
self.add_argument(
|
||||
"url",
|
||||
help="url of Arte movie's webpage",
|
||||
action="store",
|
||||
type=str,
|
||||
nargs="?",
|
||||
)
|
||||
self.add_argument(
|
||||
"-l",
|
||||
"--languages",
|
||||
"version",
|
||||
help="one of the language code proposed by Arte",
|
||||
required=False,
|
||||
action="store",
|
||||
type=str,
|
||||
nargs="?",
|
||||
)
|
||||
self.add_argument(
|
||||
"-r",
|
||||
"--resolution",
|
||||
"resolution",
|
||||
help="directory where the output files will be stored",
|
||||
required=False,
|
||||
action="store",
|
||||
type=str,
|
||||
nargs="?",
|
||||
|
@ -49,4 +39,9 @@ class Parser(argparse.ArgumentParser):
|
|||
List: liste ordonnée des arguments
|
||||
"""
|
||||
args_namespace = self.parse_args()
|
||||
return [args_namespace.url, args_namespace.languages, args_namespace.resolution]
|
||||
args_list = [
|
||||
args_namespace.url,
|
||||
args_namespace.version,
|
||||
args_namespace.resolution,
|
||||
]
|
||||
return [arg for arg in args_list if arg is not None]
|
||||
|
|
|
@ -24,9 +24,7 @@ class TestParser(TestCase):
|
|||
self.parser.parse_args(
|
||||
[
|
||||
"https://www.arte.tv/en/videos/104001-000-A/clint-eastwood/",
|
||||
"-l",
|
||||
"VOF-STMF",
|
||||
"-r",
|
||||
"216p",
|
||||
],
|
||||
)
|
||||
|
@ -34,7 +32,7 @@ class TestParser(TestCase):
|
|||
self.assertEqual(
|
||||
args,
|
||||
{
|
||||
"languages": "VOF-STMF",
|
||||
"version": "VOF-STMF",
|
||||
"resolution": "216p",
|
||||
"url": "https://www.arte.tv/en/videos/104001-000-A/clint-eastwood/",
|
||||
},
|
||||
|
@ -44,7 +42,7 @@ class TestParser(TestCase):
|
|||
"argparse.ArgumentParser.parse_args",
|
||||
return_value=argparse.Namespace(
|
||||
url="https://www.arte.tv/en/videos/104001-000-A/clint-eastwood/",
|
||||
languages="VOF-STMF",
|
||||
version="VOF-STMF",
|
||||
resolution="216p",
|
||||
),
|
||||
)
|
||||
|
|
Loading…
Reference in New Issue