Files
Terminal.Gui/UnitTests/Views/ViewDisposalTest.cs
usr c7942ae3bb Fix issue where the TestViewsDisposeCorrectly was not doing what it was supposed to do (#2964)
* add Disposal Test and fix an ssue where the CopyClipboard test was failing

* Update ViewDisposalTest.cs

* Update ViewDisposalTest.cs: Some Formatting, and adding code comments.

* Fix ViewDisposalTests (Wasn't working the way it was supposed to)

* update test

* update test

* update test

* try to fix as many conflicts as possible

* make test output prettier

* fix formatting

* Fix Subviews not being empty after disposing on all views.

* The fail cause was Application.Top not being disposed.

* Fix others containers that weren't being removed.

* Revert "The fail cause was Application.Top not being disposed."

This reverts commit 0c2183ed9e.

* Application.Top isn't null and need disposing.

* Fixes #2985. Application.RunState must be responsible for dispose the Toplevel property.

* Change the unit test with ans without Application.Shutdown method.

* Update ViewDisposeTests to actually check wether ALL views have been disposed (not just container)

* small additional check just to be safe

* Update ViewDisposalTest.cs: Formatting

* Update ViewDisposalTest.cs: Minor change to re-trigger Action

TestVKPacket is acting up again. Maybe the test is running async and is receiving scan codes from other instances?

---------

Co-authored-by: John Züchler <john.zuechler@eks-intec.de>
Co-authored-by: BDisp <bd.bdisp@gmail.com>
Co-authored-by: Tig <tig@users.noreply.github.com>
2023-11-15 09:22:22 -07:00

141 lines
4.5 KiB
C#

using SixLabors.ImageSharp.Processing.Processors.Quantization;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Reflection;
using Terminal.Gui;
using Xunit;
using Xunit.Abstractions;
namespace UnitTests.ViewsTests {
public class ViewDisposalTest {
#nullable enable
Dictionary<Type, object? []?> special_params = new Dictionary<Type, object? []?> ();
#nullable restore
readonly ITestOutputHelper output;
public ViewDisposalTest (ITestOutputHelper output)
{
{
this.output = output;
}
}
[Theory]
[InlineData (true)]
[InlineData (false)]
public void TestViewsDisposeCorrectly (bool callShutdown)
{
var refs = DoTest (callShutdown);
//var reference = refs [0];
for (var i = 0; i < 10 && refs [0].IsAlive; i++) {
GC.Collect ();
GC.WaitForPendingFinalizers ();
}
foreach (var reference in refs) {
if (reference.IsAlive) {
#if DEBUG_IDISPOSABLE
Assert.True (((View)reference.Target).WasDisposed);
#endif
string alive = ""; // Instead of just checking the subviews of the container, we now iterate through a list
foreach (var r in refs) { // of Weakreferences Referencing every View that was tested. This makes more sense because
if (r.IsAlive) { // View.Dispose removes all of its subviews, wich is why View.Subviews is always empty
if (r == refs [0]) { // after View.Dispose has run. Luckily I didnt discover any more bugs or this wouldv'e
alive += "\n View (Container)"; // been a little bit annoying to find an answer for. Thanks to BDisp for listening to
} // me and giving his best to help me fix this thing. If you take a look at the commit log
alive += ",\n--"; // you will find that he did most of the work. -a-usr
alive += r.Target.GetType ().Name;
} // NOTE: DELETE BEFORE NEXT COMMIT
}
Assert.Fail ($"Some Views didnt get Garbage Collected: {alive}");
}
}
if (!callShutdown) {
Application.Shutdown ();
}
}
void getSpecialParams ()
{
special_params.Clear ();
//special_params.Add (typeof (LineView), new object [] { Orientation.Horizontal });
}
List<WeakReference> DoTest (bool callShutdown)
{
var driver = new FakeDriver ();
Application.Init (driver, new FakeMainLoop (driver));
getSpecialParams ();
View Container = new View ();
List<WeakReference> refs = new List<WeakReference> { new WeakReference (Container, true) };
Container.Add (new View ());
Toplevel top = new ();
var state = Application.Begin (top);
var views = GetViews ();
foreach (var view in views) {
View instance;
//Create instance of view and add to container
if (special_params.ContainsKey (view)) {
instance = (View)Activator.CreateInstance (view, special_params [view]);
} else {
instance = (View)Activator.CreateInstance (view);
}
Assert.NotNull (instance);
Container.Add (instance);
refs.Add (new WeakReference (instance, true));
output.WriteLine ($"Added instance of {view}!");
}
top.Add (Container);
// make sure the application is doing to the views whatever its supposed to do to the views
for (var i = 0; i < 100; i++) {
Application.Refresh ();
}
top.Remove (Container);
Application.End (state);
Assert.True (refs.All (r => r.IsAlive));
#if DEBUG_IDISPOSABLE
Assert.True (top.WasDisposed);
Assert.False (Container.WasDisposed);
#endif
Assert.Null (Application.Top);
Container.Dispose ();
#if DEBUG_IDISPOSABLE
Assert.True (Container.WasDisposed);
#endif
if (callShutdown) {
Application.Shutdown ();
}
return refs;
}
/// <summary>
/// Get all types derived from <see cref="View"/> using reflection
/// </summary>
/// <returns></returns>
List<Type> GetViews ()
{
List<Type> valid = new ();
// Filter all types that can be instantiated, are public, arent generic, aren't the view type itself, but derive from view
foreach (var type in Assembly.GetAssembly (typeof (View)).GetTypes ().Where (T => { //body of anonymous check function
return ((!T.IsAbstract) && T.IsPublic && T.IsClass && T.IsAssignableTo (typeof (View)) && !T.IsGenericType && !(T == typeof (View)));
})) //end of body of anonymous check function
{ //body of the foreach loop
output.WriteLine ($"Found Type {type.Name}");
Assert.DoesNotContain (type, valid);
Assert.True (type.IsAssignableTo (typeof (IDisposable)));// Just to be safe
valid.Add (type);
output.WriteLine (" -Added!");
} //end body of foreach loop
return valid;
}
}
}