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

Test cases busted #54

Closed
21 of 29 tasks
MohamedBakoush opened this issue Nov 8, 2021 · 5 comments
Closed
21 of 29 tasks

Test cases busted #54

MohamedBakoush opened this issue Nov 8, 2021 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@MohamedBakoush
Copy link
Owner

MohamedBakoush commented Nov 8, 2021

Test cases no longer make sense, due to neglect, need to be redone.

Frontend

  • basics.js
  • currentVideoDownloads.js
  • favicon.js
  • folder.js
  • folderPath.js
  • index.js
  • navigationBar.js
  • notify.js
  • optionMenu.js
  • search.js
  • showAvailableVideos.js
  • videoPlayerButtons.js

Backend

  • available-videos.js
  • check-path-validity.js
  • current-download-videos.js
  • data-videos.js
  • delete-data.js
  • ffmpeg-download-compression.js
  • ffmpeg-download-image.js
  • ffmpeg-download-response.js
  • ffmpeg-download-stream.js
  • ffmpeg-download-trimed-video.js
  • ffmpeg-download-video.js
  • ffmpeg-path.js
  • ffmpeg-unfinished-videos.js
  • ffmpeg-upload-video.js
  • stream-video-image.js
  • user-settings.js
  • youtubedl-download-video.js
@MohamedBakoush MohamedBakoush self-assigned this Nov 8, 2021
@MohamedBakoush MohamedBakoush added the enhancement New feature or request label Nov 8, 2021
@MohamedBakoush
Copy link
Owner Author

res.json need to removed from

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

return won't work - backend notify (database) system should do the trick.

@MohamedBakoush
Copy link
Owner Author

delete-data.js took a bit of time to prep but at this point in time, it's ready to be tested.

@MohamedBakoush
Copy link
Owner Author

test cases for delete-data.js have finally been finished, all that's left is checking manually if deleting data is acting as it's supposed to. 🎉

@MohamedBakoush
Copy link
Owner Author

MohamedBakoush commented Feb 16, 2022

Client file structure needs to be restructured, each file should contain one major feature same as the backend for readability/maintainability.

@MohamedBakoush
Copy link
Owner Author

MohamedBakoush commented Jun 21, 2022

Great for now.

For future reference, if a feature gets added or altered a test case gets also added or altered.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant