Skip to content

Comments

Group VuMark generation tests in class#2957

Merged
adamtheturtle merged 2 commits intomainfrom
adamtheturtle/group-vumark-tests
Feb 18, 2026
Merged

Group VuMark generation tests in class#2957
adamtheturtle merged 2 commits intomainfrom
adamtheturtle/group-vumark-tests

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 18, 2026

Summary

  • Moved test_generate_instance_success into a TestGenerateInstance class
  • Applied @pytest.mark.usefixtures decorator at the class level to reduce duplication
  • Refactored long lines to comply with line length limits

Test plan

  • Existing tests continue to pass with the new class structure

🤖 Generated with Claude Code


Note

Low Risk
Low risk test-only refactor that changes pytest collection structure but not request/response assertions or production code paths.

Overview
Refactors tests/mock_vws/test_vumark_generation_api.py by moving test_generate_instance_success into a TestGenerateInstance class and applying @pytest.mark.usefixtures("verify_mock_vuforia") at the class level.

Minor readability tweaks were made (e.g., introducing target_id/body_dict locals and splitting the response Content-Type extraction into a named variable) without changing the test’s behavior or assertions.

Written by Cursor Bugbot for commit 74340ac. This will update automatically on new commits. Configure here.

Move test_generate_instance_success into a TestGenerateInstance class and apply the @pytest.mark.usefixtures decorator at the class level to reduce duplication.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@adamtheturtle adamtheturtle merged commit 40bd318 into main Feb 18, 2026
106 checks passed
@adamtheturtle adamtheturtle deleted the adamtheturtle/group-vumark-tests branch February 18, 2026 15:44
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.

1 participant