using System.Text;
using Xunit.Abstractions;
namespace DriverTests;
///
/// Tests for https://github.com/gui-cs/Terminal.Gui/issues/4466.
/// These tests validate that FillRect properly handles wide characters when overlapping existing content.
/// Specifically, they ensure that wide characters are properly invalidated and replaced when a MessageBox border or similar UI element is drawn over them, preventing visual corruption.
///
public class OutputBufferWideCharTests (ITestOutputHelper output)
{
///
/// Tests that FillRect properly invalidates wide characters when overwriting them.
/// This is the core issue in #4466 - when a MessageBox border is drawn over Chinese text,
/// the wide characters need to be properly invalidated.
///
[Fact]
[Trait ("Category", "Output")]
public void FillRect_OverwritesWideChar_InvalidatesProperly ()
{
// Arrange - Create a buffer and draw a wide character
OutputBufferImpl buffer = new ()
{
Rows = 5, Cols = 10,
CurrentAttribute = new (Color.White, Color.Black)
};
// Draw a Chinese character (2 columns wide) at position 2,1
buffer.Move (2, 1);
buffer.AddStr ("你"); // Chinese character "you", 2 columns wide
// Verify the wide character was drawn
Assert.Equal ("你", buffer.Contents! [1, 2].Grapheme);
Assert.True (buffer.Contents [1, 2].IsDirty);
// With the fix, the second column should NOT be modified by AddStr
// The wide glyph naturally renders across both columns
Assert.NotEqual ("你", buffer.Contents [1, 3].Grapheme);
// Clear dirty flags to test FillRect behavior
for (var r = 0; r < buffer.Rows; r++)
{
for (var c = 0; c < buffer.Cols; c++)
{
buffer.Contents [r, c].IsDirty = false;
}
}
// Act - Fill a rectangle that overlaps the first column of the wide character
// This simulates drawing a MessageBox border over Chinese text
buffer.FillRect (new (2, 1, 1, 1), new Rune ('│'));
// Assert
// With FIXES_4466: FillRect calls AddStr, which properly invalidates the wide character
// The wide character at [1,2] should be replaced with replacement char or the new content
Assert.Equal ("│", buffer.Contents [1, 2].Grapheme);
Assert.True (buffer.Contents [1, 2].IsDirty, "Cell [1,2] should be marked dirty after FillRect");
// The adjacent cell should also be marked dirty for proper rendering
Assert.True (buffer.Contents [1, 3].IsDirty, "Adjacent cell [1,3] should be marked dirty to ensure proper rendering");
}
///
/// Tests that FillRect handles overwriting the second column of a wide character.
/// When drawing at an odd column that's the second half of a wide glyph, the
/// wide glyph should be invalidated.
///
[Fact]
[Trait ("Category", "Output")]
public void FillRect_OverwritesSecondColumnOfWideChar_InvalidatesWideChar ()
{
// Arrange
OutputBufferImpl buffer = new ()
{
Rows = 5, Cols = 10,
CurrentAttribute = new (Color.White, Color.Black)
};
// Draw a wide character at position 2,1
buffer.Move (2, 1);
buffer.AddStr ("好"); // Chinese character, 2 columns wide
Assert.Equal ("好", buffer.Contents! [1, 2].Grapheme);
// Clear dirty flags
for (var r = 0; r < buffer.Rows; r++)
{
for (var c = 0; c < buffer.Cols; c++)
{
buffer.Contents [r, c].IsDirty = false;
}
}
// Act - Fill at the second column of the wide character (position 3)
buffer.FillRect (new (3, 1, 1, 1), new Rune ('│'));
// Assert
// With the fix: The original wide character at col 2 should be invalidated
// because we're overwriting its second column
Assert.True (buffer.Contents [1, 2].IsDirty, "Wide char at col 2 should be invalidated when its second column is overwritten");
Assert.Equal (buffer.Contents [1, 2].Grapheme, Rune.ReplacementChar.ToString ());
Assert.Equal ("│", buffer.Contents [1, 3].Grapheme);
Assert.True (buffer.Contents [1, 3].IsDirty);
}
///
/// Tests the ChineseUI scenario: Drawing a MessageBox with borders over Chinese button text.
/// This simulates the specific repro case from the issue. See: https://github.com/gui-cs/Terminal.Gui/issues/4466
///
[Fact]
[Trait ("Category", "Output")]
public void ChineseUI_MessageBox_Over_WideChars ()
{
// Arrange - Simulate the ChineseUI scenario
OutputBufferImpl buffer = new ()
{
Rows = 10, Cols = 30,
CurrentAttribute = new (Color.White, Color.Black)
};
// Draw Chinese button text (like "你好呀")
buffer.Move (5, 3);
buffer.AddStr ("你好呀"); // 3 Chinese characters, 6 columns total
// Verify initial state
Assert.Equal ("你", buffer.Contents! [3, 5].Grapheme);
Assert.Equal ("好", buffer.Contents [3, 7].Grapheme);
Assert.Equal ("呀", buffer.Contents [3, 9].Grapheme);
// Clear dirty flags to simulate the state before MessageBox draws
for (var r = 0; r < buffer.Rows; r++)
{
for (var c = 0; c < buffer.Cols; c++)
{
buffer.Contents [r, c].IsDirty = false;
}
}
// Act - Draw a MessageBox border that partially overlaps the Chinese text
// This simulates the mouse moving over the border, causing HighlightState changes
// Draw vertical line at column 8 (overlaps second char "好")
for (var row = 2; row < 6; row++)
{
buffer.FillRect (new (8, row, 1, 1), new Rune ('│'));
}
// Assert - The wide characters should be properly handled
// With the fix: Wide characters are properly invalidated
// The first character "你" at col 5 should be unaffected
Assert.Equal ("你", buffer.Contents [3, 5].Grapheme);
// The second character "好" at col 7 had its second column overwritten
// so it should be replaced with replacement char
Assert.Equal (buffer.Contents [3, 7].Grapheme, Rune.ReplacementChar.ToString ());
Assert.True (buffer.Contents [3, 7].IsDirty, "Invalidated wide char should be marked dirty");
// The border should be drawn at col 8
Assert.Equal ("│", buffer.Contents [3, 8].Grapheme);
Assert.True (buffer.Contents [3, 8].IsDirty);
// The third character "呀" at col 9 should be unaffected
Assert.Equal ("呀", buffer.Contents [3, 9].Grapheme);
}
///
/// Tests that FillRect works correctly with single-width characters (baseline behavior).
/// This should work the same with or without FIXES_4466.
///
[Fact]
[Trait ("Category", "Output")]
public void FillRect_SingleWidthChars_WorksCorrectly ()
{
// Arrange
OutputBufferImpl buffer = new ()
{
Rows = 5, Cols = 10,
CurrentAttribute = new (Color.White, Color.Black)
};
// Draw some ASCII text
buffer.Move (2, 1);
buffer.AddStr ("ABC");
Assert.Equal ("A", buffer.Contents! [1, 2].Grapheme);
Assert.Equal ("B", buffer.Contents [1, 3].Grapheme);
Assert.Equal ("C", buffer.Contents [1, 4].Grapheme);
// Clear dirty flags
for (var r = 0; r < buffer.Rows; r++)
{
for (var c = 0; c < buffer.Cols; c++)
{
buffer.Contents [r, c].IsDirty = false;
}
}
// Act - Overwrite with FillRect
buffer.FillRect (new (3, 1, 1, 1), new Rune ('X'));
// Assert - This should work the same regardless of FIXES_4466
Assert.Equal ("A", buffer.Contents [1, 2].Grapheme);
Assert.Equal ("X", buffer.Contents [1, 3].Grapheme);
Assert.True (buffer.Contents [1, 3].IsDirty);
Assert.Equal ("C", buffer.Contents [1, 4].Grapheme);
}
///
/// Tests FillRect with wide characters at buffer boundaries.
///
[Fact]
[Trait ("Category", "Output")]
public void FillRect_WideChar_AtBufferBoundary ()
{
// Arrange
OutputBufferImpl buffer = new ()
{
Rows = 5, Cols = 10,
CurrentAttribute = new (Color.White, Color.Black)
};
// Draw a wide character at the right edge (col 8, which would extend to col 9)
buffer.Move (8, 1);
buffer.AddStr ("山"); // Chinese character "mountain", 2 columns wide
Assert.Equal ("山", buffer.Contents! [1, 8].Grapheme);
// Clear dirty flags
for (var r = 0; r < buffer.Rows; r++)
{
for (var c = 0; c < buffer.Cols; c++)
{
buffer.Contents [r, c].IsDirty = false;
}
}
// Act - FillRect at the wide character position
buffer.FillRect (new (8, 1, 1, 1), new Rune ('│'));
// Assert
Assert.Equal ("│", buffer.Contents [1, 8].Grapheme);
Assert.True (buffer.Contents [1, 8].IsDirty);
// Adjacent cell should be marked dirty
Assert.True (
buffer.Contents [1, 9].IsDirty,
"Cell after wide char replacement should be marked dirty");
}
///
/// Tests OutputBase.Write method marks cells dirty correctly for wide characters.
/// This tests the other half of the fix in OutputBase.cs.
///
[Fact]
[Trait ("Category", "Output")]
public void OutputBase_Write_WideChar_MarksCellsDirty ()
{
// Arrange
OutputBufferImpl buffer = new ()
{
Rows = 5, Cols = 20,
CurrentAttribute = new (Color.White, Color.Black)
};
// Draw a line with wide characters
buffer.Move (0, 1);
buffer.AddStr ("你好"); // Two wide characters
// Mark all as not dirty to simulate post-Write state
for (var r = 0; r < buffer.Rows; r++)
{
for (var c = 0; c < buffer.Cols; c++)
{
buffer.Contents! [r, c].IsDirty = false;
}
}
// Verify initial state
Assert.Equal ("你", buffer.Contents! [1, 0].Grapheme);
Assert.Equal ("好", buffer.Contents [1, 2].Grapheme);
// Act - Now overwrite the first wide char by writing at its position
buffer.Move (0, 1);
buffer.AddStr ("A"); // Single width char
// Assert
// With the fix: The first cell is replaced with 'A' and marked dirty
Assert.Equal ("A", buffer.Contents [1, 0].Grapheme);
Assert.True (buffer.Contents [1, 0].IsDirty);
// The adjacent cell (col 1) should be marked dirty for proper rendering
Assert.True (
buffer.Contents [1, 1].IsDirty,
"Adjacent cell should be marked dirty after writing single-width char over wide char");
// The second wide char should remain
Assert.Equal ("好", buffer.Contents [1, 2].Grapheme);
}
///
/// Tests that filling a rectangle with spaces properly handles wide character cleanup.
/// This simulates clearing a region that contains wide characters.
///
[Fact]
[Trait ("Category", "Output")]
public void FillRect_WithSpaces_OverWideChars ()
{
// Arrange
OutputBufferImpl buffer = new ()
{
Rows = 5, Cols = 15,
CurrentAttribute = new (Color.White, Color.Black)
};
// Draw a line of mixed content
buffer.Move (2, 2);
buffer.AddStr ("A你B好C");
// Verify setup
Assert.Equal ("A", buffer.Contents! [2, 2].Grapheme);
Assert.Equal ("你", buffer.Contents [2, 3].Grapheme);
Assert.Equal ("B", buffer.Contents [2, 5].Grapheme);
Assert.Equal ("好", buffer.Contents [2, 6].Grapheme);
Assert.Equal ("C", buffer.Contents [2, 8].Grapheme);
// Clear dirty flags
for (var r = 0; r < buffer.Rows; r++)
{
for (var c = 0; c < buffer.Cols; c++)
{
buffer.Contents [r, c].IsDirty = false;
}
}
// Act - Fill the region with spaces (simulating clearing)
buffer.FillRect (new (3, 2, 4, 1), new Rune (' '));
// Assert
// With the fix: Wide characters are properly handled
Assert.Equal (" ", buffer.Contents [2, 3].Grapheme);
Assert.True (buffer.Contents [2, 3].IsDirty);
// Wide character '你' at col 3 was replaced, so col 4 should be marked dirty
Assert.True (
buffer.Contents [2, 4].IsDirty,
"Cell after replaced wide char should be dirty");
Assert.Equal (" ", buffer.Contents [2, 4].Grapheme);
Assert.Equal (" ", buffer.Contents [2, 5].Grapheme);
Assert.Equal (" ", buffer.Contents [2, 6].Grapheme);
// Cell 7 should be dirty because '好' was partially overwritten
Assert.True (
buffer.Contents [2, 7].IsDirty,
"Adjacent cell should be dirty after wide char replacement");
}
}