From beb890d4fe699cf718748884c1e642e28239c1fc Mon Sep 17 00:00:00 2001
From: Youssef Victor <youssefvictor00@gmail.com>
Date: Fri, 6 Dec 2024 20:59:08 +0100
Subject: [PATCH] Use `ValidationResult.ValidTask` instead of
 `Task.FromResult(ValidationResult.Valid())` (#4262)

---
 .../CrashDumpEnvironmentVariableProvider.cs                     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpEnvironmentVariableProvider.cs b/src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpEnvironmentVariableProvider.cs
index ee85390bf8..ae351902ac 100644
--- a/src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpEnvironmentVariableProvider.cs
+++ b/src/Platform/Microsoft.Testing.Extensions.CrashDump/CrashDumpEnvironmentVariableProvider.cs
@@ -198,7 +198,7 @@ public Task<ValidationResult> ValidateTestHostEnvironmentVariablesAsync(IReadOnl
             }
         }
 
-        return Task.FromResult(errors.Length > 0 ? ValidationResult.Invalid(errors.ToString()) : ValidationResult.Valid());
+        return errors.Length > 0 ? Task.FromResult(ValidationResult.Invalid(errors.ToString())) : ValidationResult.ValidTask;
 
         static void AddError(StringBuilder errors, string variableName, string? expectedValue, string? actualValue)
         {