fix: don't modify config of external repos

The git config files of external repos are read-only.

This is the only place where we make reference to a remote named
'origin', so this also closes kemitix/git-next#85.

Closes kemitix/git-next#85
This commit is contained in:
Paul Campbell 2024-07-12 08:12:45 +01:00
parent 5f36282667
commit 57a614bad3
2 changed files with 3 additions and 3 deletions

View file

@ -138,7 +138,7 @@ impl RepoDetails {
#[tracing::instrument]
pub fn write_remote_url(&self, url: &RemoteUrl) -> Result<(), kxio::fs::Error> {
if self.gitdir.storage_path_type() != StoragePathType::Internal {
// return Err(Not an internal repo)
return Ok(());
}
let fs = self.gitdir.as_fs();
// load config file

View file

@ -354,7 +354,7 @@ async fn should_reject_message_with_expired_token() -> TestResult {
}
#[test_log::test(actix::test)]
// NOTE: failed then passed on retry: count = 3
// NOTE: failed then passed on retry: count = 6
async fn should_send_validate_repo_when_retryable_error() -> TestResult {
//given
let fs = given::a_filesystem();
@ -372,7 +372,7 @@ async fn should_send_validate_repo_when_retryable_error() -> TestResult {
);
addr.send(crate::messages::ValidateRepo::new(MessageToken::default()))
.await?;
tokio::time::sleep(std::time::Duration::from_millis(1)).await;
tokio::time::sleep(std::time::Duration::from_millis(2)).await;
System::current().stop();
//then