Skip to content

Commit 8fe31de

Browse files
committed
Apply suggestions from code review
1 parent 3f9237c commit 8fe31de

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

__tests__/commands/dev-env-sync-sql.ts

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -243,10 +243,13 @@ describe( 'commands/DevEnvSyncSQLCommand', () => {
243243
);
244244
await cmd.generateExport();
245245

246-
expect.objectContaining( {
247-
error_type: 'live_backup_copy',
248-
error_message: 'ops!',
249-
} );
246+
expect( mockTracker ).toHaveBeenCalledWith(
247+
'error',
248+
expect.objectContaining( {
249+
error_type: 'live_backup_copy',
250+
error_message: 'oooops!',
251+
} )
252+
);
250253
expect( exit.withError ).toHaveBeenCalledWith( 'Error creating backup copy: oooops!' );
251254
} );
252255
} );

src/lib/live-backup-copy.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ export async function startLiveBackupCopy( {
8181
variables: {
8282
input: {
8383
id: appId,
84-
tool: SQLDumpTool.MYSQLDUMP,
84+
tool: config.tool,
8585
environmentId,
8686
tables,
8787
type: config.type,

0 commit comments

Comments
 (0)