Skip to content

Commit 9ad50fb

Browse files
committed
Remove unnecessary source texts
1 parent fbe3e1e commit 9ad50fb

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

src/Razor/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/Completion/Delegation/DelegatedCompletionItemResolverTest.NetFx.cs

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -197,23 +197,22 @@ Task FooAsync()
197197
""";
198198
TestFileMarkupParser.GetPosition(input, out var documentContent, out _);
199199
var originalSourceText = SourceText.From(documentContent);
200-
var expectedSourceText = SourceText.From(
201-
"""
200+
var expected = """
202201
@{
203202
async Task FooAsync()
204203
{
205204
await
206205
}
207206
}
208-
""");
207+
""";
209208

210209
// Act
211210
var resolvedItem = await ResolveCompletionItemAsync(input, itemToResolve: "await", supportsVisualStudioExtensions: true, DisposalToken);
212211

213212
// Assert
214213
var textChange = originalSourceText.GetTextChange(resolvedItem.TextEdit.Value.First);
215214
var actualSourceText = originalSourceText.WithChanges(textChange);
216-
Assert.True(expectedSourceText.ContentEquals(actualSourceText));
215+
AssertEx.EqualOrDiff(expected, actualSourceText.ToString());
217216
}
218217

219218
[Fact]
@@ -232,16 +231,15 @@ Task FooAsync()
232231

233232
// Admittedly the result here is not perfect, but only because our tests don't implement the full LSP editor logic. The key thing
234233
// is the addition of the using directive.
235-
var expectedSourceText = SourceText.From(
236-
"""
234+
var expected = """
237235
@using System.Text
238236
@{
239237
Task FooAsync()
240238
{
241239
String
242240
}
243241
}
244-
""");
242+
""";
245243

246244
var codeDocument = CreateCodeDocument(input.Text, filePath: "C:/path/to/file.razor");
247245
// Roslyn won't send unimported types if SupportsVisualStudioExtensions is true
@@ -275,7 +273,7 @@ Task FooAsync()
275273
var originalSourceText = SourceText.From(input.Text);
276274
var textChange = originalSourceText.GetTextChange(resolvedItem.AdditionalTextEdits.Single());
277275
var actualSourceText = originalSourceText.WithChanges(textChange);
278-
AssertEx.EqualOrDiff(expectedSourceText.ToString(), actualSourceText.ToString());
276+
AssertEx.EqualOrDiff(expected, actualSourceText.ToString());
279277
}
280278

281279
[Fact]
@@ -290,23 +288,22 @@ public async Task ResolveAsync_CSharp_OverrideCompletion()
290288
""";
291289
TestFileMarkupParser.GetPosition(input, out var documentContent, out _);
292290
var originalSourceText = SourceText.From(documentContent);
293-
var expectedSourceText = SourceText.From(
294-
"""
291+
var expected = """
295292
@using System.Threading.Tasks
296293
@code {
297294
protected override Task OnInitializedAsync()
298295
{
299296
return base.OnInitializedAsync();
300297
}
301298
}
302-
""");
299+
""";
303300

304301
// Act
305302
var resolvedItem = await ResolveCompletionItemAsync(input, itemToResolve: "OnInitializedAsync()", supportsVisualStudioExtensions: false, DisposalToken);
306303

307304
var textChange = originalSourceText.GetTextChange((TextEdit)resolvedItem.Command.Arguments[1]);
308305
var actualSourceText = originalSourceText.WithChanges(textChange);
309-
AssertEx.EqualOrDiff(expectedSourceText.ToString(), actualSourceText.ToString());
306+
AssertEx.EqualOrDiff(expected, actualSourceText.ToString());
310307
}
311308

312309
[Fact]

0 commit comments

Comments
 (0)