@@ -215,23 +210,18 @@
@* Keyset paging — the Task 4 query response carries a (CreatedAtUtc, Id)
cursor rather than page numbers, so we keep a stack of cursors to step
- backwards and the response's NextAfter* cursor to step forwards. *@
-
-
- @* No "of N" total: keyset paging has no cheap total-count, so
- the label is intentionally page-number-only. Do not "fix"
- this by adding a total — that would require a COUNT(*). *@
- Page @(_cursorStack.Count + 1) · @_siteCalls.Count rows
-
-
-
-
-
-
+ backwards and the response's NextAfter* cursor to step forwards. All
+ cursor logic stays in this code-behind; KeysetPager is presentational.
+ No "of N" total: keyset paging has no cheap total-count, so the label is
+ intentionally page-number-only. Do not "fix" this by adding a total —
+ that would require a COUNT(*). *@
+
}
@* ── Trends (K14: collapsible KPI-history charts) ──
@@ -292,7 +282,7 @@
@if (_detailSiteCall != null)
{
var d = _detailSiteCall;
-
diff --git a/tests/ZB.MOM.WW.ScadaBridge.CentralUI.Tests/Pages/SiteCallsReportPageTests.cs b/tests/ZB.MOM.WW.ScadaBridge.CentralUI.Tests/Pages/SiteCallsReportPageTests.cs
index 13b0b2ea..bd35a2fa 100644
--- a/tests/ZB.MOM.WW.ScadaBridge.CentralUI.Tests/Pages/SiteCallsReportPageTests.cs
+++ b/tests/ZB.MOM.WW.ScadaBridge.CentralUI.Tests/Pages/SiteCallsReportPageTests.cs
@@ -307,9 +307,9 @@ public class SiteCallsReportPageTests : BunitContext
var cut = Render();
cut.WaitForState(() => cut.Markup.Contains("ERP.GetOrder"));
- var next = cut.Find("[data-test='site-calls-next']");
+ var next = cut.Find("[data-test='keyset-next']");
Assert.True(next.HasAttribute("disabled"));
- var prev = cut.Find("[data-test='site-calls-prev']");
+ var prev = cut.Find("[data-test='keyset-prev']");
Assert.True(prev.HasAttribute("disabled"));
}
@@ -338,7 +338,7 @@ public class SiteCallsReportPageTests : BunitContext
var cut = Render();
cut.WaitForState(() => cut.Markup.Contains("ERP.Op0"));
- var next = cut.Find("[data-test='site-calls-next']");
+ var next = cut.Find("[data-test='keyset-next']");
Assert.False(next.HasAttribute("disabled"));
next.Click();
@@ -381,7 +381,7 @@ public class SiteCallsReportPageTests : BunitContext
cut.WaitForState(() => cut.Markup.Contains("ERP.Op0"));
// Step forward — query 2 carries the keyset cursor.
- var next = cut.Find("[data-test='site-calls-next']");
+ var next = cut.Find("[data-test='keyset-next']");
next.Click();
cut.WaitForAssertion(() =>
{
@@ -390,7 +390,7 @@ public class SiteCallsReportPageTests : BunitContext
});
// Previous is now live (the back-stack has one entry); click it.
- var prev = cut.Find("[data-test='site-calls-prev']");
+ var prev = cut.Find("[data-test='keyset-prev']");
Assert.False(prev.HasAttribute("disabled"));
prev.Click();
@@ -402,7 +402,7 @@ public class SiteCallsReportPageTests : BunitContext
Assert.Null(_queryRequests[2].AfterCreatedAtUtc);
Assert.Null(_queryRequests[2].AfterId);
// Back on page 1, the back-stack is empty again so Previous re-disables.
- Assert.True(cut.Find("[data-test='site-calls-prev']").HasAttribute("disabled"));
+ Assert.True(cut.Find("[data-test='keyset-prev']").HasAttribute("disabled"));
});
}