Skip to content

Commit b80df1b

Browse files
authored
Merge pull request #67030 from CyrusNajmabadi/disablePreviewDiagnostics
Disable diagnostics in the preview window
2 parents dd8fb67 + 3e57878 commit b80df1b

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

src/EditorFeatures/Core/Diagnostics/AbstractPushOrPullDiagnosticsTaggerProvider.SingleDiagnosticKindPullTaggerProvider.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,9 @@ private async Task ProduceTagsAsync(
9595
if (document == null)
9696
return;
9797

98+
if (document.Project.Solution.Workspace.Kind == WorkspaceKind.Preview)
99+
return;
100+
98101
var snapshot = documentSpanToTag.SnapshotSpan.Snapshot;
99102

100103
var project = document.Project;

src/EditorFeatures/Test/Preview/PreviewWorkspaceTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ public async Task TestPreviewDiagnosticTagger()
157157
}
158158

159159
[WpfFact]
160-
public async Task TestPreviewDiagnosticTaggerInPreviewPane()
160+
public async Task TestPreviewDiagnosticTaggerInPreviewPaneDoesNotWork()
161161
{
162162
// TODO: WPF required due to https://github.com/dotnet/roslyn/issues/46153
163163
using var workspace = TestWorkspace.CreateCSharp("class { }", composition: EditorTestCompositions.EditorFeaturesWpf);
@@ -173,7 +173,7 @@ public async Task TestPreviewDiagnosticTaggerInPreviewPane()
173173
var newDocument = oldDocument.WithText(oldText.WithChanges(new TextChange(new TextSpan(0, oldText.Length), "class C { }")));
174174

175175
// create a diff view
176-
WpfTestRunner.RequireWpfFact($"{nameof(TestPreviewDiagnosticTaggerInPreviewPane)} creates a {nameof(DifferenceViewerPreview)}");
176+
WpfTestRunner.RequireWpfFact($"{nameof(TestPreviewDiagnosticTaggerInPreviewPaneDoesNotWork)} creates a {nameof(DifferenceViewerPreview)}");
177177

178178
var previewFactoryService = (PreviewFactoryService)workspace.ExportProvider.GetExportedValue<IPreviewFactoryService>();
179179
using var diffView = await previewFactoryService.CreateChangedDocumentPreviewViewAsync(oldDocument, newDocument, CancellationToken.None);
@@ -211,7 +211,7 @@ public async Task TestPreviewDiagnosticTaggerInPreviewPane()
211211
// check left buffer
212212
var leftSnapshot = leftBuffer.CurrentSnapshot;
213213
var leftSpans = leftTagger.GetTags(leftSnapshot.GetSnapshotSpanCollection()).ToList();
214-
Assert.Equal(1, leftSpans.Count);
214+
Assert.Equal(0, leftSpans.Count);
215215

216216
// check right buffer
217217
var rightSnapshot = rightBuffer.CurrentSnapshot;

0 commit comments

Comments
 (0)