feat(stepfunctions-tasks): await the eval so async ops can be passed to tasks.EvaluateExpression

[3p3r]

Currently the way EvaluateExpression is set up, if you pass an async expression to it, it never waits for the execution to complete. By adding an await before the eval() call, async ops can be passed in. This also works for sync ops same as before (await works with sync code as well).

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[gitpod-io]

[mergify]

Title does not follow the guidelines of Conventional Commits. Please adjust title before merge.

[BenChaimberg]

Thanks for your contribution! Can you just give me an example of such an asynchronous operation?

[3p3r]

@BenChaimberg absolutely! here is a snippet I have in my step function which is currently not possible without this patch:

new tasks.EvaluateExpression(this, 'test', {
  expression: `(async () => {
    const AWS = require("aws-sdk");
    const iam = new AWS.IAM();
    return await iam.listRoles().promise();
  })()`,
  resultPath: "$.roles"
})

The addition of the await before the eval() in the patch allows me to run this async code and actually wait for it to finish processing before control is handed back to my state machine.

[mergify]

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
• Commit ID: d70093d
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[mergify]

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

[jorge-ortega]

This change seems to have been reverted by this commit. Was this intentional?