Update config handling and add e2e tests

- Add tests for local list get and set of OpenAI model
- Add `get_config_path` function for retrieving either local or user config path
- Update `delete` and `set` functions to use `get_config_path` function
- Add async-std dependency and update anyhow and async-trait versions

[e2e/test_config_local_list_get_set.sh]
- Add tests for local list get and set of OpenAI model
- Assert that local config set and delete fail when not using --local flag
- Assert that local config get returns the correct value after setting and deleting with --local flag
[src/actions/config.rs]
- Add `async_std::path::Path` to the imports
- Change the `get_config_path` function to handle both local and user config paths
- Add `get_config_path` function for retrieving either local or user config path
- Add error handling to the `delete` and `set` functions for when no config path is found
- Update the `delete` and `set` functions to use the `get_config_path` function
[Cargo.toml]
- Add async-std dependency
- Update anyhow and async-trait versions

Author: zurawiki

Cargo.toml

@@ -18,6 +18,7 @@ path = "src/main.rs"
 
 [dependencies]
 anyhow = "1.0.69"
+async-std = "1.12.0"
 async-trait = "0.1.64"
 clap = { version = "4.1.6", features = ["derive"] }
 colored = "2.0.0"

e2e/test_config_local_list_get_set.sh

@@ -24,3 +24,35 @@ export TEMPDIR=$(mktemp -d)
     # assert is default
 )
 rm -rf "${TEMPDIR}"
+
+
+export TEMPDIR=$(mktemp -d)
+(
+    cd "${TEMPDIR}"
+
+    gptcommit config list
+    # assert is valid TOML
+
+    gptcommit config get openai.model
+    # assert default = text-davinci-003
+    set +e
+    gptcommit config set --local openai.model foo
+    # TODO assert output
+    test $? -ne 0 || exit $?
+    set -e
+    gptcommit config set openai.model bar
+    gptcommit config get openai.model
+    # assert is foo
+
+    gptcommit config delete openai.model
+    gptcommit config get openai.model
+    # assert still is foo
+    set +e
+    gptcommit config delete --local openai.model
+    # TODO assert output
+    test $? -ne 0 || exit $?
+    set -e
+    gptcommit config get openai.model
+    # assert is default
+)
+rm -rf "${TEMPDIR}"

src/actions/config.rs

@@ -1,5 +1,6 @@
 use std::{collections::VecDeque, fs, path::PathBuf};
 
+use async_std::path::Path;
 use clap::{Args, Subcommand};
 use toml::Value;
 
@@ -50,14 +51,26 @@ pub(crate) async fn main(settings: Settings, args: ConfigArgs) -> Result<()> {
     }
 }
 
+fn get_config_path(local: bool) -> Result<PathBuf> {
+    if local {
+        if let Some(config_path) = get_local_config_path() {
+            Ok(config_path)
+        } else {
+            bail!("No repo-local config found. Please run `git init` to create a repo first");
+        }
+    } else {
+                if let Some(config_path) = get_user_config_path() {
+            Ok(config_path)
+        } else {
+            bail!("No user config found.");
+        }
+    }
+}
+
 async fn delete(_settings: Settings, full_key: String, local: bool) -> Result<()> {
     let settings = &Settings::from_clear(&full_key)?;
     let toml_string = toml::to_string_pretty(settings).unwrap();
-    let config_path: PathBuf = if local {
-        get_local_config_path().expect("Could not find repo-local config path")
-    } else {
-        get_user_config_path().expect("Could not find user config path")
-    };
+    let config_path = get_config_path(local)?;
     fs::write(&config_path, toml_string)?;
     println!("Cleared {full_key}");
     println!("Config saved to {}", config_path.display());
@@ -67,11 +80,7 @@ async fn delete(_settings: Settings, full_key: String, local: bool) -> Result<()
 async fn set(_settings: Settings, full_key: String, value: String, local: bool) -> Result<()> {
     let settings = &Settings::from_set_override(&full_key, &value)?;
     let toml_string = toml::to_string_pretty(settings).unwrap();
-    let config_path: PathBuf = if local {
-        get_local_config_path().expect("Could not find repo-local config path")
-    } else {
-        get_user_config_path().expect("Could not find user config path")
-    };
+    let config_path = get_config_path(local)?;
     fs::write(&config_path, toml_string)?;
     println!("{full_key} = {value}");
     println!("Config saved to {}", config_path.display());