Use --hashing-algorithm sha512 in a test

Author: michaelklishin

src/commands.rs

@@ -915,7 +915,7 @@ pub fn declare_user(client: APIClient, command_args: &ArgMatches) -> ClientResul
             .unwrap();
         let salt = password_hashing::salt();
         let hash = hashing_algo.salt_and_hash(&salt, password).unwrap();
-        String::from_utf8_lossy(hash.as_slice()).to_string()
+        String::from_utf8(hash).unwrap().to_string()
     } else {
         provided_hash.to_string()
     };

tests/users_tests.rs

@@ -86,9 +86,12 @@ fn test_list_users_with_table_styles() -> Result<(), Box<dyn std::error::Error>>
 }
 
 #[test]
-fn test_create_user() -> Result<(), Box<dyn std::error::Error>> {
-    let username = "test_create_user.1";
-    let password = "pa$$w0rd///8*9";
+fn test_create_user_using_sha256_for_hashing() -> Result<(), Box<dyn std::error::Error>> {
+    let username = "test_create_user_using_sha256_for_hashing.1";
+    let password = "pa$$w0rd_9w798f__sd8f7";
+
+    run_succeeds(["users", "delete", "--name", username, "--idempotently"]);
+
     run_succeeds([
         "users",
         "declare",
@@ -107,5 +110,33 @@ fn test_create_user() -> Result<(), Box<dyn std::error::Error>> {
 
     run_succeeds(["list", "users"]).stdout(predicate::str::contains(username).not());
 
+    Ok(())
+}
+
+#[test]
+fn test_create_user_using_sha512_for_hashing() -> Result<(), Box<dyn std::error::Error>> {
+    let username = "test_create_user_using_sha512_for_hashing.1";
+    let password = "pa$$w0rd///8*9";
+
+    run_succeeds(["users", "delete", "--name", username, "--idempotently"]);
+
+    run_succeeds([
+        "users",
+        "declare",
+        "--name",
+        username,
+        "--password",
+        password,
+        "--hashing-algorithm",
+        "sha512",
+        "--tags",
+        "administrator",
+    ]);
+    // unless the node is also configured to use SHA-512, we cannot try this
+    // password the same way we do in the SHA-256 version, by passing in --username and --password
+    run_succeeds(["users", "delete", "--name", username]);
+
+    run_succeeds(["list", "users"]).stdout(predicate::str::contains(username).not());
+
     Ok(())
 }