From 06b5a5813874b408d471bb0cbcc8456cc411114d Mon Sep 17 00:00:00 2001
From: Nathan Flurry <git@nathanflurry.com>
Date: Tue, 17 Sep 2024 00:49:03 -0700
Subject: [PATCH] fix(backend): fix incorrect status/stop commands

---
 .../cli/commands/db/instance/status.ts        | 27 ++++++++++++++++---
 .../backend/cli/commands/db/instance/stop.ts  | 25 +++--------------
 2 files changed, 27 insertions(+), 25 deletions(-)

diff --git a/packages/backend/cli/commands/db/instance/status.ts b/packages/backend/cli/commands/db/instance/status.ts
index 1bba5910..7f83d529 100644
--- a/packages/backend/cli/commands/db/instance/status.ts
+++ b/packages/backend/cli/commands/db/instance/status.ts
@@ -1,8 +1,9 @@
 import { z } from "zod";
 import { globalOptsSchema, initProject } from "../../../common.ts";
 import { getDefaultPostgresManager } from "../../../../toolchain/postgres/mod.ts";
-import { stop } from "../../../../toolchain/postgres/manager.ts";
-import { success, warn } from "../../../../toolchain/term/status.ts";
+import { Status, status } from "../../../../toolchain/postgres/manager.ts";
+import { info, warn } from "../../../../toolchain/term/status.ts";
+import { UnreachableError } from "../../../../toolchain/error/mod.ts";
 
 export const optsSchema = globalOptsSchema;
 
@@ -12,8 +13,26 @@ export async function execute(opts: Opts) {
 	const project = await initProject(opts);
 	const manager = await getDefaultPostgresManager(project);
 	if (manager) {
-		await stop(manager);
-		success("Postgres instance stopped");
+		let statusText: string;
+		const currentStatus = await status(manager);
+		if (currentStatus === Status.NotInstalled) {
+			statusText = "Not installed";
+		} else if (currentStatus === Status.Installed) {
+			statusText = "Installed";
+		} else if (currentStatus === Status.Initialized) {
+			statusText = "Initialized";
+		} else if (currentStatus === Status.DefaultDatabaseNotCreated) {
+			statusText = "Default database not created";
+		} else if (currentStatus === Status.Stopped) {
+			statusText = "Stopped";
+		} else if (currentStatus === Status.Started) {
+			statusText = "Started";
+		} else if (currentStatus === Status.Connectable) {
+			statusText = "Connectable";
+		} else {
+			throw new UnreachableError(currentStatus);
+		}
+		info("Status", statusText);
 	} else {
 		warn("Postgres is disabled");
 	}
diff --git a/packages/backend/cli/commands/db/instance/stop.ts b/packages/backend/cli/commands/db/instance/stop.ts
index 2a80a071..1bba5910 100644
--- a/packages/backend/cli/commands/db/instance/stop.ts
+++ b/packages/backend/cli/commands/db/instance/stop.ts
@@ -1,9 +1,8 @@
 import { z } from "zod";
 import { globalOptsSchema, initProject } from "../../../common.ts";
 import { getDefaultPostgresManager } from "../../../../toolchain/postgres/mod.ts";
-import { Status, status } from "../../../../toolchain/postgres/manager.ts";
-import { info, warn } from "../../../../toolchain/term/status.ts";
-import { UnreachableError } from "../../../../toolchain/error/mod.ts";
+import { stop } from "../../../../toolchain/postgres/manager.ts";
+import { success, warn } from "../../../../toolchain/term/status.ts";
 
 export const optsSchema = globalOptsSchema;
 
@@ -13,24 +12,8 @@ export async function execute(opts: Opts) {
 	const project = await initProject(opts);
 	const manager = await getDefaultPostgresManager(project);
 	if (manager) {
-		let statusText: string;
-		const currentStatus = await status(manager);
-		if (currentStatus === Status.NotInstalled) {
-			statusText = "Not installed";
-		} else if (currentStatus === Status.Installed) {
-			statusText = "Installed";
-		} else if (currentStatus === Status.Initialized) {
-			statusText = "Initialized";
-		} else if (currentStatus === Status.DefaultDatabaseNotCreated) {
-			statusText = "Default database not created";
-		} else if (currentStatus === Status.Stopped) {
-			statusText = "Stopped";
-		} else if (currentStatus === Status.Started) {
-			statusText = "Started";
-		} else {
-			throw new UnreachableError(currentStatus);
-		}
-		info("Status", statusText);
+		await stop(manager);
+		success("Postgres instance stopped");
 	} else {
 		warn("Postgres is disabled");
 	}