Skip to content

Commit 65f926f

Browse files
authored
Merge pull request #54 from NixOS/small-cleanup
Small cleanup
2 parents 6b1b172 + 5a08e59 commit 65f926f

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/action/base/create_user.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ async fn execute_dscl_retry_on_specific_errors(dscl_args: &[&str]) -> Result<(),
268268
);
269269
break;
270270
} else if retry_tokens == 0 {
271-
return Err(ActionErrorKind::command_output(&command, output))?;
271+
return Err(ActionErrorKind::command_output(&command, output));
272272
} else {
273273
if output.status.code() == Some(140) && stderr.contains("-14988 (eNotYetImplemented)") {
274274
// Retry due to buggy macOS user behavior?
@@ -384,7 +384,7 @@ pub async fn delete_user_macos(name: &str) -> Result<(), ActionErrorKind> {
384384
},
385385
Some(185) if stderr.contains("-14009 (eDSUnknownNodeName)") => {
386386
// The user has already been deleted
387-
tracing::debug!("User already deleted: /Users/{}`", name);
387+
tracing::debug!("User already deleted: /Users/{}", name);
388388
},
389389
_ => {
390390
// Something went wrong

src/action/common/place_nix_configuration.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -566,13 +566,13 @@ mod tests {
566566
.expect("place nix config should succeed");
567567

568568
let standard_conf = tokio::fs::read_to_string(nix_conf_path).await.unwrap();
569-
assert!(standard_conf.contains("trusted-user"), "trusted-user setting should exist in standard conf so that we don't break cachix users");
569+
assert!(standard_conf.contains("trusted-users"), "trusted-users setting should exist in standard conf so that we don't break cachix users");
570570

571571
let custom_conf = tokio::fs::read_to_string(nix_custom_conf_path)
572572
.await
573573
.unwrap();
574574
assert!(
575-
custom_conf.contains("trusted-user"),
575+
custom_conf.contains("trusted-users"),
576576
"trusted-user setting should exist in custom conf as well"
577577
);
578578

0 commit comments

Comments
 (0)