mirror of
https://github.com/Cleanuparr/Cleanuparr.git
synced 2026-10-28 10:48:49 -04:00
fixed cycle searched count counting queued items
This commit is contained in:
1 parent
8ba3f0eaa0
commit
d5ea35445f
2 files changed
+43
-4
No files matched your search
@@ -86,18 +86,55 @@ public class SearchStatsControllerTests : IDisposable
|
||||
var radarr = SeekerTestDataFactory.AddRadarrInstance(_dataContext);
|
||||
Guid currentCycleId = Guid.NewGuid();
|
||||
AddInstanceConfig(radarr.Id, currentCycleId);
|
||||
AddCycle(currentCycleId, radarr.Id, eligibleNow: 10, remainingUnits: 4, startedAt: Now.AddDays(-2));
|
||||
AddCycle(currentCycleId, radarr.Id, eligibleNow: 10, remainingUnits: 4, startedAt: Now.AddDays(-2), searchedUnits: 3);
|
||||
|
||||
SearchStatsSummaryResponse summary = Summary(await _controller.GetSummary());
|
||||
|
||||
InstanceSearchStat stat = summary.PerInstanceStats.ShouldHaveSingleItem();
|
||||
stat.CycleItemsTotal.ShouldBe(10);
|
||||
stat.CycleItemsRemaining.ShouldBe(4);
|
||||
stat.CycleItemsSearched.ShouldBe(6);
|
||||
stat.CycleItemsSearched.ShouldBe(3);
|
||||
stat.CycleStartedAt.ShouldBe(Now.AddDays(-2));
|
||||
stat.CycleCompletedAt.ShouldBeNull();
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task GetSummary_PerInstanceStats_CycleItemsSearched_ExcludesQueuedItems()
|
||||
{
|
||||
var radarr = SeekerTestDataFactory.AddRadarrInstance(_dataContext);
|
||||
Guid currentCycleId = Guid.NewGuid();
|
||||
AddInstanceConfig(radarr.Id, currentCycleId);
|
||||
|
||||
// 5 eligible, 2 remaining, 1 actually searched: 2 are sitting in the download queue
|
||||
AddCycle(currentCycleId, radarr.Id, eligibleNow: 5, remainingUnits: 2, startedAt: Now.AddDays(-1), searchedUnits: 1);
|
||||
|
||||
SearchStatsSummaryResponse summary = Summary(await _controller.GetSummary());
|
||||
|
||||
InstanceSearchStat stat = summary.PerInstanceStats.ShouldHaveSingleItem();
|
||||
stat.CycleItemsSearched.ShouldBe(1);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task GetSummary_PerInstanceStats_DryRunOn_NoDryCycle_ValuesAreZeroOrNull()
|
||||
{
|
||||
var radarr = SeekerTestDataFactory.AddRadarrInstance(_dataContext);
|
||||
(await _dataContext.GeneralConfigs.FirstAsync()).DryRun = true;
|
||||
await _dataContext.SaveChangesAsync();
|
||||
|
||||
Guid currentCycleId = Guid.NewGuid();
|
||||
AddInstanceConfig(radarr.Id, currentCycleId);
|
||||
AddCycle(currentCycleId, radarr.Id, eligibleNow: 10, remainingUnits: 4, startedAt: Now.AddDays(-2), isDryRun: false);
|
||||
|
||||
SearchStatsSummaryResponse summary = Summary(await _controller.GetSummary());
|
||||
|
||||
InstanceSearchStat stat = summary.PerInstanceStats.ShouldHaveSingleItem();
|
||||
stat.CycleItemsSearched.ShouldBe(0);
|
||||
stat.CycleItemsTotal.ShouldBe(0);
|
||||
stat.CycleItemsRemaining.ShouldBe(0);
|
||||
stat.CycleStartedAt.ShouldBeNull();
|
||||
stat.CycleCompletedAt.ShouldBeNull();
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task GetSummary_PerInstanceStats_LastCycle_IsTheNewestCompletedOtherCycle()
|
||||
{
|
||||
@@ -176,7 +213,8 @@ public class SearchStatsControllerTests : IDisposable
|
||||
int remainingUnits,
|
||||
DateTimeOffset startedAt,
|
||||
DateTimeOffset? completedAt = null,
|
||||
bool isDryRun = false)
|
||||
bool isDryRun = false,
|
||||
int searchedUnits = 0)
|
||||
{
|
||||
_eventsContext.SeekerCycles.Add(new SeekerCycle
|
||||
{
|
||||
@@ -188,6 +226,7 @@ public class SearchStatsControllerTests : IDisposable
|
||||
EligibleAtStart = eligibleNow,
|
||||
EligibleNow = eligibleNow,
|
||||
RemainingUnits = remainingUnits,
|
||||
SearchedUnits = searchedUnits,
|
||||
IsDryRun = isDryRun,
|
||||
});
|
||||
_eventsContext.SaveChanges();
|
||||
|
||||
@@ -104,7 +104,7 @@ public sealed class SearchStatsController : ControllerBase
|
||||
LastSearchedAt = history?.LastSearchedAt,
|
||||
LastProcessedAt = ic.LastProcessedAt,
|
||||
CurrentCycleId = current?.Id ?? ic.CurrentCycleId,
|
||||
CycleItemsSearched = current is null ? 0 : Math.Max(0, current.EligibleNow - current.RemainingUnits),
|
||||
CycleItemsSearched = current?.SearchedUnits ?? 0,
|
||||
CycleItemsTotal = current?.EligibleNow ?? 0,
|
||||
CycleItemsRemaining = current?.RemainingUnits ?? 0,
|
||||
CycleStartedAt = current?.StartedAt,
|
||||
|
||||
Reference in new issue
Block a user