Skip to content

Commit

Permalink
Make map-syntax target case insensitive
Browse files Browse the repository at this point in the history
  • Loading branch information
Keith Hall committed Feb 9, 2025
1 parent ac40f7c commit c3e4cf6
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/assets.rs
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ impl HighlightingAssets {

if let Some(MappingTarget::MapTo(syntax_name)) = syntax_match {
return self
.find_syntax_by_name(syntax_name)?
.find_syntax_by_token(syntax_name)?
.ok_or_else(|| Error::UnknownSyntax(syntax_name.to_owned()));
}

Expand Down Expand Up @@ -259,6 +259,13 @@ impl HighlightingAssets {
.map(|syntax| SyntaxReferenceInSet { syntax, syntax_set }))
}

fn find_syntax_by_token(&self, token: &str) -> Result<Option<SyntaxReferenceInSet>> {
let syntax_set = self.get_syntax_set()?;
Ok(syntax_set
.find_syntax_by_token(token)
.map(|syntax| SyntaxReferenceInSet { syntax, syntax_set }))
}

fn get_syntax_for_file_name(
&self,
file_name: &OsStr,
Expand Down
21 changes: 21 additions & 0 deletions tests/integration_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2751,6 +2751,27 @@ fn highlighting_independant_from_map_syntax_case() {
.stderr("");
}

#[test]
fn map_syntax_target_syntax_case_insensitive() {
let expected = bat()
.arg("-f")
.arg("--map-syntax=*.config:json")
.arg("map-syntax_case.Config")
.assert()
.get_output()
.stdout
.clone();

bat()
.arg("-f")
.arg("--map-syntax=*.config:json")
.arg("map-syntax_case.Config")
.assert()
.success()
.stdout(expected)
.stderr("");
}

#[test]
fn strip_ansi_always_strips_ansi() {
bat()
Expand Down

0 comments on commit c3e4cf6

Please sign in to comment.