Add exact-match integration tests for note searches#911
Merged
multiplex55 merged 1 commit intomasterfrom Feb 24, 2026
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
LauncherApp::search()honorssettings.match_exact = truefor note-related commands and returns resolved note actions rather than onlyquery:*suggestions.note today,note search <term>, and a transformed command path (note new Hello World) behave deterministically under exact-match mode.Description
setup_notes_envto seedML_NOTES_DIR/HOMEand callsave_notes()for deterministic note test setup.note_today_returns_resolved_note_action_when_exact_match_enabledthat assertsnote todayyields at least one result whoseactionstarts withnote:new:and not onlyquery:*whenmatch_exactis enabled.note_search_matches_note_content_when_exact_match_enabledthat seeds notes (one containingneedle) and assertsnote search needlereturnsnote:open:betaand not onlyquery:*whenmatch_exactis enabled.note_new_generates_slugged_action_when_exact_match_enabledthat assertsnote new Hello Worldproducesnote:new:hello-worldand notquery:*under exact-match mode.set_current_dirpatterns andLauncherApp::new(...)scaffolding used elsewhere in tests for consistency.Testing
cargo test --test plugin_exact_matchto run the new tests, but the build failed due to a missing system dependency required by a transitive crate (alsa-sys), specifically the systemalsapkg-config metadata/library; no test assertions were executed to completion in this environment.Codex Task