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

Added CRUD tests to GameLibrary.Tests #26

Merged
merged 6 commits into from
Nov 23, 2024
Merged

Added CRUD tests to GameLibrary.Tests #26

merged 6 commits into from
Nov 23, 2024

Conversation

Ahmad-Alqadiri
Copy link
Contributor

@Ahmad-Alqadiri Ahmad-Alqadiri commented Nov 22, 2024

✨ Description

Please try to check; create, edit, etc., and exactly if tests are passed. Thanks

Linked Issue:

πŸ”„ Type of Change

  • 🐞 Bug fix
  • ✨ New feature
  • πŸ”§ Improvement

βœ… Checklist

  • πŸ“ Code follows project style guidelines
  • πŸ”— Relevant issue is linked and moved to 'In Progress'
  • πŸ§ͺ All relevant tests have been added or updated
  • βœ”οΈ Continuous integration (CI) tests have passed
  • πŸ‘€ Ready for review

πŸ“„ Additional Information

GiGwebs and others added 6 commits November 17, 2024 02:59
I Completed the Games list & details pages and added features like Favoriting, Rating, Game usage or ownership status ... etc.
Added test for Create model
Added test for Editing model but still one function is integrated and protected function, so I can't test it.
Added file to test delete model.
Fixed some lines of the codes in different files.
Cleaned some lines of the code also in different files.
Added Index game tests
Added details game tests
Added index game tests
cleaned and prettier the code looks like.
Update branch to match main current state.

Do not worry, all changes are retained and functional.
@RespectMathias RespectMathias merged commit 49aad2d into main Nov 23, 2024
@RespectMathias
Copy link
Contributor

Merge Committed into main.

@RespectMathias RespectMathias deleted the Add-test-CURD branch November 23, 2024 06:58
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.

3 participants