Skip to content

Conversation

@utkarsh14
Copy link
Collaborator

@utkarsh14 utkarsh14 commented Aug 13, 2025

Run tests based on file changes

Ticket

Fixes: No tickets created

Description, Motivation and Context

Run tests based on files changed- divide testing portions into following modules:

  • Common - Form, surveys, module questionnaire etc
  • Assignment- Assignment , email, mapping , targets , enumerators etc
  • DQ
  • Docs
  • Media files

How Has This Been Tested?

Local and github actions:

Action 1 : when changes happened on dq and docs only those tests run

Action 2: When changes included forms all tests run

Checklist:

This checklist is a useful reminder of small things that can easily be forgotten.
Put an x in all the items that apply and remove any items that are not relevant to this PR.

  • My code follows the style guidelines and standard practices for this project
  • I have reviewed my own code to ensure good quality
  • I have tested the functionality of my code to ensure it works as intended
  • I have resolved merge conflicts
  • I have written good commit messages
  • I have created migration scripts from the latest dev changes. This will prevent migration script version conflicts (if applicable)
  • I have scrutinized and edited the migration script with reference to changes Alembic won't auto-detect (if applicable). Note that this includes manually adding the creation of new CHECK constraints.
  • I have updated the automated tests (if applicable)
  • I have updated the README file (if applicable)
  • I have updated the OpenAPI documentation (if applicable)

@utkarsh14 utkarsh14 requested a review from jeenut27 August 13, 2025 05:51
@utkarsh14
Copy link
Collaborator Author

utkarsh14 commented Aug 13, 2025

@jeenut27 - I have not added test files to changes list, I wanted to discuss if we need to add them module wise to changes list

Since the only changes in current PR is GA file and dockerfile its going to run all tests, similarly if only a test file would change it would still run all tests.

@utkarsh14 utkarsh14 marked this pull request as ready for review August 13, 2025 05:53
Copy link
Collaborator

@jeenut27 jeenut27 left a comment

Choose a reason for hiding this comment

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

Looks good

@utkarsh14 utkarsh14 merged commit b0efcd2 into dev Aug 13, 2025
4 of 5 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.

3 participants