Skip to content

Commit 28e2a03

Browse files
authored
* Remove workaround for spectreconsole/spectre.console#608 * Fix tests
1 parent a78abda commit 28e2a03

File tree

3 files changed

+12
-15
lines changed

3 files changed

+12
-15
lines changed

ConsoleMarkdownRenderer.Tests/DisplayTests.cs

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ public void DisplayTests_ExitWorks()
2525
AssertCrossPlatStringMatch(@"- [](sub/sub.md)
2626
2727
> Done
28-
{}(sub/sub.md)", TrimmedConsoleOutput);
28+
[](sub/sub.md)", TrimmedConsoleOutput);
2929
}
3030

3131
[TestMethod]
@@ -39,7 +39,7 @@ public void DisplayTests_CanPassText()
3939
AssertCrossPlatStringMatch(@"- [](sub/sub.md)
4040
4141
> Done
42-
{}(sub/sub.md)", TrimmedConsoleOutput);
42+
[](sub/sub.md)", TrimmedConsoleOutput);
4343
}
4444

4545
[TestMethod]
@@ -55,12 +55,12 @@ public void DisplayTests_CanFollowingLinks()
5555
AssertCrossPlatStringMatch(@"- [](sub/sub.md)
5656
5757
> Done
58-
{}(sub/sub.md) Done
59-
> {}(sub/sub.md) -[](../start.md)
58+
[](sub/sub.md) Done
59+
> [](sub/sub.md) -[](../start.md)
6060
6161
> Done
6262
Back
63-
{}(../start.md)", TrimmedConsoleOutput);
63+
[](../start.md)", TrimmedConsoleOutput);
6464
}
6565

6666
[TestMethod]
@@ -79,17 +79,17 @@ public void DisplayTests_BackWorks()
7979
AssertCrossPlatStringMatch(@"- [](sub/sub.md)
8080
8181
> Done
82-
{}(sub/sub.md) Done
83-
> {}(sub/sub.md) -[](../start.md)
82+
[](sub/sub.md) Done
83+
> [](sub/sub.md) -[](../start.md)
8484
8585
> Done
8686
Back
87-
{}(../start.md) Done
87+
[](../start.md) Done
8888
> Back
89-
{}(../start.md) - [](sub/sub.md)
89+
[](../start.md) - [](sub/sub.md)
9090
9191
> Done
92-
{}(sub/sub.md)", TrimmedConsoleOutput);
92+
[](sub/sub.md)", TrimmedConsoleOutput);
9393
}
9494

9595
[TestMethod]

Displayer.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ private static void DisplayMarkdown(string text, Uri baseUri, DisplayOptions? op
227227
{
228228
done => "Done",
229229
back => "Back",
230-
_ => links[i].ToString(),
230+
_ => Markup.Escape(links[i].ToString()),
231231
};
232232

233233
var needToPrompt = true;

ObjectRenderers/LinkItem.cs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,6 @@ public LinkItem(LinkInline link, string content)
2424
public readonly LinkInline Link;
2525

2626
public override string ToString()
27-
=> $"{(Link.IsImage ? "!" : string.Empty)}[{Content}]({Link.Url})"
28-
// Workaround for https://github.com/spectreconsole/spectre.console/issues/608
29-
.Replace('[', '{')
30-
.Replace(']', '}');
27+
=> $"{(Link.IsImage ? "!" : string.Empty)}[{Content}]({Link.Url})";
3128
}
3229
}

0 commit comments

Comments
 (0)