Refactor sfn->sfn context injection code by merging two functions (2)

DataDog / datadog-ci

Use Datadog from your CI.

https://datadoghq.com

Apache License 2.0

129 stars 55 forks source link

Refactor sfn->sfn context injection code by merging two functions (2) #1467

Closed lym953 closed 1 month ago

lym953 commented 1 month ago

What

Merge addTraceContextToStepFunctionParameters() into its callsites, and removing the extra if checks.

Why

Depending on the original state Input, we will have different instrumentation strategies, so a single addTraceContextToStepFunctionParameters() will no longer work.

Testing

Run npm test to pass existing tests

Review checklist

[ ] Feature or bugfix MUST have appropriate tests (unit, integration)

datadog-datadog-prod-us1[bot] commented 1 month ago

Datadog Report

Branch report: yiming.luo/fix-step-func-14 Commit report: 682a7d6 Test service: datadog-ci-tests

:white_check_mark: 0 Failed, 432 Passed, 0 Skipped, 1m 32.14s Total duration (2m 1.75s time saved)