Fix multiple data-driven tests not all being picked up in a [DependsOn] (#2436)

* Fix multiple data-driven tests not all being picked up in a [DependsOn]

* Engine test

Author: thomhurst

TUnit.Core/TestContext.cs

@@ -1,4 +1,5 @@
 using System.Diagnostics;
+using System.Diagnostics.CodeAnalysis;
 
 namespace TUnit.Core;
 
@@ -72,6 +73,9 @@ internal TestContext(IServiceProvider serviceProvider, TestDetails testDetails,
     /// </summary>
     public TestDetails TestDetails { get; }
 
+    [field: AllowNull, MaybeNull]
+    public TestContext[] Dependencies => field ??= InternalDiscoveredTest.Dependencies.Select(x => x.Test.TestContext).ToArray();
+
     /// <summary>
     /// Gets or sets the current retry attempt for the test.
     /// </summary>

TUnit.Engine.Tests/DependencyCountTests.cs

@@ -0,0 +1,20 @@
+using Shouldly;
+using TUnit.Engine.Tests.Enums;
+
+namespace TUnit.Engine.Tests;
+
+public class DependencyCountTests(TestMode testMode) : InvokableTestBase(testMode)
+{
+    public async Task Test()
+    {
+        await RunTestsWithFilter(
+            "/*/*/DependencyCountTests/*",
+            [
+                result => result.ResultSummary.Outcome.ShouldBe("Completed"),
+                result => result.ResultSummary.Counters.Total.ShouldBe(8),
+                result => result.ResultSummary.Counters.Passed.ShouldBe(8),
+                result => result.ResultSummary.Counters.Failed.ShouldBe(0),
+                result => result.ResultSummary.Counters.NotExecuted.ShouldBe(0),
+            ]);
+    }
+}

TUnit.Engine/Services/DependencyCollector.cs

@@ -122,7 +122,9 @@ public bool Equals(TestDetails? x, TestDetails? y)
                 return false;
             }
 
-            return x.IsSameTest(y);
+            return x.IsSameTest(y) && 
+                   x.TestMethodParameterTypes.SequenceEqual(y.TestMethodParameterTypes) &&
+                   x.TestMethodArguments.SequenceEqual(y.TestMethodArguments);
         }
 
         public int GetHashCode(TestDetails obj)

TUnit.PublicAPI/Tests.Core_Library_Has_No_API_Changes.DotNet2_0.verified.txt

@@ -790,6 +790,7 @@ namespace TUnit.Core
         public System.Collections.Generic.List<TUnit.Core.ArgumentDisplayFormatter> ArgumentDisplayFormatters { get; }
         public System.Threading.CancellationToken CancellationToken { get; }
         public int CurrentRetryAttempt { get; }
+        public TUnit.Core.TestContext[] Dependencies { get; }
         public TUnit.Core.TestContextEvents Events { get; }
         public bool IsRegistered { get; }
         public System.Collections.Generic.Dictionary<string, object?> ObjectBag { get; }

TUnit.PublicAPI/Tests.Core_Library_Has_No_API_Changes.DotNet8_0.verified.txt

@@ -836,6 +836,7 @@ namespace TUnit.Core
         public System.Collections.Generic.List<TUnit.Core.ArgumentDisplayFormatter> ArgumentDisplayFormatters { get; }
         public System.Threading.CancellationToken CancellationToken { get; }
         public int CurrentRetryAttempt { get; }
+        public TUnit.Core.TestContext[] Dependencies { get; }
         public TUnit.Core.TestContextEvents Events { get; }
         public bool IsRegistered { get; }
         public System.Collections.Generic.Dictionary<string, object?> ObjectBag { get; }

TUnit.PublicAPI/Tests.Core_Library_Has_No_API_Changes.DotNet9_0.verified.txt

@@ -836,6 +836,7 @@ namespace TUnit.Core
         public System.Collections.Generic.List<TUnit.Core.ArgumentDisplayFormatter> ArgumentDisplayFormatters { get; }
         public System.Threading.CancellationToken CancellationToken { get; }
         public int CurrentRetryAttempt { get; }
+        public TUnit.Core.TestContext[] Dependencies { get; }
         public TUnit.Core.TestContextEvents Events { get; }
         public bool IsRegistered { get; }
         public System.Collections.Generic.Dictionary<string, object?> ObjectBag { get; }

TUnit.TestProject/DependencyCountTests.cs

@@ -0,0 +1,43 @@
+namespace TUnit.TestProject;
+
+public class DependencyCountTests
+{
+    [Test]
+    [Arguments(1)]
+    [Arguments(2)]
+    [Arguments(3)]
+    public Task Test1(int value)
+    {
+        return Task.CompletedTask;
+    }
+
+    [Test, DependsOn(nameof(Test1))]
+    public async Task Test2()
+    {
+        await Assert.That(TestContext.Current!.Dependencies).HasCount().EqualTo(3);
+    }
+    
+    [Test]
+    public Task Test3()
+    {
+        return Task.CompletedTask;
+    }
+    
+    [Test, DependsOn(nameof(Test3))]
+    public async Task Test4()
+    {
+        await Assert.That(TestContext.Current!.Dependencies).HasCount().EqualTo(1);
+    }
+    
+    [Test, DependsOn(nameof(Test1))]
+    public Task Test5()
+    {
+        return Task.CompletedTask;
+    }
+    
+    [Test, DependsOn(nameof(Test5))]
+    public async Task Test6()
+    {
+        await Assert.That(TestContext.Current!.Dependencies).HasCount().EqualTo(4);
+    }
+}