Skip to content

Commit ef85e6e

Browse files
committed
fix: remove strict editor validation in update method to fix integration tests
The integration tests were failing because the update method was too strict in validating the editor. In test environments, the editor might not be in visibleTextEditors list. Keeping the validation only for scrolling operations where it makes sense.
1 parent 5afee2a commit ef85e6e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/integrations/editor/DiffViewProvider.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ export class DiffViewProvider {
119119
const diffEditor = this.activeDiffEditor
120120
const document = diffEditor?.document
121121

122-
if (!diffEditor || !document || !this.isEditorValid(diffEditor)) {
122+
if (!diffEditor || !document) {
123123
throw new Error("User closed text editor, unable to edit file...")
124124
}
125125

0 commit comments

Comments
 (0)