mirror of
https://github.com/BerriAI/litellm.git
synced 2026-10-08 03:08:45 +00:00
style(ui): drop narration comments from the usage gating tests
Both restated what the test name and the surrounding setup already say, so they were maintenance cost without explanatory value. The reasoning they carried lives in the commit that added the gates.
This commit is contained in:
parent
6a540a1bf8
commit
e3d3177ff1
2 changed files with 0 additions and 4 deletions
|
|
@ -863,8 +863,6 @@ describe("UsagePage", () => {
|
|||
});
|
||||
});
|
||||
|
||||
// The select hides both views from a non-admin, so this drives the section
|
||||
// gate directly through the mocked select, which always offers every option.
|
||||
it.each(["organization", "agent"])("should not render the %s usage view for an internal user", async (usageView) => {
|
||||
mockUseAuthorized.mockReturnValue(nonAdminSession);
|
||||
|
||||
|
|
|
|||
|
|
@ -127,8 +127,6 @@ describe("UsageViewSelect", () => {
|
|||
expect(screen.getByRole("option", { name: optionName })).toBeInTheDocument();
|
||||
});
|
||||
|
||||
// Neither /organization/daily/activity nor /agent/daily/activity admits an
|
||||
// internal user, so the option that fires them must not be selectable.
|
||||
it.each(["Organization Usage", "Agent Usage (A2A)"])("should hide %s from an internal user", (optionName) => {
|
||||
render(<UsageViewSelect value="global" onChange={mockOnChange} userRole="Internal User" canViewTagUsage={true} />);
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue