fixed bug in cli where no transcript could be retrieved if no language was specified
This commit is contained in:
		
							parent
							
								
									0901fe3053
								
							
						
					
					
						commit
						7159f60823
					
				|  | @ -1,12 +1,7 @@ | ||||||
| 
 | 
 | ||||||
| # YouTube Transcript/Subtitle API (including automatically generated subtitles and subtitle translations)   | # YouTube Transcript/Subtitle API (including automatically generated subtitles and subtitle translations)   | ||||||
|    |    | ||||||
| [](https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=BAENLEW8VUJ6G&source=url)   | [](https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=BAENLEW8VUJ6G&source=url) [](https://travis-ci.org/jdepoix/youtube-transcript-api) [](https://coveralls.io/github/jdepoix/youtube-transcript-api?branch=master) [](http://opensource.org/licenses/MIT) [](https://pypi.org/project/youtube-transcript-api/) [](https://pypi.org/project/youtube-transcript-api/) | ||||||
| [](https://travis-ci.org/jdepoix/youtube-transcript-api)   |  | ||||||
| [](https://coveralls.io/github/jdepoix/youtube-transcript-api?branch=master)   |  | ||||||
| [](http://opensource.org/licenses/MIT)   |  | ||||||
| [](https://pypi.org/project/youtube-transcript-api/)   |  | ||||||
| [](https://pypi.org/project/youtube-transcript-api/)   |  | ||||||
| 
 | 
 | ||||||
| This is an python API which allows you to get the transcripts/subtitles for a given YouTube video. It also works for automatically generated subtitles, supports translating subtitles and it does not require a headless browser, like other selenium based solutions do! | This is an python API which allows you to get the transcripts/subtitles for a given YouTube video. It also works for automatically generated subtitles, supports translating subtitles and it does not require a headless browser, like other selenium based solutions do! | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
|  | @ -72,7 +72,7 @@ class YouTubeTranscriptCli(): | ||||||
|         parser.add_argument( |         parser.add_argument( | ||||||
|             '--languages', |             '--languages', | ||||||
|             nargs='*', |             nargs='*', | ||||||
|             default=[], |             default=['en',], | ||||||
|             type=str, |             type=str, | ||||||
|             help=( |             help=( | ||||||
|                 'A list of language codes in a descending priority. For example, if this is set to "de en" it will ' |                 'A list of language codes in a descending priority. For example, if this is set to "de en" it will ' | ||||||
|  |  | ||||||
|  | @ -77,7 +77,7 @@ class TestYouTubeTranscriptCli(TestCase): | ||||||
|         parsed_args = YouTubeTranscriptCli('v1 v2'.split())._parse_args() |         parsed_args = YouTubeTranscriptCli('v1 v2'.split())._parse_args() | ||||||
|         self.assertEqual(parsed_args.video_ids, ['v1', 'v2']) |         self.assertEqual(parsed_args.video_ids, ['v1', 'v2']) | ||||||
|         self.assertEqual(parsed_args.json, False) |         self.assertEqual(parsed_args.json, False) | ||||||
|         self.assertEqual(parsed_args.languages, []) |         self.assertEqual(parsed_args.languages, ['en']) | ||||||
| 
 | 
 | ||||||
|     def test_argument_parsing__fail_without_video_ids(self): |     def test_argument_parsing__fail_without_video_ids(self): | ||||||
|         with self.assertRaises(SystemExit): |         with self.assertRaises(SystemExit): | ||||||
|  | @ -87,12 +87,12 @@ class TestYouTubeTranscriptCli(TestCase): | ||||||
|         parsed_args = YouTubeTranscriptCli('v1 v2 --json'.split())._parse_args() |         parsed_args = YouTubeTranscriptCli('v1 v2 --json'.split())._parse_args() | ||||||
|         self.assertEqual(parsed_args.video_ids, ['v1', 'v2']) |         self.assertEqual(parsed_args.video_ids, ['v1', 'v2']) | ||||||
|         self.assertEqual(parsed_args.json, True) |         self.assertEqual(parsed_args.json, True) | ||||||
|         self.assertEqual(parsed_args.languages, []) |         self.assertEqual(parsed_args.languages, ['en']) | ||||||
| 
 | 
 | ||||||
|         parsed_args = YouTubeTranscriptCli('--json v1 v2'.split())._parse_args() |         parsed_args = YouTubeTranscriptCli('--json v1 v2'.split())._parse_args() | ||||||
|         self.assertEqual(parsed_args.video_ids, ['v1', 'v2']) |         self.assertEqual(parsed_args.video_ids, ['v1', 'v2']) | ||||||
|         self.assertEqual(parsed_args.json, True) |         self.assertEqual(parsed_args.json, True) | ||||||
|         self.assertEqual(parsed_args.languages, []) |         self.assertEqual(parsed_args.languages, ['en']) | ||||||
| 
 | 
 | ||||||
|     def test_argument_parsing__languages(self): |     def test_argument_parsing__languages(self): | ||||||
|         parsed_args = YouTubeTranscriptCli('v1 v2 --languages de en'.split())._parse_args() |         parsed_args = YouTubeTranscriptCli('v1 v2 --languages de en'.split())._parse_args() | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue