Track dev-env-import-sql commands properly (#1557)

Automattic · Nov 13, 2023 · 80de030 · 80de030
1 parent b4e542e
commit 80de030
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/bin/vip-dev-env-import-sql.js b/src/bin/vip-dev-env-import-sql.js
@@ -8,7 +8,7 @@ import {
 	handleCLIException,
 	getEnvironmentName,
 } from '../lib/dev-environment/dev-environment-cli';
-import { trackEvent } from '../lib/tracker';
+import { makeCommandTracker } from '../lib/tracker';
 
 const examples = [
 	{
@@ -44,12 +44,15 @@ command( {
 		const slug = await getEnvironmentName( opt );
 		const cmd = new DevEnvImportSQLCommand( fileName, opt, slug );
 		const trackingInfo = getEnvTrackingInfo( cmd.slug );
+		const trackerFn = makeCommandTracker( 'dev_env_import_sql', trackingInfo );
+		await trackerFn( 'execute' );
 
 		try {
 			await cmd.run();
-			await trackEvent( 'dev_env_import_sql_command_success', trackingInfo );
+			await trackerFn( 'success' );
 		} catch ( error ) {
-			await handleCLIException( error, 'dev_env_import_sql_command_error', trackingInfo );
+			await handleCLIException( error );
+			await trackerFn( 'error', { message: error.message, stack: error.stack } );
 			process.exitCode = 1;
 		}
 	} );
diff --git a/src/commands/dev-env-import-sql.js b/src/commands/dev-env-import-sql.js
@@ -18,7 +18,6 @@ export class DevEnvImportSQLCommand {
 	fileName;
 	options;
 	slug;
-	trackingInfo;
 
 	constructor( fileName, options, slug ) {
 		this.fileName = fileName;