Hopiu/lychee
1
0
Fork 0
mirror of https://github.com/Hopiu/lychee.git synced 2026-04-05 14:01:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: Add accept option to merged config (#1344)

Browse source
This commit is contained in:
Techassi 2024-01-09 20:55:39 +01:00 committed by GitHub
parent 8aa9081f1a
commit 0d0be52844
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 24 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
lychee-bin/src/options.rs
View file

@ -148,7 +148,7 @@ impl LycheeOptions {
}
#[allow(clippy::struct_excessive_bools)]
#[derive(Parser, Debug, Deserialize, Clone)]
#[derive(Parser, Debug, Deserialize, Clone, Default)]
pub(crate) struct Config {
/// Verbose program output
#[clap(flatten)]
@ -452,6 +452,7 @@ impl Config {
require_https: false;
cookie_jar: None;
include_fragments: false;
accept: AcceptSelector::default();
}
if self
@ -469,3 +470,25 @@ impl Config {
}
}
}
#[cfg(test)]
mod tests {
use super::*;
#[test]
fn test_accept_status_codes() {
let toml = Config {
accept: AcceptSelector::from_str("200..=204, 429, 500").unwrap(),
..Default::default()
};
let mut cli = Config::default();
cli.merge(toml);
assert!(cli.accept.contains(429));
assert!(cli.accept.contains(200));
assert!(cli.accept.contains(203));
assert!(cli.accept.contains(204));
assert!(!cli.accept.contains(205));
}
}