Skip to content

Commit a024eeb

Browse files
authored
Support removal of promise() for ExpressionStatement (#405)
1 parent 219abe1 commit a024eeb

File tree

8 files changed

+24
-0
lines changed

8 files changed

+24
-0
lines changed

.changeset/happy-glasses-turn.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"aws-sdk-js-codemod": patch
3+
---
4+
5+
Support removal of promise() for ExpressionStatement

src/transforms/v2-to-v3/__fixtures__/api-promise/global-import.input.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ import AWS from "aws-sdk";
22

33
const client = new AWS.DynamoDB();
44

5+
// ExpressionStatement
6+
client.listTables().promise();
7+
58
// async/await
69
try {
710
await client.listTables().promise();

src/transforms/v2-to-v3/__fixtures__/api-promise/global-import.output.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ import { DynamoDB } from "@aws-sdk/client-dynamodb";
22

33
const client = new DynamoDB();
44

5+
// ExpressionStatement
6+
client.listTables();
7+
58
// async/await
69
try {
710
await client.listTables();

src/transforms/v2-to-v3/__fixtures__/api-promise/service-import.input.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ import DynamoDBClient from "aws-sdk/clients/dynamodb";
22

33
const client = new DynamoDBClient();
44

5+
// ExpressionStatement
6+
client.listTables().promise();
7+
58
// async/await
69
try {
710
await client.listTables().promise();

src/transforms/v2-to-v3/__fixtures__/api-promise/service-import.output.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ import { DynamoDB as DynamoDBClient } from "@aws-sdk/client-dynamodb";
22

33
const client = new DynamoDBClient();
44

5+
// ExpressionStatement
6+
client.listTables();
7+
58
// async/await
69
try {
710
await client.listTables();

src/transforms/v2-to-v3/__fixtures__/api-promise/service-require.input.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ const DynamoDBClient = require("aws-sdk/clients/dynamodb");
22

33
const client = new DynamoDBClient();
44

5+
// ExpressionStatement
6+
client.listTables().promise();
7+
58
// async/await
69
try {
710
await client.listTables().promise();

src/transforms/v2-to-v3/__fixtures__/api-promise/service-require.output.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ const {
44

55
const client = new DynamoDBClient();
66

7+
// ExpressionStatement
8+
client.listTables();
9+
710
// async/await
811
try {
912
await client.listTables();

src/transforms/v2-to-v3/apis/removePromiseForCallExpression.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ export const removePromiseForCallExpression = (callExpression: ASTPath<CallExpre
1111
}
1212
case "ArrowFunctionExpression":
1313
case "AwaitExpression":
14+
case "ExpressionStatement":
1415
case "ObjectProperty":
1516
case "ReturnStatement":
1617
case "VariableDeclarator": {

0 commit comments

Comments
 (0)