Skip to content

Commit d7f85ee

Browse files
committed
fix: clarify whoami api key source
1 parent 7954d02 commit d7f85ee

2 files changed

Lines changed: 28 additions & 1 deletion

File tree

src/cortex-cli/src/cli/handlers.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -551,7 +551,7 @@ pub async fn run_whoami() -> Result<()> {
551551
&& !token.is_empty()
552552
{
553553
println!(
554-
"Authenticated via CORTEX_API_KEY: {}",
554+
"Authenticated via CORTEX_API_KEY environment variable: {}",
555555
safe_format_key(&token)
556556
);
557557
return Ok(());
Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
use std::process::Command;
2+
3+
#[test]
4+
fn whoami_cortex_api_key_names_environment_variable_source() {
5+
let output = Command::new(env!("CARGO_BIN_EXE_Cortex"))
6+
.arg("whoami")
7+
.env("CORTEX_API_KEY", "test-key-for-whoami")
8+
.env_remove("CORTEX_AUTH_TOKEN")
9+
.output()
10+
.expect("run Cortex whoami");
11+
12+
let stdout = String::from_utf8_lossy(&output.stdout);
13+
let stderr = String::from_utf8_lossy(&output.stderr);
14+
15+
assert!(
16+
output.status.success(),
17+
"whoami failed\nstdout:\n{stdout}\nstderr:\n{stderr}"
18+
);
19+
assert!(
20+
stdout.contains("Authenticated via CORTEX_API_KEY environment variable:"),
21+
"expected environment variable source label\nstdout:\n{stdout}\nstderr:\n{stderr}"
22+
);
23+
assert!(
24+
!stdout.contains("Authenticated via CORTEX_API_KEY:"),
25+
"stale auth-source wording should not be present\nstdout:\n{stdout}\nstderr:\n{stderr}"
26+
);
27+
}

0 commit comments

Comments
 (0)