feat(FN-4585): add mobile reliability view to MobileNavBar
Adds mobile reliability navigation to the dashboard's MobileNavBar, including a regression test to guard against feature-access issues on small screens. Also includes the corresponding changeset for publishing the change. Fusion-Task-Id: FN-4585
This commit is contained in:
@@ -159,11 +159,30 @@ describe("Mobile Feature Access Regression Guard", () => {
|
||||
expect(screen.getByTestId("mobile-more-item-schedules")).toBeDefined();
|
||||
expect(screen.getByTestId("mobile-more-item-github")).toBeDefined();
|
||||
expect(screen.getByTestId("mobile-more-item-usage")).toBeDefined();
|
||||
expect(screen.getByTestId("mobile-more-item-reliability")).toBeDefined();
|
||||
expect(screen.queryByTestId("mobile-more-item-chat")).toBeNull();
|
||||
expect(screen.queryByTestId("mobile-more-item-nodes")).toBeNull();
|
||||
expect(screen.getByTestId("mobile-more-item-settings")).toBeDefined();
|
||||
});
|
||||
|
||||
it("reliability view is reachable from mobile More sheet", () => {
|
||||
const props = createDefaultMobileNavProps();
|
||||
render(<MobileNavBar {...props} />);
|
||||
|
||||
fireEvent.click(screen.getByTestId("mobile-nav-tab-more"));
|
||||
|
||||
const reliabilityItem = screen.getByTestId("mobile-more-item-reliability");
|
||||
expect(reliabilityItem).toBeDefined();
|
||||
fireEvent.click(reliabilityItem);
|
||||
expect(props.onChangeView).toHaveBeenCalledWith("reliability");
|
||||
});
|
||||
|
||||
it("more tab is active when reliability view is open", () => {
|
||||
render(<MobileNavBar {...createDefaultMobileNavProps()} view="reliability" />);
|
||||
|
||||
expect(screen.getByTestId("mobile-nav-tab-more").className).toContain("mobile-nav-tab--active");
|
||||
});
|
||||
|
||||
it("nodes view is reachable from mobile More sheet when enabled", () => {
|
||||
const props = createDefaultMobileNavProps();
|
||||
render(<MobileNavBar {...props} experimentalFeatures={{ nodesView: true }} />);
|
||||
|
||||
@@ -229,6 +229,7 @@ export function MobileNavBar({
|
||||
|
||||
const isMoreActive =
|
||||
view === "documents"
|
||||
|| view === "reliability"
|
||||
|| (Boolean(experimentalFeatures?.evalsView) && view === "evals")
|
||||
|| view === "research"
|
||||
|| view === "insights"
|
||||
@@ -606,6 +607,16 @@ export function MobileNavBar({
|
||||
<FileText />
|
||||
<span>Documents</span>
|
||||
</button>
|
||||
|
||||
<button
|
||||
type="button"
|
||||
className="mobile-more-item"
|
||||
data-testid="mobile-more-item-reliability"
|
||||
onClick={() => handleMoreAction(() => onChangeView("reliability"))}
|
||||
>
|
||||
<Activity />
|
||||
<span>Reliability</span>
|
||||
</button>
|
||||
{experimentalFeatures?.evalsView && (
|
||||
<button
|
||||
type="button"
|
||||
|
||||
Reference in New Issue
Block a user