Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Controllers Tests For Course Management #52

Merged

Conversation

MoustafaAmer12
Copy link
Collaborator

Controllers Tests

Course Controller

  • Testing For Get/Post/Put/Delete endpoints
  • Testing For Get All Courses For Instructor

Module Controller

  • Testing For Post/Put/Delete endpoints
  • Testing For Get course's modules and lessons

Lesson Controller

  • Testing For Post/Put/Delete

Tests Include:

1- Success Tests
2- Unauthorized Access
3- Validation for parameters
4- Entity Ownership
5- Entity Not Found

Copy link

github-actions bot commented Jan 1, 2025

Code Coverage Summary

Overall Project 82.94% 🍏

There is no coverage information present for the Files changed

Copy link
Collaborator

@asserelzeki6 asserelzeki6 left a comment

Choose a reason for hiding this comment

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

I noticed you make the same mock functions in every test file can you make them generalize them for more modularity ??

@MoustafaAmer12
Copy link
Collaborator Author

MoustafaAmer12 commented Jan 1, 2025

I noticed you make the same mock functions in every test file can you make them generalize them for more modularity ??

The issue here is that I require the id's for the path variable in the test end points, do you know whether there is an approach to store the id ?

@Elshaarawy-1 Elshaarawy-1 self-requested a review January 1, 2025 08:39
@ahmedyoussefg ahmedyoussefg merged commit a2c460e into devel Jan 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants