mirror of
https://github.com/zed-industries/zed.git
synced 2026-05-25 14:44:28 +00:00
Many editors such as vim and emacs support "modelines", a comment at the beginning of the file that allows the file type to be explicitly specified along with per-file specific settings - The amount of configurations, style and settings mapping cannot be handled in one go, so this opens up a lot of potential improvements. - I left out the possiblity to have "zed" specific modelines for now, but this could be potentially interesting. - Mapping the mode or filetype to zed language names isn't obvious either. We may want to make it configurable. This is my first contribution to zed, be kind. I struggled a bit to find the right place to add those settings. I use a similar approach as done with editorconfig (merge_with_editorconfig). There might be better ways. Closes #4762 Release Notes: - Add basic emacs/vim modeline support. Supersedes #41899, changes: - limit reading to the first and last 1kb - add documentation - more variables handled - add Arc around ModelineSettings to avoid extra cloning - changed the way mode -> language mapping is done, thanks to `modeline_aliases` language config - drop vim ex: support - made "Local Variables:" handling a separate commit, so we can drop it easily - various code style improvements --------- Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Conrad Irwin <conrad.irwin@gmail.com>
156 lines
4 KiB
Rust
156 lines
4 KiB
Rust
use project::search::SearchQuery;
|
|
use text::Rope;
|
|
use util::{
|
|
paths::{PathMatcher, PathStyle},
|
|
rel_path::RelPath,
|
|
};
|
|
|
|
#[test]
|
|
fn path_matcher_creation_for_valid_paths() {
|
|
for valid_path in [
|
|
"file",
|
|
"Cargo.toml",
|
|
".DS_Store",
|
|
"~/dir/another_dir/",
|
|
"./dir/file",
|
|
"dir/[a-z].txt",
|
|
] {
|
|
let path_matcher = PathMatcher::new(&[valid_path.to_owned()], PathStyle::local())
|
|
.unwrap_or_else(|e| panic!("Valid path {valid_path} should be accepted, but got: {e}"));
|
|
assert!(
|
|
path_matcher.is_match(&RelPath::new(valid_path.as_ref(), PathStyle::local()).unwrap()),
|
|
"Path matcher for valid path {valid_path} should match itself"
|
|
)
|
|
}
|
|
}
|
|
|
|
#[test]
|
|
fn path_matcher_creation_for_globs() {
|
|
for invalid_glob in ["dir/[].txt", "dir/[a-z.txt", "dir/{file"] {
|
|
match PathMatcher::new(&[invalid_glob.to_owned()], PathStyle::local()) {
|
|
Ok(_) => panic!("Invalid glob {invalid_glob} should not be accepted"),
|
|
Err(_expected) => {}
|
|
}
|
|
}
|
|
|
|
for valid_glob in [
|
|
"dir/?ile",
|
|
"dir/*.txt",
|
|
"dir/**/file",
|
|
"dir/[a-z].txt",
|
|
"{dir,file}",
|
|
] {
|
|
match PathMatcher::new(&[valid_glob.to_owned()], PathStyle::local()) {
|
|
Ok(_expected) => {}
|
|
Err(e) => panic!("Valid glob should be accepted, but got: {e}"),
|
|
}
|
|
}
|
|
}
|
|
|
|
#[test]
|
|
fn test_case_sensitive_pattern_items() {
|
|
let case_sensitive = false;
|
|
let search_query = SearchQuery::regex(
|
|
"test\\C",
|
|
false,
|
|
case_sensitive,
|
|
false,
|
|
false,
|
|
Default::default(),
|
|
Default::default(),
|
|
false,
|
|
None,
|
|
)
|
|
.expect("Should be able to create a regex SearchQuery");
|
|
|
|
assert_eq!(
|
|
search_query.case_sensitive(),
|
|
true,
|
|
"Case sensitivity should be enabled when \\C pattern item is present in the query."
|
|
);
|
|
|
|
let case_sensitive = true;
|
|
let search_query = SearchQuery::regex(
|
|
"test\\c",
|
|
true,
|
|
case_sensitive,
|
|
false,
|
|
false,
|
|
Default::default(),
|
|
Default::default(),
|
|
false,
|
|
None,
|
|
)
|
|
.expect("Should be able to create a regex SearchQuery");
|
|
|
|
assert_eq!(
|
|
search_query.case_sensitive(),
|
|
false,
|
|
"Case sensitivity should be disabled when \\c pattern item is present, even if initially set to true."
|
|
);
|
|
|
|
let case_sensitive = false;
|
|
let search_query = SearchQuery::regex(
|
|
"test\\c\\C",
|
|
false,
|
|
case_sensitive,
|
|
false,
|
|
false,
|
|
Default::default(),
|
|
Default::default(),
|
|
false,
|
|
None,
|
|
)
|
|
.expect("Should be able to create a regex SearchQuery");
|
|
|
|
assert_eq!(
|
|
search_query.case_sensitive(),
|
|
true,
|
|
"Case sensitivity should be enabled when \\C is the last pattern item, even after a \\c."
|
|
);
|
|
|
|
let case_sensitive = false;
|
|
let search_query = SearchQuery::regex(
|
|
"tests\\\\C",
|
|
false,
|
|
case_sensitive,
|
|
false,
|
|
false,
|
|
Default::default(),
|
|
Default::default(),
|
|
false,
|
|
None,
|
|
)
|
|
.expect("Should be able to create a regex SearchQuery");
|
|
|
|
assert_eq!(
|
|
search_query.case_sensitive(),
|
|
false,
|
|
"Case sensitivity should not be enabled when \\C pattern item is preceded by a backslash."
|
|
);
|
|
}
|
|
|
|
#[gpui::test]
|
|
async fn test_multiline_regex(cx: &mut gpui::TestAppContext) {
|
|
let search_query = SearchQuery::regex(
|
|
"^hello$\n",
|
|
false,
|
|
false,
|
|
false,
|
|
false,
|
|
Default::default(),
|
|
Default::default(),
|
|
false,
|
|
None,
|
|
)
|
|
.expect("Should be able to create a regex SearchQuery");
|
|
|
|
use language::Buffer;
|
|
let text = Rope::from("hello\nworld\nhello\nworld");
|
|
let snapshot = cx
|
|
.update(|app| Buffer::build_snapshot(text, None, None, None, app))
|
|
.await;
|
|
|
|
let results = search_query.search(&snapshot, None).await;
|
|
assert_eq!(results, vec![0..6, 12..18]);
|
|
}
|