Refactor JobRepository to remove wait timeouts. - #94

Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-dotnet@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/

Test method 'HeartbeatMonitorDelayWaitAsync_WhenCallerCancels_PropagatesCancellation' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_ZeroTimeout_CancelledToken_WhenUnset_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_ZeroTimeout_CancelledToken_WhenSet_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'HeartbeatMonitorDelayWaitAsync_WhenCallerCancelsDuringSleep_PropagatesCancellation' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'IdempotencyMonitorDelayWaitAsync_WhenDisposed_ReturnsWithoutSleeping' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'IdempotencyMonitorDelayWaitAsync_CompletesNormallyWhenWatchedJobsExist' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_DefaultOverload_WhenAlreadySet_ReturnsTrue' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_CancelledToken_WhenAlreadySet_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_CancelledToken_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_CancelledToken_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'HeartbeatMonitorDelayWaitAsync_WhenCallerCancels_PropagatesCancellation' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_ZeroTimeout_CancelledToken_WhenSet_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'GetNextAsync_WhenCancelledWhileWaiting_ThrowsOperationCanceledException' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'HeartbeatMonitorDelayWaitAsync_WhenCallerCancelsDuringSleep_PropagatesCancellation' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'IdempotencyMonitorDelayWaitAsync_WhenDisposed_ReturnsWithoutSleeping' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_DefaultOverload_WhenAlreadySet_ReturnsTrue' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'IdempotencyMonitorDelayWaitAsync_CompletesNormallyWhenWatchedJobsExist' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_CancelledToken_WhenAlreadySet_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)

Test method 'Test_WaitAsync_CancelledToken_Throws' has a Timeout but does not reference TestContext.Current.CancellationToken. Tests with a Timeout should reference TestContext.Current.CancellationToken so they can terminate promptly when the timeout is exceeded. (https://xunit.net/xunit.analyzers/rules/xUnit1069)