Skip to content

Run import performance test also for v3 Locations#14404

Draft
valentijnscholten wants to merge 5 commits intoDefectDojo:devfrom
valentijnscholten:performance-test-v3-updates
Draft

Run import performance test also for v3 Locations#14404
valentijnscholten wants to merge 5 commits intoDefectDojo:devfrom
valentijnscholten:performance-test-v3-updates

Conversation

@valentijnscholten
Copy link
Member

@valentijnscholten valentijnscholten commented Feb 27, 2026

Summary

  • Add TestDojoImporterPerformanceSmallLocations with V3_FEATURE_LOCATIONS for v3 importer performance testing
  • Update update_performance_test_counts.py to run both v2 (TestDojoImporterPerformanceSmall) and v3 (TestDojoImporterPerformanceSmallLocations) test classes
  • Added instructions to CONTRIBUTING.md on how to automagically update the query counts after code changes

- Add TestDojoImporterPerformanceSmallLocations with V3_FEATURE_LOCATIONS
- Update update_performance_test_counts.py to run both v2 and v3 test classes
- Add --no-keepdb and EXTRA_ARGS to run-unittest.sh for test flexibility
@valentijnscholten valentijnscholten added this to the 2.56.0 milestone Feb 27, 2026
@valentijnscholten valentijnscholten changed the title Add v3 Locations performance test and update script support Run import performance test also for v3 Locations Feb 27, 2026
@github-actions github-actions bot added the docs label Feb 27, 2026
mtesauro
mtesauro previously approved these changes Feb 28, 2026
Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

What's up with the REST tests lately?

@valentijnscholten
Copy link
Member Author

The tests pass locally, but not GHA. Maybe let's wait for the locations code to settle down before merging this?

@valentijnscholten valentijnscholten marked this pull request as draft March 1, 2026 10:18
@Maffooch Maffooch modified the milestones: 2.56.0, 2.56.1 Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants