Browse Source

Fix Disposing unit tests that sometimes throws because some instances aren't cleared on others unit tests classes. (#3142)

BDisp 1 year ago
parent
commit
d69de86ba4
1 changed files with 9 additions and 0 deletions
  1. 9 0
      UnitTests/Input/ResponderTests.cs

+ 9 - 0
UnitTests/Input/ResponderTests.cs

@@ -139,6 +139,9 @@ public class ResponderTests {
 	[Fact]
 	[Fact]
 	public void Responder_Not_Notifying_Dispose ()
 	public void Responder_Not_Notifying_Dispose ()
 	{
 	{
+		// Only clear before because need to test after assert
+		Responder.Instances.Clear ();
+
 		var container1 = new View () { Id = "Container1" };
 		var container1 = new View () { Id = "Container1" };
 
 
 		var view = new View () { Id = "View" };
 		var view = new View () { Id = "View" };
@@ -175,6 +178,9 @@ public class ResponderTests {
 	[Fact]
 	[Fact]
 	public void Disposing_Event_Notify_All_Subscribers_On_The_Second_Container ()
 	public void Disposing_Event_Notify_All_Subscribers_On_The_Second_Container ()
 	{
 	{
+		// Only clear before because need to test after assert
+		Responder.Instances.Clear ();
+
 		var container1 = new View () { Id = "Container1" };
 		var container1 = new View () { Id = "Container1" };
 
 
 		var view = new View () { Id = "View" };
 		var view = new View () { Id = "View" };
@@ -212,6 +218,9 @@ public class ResponderTests {
 	[Fact]
 	[Fact]
 	public void Disposing_Event_Notify_All_Subscribers_On_The_First_Container ()
 	public void Disposing_Event_Notify_All_Subscribers_On_The_First_Container ()
 	{
 	{
+		// Only clear before because need to test after assert
+		Responder.Instances.Clear ();
+
 		var container1 = new View () { Id = "Container1" };
 		var container1 = new View () { Id = "Container1" };
 		var count = 0;
 		var count = 0;