Skip to content

Commit 0e8a0ae

Browse files
SLVS-2714 Fix: Certain whitespace changes do not trigger reanalysis (#6530)
Part of SLVS-2604
1 parent 8025258 commit 0e8a0ae

File tree

2 files changed

+7
-17
lines changed

2 files changed

+7
-17
lines changed

src/Integration.Vsix.UnitTests/SonarLintTagger/TextBufferIssueTrackerTests.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -258,20 +258,20 @@ public void UpdateAnalysisState_CriticalException_IsNotSuppressed()
258258
CreateTextChange("", ""),
259259
CreateTextChange("\r\n", "\t\nSOMETEXT"),
260260
];
261-
private static object[][] OnTextBufferChangedOnBackground_UpdatesIfNotWhitespace_Params =>
261+
private static object[][] OnTextBufferChangedOnBackground_Updates_Params =>
262262
[
263-
[EmptyAndWhitespaceChanges, false],
264-
[NonEmptyChanges, true]
263+
[EmptyAndWhitespaceChanges],
264+
[NonEmptyChanges]
265265
];
266-
[DynamicData(nameof(OnTextBufferChangedOnBackground_UpdatesIfNotWhitespace_Params))]
266+
[DynamicData(nameof(OnTextBufferChangedOnBackground_Updates_Params))]
267267
[DataTestMethod]
268-
public void OnTextBufferChangedOnBackground_UpdatesIfNotWhitespace(List<ITextChange> changes, bool didUpdate)
268+
public void OnTextBufferChangedOnBackground_Updates(List<ITextChange> changes)
269269
{
270270
ClearMocks();
271271
mockedJavascriptDocumentFooJs.TextBuffer.CurrentSnapshot.Version.Changes.Returns(new TestableNormalizedTextChangeCollection(changes));
272272
RaiseTextBufferChangedOnBackground(currentTextBuffer: mockDocumentTextBuffer, CreateTextSnapshotMock());
273273

274-
documentTrackerUpdater.Received(didUpdate ? 1 : 0).OnDocumentUpdated(testSubject);
274+
documentTrackerUpdater.Received(1).OnDocumentUpdated(testSubject);
275275
VerifyMetadataNotUpdated();
276276
VerifyIssueConsumerNotCreated();
277277
VerifyIssueConsumerNotRemoved();

src/Integration.Vsix/SonarLintTagger/TextBufferIssueTracker.cs

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -197,15 +197,5 @@ private void CreateIssueConsumer(FileSnapshot fileSnapshot)
197197
issueConsumerStorage.Set(FilePath, issueConsumer);
198198
}
199199

200-
private void TextBuffer_OnChangedOnBackground(object sender, TextContentChangedEventArgs e)
201-
{
202-
var normalizedTextChangeCollection = e.Changes;
203-
204-
if (normalizedTextChangeCollection.All(x => string.IsNullOrWhiteSpace(x.NewText) && string.IsNullOrWhiteSpace(x.OldText)))
205-
{
206-
return;
207-
}
208-
209-
documentTrackerUpdater.OnDocumentUpdated(this);
210-
}
200+
private void TextBuffer_OnChangedOnBackground(object sender, TextContentChangedEventArgs e) => documentTrackerUpdater.OnDocumentUpdated(this);
211201
}

0 commit comments

Comments
 (0)