Skip to content
This repository has been archived by the owner on May 25, 2023. It is now read-only.

Ffmpeg download response #84

Merged
merged 11 commits into from Jan 24, 2022
Merged

Ffmpeg download response #84

merged 11 commits into from Jan 24, 2022

Conversation

MohamedBakoush
Copy link
Owner

@MohamedBakoush MohamedBakoush commented Jan 24, 2022 •
edited

This is meant to remove res.json(), to make testing code possible for

  • getVideoLinkFromUrl (youtubedl-download-video.js)
  • downloadUploadedVideo, uploadVideoFile (ffmpeg-upload-video.js)
  • downloadVideo, trimVideo (ffmpeg-download-video.js)
  • downloadVideoStream (ffmpeg-download-stream.js)

ffmpeg-download-response.js code tested
4 function - 15 test cases

npm test ffmpeg-download-response

#54

@MohamedBakoush MohamedBakoush self-assigned this Jan 24, 2022
@MohamedBakoush MohamedBakoush merged commit 14c41d1 into master Jan 24, 2022
@MohamedBakoush MohamedBakoush deleted the ffmpeg-download-response branch January 24, 2022 22:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@MohamedBakoush MohamedBakoush

Labels
None yet
Projects
No open projects
ffmpeg download response
Awaiting triage
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@MohamedBakoush