vinzenz/hyperhive
vinzenz/hyperhive
Watch
0
0
Fork
You've already forked hyperhive
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

hive-forge: fix ci-log clippy cast_possible_truncation (use try_from)

Browse source
This commit is contained in:
damocles 2026-06-16 11:51:01 +02:00
commit a69844d463
1 changed files with 14 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

22
hive-forge/src/verbs/ci_log.rs
View file

@ -124,7 +124,11 @@ pub fn run(client: &Client, args: Args) -> Result<()> {
); );
} }
for sl in &steps_log { for sl in &steps_log {
let idx = sl.get("step").and_then(Value::as_u64).unwrap_or_default() as usize; let idx = sl
.get("step")
.and_then(Value::as_u64)
.and_then(|n| usize::try_from(n).ok())
.unwrap_or_default();
println!("=== step {idx}: {} ===", step_name(steps.get(idx))); println!("=== step {idx}: {} ===", step_name(steps.get(idx)));
if let Some(lines) = sl.get("lines").and_then(Value::as_array) { if let Some(lines) = sl.get("lines").and_then(Value::as_array) {
for line in lines { for line in lines {
@ -141,11 +145,10 @@ pub fn run(client: &Client, args: Args) -> Result<()> {
#[cfg(test)] #[cfg(test)]
mod tests { mod tests {
use super::*; use super::*;
use serde_json::json;
#[test] #[test]
fn steps_of_reads_current_job_steps() { fn steps_of_reads_current_job_steps() {
let view = json!({ let view = serde_json::json!({
"state": { "currentJob": { "steps": [ "state": { "currentJob": { "steps": [
{ "summary": "Set up job" }, { "summary": "Set up job" },
{ "summary": "Run tests" } { "summary": "Run tests" }
@ -159,18 +162,21 @@ mod tests {
#[test] #[test]
fn steps_of_missing_is_empty() { fn steps_of_missing_is_empty() {
assert!(steps_of(&json!({})).is_empty()); assert!(steps_of(&serde_json::json!({})).is_empty());
assert!(steps_of(&json!({ "state": {} })).is_empty()); assert!(steps_of(&serde_json::json!({ "state": {} })).is_empty());
} }
#[test] #[test]
fn step_name_falls_back() { fn step_name_falls_back() {
assert_eq!(step_name(Some(&json!({ "name": "compile" }))), "compile"); assert_eq!(
assert_eq!(step_name(Some(&json!({}))), "step"); step_name(Some(&serde_json::json!({ "name": "compile" }))),
"compile"
);
assert_eq!(step_name(Some(&serde_json::json!({}))), "step");
assert_eq!(step_name(None), "step"); assert_eq!(step_name(None), "step");
// `summary` wins over `name` when both are present. // `summary` wins over `name` when both are present.
assert_eq!( assert_eq!(
step_name(Some(&json!({ "summary": "a", "name": "b" }))), step_name(Some(&serde_json::json!({ "summary": "a", "name": "b" }))),
"a" "a"
); );
} }