tk/spectre.console
1
0
Fork 0
mirror of https://github.com/nsnail/spectre.console.git synced 2025-04-16 00:42:51 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix @ being used in figlet font

Browse Source
This commit is contained in:
Salvage 2022-09-21 19:03:11 +02:00 committed by Patrik Svensson
parent fd8d271a10
commit 999c59909b
6 changed files with 1261 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Spectre.Console/Widgets/Figlet/FigletFontParser.cs
View File

@ -35,7 +35,7 @@ internal static class FigletFontParser
throw new InvalidOperationException("Unknown index for FIGlet character");
}
buffer.Add(line.Replace(header.Hardblank, ' ').ReplaceExact("@", string.Empty));
buffer.Add(line.Replace(header.Hardblank, ' ').TrimEnd('@'));
if (line.EndsWith("@@", StringComparison.Ordinal))
{

1239
test/Spectre.Console.Tests/Data/poison.flf Normal file
View File

File diff suppressed because it is too large Load Diff

12
test/Spectre.Console.Tests/Expectations/Widgets/Figlet/Load_Stream.Output_fontfile=poison.flf.verified.txt Normal file
View File

@ -0,0 +1,12 @@

@@@@@@@ @@@@@@ @@@@@@@ @@@@@@@ @@@ @@@ @@@ @@@ @@@ @@@ @@@@@@ @@@@@@ @@@ @@@ @@@@@@@@ @@@@@@@ @@@@@@@@
@@@@@@@@ @@@@@@@@ @@@@@@@ @@@@@@@@ @@@ @@@ @@@ @@@ @@@ @@@ @@@@@@@@ @@@@@@@ @@@ @@@ @@@@@@@@ @@@@@@@@ @@@@@@@@
@@! @@@ @@! @@@ @@! @@! @@@ @@! @@! !@@ @@! @@! @@! @@! @@@ !@@ @@! @@@ @@! @@! @@@ @@!
!@! @!@ !@! @!@ !@! !@! @!@ !@! !@! @!! !@! !@! !@! !@! @!@ !@! !@! @!@ !@! !@! @!@ !@!
@!@@!@! @!@!@!@! @!! @!@!!@! !!@ @!@@!@! @!! !!@ @!@ @!@!@!@! !!@@!! @!@!@!@! @!!!:! @!@!!@! @!!!:!
!!@!!! !!!@!!!! !!! !!@!@! !!! !!@!!! !@! !!! !@! !!!@!!!! !!@!!! !!!@!!!! !!!!!: !!@!@! !!!!!:
!!: !!: !!! !!: !!: :!! !!: !!: :!! !!: !!: !!: !!: !!! !:! !!: !!! !!: !!: :!! !!:
:!: :!: !:! :!: :!: !:! :!: :!: !:! :!: :!: :!: :!: !:! !:! :!: !:! :!: :!: !:! :!:
:: :: ::: :: :: ::: :: :: ::: :::: :: ::: :: ::: :::: :: :: ::: :: :::: :: ::: :: ::::
: : : : : : : : : : ::: :: : : : : : : :: : : : : : : :: :: : : : : :: ::

0
test/Spectre.Console.Tests/Expectations/Widgets/Figlet/Load_Stream.Output.verified.txt → test/Spectre.Console.Tests/Expectations/Widgets/Figlet/Load_Stream.Output_fontfile=starwars.flf.verified.txt
View File

2
test/Spectre.Console.Tests/Spectre.Console.Tests.csproj
View File

@ -9,6 +9,8 @@
<AdditionalFiles Include="..\..\src\stylecop.json" Link="Properties/stylecop.json" />
<None Remove="Data\starwars.flf" />
<EmbeddedResource Include="Data\starwars.flf" />
<None Remove="Data\poison.flf" />
<EmbeddedResource Include="Data\poison.flf" />
</ItemGroup>
<ItemGroup>

11
test/Spectre.Console.Tests/Unit/Widgets/FigletTests.cs
View File

@ -4,20 +4,23 @@ namespace Spectre.Console.Tests.Unit;
[ExpectationPath("Widgets/Figlet")]
public sealed class FigletTests
{
[Fact]
[Theory]
[InlineData("starwars.flf")]
[InlineData("poison.flf")]
[Expectation("Load_Stream")]
public async Task Should_Load_Font_From_Stream()
public async Task Should_Load_Font_From_Stream(string fontfile)
{
// Given
var console = new TestConsole().Width(180);
var font = FigletFont.Load(EmbeddedResourceReader.LoadResourceStream("Spectre.Console.Tests/Data/starwars.flf"));
var font = FigletFont.Load(EmbeddedResourceReader.LoadResourceStream($"Spectre.Console.Tests/Data/{fontfile}"));
var text = new FigletText(font, "Patrik was here");
// When
console.Write(text);
// Then
await Verifier.Verify(console.Output);
await Verifier.Verify(console.Output)
.UseParameters(fontfile);
}
[Fact]