snake_case + make errors generic

dust-tt · Dec 12, 2024 · e45fa09 · e45fa09
1 parent ed4bcc5
commit e45fa09
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/core/src/databases/remote_databases/get_remote_database.rs b/core/src/databases/remote_databases/get_remote_database.rs
@@ -17,8 +17,8 @@ pub async fn get_remote_database(
             let db = SnowflakeRemoteDatabase::new(content)?;
             Ok(Box::new(db) as Box<dyn RemoteDatabase + Sync + Send>)
         }
-        CredentialProvider::Modjo => {
-            anyhow::bail!("Modjo is not a remote database provider")
+        _ => {
+            anyhow::bail!("{:?} is not a supported remote database provider", provider)
         }
     }
 }
diff --git a/core/src/oauth/credential.rs b/core/src/oauth/credential.rs
@@ -109,7 +109,7 @@ impl Credential {
                 vec!["account", "warehouse", "username", "password", "role"]
             }
             CredentialProvider::Modjo => {
-                vec!["apiKey"]
+                vec!["api_key"]
             }
         };
 

diff --git a/types/src/oauth/lib.ts b/types/src/oauth/lib.ts
@@ -75,7 +75,7 @@ export const SnowflakeCredentialsSchema = t.type({
 export type SnowflakeCredentials = t.TypeOf<typeof SnowflakeCredentialsSchema>;
 
 export const ApiKeyCredentialsSchema = t.type({
-  apiKey: t.string,
+  api_key: t.string,
 });
 export type ModjoCredentials = t.TypeOf<typeof ApiKeyCredentialsSchema>;