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

api/core.py test coverage #992

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nitinawari
Copy link
Contributor

@nitinawari nitinawari commented Jan 20, 2025

Proposed change

Your PR description goes here.

Type of change

  • New core framework functionality
  • Bugfix (non-breaking change which fixes an issue)
  • Code refactoring without any functionality changes
  • New or existing module/payload change
  • Localization improvement
  • Dependency upgrade
  • Documentation improvement
  • test coverage

Checklist

  • I've followed the contributing guidelines
  • I've run make pre-commit, it didn't generate any changes
  • I've run make test, all tests passed locally

@nitinawari
Copy link
Contributor Author

@securestep9 @arkid15r I started writing test cases for each file to reach the maximum test coverage. For this file, I have achieved 100% test coverage. I need a review of the code quality and syntax style so that I can apply the same approach to the other tests.

@nitinawari nitinawari changed the title Api core test coverage api/core.py test coverage Jan 20, 2025
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.

test coverage for api/core.py
1 participant