Skip to content

2021csb1167/sof 149 test youtube audio extract python file#179

Merged
Arker123 merged 20 commits intoArker123:mainfrom
Dhruv-Singh-Negi:2021csb1167/sof-149-test-youtube-audio-extract-python-file
Apr 14, 2024
Merged

2021csb1167/sof 149 test youtube audio extract python file#179
Arker123 merged 20 commits intoArker123:mainfrom
Dhruv-Singh-Negi:2021csb1167/sof-149-test-youtube-audio-extract-python-file

Conversation

@Dhruv-Singh-Negi
Copy link
Collaborator

Implemented Unit Tests for YouTube Audio Extractor ensure functionality and reliability:

  1. test_argument_parser(): Confirms argument parsing for video URLs and flags.
  2. test_generate(): Validates audio download and transcript generation using mocks.
  3. test_get_transcript_from_video(): Ensures correct transcript and language are returned.
  4. test_main(): Tests main function's integration and argument processing.

Mocks are used to isolate functionality and avoid external dependencies, ensuring robust testing.

@linear
Copy link

linear bot commented Apr 2, 2024

@Arker123 Arker123 added draft and removed draft labels Apr 9, 2024
@Arker123 Arker123 enabled auto-merge April 10, 2024 08:29
@Arker123 Arker123 added the tests label Apr 10, 2024
@Arker123
Copy link
Owner

Some of the tests aren't working properly. Can you add these tests to the workflows, too?

@Dhruv-Singh-Negi
Copy link
Collaborator Author

Some of the tests aren't working properly. Can you add these tests to the workflows, too?

Done, Review again.

Copy link
Owner

@Arker123 Arker123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you very much! nice work!

Copy link
Collaborator

@Dhananjay-Goel Dhananjay-Goel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Arker123 Arker123 merged commit 9fb1834 into Arker123:main Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants