Skip to content

Conversation

@clorenquecan
Copy link
Collaborator

This pull request introduces model test coverage for various apps.

Model Test Coverage

  • Organized tests within dedicated tests directories inside each app in files test_model.py.
  • Included tests for __str__ methods to ensure expected behavior.

Fixtures & Data Setup

  • I used the existing factories and fixtures, and defined the missing trait fixture in the conftest.py

Note: The test cases for the places, and biodiversity apps were already merged in a previous PRs.
I also install and run pre-commit locally as per @dr-rompecabezas suggestion (Thanks very useful!)

Closes #22

Copy link
Collaborator Author

@clorenquecan clorenquecan left a comment

Choose a reason for hiding this comment

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

Adding back TrailFactory

Copy link
Collaborator

@dr-rompecabezas dr-rompecabezas left a comment

Choose a reason for hiding this comment

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

Thanks, @clorenquecan!

@dr-rompecabezas dr-rompecabezas merged commit f88fdf2 into OmdenaAI:main May 1, 2025
2 checks 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.

Add tests for model methods

2 participants