Skip to content

Add normalizeGraphObjectKey to createMockExecutionContext#1123

Open
Gonzalo-Avalos-Ribas wants to merge 2 commits intomainfrom
NO-TICKET-add-normalize-graph-object-key-to-mock-execution-context
Open

Add normalizeGraphObjectKey to createMockExecutionContext#1123
Gonzalo-Avalos-Ribas wants to merge 2 commits intomainfrom
NO-TICKET-add-normalize-graph-object-key-to-mock-execution-context

Conversation

@Gonzalo-Avalos-Ribas
Copy link
Contributor

@Gonzalo-Avalos-Ribas Gonzalo-Avalos-Ribas commented Sep 24, 2024

Adding normalizeGraphObjectKey to createMockExecutionContext. This previously made executeStepWithDependencies to not call the normalizeGraphObjectKey declared in the invocationConfig. That made some integrations behave differently when running tests than when actual collection running.

@Gonzalo-Avalos-Ribas Gonzalo-Avalos-Ribas requested a review from a team as a code owner September 24, 2024 18:21
entities: preContext.jobState.collectedEntities,
relationships: preContext.jobState.collectedRelationships,
setData: preContext.jobState.collectedData,
normalizeGraphObjectKey: invocationConfig.normalizeGraphObjectKey,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the only change, only affects executeStepsWithDependencies

@mscottford mscottford force-pushed the NO-TICKET-add-normalize-graph-object-key-to-mock-execution-context branch from c131d50 to efda5d6 Compare February 19, 2026 15:23
@mscottford mscottford force-pushed the NO-TICKET-add-normalize-graph-object-key-to-mock-execution-context branch from efda5d6 to 7868d8e Compare February 19, 2026 19:09
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