| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688 |
- #nullable enable
- using UnitTests;
- namespace ViewBaseTests.Drawing;
- [Trait ("Category", "Output")]
- public class NeedsDrawTests : FakeDriverBase
- {
- [Fact]
- public void NeedsDraw_False_If_Width_Height_Zero ()
- {
- View view = new () { Width = 0, Height = 0 };
- view.BeginInit ();
- view.EndInit ();
- Assert.False (view.NeedsDraw);
- //Assert.False (view.SubViewNeedsDraw);
- }
- [Fact]
- public void NeedsDraw_True_Initially_If_Width_Height_Not_Zero ()
- {
- View superView = new () { Driver = CreateFakeDriver (), Width = 1, Height = 1 };
- View view1 = new () { Width = 1, Height = 1 };
- View view2 = new () { Width = 1, Height = 1 };
- superView.Add (view1, view2);
- superView.BeginInit ();
- superView.EndInit ();
- Assert.True (superView.NeedsDraw);
- Assert.True (superView.SubViewNeedsDraw);
- Assert.True (view1.NeedsDraw);
- Assert.True (view2.NeedsDraw);
- superView.Layout (); // NeedsDraw is always false if Layout is needed
- superView.Draw ();
- Assert.False (superView.NeedsDraw);
- Assert.False (superView.SubViewNeedsDraw);
- Assert.False (view1.NeedsDraw);
- Assert.False (view2.NeedsDraw);
- superView.SetNeedsDraw ();
- Assert.True (superView.NeedsDraw);
- Assert.True (superView.SubViewNeedsDraw);
- Assert.True (view1.NeedsDraw);
- Assert.True (view2.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_True_After_Constructor ()
- {
- var view = new View { Width = 2, Height = 2 };
- Assert.True (view.NeedsDraw);
- view = new () { Width = 2, Height = 2, BorderStyle = LineStyle.Single };
- Assert.True (view.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_True_After_BeginInit ()
- {
- var view = new View { Width = 2, Height = 2, BorderStyle = LineStyle.Single };
- Assert.True (view.NeedsDraw);
- view.BeginInit ();
- Assert.True (view.NeedsDraw);
- view.NeedsDraw = false;
- view.BeginInit ();
- Assert.False (view.NeedsDraw); // Because layout is still needed
- view.Layout ();
- // NeedsDraw is true after layout and NeedsLayout is false if SubViewsLaidOut doesn't call SetNeedsLayout
- Assert.True (view.NeedsDraw);
- Assert.False (view.NeedsLayout);
- }
- [Fact]
- public void NeedsDraw_True_After_EndInit_Where_Call_Layout ()
- {
- var view = new View { Width = 2, Height = 2, BorderStyle = LineStyle.Single };
- Assert.True (view.NeedsDraw);
- view.BeginInit ();
- Assert.True (view.NeedsDraw);
- view.EndInit ();
- Assert.True (view.NeedsDraw);
- view = new () { Width = 2, Height = 2, BorderStyle = LineStyle.Single };
- view.BeginInit ();
- view.NeedsDraw = false;
- view.EndInit ();
- Assert.True (view.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_After_SetLayoutNeeded_And_Layout ()
- {
- var view = new View { Driver = CreateFakeDriver (), Width = 2, Height = 2 };
- Assert.True (view.NeedsDraw);
- Assert.False (view.NeedsLayout);
- view.Draw ();
- Assert.False (view.NeedsDraw);
- Assert.False (view.NeedsLayout);
- view.SetNeedsLayout ();
- Assert.False (view.NeedsDraw);
- Assert.True (view.NeedsLayout);
- view.Layout ();
- Assert.True (view.NeedsDraw);
- Assert.False (view.NeedsLayout);
- }
- [Fact]
- public void NeedsDraw_False_After_SetRelativeLayout_Absolute_Dims ()
- {
- var view = new View { Driver = CreateFakeDriver (), Width = 2, Height = 2 };
- Assert.True (view.NeedsDraw);
- view.Draw ();
- Assert.False (view.NeedsDraw);
- Assert.False (view.NeedsLayout);
- // SRL won't change anything since the view frame wasn't changed
- view.SetRelativeLayout (new (100, 100));
- Assert.False (view.NeedsDraw);
- view.SetNeedsLayout ();
- // SRL won't change anything since the view frame wasn't changed
- // SRL doesn't depend on NeedsLayout, but LayoutSubViews does
- view.SetRelativeLayout (new (100, 100));
- Assert.False (view.NeedsDraw);
- Assert.True (view.NeedsLayout);
- view.Layout ();
- Assert.True (view.NeedsDraw);
- Assert.False (view.NeedsLayout);
- view.NeedsDraw = false;
- // SRL won't change anything since the view frame wasn't changed. However, Layout has not been called
- view.SetRelativeLayout (new (10, 10));
- Assert.False (view.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_False_After_SetRelativeLayout_Relative_Dims ()
- {
- var view = new View { Width = Dim.Percent (50), Height = Dim.Percent (50) };
- View superView = new ()
- {
- Id = "superView",
- Width = Dim.Fill (),
- Height = Dim.Fill ()
- };
- // A layout wasn't called yet, so NeedsDraw is still empty
- Assert.False (superView.NeedsDraw);
- superView.Add (view);
- // A layout wasn't called yet, so NeedsDraw is still empty
- Assert.False (view.NeedsDraw);
- Assert.False (superView.NeedsDraw);
- superView.BeginInit ();
- Assert.False (view.NeedsDraw);
- Assert.False (superView.NeedsDraw);
- superView.EndInit (); // Call Layout
- Assert.True (view.NeedsDraw);
- Assert.True (superView.NeedsDraw);
- superView.SetRelativeLayout (new (100, 100));
- Assert.True (view.NeedsDraw);
- Assert.True (superView.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_False_After_SetRelativeLayout_10x10 ()
- {
- View superView = new ()
- {
- Id = "superView",
- Width = Dim.Fill (),
- Height = Dim.Fill ()
- };
- Assert.False (superView.NeedsDraw);
- superView.Layout ();
- Assert.True (superView.NeedsDraw);
- superView.NeedsDraw = false;
- superView.SetRelativeLayout (new (10, 10));
- Assert.True (superView.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_True_After_LayoutSubViews ()
- {
- var view = new View { Width = 2, Height = 2, BorderStyle = LineStyle.Single };
- Assert.True (view.NeedsDraw);
- view.BeginInit ();
- Assert.True (view.NeedsDraw);
- view.EndInit ();
- Assert.True (view.NeedsDraw);
- view.SetRelativeLayout (new (100, 100));
- Assert.True (view.NeedsDraw);
- view.LayoutSubViews ();
- Assert.True (view.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_False_After_Draw ()
- {
- var view = new View { Driver = CreateFakeDriver (), Width = 2, Height = 2, BorderStyle = LineStyle.Single };
- Assert.True (view.NeedsDraw);
- view.BeginInit ();
- Assert.True (view.NeedsDraw);
- view.EndInit ();
- Assert.True (view.NeedsDraw);
- view.SetRelativeLayout (new (100, 100));
- Assert.True (view.NeedsDraw);
- view.LayoutSubViews ();
- Assert.True (view.NeedsDraw);
- view.Draw ();
- Assert.False (view.NeedsDraw);
- }
- [Fact]
- public void NeedsDrawRect_Is_Viewport_Relative ()
- {
- View superView = new ()
- {
- Id = "superView",
- Width = 10,
- Height = 10
- };
- Assert.Equal (new (0, 0, 10, 10), superView.Frame);
- Assert.Equal (new (0, 0, 10, 10), superView.Viewport);
- Assert.Equal (new (0, 0, 10, 10), superView.NeedsDrawRect);
- var view = new View
- {
- Id = "view"
- };
- view.Frame = new (0, 1, 2, 3);
- Assert.Equal (new (0, 1, 2, 3), view.Frame);
- Assert.Equal (new (0, 0, 2, 3), view.Viewport);
- Assert.Equal (new (0, 0, 2, 3), view.NeedsDrawRect);
- superView.Add (view);
- Assert.Equal (new (0, 0, 10, 10), superView.Frame);
- Assert.Equal (new (0, 0, 10, 10), superView.Viewport);
- Assert.Equal (new (0, 0, 10, 10), superView.NeedsDrawRect);
- Assert.Equal (new (0, 1, 2, 3), view.Frame);
- Assert.Equal (new (0, 0, 2, 3), view.Viewport);
- Assert.Equal (new (0, 0, 2, 3), view.NeedsDrawRect);
- view.Frame = new (3, 3, 5, 5);
- Assert.Equal (new (3, 3, 5, 5), view.Frame);
- Assert.Equal (new (0, 0, 5, 5), view.Viewport);
- Assert.Equal (new (0, 0, 5, 5), view.NeedsDrawRect);
- view.Frame = new (3, 3, 6, 6); // Grow right/bottom 1
- Assert.Equal (new (3, 3, 6, 6), view.Frame);
- Assert.Equal (new (0, 0, 6, 6), view.Viewport);
- Assert.Equal (new (0, 0, 6, 6), view.NeedsDrawRect);
- view.Frame = new (3, 3, 5, 5); // Shrink right/bottom 1
- Assert.Equal (new (3, 3, 5, 5), view.Frame);
- Assert.Equal (new (0, 0, 5, 5), view.Viewport);
- Assert.Equal (new (0, 0, 5, 5), view.NeedsDrawRect);
- view.SetContentSize (new (10, 10));
- Assert.Equal (new (3, 3, 5, 5), view.Frame);
- Assert.Equal (new (0, 0, 5, 5), view.Viewport);
- Assert.Equal (new (0, 0, 5, 5), view.NeedsDrawRect);
- view.Viewport = new (1, 1, 5, 5); // Scroll up/left 1
- Assert.Equal (new (3, 3, 5, 5), view.Frame);
- Assert.Equal (new (1, 1, 5, 5), view.Viewport);
- Assert.Equal (new (0, 0, 5, 5), view.NeedsDrawRect);
- view.Frame = new (3, 3, 6, 6); // Grow right/bottom 1
- Assert.Equal (new (3, 3, 6, 6), view.Frame);
- Assert.Equal (new (1, 1, 6, 6), view.Viewport);
- Assert.Equal (new (1, 1, 6, 6), view.NeedsDrawRect);
- view.Frame = new (3, 3, 5, 5);
- Assert.Equal (new (3, 3, 5, 5), view.Frame);
- Assert.Equal (new (1, 1, 5, 5), view.Viewport);
- Assert.Equal (new (1, 1, 5, 5), view.NeedsDrawRect);
- }
-
- [Fact]
- public void ClearNeedsDraw_ClearsOwnFlags ()
- {
- // Verify that ClearNeedsDraw properly clears the view's own flags
- IDriver driver = CreateFakeDriver (80, 25);
- driver.Clip = new Region (driver.Screen);
- var view = new View
- {
- X = 0,
- Y = 0,
- Width = 20,
- Height = 20,
- Driver = driver
- };
- view.BeginInit ();
- view.EndInit ();
- view.LayoutSubViews ();
- Assert.True (view.NeedsDraw);
- Assert.Equal (view.Viewport, view.NeedsDrawRect);
- view.Draw ();
- Assert.False (view.NeedsDraw);
- Assert.Equal (Rectangle.Empty, view.NeedsDrawRect);
- Assert.False (view.SubViewNeedsDraw);
- }
- [Fact]
- public void ClearNeedsDraw_ClearsAdornments ()
- {
- // Verify that ClearNeedsDraw clears adornment flags
- IDriver driver = CreateFakeDriver (80, 25);
- driver.Clip = new Region (driver.Screen);
- var view = new View
- {
- X = 0,
- Y = 0,
- Width = 20,
- Height = 20,
- Driver = driver
- };
- view.Border!.Thickness = new Thickness (1);
- view.Padding!.Thickness = new Thickness (1);
- view.BeginInit ();
- view.EndInit ();
- view.LayoutSubViews ();
- Assert.True (view.Border!.NeedsDraw);
- Assert.True (view.Padding!.NeedsDraw);
- view.Draw ();
- Assert.False (view.Border!.NeedsDraw);
- Assert.False (view.Padding!.NeedsDraw);
- }
- [Fact]
- public void ClearNeedsDraw_PropagatesDownToAllSubViews ()
- {
- // Verify that ClearNeedsDraw clears flags on all descendants
- IDriver driver = CreateFakeDriver (80, 25);
- driver.Clip = new Region (driver.Screen);
- var topView = new View
- {
- X = 0,
- Y = 0,
- Width = 100,
- Height = 100,
- Driver = driver
- };
- var middleView = new View { X = 10, Y = 10, Width = 50, Height = 50 };
- var bottomView = new View { X = 5, Y = 5, Width = 20, Height = 20 };
- topView.Add (middleView);
- middleView.Add (bottomView);
- topView.BeginInit ();
- topView.EndInit ();
- topView.LayoutSubViews ();
- Assert.True (topView.NeedsDraw);
- Assert.True (middleView.NeedsDraw);
- Assert.True (bottomView.NeedsDraw);
- topView.Draw ();
- Assert.False (topView.NeedsDraw);
- Assert.False (topView.SubViewNeedsDraw);
- Assert.False (middleView.NeedsDraw);
- Assert.False (middleView.SubViewNeedsDraw);
- Assert.False (bottomView.NeedsDraw);
- }
- #region NeedsDraw Tests
- [Fact]
- public void NeedsDraw_InitiallyFalse_WhenNotVisible ()
- {
- var view = new View { Visible = false };
- view.BeginInit ();
- view.EndInit ();
- Assert.False (view.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_TrueAfterSetNeedsDraw ()
- {
- var view = new View { X = 0, Y = 0, Width = 10, Height = 10 };
- view.BeginInit ();
- view.EndInit ();
- view.LayoutSubViews ();
- view.SetNeedsDraw ();
- Assert.True (view.NeedsDraw);
- }
- [Fact]
- public void NeedsDraw_ClearedAfterDraw ()
- {
- IDriver driver = CreateFakeDriver (80, 25);
- driver.Clip = new Region (driver.Screen);
- var view = new View
- {
- X = 0,
- Y = 0,
- Width = 10,
- Height = 10,
- Driver = driver
- };
- view.BeginInit ();
- view.EndInit ();
- view.LayoutSubViews ();
- view.SetNeedsDraw ();
- Assert.True (view.NeedsDraw);
- view.Draw ();
- Assert.False (view.NeedsDraw);
- }
- [Fact]
- public void SetNeedsDraw_WithRectangle_UpdatesNeedsDrawRect ()
- {
- var view = new View { Driver = CreateFakeDriver (), X = 0, Y = 0, Width = 20, Height = 20 };
- view.BeginInit ();
- view.EndInit ();
- view.LayoutSubViews ();
- // After layout, view will have NeedsDrawRect set to the viewport
- // We need to clear it first
- view.Draw ();
- Assert.False (view.NeedsDraw);
- Assert.Equal (Rectangle.Empty, view.NeedsDrawRect);
- var rect = new Rectangle (5, 5, 10, 10);
- view.SetNeedsDraw (rect);
- Assert.True (view.NeedsDraw);
- Assert.Equal (rect, view.NeedsDrawRect);
- }
- [Fact]
- public void SetNeedsDraw_MultipleRectangles_Expands ()
- {
- IDriver driver = CreateFakeDriver (80, 25);
- driver.Clip = new Region (driver.Screen);
- var view = new View { X = 0, Y = 0, Width = 30, Height = 30, Driver = driver };
- view.BeginInit ();
- view.EndInit ();
- view.LayoutSubViews ();
- // After layout, clear NeedsDraw
- view.Draw ();
- Assert.False (view.NeedsDraw);
- view.SetNeedsDraw (new Rectangle (5, 5, 10, 10));
- view.SetNeedsDraw (new Rectangle (15, 15, 10, 10));
- // Should expand to cover the entire viewport when we have overlapping regions
- // The current implementation expands to viewport size
- Rectangle expected = new Rectangle (0, 0, 30, 30);
- Assert.Equal (expected, view.NeedsDrawRect);
- }
- [Fact]
- public void SetNeedsDraw_NotVisible_DoesNotSet ()
- {
- var view = new View
- {
- X = 0,
- Y = 0,
- Width = 10,
- Height = 10,
- Visible = false
- };
- view.BeginInit ();
- view.EndInit ();
- view.SetNeedsDraw ();
- Assert.False (view.NeedsDraw);
- }
- [Fact]
- public void SetNeedsDraw_PropagatesToSuperView ()
- {
- var parent = new View { X = 0, Y = 0, Width = 50, Height = 50 };
- var child = new View { X = 10, Y = 10, Width = 20, Height = 20 };
- parent.Add (child);
- parent.BeginInit ();
- parent.EndInit ();
- parent.LayoutSubViews ();
- child.SetNeedsDraw ();
- Assert.True (child.NeedsDraw);
- Assert.True (parent.SubViewNeedsDraw);
- }
- [Fact]
- public void SetNeedsDraw_SetsAdornmentsNeedsDraw ()
- {
- var view = new View { X = 0, Y = 0, Width = 20, Height = 20 };
- view.Border!.Thickness = new Thickness (1);
- view.Padding!.Thickness = new Thickness (1);
- view.BeginInit ();
- view.EndInit ();
- view.LayoutSubViews ();
- view.SetNeedsDraw ();
- Assert.True (view.Border!.NeedsDraw);
- Assert.True (view.Padding!.NeedsDraw);
- }
- [Fact (Skip = "This is a real bug discovered in PR #4431 that needs to be fixed")]
- public void IndividualViewDraw_DoesNotClearSuperViewSubViewNeedsDraw ()
- {
- // This test validates that individual view Draw() calls should NOT clear the superview's
- // SubViewNeedsDraw flag when sibling subviews still need drawing.
- //
- // This is the core behavior that enables the fix in the static Draw method.
- IDriver driver = CreateFakeDriver ();
- driver.Clip = new (driver.Screen);
- View superview = new ()
- {
- X = 0,
- Y = 0,
- Width = 50,
- Height = 50,
- Driver = driver,
- Id = "SuperView"
- };
- View subview1 = new () { X = 0, Y = 0, Width = 10, Height = 10, Id = "SubView1" };
- View subview2 = new () { X = 0, Y = 10, Width = 10, Height = 10, Id = "SubView2" };
- superview.Add (subview1, subview2);
- superview.BeginInit ();
- superview.EndInit ();
- superview.LayoutSubViews ();
- Assert.True (superview.SubViewNeedsDraw);
- Assert.True (subview1.NeedsDraw);
- Assert.True (subview2.NeedsDraw);
- // Draw only subview1 (NOT using the static Draw method)
- subview1.Draw ();
- // SubView1 should be cleared
- Assert.False (subview1.NeedsDraw);
- // SubView2 still needs drawing
- Assert.True (subview2.NeedsDraw);
- // THE KEY ASSERTION: SuperView's SubViewNeedsDraw should STILL be true
- // because subview2 still needs drawing
- //
- // This behavior is REQUIRED for the static Draw fix to work properly.
- // ClearNeedsDraw() does NOT clear SuperView.SubViewNeedsDraw anymore.
- Assert.True (superview.SubViewNeedsDraw,
- "SuperView's SubViewNeedsDraw must remain true when subview2 still needs drawing");
- // Now draw subview2
- subview2.Draw ();
- Assert.False (subview2.NeedsDraw);
- // SuperView's SubViewNeedsDraw should STILL be true because only the superview
- // itself (or the static Draw method on all subviews) should clear it
- Assert.True (superview.SubViewNeedsDraw,
- "SuperView's SubViewNeedsDraw should only be cleared by superview.Draw() or static Draw() on all subviews");
- }
- #endregion
- #region SubViewNeedsDraw Tests
- [Fact]
- public void SubViewNeedsDraw_InitiallyFalse ()
- {
- IDriver driver = CreateFakeDriver (80, 25);
- driver.Clip = new Region (driver.Screen);
- var view = new View { Width = 10, Height = 10, Driver = driver };
- view.BeginInit ();
- view.EndInit ();
- view.Draw (); // Draw once to clear initial NeedsDraw
- Assert.False (view.SubViewNeedsDraw);
- }
- [Fact]
- public void SetSubViewNeedsDraw_PropagatesUp ()
- {
- var grandparent = new View { X = 0, Y = 0, Width = 100, Height = 100 };
- var parent = new View { X = 10, Y = 10, Width = 50, Height = 50 };
- var child = new View { X = 5, Y = 5, Width = 20, Height = 20 };
- grandparent.Add (parent);
- parent.Add (child);
- grandparent.BeginInit ();
- grandparent.EndInit ();
- grandparent.LayoutSubViews ();
- child.SetSubViewNeedsDraw ();
- Assert.True (child.SubViewNeedsDraw);
- Assert.True (parent.SubViewNeedsDraw);
- Assert.True (grandparent.SubViewNeedsDraw);
- }
- [Fact]
- public void SubViewNeedsDraw_ClearedAfterDraw ()
- {
- IDriver driver = CreateFakeDriver (80, 25);
- driver.Clip = new Region (driver.Screen);
- var parent = new View
- {
- X = 0,
- Y = 0,
- Width = 50,
- Height = 50,
- Driver = driver
- };
- var child = new View { X = 10, Y = 10, Width = 20, Height = 20 };
- parent.Add (child);
- parent.BeginInit ();
- parent.EndInit ();
- parent.LayoutSubViews ();
- child.SetNeedsDraw ();
- Assert.True (parent.SubViewNeedsDraw);
- parent.Draw ();
- Assert.False (parent.SubViewNeedsDraw);
- Assert.False (child.SubViewNeedsDraw);
- }
- #endregion
- }
|