fix and add test

Author: katieperry4

packages/cypress/cypress/support/commands/odh.ts

@@ -1131,6 +1131,10 @@ declare global {
           type: 'POST /maas/api/v1/api-keys',
           response: { data: OdhResponse<CreateAPIKeyResponse> },
         ) => Cypress.Chainable<null>) &
+        ((
+          type: 'GET /maas/api/v1/is-maas-admin',
+          response: OdhResponse<{ data: { allowed: boolean } }>,
+        ) => Cypress.Chainable<null>) &
         ((
           type: 'GET /maas/api/v1/user',
           response: OdhResponse<{ data: { userId: string; clusterAdmin: boolean } }>,

packages/cypress/cypress/tests/mocked/modelsAsAService/maasApiKeys.cy.ts

@@ -1,7 +1,7 @@
 import { mockDashboardConfig, mockDscStatus } from '@odh-dashboard/internal/__mocks__';
 import { DataScienceStackComponent } from '@odh-dashboard/internal/concepts/areas/types';
 import type { APIKey } from '@odh-dashboard/maas/types/api-key';
-import { asProductAdminUser } from '../../../utils/mockUsers';
+import { asClusterAdminUser, asProjectAdminUser } from '../../../utils/mockUsers';
 import {
   apiKeysPage,
   bulkRevokeAPIKeyModal,
@@ -22,7 +22,7 @@ const mockSearchResponse = (keys: APIKey[]) => ({
 
 describe('API Keys Page', () => {
   beforeEach(() => {
-    asProductAdminUser();
+    asClusterAdminUser();
     cy.interceptOdh(
       'GET /api/config',
       mockDashboardConfig({
@@ -33,6 +33,7 @@ describe('API Keys Page', () => {
     cy.interceptOdh('GET /maas/api/v1/user', {
       data: { userId: 'test-user', clusterAdmin: false },
     });
+    cy.interceptOdh('GET /maas/api/v1/is-maas-admin', { data: { allowed: true } });
     cy.interceptOdh('GET /maas/api/v1/namespaces', { data: [] });
 
     cy.interceptOdh(
@@ -92,6 +93,13 @@ describe('API Keys Page', () => {
       .should('contain.text', 'production-backend');
   });
 
+  it('should not display the username filter for non-MaaS admins', () => {
+    asProjectAdminUser();
+    apiKeysPage.visit();
+    apiKeysPage.findFilterInput().should('not.exist');
+    apiKeysPage.findUsernameFilterTooltip().should('not.exist');
+  });
+
   it('should filter api keys by status', () => {
     const nonActiveKeys = mockAPIKeys().filter((k) => k.status !== 'active');