test events

Author: edmundhung

packages/wrangler/src/index.ts

@@ -713,7 +713,7 @@ export async function main(argv: string[]): Promise<void> {
 
 		const durationMs = Date.now() - startTime;
 
-		void dispatcher?.sendNewEvent("wrangler command completed", {
+		await dispatcher?.sendNewEvent("wrangler command completed", {
 			command,
 			args: metricsArgs,
 			durationMs,
@@ -829,7 +829,7 @@ export async function main(argv: string[]): Promise<void> {
 
 		const durationMs = Date.now() - startTime;
 
-		void dispatcher?.sendNewEvent("wrangler command errored", {
+		await dispatcher?.sendNewEvent("wrangler command errored", {
 			command,
 			args: metricsArgs,
 			durationMs,

packages/wrangler/src/metrics/metrics-config.ts

@@ -146,6 +146,10 @@ export function updateMetricsPermission(enabled: boolean) {
  * Get the path to the metrics config file.
  */
 function getMetricsConfigPath(): string {
+	console.log(
+		"getMetricsConfigPath",
+		path.resolve(getGlobalWranglerConfigPath(), "metrics.json")
+	);
 	return path.resolve(getGlobalWranglerConfigPath(), "metrics.json");
 }
 

packages/wrangler/src/metrics/metrics-dispatcher.ts

@@ -102,7 +102,7 @@ export function getMetricsDispatcher(options: MetricsConfigOptions) {
 
 		// Do not await this fetch call.
 		// Just fire-and-forget, otherwise we might slow down the rest of Wrangler.
-		fetch(`${SPARROW_URL}/api/v1/event`, {
+		await fetch(`${SPARROW_URL}/api/v1/event`, {
 			method: "POST",
 			headers: {
 				Accept: "*/*",