Skip to content

Commit

Permalink
Implement Display instead ToString for Container State as per clippy …
Browse files Browse the repository at this point in the history
…warning
  • Loading branch information
piotr-iohk committed Sep 17, 2024
1 parent a84a08a commit ffdaf11
Showing 1 changed file with 15 additions and 13 deletions.
28 changes: 15 additions & 13 deletions src/docker/manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ use crate::{
utils::run_command,
};
use serde::{Deserialize, Serialize};
use std::fmt;
use std::fs::File;
use std::io::Write;
use std::{
Expand Down Expand Up @@ -195,7 +196,7 @@ impl DockerManager {

if let Some(state) = filter {
cmd.push("--filter".to_string());
cmd.push(format!("status={}", state.to_string()));
cmd.push(format!("status={}", state));
}

// Convert Vec<String> to Vec<&str> for compatibility with run_docker_compose
Expand Down Expand Up @@ -374,17 +375,18 @@ impl DockerManager {
}
}

impl ToString for ContainerState {
fn to_string(&self) -> String {
match self {
ContainerState::Created => "created".into(),
ContainerState::Exited => "exited".into(),
ContainerState::Running => "running".into(),
ContainerState::Paused => "paused".into(),
ContainerState::Restarting => "restarting".into(),
ContainerState::Removing => "removing".into(),
ContainerState::Dead => "dead".into(),
ContainerState::Unknown => "unknown".into(),
}
impl fmt::Display for ContainerState {
fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
let state_str = match self {
ContainerState::Created => "created",
ContainerState::Exited => "exited",
ContainerState::Running => "running",
ContainerState::Paused => "paused",
ContainerState::Restarting => "restarting",
ContainerState::Removing => "removing",
ContainerState::Dead => "dead",
ContainerState::Unknown => "unknown",
};
write!(f, "{}", state_str)
}
}

0 comments on commit ffdaf11

Please sign in to comment.