diff --git a/Cargo.lock b/Cargo.lock index 651671a..da1f638 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -541,6 +541,7 @@ name = "query-docker-tags" version = "0.1.0" dependencies = [ "chrono", + "lazy_static", "regex", "reqwest", "serde", diff --git a/Cargo.toml b/Cargo.toml index 988c3ba..741dd04 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -14,6 +14,7 @@ chrono = "0.4.19" tui = "0.16" termion = "1.5" regex = "1.5.4" +lazy_static = "1.4.0" [profile.release] lto = "yes" diff --git a/src/repo.rs b/src/repo.rs index 5ead10c..d14d308 100644 --- a/src/repo.rs +++ b/src/repo.rs @@ -16,9 +16,18 @@ pub enum Repo { Project(String), } -// pub fn extract_yaml(repo: &str) -> (String, String, String, String) { -// let regex = regex::Regex::new(r"( *image *:) *([^/])??/([^/:]):?(.*)?"); -// } +/// check if yaml line matches and returns the split of repo string and rest +pub fn match_yaml_image(input: &str) -> Option<(&str, &str)> { + lazy_static::lazy_static! { + static ref REGEX: regex::Regex = regex::Regex::new(r"^( +image *: *)([a-z0-9\./:]+)").unwrap(); + } + let caps = match REGEX.captures(input) { + Some(caps) => caps, + None => return None, + }; + + Some((caps.get(1).unwrap().as_str(), caps.get(2).unwrap().as_str())) +} pub fn split_repo(repo: &str) -> Result { let split_tag: Vec<&str> = repo.split(":").collect(); @@ -140,4 +149,27 @@ mod tests { )) ); } + + #[test] + fn test_match_yaml_image() { + use crate::repo::match_yaml_image as test_fn; + assert_eq!(test_fn(""), None); + assert_eq!(test_fn("version: '2'"), None); + assert_eq!(test_fn("image: "), None); + assert_eq!(test_fn(" image: "), None); + assert_eq!(test_fn(" image: nginx"), Some((" image: ", "nginx"))); + assert_eq!( + test_fn(" image: library/nginx"), + Some((" image: ", "library/nginx")) + ); + assert_eq!( + test_fn(" image: ghcr.io/library/nginx"), + Some((" image: ", "ghcr.io/library/nginx")) + ); + assert_eq!(test_fn("# image: nginx"), None); + assert_eq!( + test_fn(" image: nginx #comment"), + Some((" image: ", "nginx")) + ); + } } diff --git a/src/widget/service_switcher.rs b/src/widget/service_switcher.rs index 5df2057..b0308f8 100644 --- a/src/widget/service_switcher.rs +++ b/src/widget/service_switcher.rs @@ -4,10 +4,10 @@ use std::io::BufRead; use std::io::BufReader; use std::io::Write; -use regex::Regex; use tui::style::{Color, Style}; use tui::widgets::{Block, Borders, List, ListState}; +use crate::repo; use crate::ui::State; #[derive(Debug)] @@ -28,7 +28,6 @@ impl fmt::Display for Error { pub struct ServiceSwitcher { list: Vec, state: ListState, - regex: Regex, changed: bool, } @@ -47,7 +46,6 @@ impl ServiceSwitcher { Self { list, state: ListState::default(), - regex: Regex::new(r"( *image *): *([^:]*):?([^:]?) *").unwrap(), changed: false, } } @@ -103,7 +101,7 @@ impl ServiceSwitcher { } //check if line matches - if self.regex.is_match(&self.list[i]) { + if repo::match_yaml_image(&self.list[i]).is_some() { self.state.select(Some(i)); return true; } @@ -135,7 +133,7 @@ impl ServiceSwitcher { } //check if line matches - if self.regex.is_match(&self.list[i]) { + if repo::match_yaml_image(&self.list[i]).is_some() { self.state.select(Some(i)); return true; } @@ -152,14 +150,10 @@ impl ServiceSwitcher { pub fn extract_repo(&self) -> Result { match self.state.selected() { None => return Err(Error::NoneSelected), - Some(i) => { - let caps = match self.regex.captures(&self.list[i]) { - None => return Err(Error::Parsing(String::from("Nothing found"))), - Some(cap) => cap, - }; - let result: String = caps.get(2).unwrap().as_str().to_string(); - return Ok(result); - } + Some(i) => match repo::match_yaml_image(&self.list[i]) { + None => return Err(Error::Parsing(String::from("Nothing found"))), + Some((_, repo)) => return Ok(repo.to_string()), + }, } } @@ -167,16 +161,10 @@ impl ServiceSwitcher { pub fn change_current_line(&mut self, repo_with_tag: String) { match self.state.selected() { None => (), - Some(i) => { - let caps = match self.regex.captures(&self.list[i]) { - None => return, - Some(cap) => cap, - }; - let mut line = caps.get(1).unwrap().as_str().to_string(); - line.push_str(": "); - line.push_str(&repo_with_tag); - self.list[i] = line; - } + Some(i) => match repo::match_yaml_image(&self.list[i]) { + None => return, + Some((front, _)) => self.list[i] = format!("{}{}", front, repo_with_tag), + }, } self.changed = true; }