Use helix_view::apply_transaction in integration-tests
`helix_view::apply_transaction` closes over `Document::apply` and `View::apply` to ensure that jumplist entries are updated when a document changes from a transaction. `Document::apply` shouldn't be called directly - this helper function should be used instead.
This commit is contained in:
parent
4cff625054
commit
1243db11a5
1 changed files with 7 additions and 8 deletions
|
@ -125,13 +125,12 @@ pub async fn test_key_sequence_with_input_text<T: Into<TestCase>>(
|
|||
let sel = doc.selection(view.id).clone();
|
||||
|
||||
// replace the initial text with the input text
|
||||
doc.apply(
|
||||
&Transaction::change_by_selection(doc.text(), &sel, |_| {
|
||||
(0, doc.text().len_chars(), Some((&test_case.in_text).into()))
|
||||
})
|
||||
.with_selection(test_case.in_selection.clone()),
|
||||
view.id,
|
||||
);
|
||||
let transaction = Transaction::change_by_selection(doc.text(), &sel, |_| {
|
||||
(0, doc.text().len_chars(), Some((&test_case.in_text).into()))
|
||||
})
|
||||
.with_selection(test_case.in_selection.clone());
|
||||
|
||||
helix_view::apply_transaction(&transaction, doc, view);
|
||||
|
||||
test_key_sequence(
|
||||
&mut app,
|
||||
|
@ -286,7 +285,7 @@ impl AppBuilder {
|
|||
.with_selection(selection);
|
||||
|
||||
// replace the initial text with the input text
|
||||
doc.apply(&trans, view.id);
|
||||
helix_view::apply_transaction(&trans, doc, view);
|
||||
}
|
||||
|
||||
Ok(app)
|
||||
|
|
Loading…
Reference in a new issue