[v15] fix connection tester error for Aurora databases (#49374)

* fix connection tester error for Aurora databases

Make database name a required field in the connection tester.

* fix cockroachdb engine name typo

* update required db name helper

Author: GavinFrazar

web/packages/teleport/src/Discover/Database/TestConnection/TestConnection.tsx

@@ -41,7 +41,7 @@ import {
   StyledBox,
   useConnectionDiagnostic,
 } from '../../Shared';
-import { DatabaseEngine } from '../../SelectResource';
+import { DatabaseEngine, getDatabaseProtocol } from '../../SelectResource';
 
 export function TestConnection() {
   const { resourceSpec, agentMeta } = useDiscover();
@@ -183,8 +183,7 @@ export function TestConnection() {
                 isDisabled={
                   attempt.status === 'processing' || nameOpts.length === 0
                 }
-                // Database name is required for Postgres.
-                isClearable={dbEngine !== DatabaseEngine.Postgres}
+                isClearable={!isDbNameRequired(dbEngine)}
               />
               <CustomInputFieldForAsterisks
                 selectedOption={selectedDbName}
@@ -240,3 +239,17 @@ function getInputValue(input: string, inputKind: 'name' | 'user') {
   }
   return input;
 }
+
+function isDbNameRequired(engine: DatabaseEngine) {
+  const protocol = getDatabaseProtocol(engine);
+  switch (protocol) {
+    case 'mongodb':
+    case 'oracle':
+    case 'postgres':
+    case 'spanner':
+    case 'sqlserver':
+      return true;
+    default:
+      return false;
+  }
+}

web/packages/teleport/src/Discover/SelectResource/databases.tsx

@@ -257,7 +257,7 @@ export const DATABASES_UNGUIDED: ResourceSpec[] = [
   {
     dbMeta: {
       location: DatabaseLocation.SelfHosted,
-      engine: DatabaseEngine.CoackroachDb,
+      engine: DatabaseEngine.CockroachDb,
     },
     name: 'CockroachDB',
     keywords: selfhostedKeywords + 'cockroachdb',
@@ -404,13 +404,32 @@ export function getDatabaseProtocol(engine: DatabaseEngine): DbProtocol {
   switch (engine) {
     case DatabaseEngine.Postgres:
     case DatabaseEngine.AuroraPostgres:
+    case DatabaseEngine.Redshift:
       return 'postgres';
     case DatabaseEngine.MySql:
     case DatabaseEngine.AuroraMysql:
       return 'mysql';
+    case DatabaseEngine.MongoDb:
+      return 'mongodb';
+    case DatabaseEngine.Redis:
+      return 'redis';
+    case DatabaseEngine.CockroachDb:
+      return 'cockroachdb';
+    case DatabaseEngine.SqlServer:
+      return 'sqlserver';
+    case DatabaseEngine.Snowflake:
+      return 'snowflake';
+    case DatabaseEngine.Cassandra:
+      return 'cassandra';
+    case DatabaseEngine.ElasticSearch:
+      return 'elasticsearch';
+    case DatabaseEngine.DynamoDb:
+      return 'dynamodb';
+    case DatabaseEngine.Doc:
+      return '' as any;
+    default:
+      engine satisfies never;
   }
-
-  return '' as any;
 }
 
 export function getDefaultDatabasePort(engine: DatabaseEngine): string {

web/packages/teleport/src/Discover/SelectResource/types.ts

@@ -50,7 +50,7 @@ export enum DatabaseEngine {
   AuroraMysql,
   MongoDb,
   Redis,
-  CoackroachDb,
+  CockroachDb,
   SqlServer,
   Snowflake,
   Cassandra,