From 44463681f9633555bf59899ac00650cb616bc790 Mon Sep 17 00:00:00 2001 From: LD-Reborn Date: Sun, 16 Nov 2025 01:28:01 +0100 Subject: [PATCH] Replaced RetryFact with xRetry's RetryFact --- .../AssetsPageTests.cs | 1 + .../Berufsschule_HAM.E2ETests.csproj | 1 + .../GroupsPageTests.cs | 1 + .../HomePageTests.cs | 1 + .../InventoryPageTests.cs | 1 + .../LocationsPageTests.cs | 1 + .../RetryFactAttribute.cs | 79 ------------------- .../UsersPageTests.cs | 1 + 8 files changed, 7 insertions(+), 79 deletions(-) delete mode 100644 tests/Berufsschule_HAM.E2ETests/RetryFactAttribute.cs diff --git a/tests/Berufsschule_HAM.E2ETests/AssetsPageTests.cs b/tests/Berufsschule_HAM.E2ETests/AssetsPageTests.cs index 7f45e3c..82f333c 100644 --- a/tests/Berufsschule_HAM.E2ETests/AssetsPageTests.cs +++ b/tests/Berufsschule_HAM.E2ETests/AssetsPageTests.cs @@ -3,6 +3,7 @@ using OpenQA.Selenium; using OpenQA.Selenium.Chrome; using OpenQA.Selenium.Support.Extensions; using System.Diagnostics; +using xRetry; namespace Berufsschule_HAM.E2ETests; diff --git a/tests/Berufsschule_HAM.E2ETests/Berufsschule_HAM.E2ETests.csproj b/tests/Berufsschule_HAM.E2ETests/Berufsschule_HAM.E2ETests.csproj index c3d5cbc..1c0e52f 100644 --- a/tests/Berufsschule_HAM.E2ETests/Berufsschule_HAM.E2ETests.csproj +++ b/tests/Berufsschule_HAM.E2ETests/Berufsschule_HAM.E2ETests.csproj @@ -18,6 +18,7 @@ runtime; build; native; contentfiles; analyzers; buildtransitive all + diff --git a/tests/Berufsschule_HAM.E2ETests/GroupsPageTests.cs b/tests/Berufsschule_HAM.E2ETests/GroupsPageTests.cs index 78d9e9b..043a7bf 100644 --- a/tests/Berufsschule_HAM.E2ETests/GroupsPageTests.cs +++ b/tests/Berufsschule_HAM.E2ETests/GroupsPageTests.cs @@ -2,6 +2,7 @@ using OpenQA.Selenium; using OpenQA.Selenium.Chrome; using System.Diagnostics; +using xRetry; namespace Berufsschule_HAM.E2ETests; diff --git a/tests/Berufsschule_HAM.E2ETests/HomePageTests.cs b/tests/Berufsschule_HAM.E2ETests/HomePageTests.cs index aed83af..a75e4db 100644 --- a/tests/Berufsschule_HAM.E2ETests/HomePageTests.cs +++ b/tests/Berufsschule_HAM.E2ETests/HomePageTests.cs @@ -3,6 +3,7 @@ using OpenQA.Selenium; using OpenQA.Selenium.Chrome; using SeleniumExtras.WaitHelpers; using System.Diagnostics; +using xRetry; namespace Berufsschule_HAM.E2ETests; diff --git a/tests/Berufsschule_HAM.E2ETests/InventoryPageTests.cs b/tests/Berufsschule_HAM.E2ETests/InventoryPageTests.cs index 3158f14..8ec9191 100644 --- a/tests/Berufsschule_HAM.E2ETests/InventoryPageTests.cs +++ b/tests/Berufsschule_HAM.E2ETests/InventoryPageTests.cs @@ -3,6 +3,7 @@ using OpenQA.Selenium; using OpenQA.Selenium.Chrome; using SeleniumExtras.WaitHelpers; using System.Diagnostics; +using xRetry; namespace Berufsschule_HAM.E2ETests; diff --git a/tests/Berufsschule_HAM.E2ETests/LocationsPageTests.cs b/tests/Berufsschule_HAM.E2ETests/LocationsPageTests.cs index cca68f3..0c43421 100644 --- a/tests/Berufsschule_HAM.E2ETests/LocationsPageTests.cs +++ b/tests/Berufsschule_HAM.E2ETests/LocationsPageTests.cs @@ -2,6 +2,7 @@ using OpenQA.Selenium; using OpenQA.Selenium.Chrome; using System.Diagnostics; +using xRetry; namespace Berufsschule_HAM.E2ETests; diff --git a/tests/Berufsschule_HAM.E2ETests/RetryFactAttribute.cs b/tests/Berufsschule_HAM.E2ETests/RetryFactAttribute.cs deleted file mode 100644 index 8799e40..0000000 --- a/tests/Berufsschule_HAM.E2ETests/RetryFactAttribute.cs +++ /dev/null @@ -1,79 +0,0 @@ -using System; -using Xunit; -using Xunit.Abstractions; -using Xunit.Sdk; - -namespace Berufsschule_HAM.E2ETests; - -public class RetryFactAttribute(int maxRetries = 5) : FactAttribute -{ - public int MaxRetries { get; } = maxRetries; -} - -public class RetryTestCase : XunitTestCase -{ - private int _maxRetries; - - [Obsolete("Called by the de-serializer", true)] - public RetryTestCase() { } - - public RetryTestCase( - IMessageSink diagnosticMessageSink, - TestMethodDisplay defaultMethodDisplay, - TestMethodDisplayOptions defaultMethodDisplayOptions, - ITestMethod testMethod, - int maxRetries) - : base(diagnosticMessageSink, defaultMethodDisplay, defaultMethodDisplayOptions, testMethod) - { - _maxRetries = maxRetries; - } - - public override async Task RunAsync( - IMessageSink diagnosticMessageSink, - IMessageBus messageBus, - object[] constructorArguments, - ExceptionAggregator aggregator, - CancellationTokenSource cancellationTokenSource) - { - var summary = new RunSummary(); - - for (int i = 1; i <= _maxRetries; i++) - { - try - { - var result = await base.RunAsync( - diagnosticMessageSink, messageBus, constructorArguments, aggregator, cancellationTokenSource); - - summary.Aggregate(result); - - if (result.Failed == 0) - break; // success, stop retrying - } catch (Exception) {} - } - - return summary; - } -} - -public class RetryFactDiscoverer : IXunitTestCaseDiscoverer -{ - private readonly IMessageSink _diagnosticMessageSink; - - public RetryFactDiscoverer(IMessageSink diagnosticMessageSink) - { - _diagnosticMessageSink = diagnosticMessageSink; - } - - public IEnumerable Discover(ITestFrameworkDiscoveryOptions discoveryOptions, - ITestMethod testMethod, - IAttributeInfo factAttribute) - { - var maxRetries = factAttribute.GetNamedArgument("MaxRetries"); - yield return new RetryTestCase( - _diagnosticMessageSink, - discoveryOptions.MethodDisplayOrDefault(), - discoveryOptions.MethodDisplayOptionsOrDefault(), - testMethod, - maxRetries); - } -} \ No newline at end of file diff --git a/tests/Berufsschule_HAM.E2ETests/UsersPageTests.cs b/tests/Berufsschule_HAM.E2ETests/UsersPageTests.cs index 082c16f..bfeda90 100644 --- a/tests/Berufsschule_HAM.E2ETests/UsersPageTests.cs +++ b/tests/Berufsschule_HAM.E2ETests/UsersPageTests.cs @@ -3,6 +3,7 @@ using OpenQA.Selenium; using OpenQA.Selenium.Chrome; using OpenQA.Selenium.Support.Extensions; using System.Diagnostics; +using xRetry; namespace Berufsschule_HAM.E2ETests;