Merge branch 'release/2.2'

This commit is contained in:
N. Taylor Mullen 2018-07-30 10:47:41 -07:00
commit fd170c1cea
2 changed files with 28 additions and 1 deletions

View File

@ -64,7 +64,7 @@ namespace Microsoft.VisualStudio.Editor.Razor
var previousLineEndIndex = GetPreviousLineEndIndex(syntaxTreeSnapshot, line);
var simulatedChange = new SourceChange(previousLineEndIndex, 0, string.Empty);
var owningSpan = syntaxTree.Root.LocateOwner(simulatedChange);
if (owningSpan.Kind == SpanKindInternal.Code)
if (owningSpan == null || owningSpan.Kind == SpanKindInternal.Code)
{
// Example,
// @{\n

View File

@ -167,6 +167,33 @@ namespace Microsoft.VisualStudio.Editor.Razor
Assert.Equal(0, indentLevel);
}
// This test verifies that we still operate on SyntaxTree's that have gaps in them. The gaps are temporary
// until our work with the parser has been completed.
[Fact]
public void GetDesiredIndentation_ReturnsNull_IfOwningSpanDoesNotExist()
{
// Arrange
var source = new StringTextSnapshot($@"
<div>
<div>
</div>
</div>
");
var syntaxTree = GetSyntaxTree(new StringTextSnapshot("something else"));
var service = new DefaultRazorIndentationFactsService();
// Act
var indentation = service.GetDesiredIndentation(
syntaxTree,
source,
source.GetLineFromLineNumber(3),
indentSize: 4,
tabSize: 1);
// Assert
Assert.Null(indentation);
}
[Fact]
public void GetDesiredIndentation_ReturnsNull_IfOwningSpanIsCode()
{