Skip to content

Conversation

@paigewilliams
Copy link
Contributor

Resolves the TODO for Write TEKDB/test_views.RelatedTest

Adds tests for the various models that have a get_related_objects method. This includes the following models:

  • Record
  • Places
  • Resources
  • ResourcesActivityEvents
  • Citations
  • Media

Bumps test coverage from 65% to 68% 🎉

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR resolves the TODO for implementing comprehensive tests for the get_related view across different model types. The implementation adds test coverage for the get_related_objects method for Record, Places, Resources, ResourcesActivityEvents, Citations, and Media models, increasing overall test coverage from 65% to 68%.

Key Changes:

  • Added create_get_related_request helper function for generating test requests
  • Implemented RelatedTest.setUp with fixture data and test objects for all tested models
  • Added 7 test methods covering authentication and related objects retrieval for each model type

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@paigewilliams paigewilliams requested a review from Copilot October 16, 2025 17:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@paigewilliams paigewilliams merged commit 4e506e8 into main Oct 16, 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.

2 participants