Skip to content

Auto Create Context: Fetch all objects for correct jira project associations#14259

Open
Maffooch wants to merge 3 commits intobugfixfrom
auto-create-context-patch
Open

Auto Create Context: Fetch all objects for correct jira project associations#14259
Maffooch wants to merge 3 commits intobugfixfrom
auto-create-context-patch

Conversation

@Maffooch
Copy link
Contributor

@Maffooch Maffooch commented Feb 6, 2026

When using auto create context, the previous implementation was only fetching products successfully if they already existed, but did not fetch an engagement or test. Because of this, jira settings on an engagement that were not inherited from a product were never considered when determining if the the import/reimport should be pushed to jira.

Until this case occured, it appeared as if auto create context was functional because existing engagements/tests were being fetched at the serializer level after the auto create context process occurred at the viewset level

@github-actions github-actions bot added the apiv2 label Feb 6, 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.

Approved

@Maffooch Maffooch requested a review from dogboat February 7, 2026 07:46
import_reimport_config = self._test_setup_jira_project_for_engagement()
# Not run the import again
import1 = self.reimport_scan_with_params(
import_reimport_config.pop("test_id", None),
Copy link
Member

Choose a reason for hiding this comment

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

i don't think there is a test_id for this scenario

Suggested change
import_reimport_config.pop("test_id", None),
None

Copy link
Member

@valentijnscholten valentijnscholten left a comment

Choose a reason for hiding this comment

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

Surprised this only pops up now, maybe now many people are using autocreate?

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