{children}
),
MultiSelectTrigger: ({ children }: { children: React.ReactNode }) => (
- {children}
+ {children}
),
MultiSelectValue: ({ placeholder }: { placeholder: string }) => (
{placeholder}
@@ -145,4 +145,26 @@ describe("ProviderTypeSelector", () => {
).toHaveAttribute("aria-disabled", "true");
expect(screen.getByText("All selected")).toBeInTheDocument();
});
+
+ it("shows one icon per selected type and a count in the trigger", async () => {
+ const azure = {
+ ...providers[0],
+ id: "provider-2",
+ attributes: { ...providers[0].attributes, provider: "azure" as const },
+ };
+
+ render(
+
?
@@ -66,7 +33,7 @@ export const ProviderIconCell = ({
className,
)}
>
-
);
};
diff --git a/ui/components/icons/providers-badge/provider-type-icon.test.tsx b/ui/components/icons/providers-badge/provider-type-icon.test.tsx
new file mode 100644
index 0000000000..96b4f5f4d4
--- /dev/null
+++ b/ui/components/icons/providers-badge/provider-type-icon.test.tsx
@@ -0,0 +1,126 @@
+import { render, screen } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+
+import type { ProviderType } from "@/types/providers";
+
+import { ProviderTypeIcon, ProviderTypeIconStack } from "./provider-type-icon";
+
+// A provider type the API may return but this UI build does not know about.
+const UNKNOWN_TYPE = "future-provider" as unknown as ProviderType;
+
+// Render the lazy provider badges as plain text so we can assert on them.
+vi.mock("@/components/icons/providers-badge", () => ({
+ AWSProviderBadge: () => AWS,
+ AzureProviderBadge: () => Azure,
+ GCPProviderBadge: () => GCP,
+ KS8ProviderBadge: () => Kubernetes,
+ M365ProviderBadge: () => M365,
+ GitHubProviderBadge: () => GitHub,
+ GoogleWorkspaceProviderBadge: () => Google Workspace,
+ IacProviderBadge: () => IaC,
+ ImageProviderBadge: () => Image,
+ OracleCloudProviderBadge: () => Oracle Cloud,
+ MongoDBAtlasProviderBadge: () => MongoDB Atlas,
+ AlibabaCloudProviderBadge: () => Alibaba Cloud,
+ CloudflareProviderBadge: () => Cloudflare,
+ OpenStackProviderBadge: () => OpenStack,
+ VercelProviderBadge: () => Vercel,
+ OktaProviderBadge: () => Okta,
+}));
+
+// Render the tooltip pieces inline so the hover content is queryable in jsdom.
+vi.mock("@/components/shadcn", () => ({
+ Badge: ({ children }: { children: React.ReactNode }) => (
+ {children}
+ ),
+ Tooltip: ({ children }: { children: React.ReactNode }) => <>{children},
+ TooltipTrigger: ({ children }: { children: React.ReactNode }) => (
+ <>{children}
+ ),
+ TooltipContent: ({ children }: { children: React.ReactNode }) => (
+ {children}
+ ),
+}));
+
+describe("ProviderTypeIcon", () => {
+ it("renders the badge for the given provider type", async () => {
+ render(
+
+
+
+
+
+ );
+};
diff --git a/ui/components/providers/providers-accounts-view.test.tsx b/ui/components/providers/providers-accounts-view.test.tsx
index 9263277ea9..c8b3baf596 100644
--- a/ui/components/providers/providers-accounts-view.test.tsx
+++ b/ui/components/providers/providers-accounts-view.test.tsx
@@ -1,11 +1,36 @@
import { render, screen } from "@testing-library/react";
-import { describe, expect, it, vi } from "vitest";
+import userEvent from "@testing-library/user-event";
+import type { ReactNode } from "react";
+import { afterEach, describe, expect, it, vi } from "vitest";
import type { FilterOption, MetaDataProps, ProviderProps } from "@/types";
import type { ProvidersTableRow } from "@/types/providers-table";
+const { refreshMock, replaceMock, searchParamsValue } = vi.hoisted(() => ({
+ refreshMock: vi.fn(),
+ replaceMock: vi.fn(),
+ searchParamsValue: { current: "" },
+}));
+
+vi.mock("next/navigation", () => ({
+ usePathname: () => "/providers",
+ useRouter: () => ({
+ refresh: refreshMock,
+ replace: replaceMock,
+ }),
+ useSearchParams: () => new URLSearchParams(searchParamsValue.current),
+}));
+
+vi.mock("@/components/providers/table", () => ({
+ SkeletonTableProviders: () => ,
+}));
+
vi.mock("@/components/providers/add-provider-button", () => ({
- AddProviderButton: () => ,
+ AddProviderButton: ({ onOpenWizard }: { onOpenWizard: () => void }) => (
+
+ ),
}));
vi.mock("@/components/providers/muted-findings-config-button", () => ({
@@ -15,7 +40,12 @@ vi.mock("@/components/providers/muted-findings-config-button", () => ({
}));
vi.mock("@/components/providers/providers-filters", () => ({
- ProvidersFilters: () =>
+
+ + No Providers Configured +
+
+
+
+ {renderCta(props)}
+ + No providers have been configured. Start by setting up a provider. +
+Filters
,
+ ProvidersFilters: ({ actions }: { actions: ReactNode }) => (
+
+ Filters
+ {actions}
+
+ ),
}));
vi.mock("@/components/providers/providers-accounts-table", () => ({
@@ -23,7 +53,21 @@ vi.mock("@/components/providers/providers-accounts-table", () => ({
}));
vi.mock("@/components/providers/wizard", () => ({
- ProviderWizardModal: () => ,
+ ProviderWizardModal: ({
+ open,
+ onOpenChange,
+ }: {
+ open: boolean;
+ onOpenChange: (open: boolean) => void;
+ }) =>
+ open ? (
+
+ Provider wizard
+
+
+ ) : null,
}));
import { ProvidersAccountsView } from "./providers-accounts-view";
@@ -36,8 +80,55 @@ const metadata: MetaDataProps = {
version: "latest",
};
+const disconnectedProviders: ProviderProps[] = [
+ {
+ id: "provider-1",
+ type: "providers",
+ attributes: {
+ provider: "aws",
+ uid: "123456789012",
+ alias: "Production",
+ status: "completed",
+ resources: 0,
+ connection: {
+ connected: false,
+ last_checked_at: "2026-04-13T00:00:00Z",
+ },
+ scanner_args: {
+ only_logs: false,
+ excluded_checks: [],
+ aws_retries_max_attempts: 3,
+ },
+ inserted_at: "2026-04-13T00:00:00Z",
+ updated_at: "2026-04-13T00:00:00Z",
+ created_by: {
+ object: "user",
+ id: "user-1",
+ },
+ },
+ relationships: {
+ secret: {
+ data: null,
+ },
+ provider_groups: {
+ meta: {
+ count: 0,
+ },
+ data: [],
+ },
+ },
+ },
+];
+
describe("ProvidersAccountsView", () => {
- it("keeps the same vertical spacing between filters and table as other views", () => {
+ afterEach(() => {
+ vi.restoreAllMocks();
+ searchParamsValue.current = "";
+ window.history.replaceState({}, "", "/");
+ });
+
+ it("shows a full page empty state without filters or table when there are no providers", () => {
+ // Given/When
render(
-
- openProviderWizard()} />
-
- }
+ {hasNoProviders ? (
+ openProviderWizard()}
/>
-
+ ) : (
+
+
+ openProviderWizard()} />
+
+ }
+ />
+
+ )}
-
-
-
-
-
-);
diff --git a/ui/components/scans/scans-providers-empty-state.test.tsx b/ui/components/scans/scans-providers-empty-state.test.tsx
index dc6748a785..deee8318e3 100644
--- a/ui/components/scans/scans-providers-empty-state.test.tsx
+++ b/ui/components/scans/scans-providers-empty-state.test.tsx
@@ -1,73 +1,43 @@
import { render, screen } from "@testing-library/react";
-import userEvent from "@testing-library/user-event";
-import { afterEach, describe, expect, it, vi } from "vitest";
+import { describe, expect, it, vi } from "vitest";
+
+import { ADD_PROVIDER_HREF } from "@/lib/providers-navigation";
import { ScansProvidersEmptyState } from "./scans-providers-empty-state";
-const { replaceMock, searchParamsValue } = vi.hoisted(() => ({
- replaceMock: vi.fn(),
- searchParamsValue: { current: "" },
-}));
-
-vi.mock("next/navigation", () => ({
- usePathname: () => "/scans",
- useRouter: () => ({
- replace: replaceMock,
- }),
- useSearchParams: () => new URLSearchParams(searchParamsValue.current),
-}));
-
-vi.mock("@/components/providers/wizard", () => ({
- ProviderWizardModal: ({ open }: { open: boolean }) =>
- open ?
-
- - No Providers Configured -
-
-
-
-
- - No providers have been configured. Start by setting up a provider. -
-Provider wizard
: null,
-}));
-
vi.mock("./no-providers-connected", () => ({
NoProvidersConnected: () => No Connected Providers
,
}));
describe("ScansProvidersEmptyState", () => {
- afterEach(() => {
- vi.clearAllMocks();
- searchParamsValue.current = "";
- });
-
- it("shows the add provider message and opens the provider wizard", async () => {
- const user = userEvent.setup();
-
+ it("shows the add provider message with a providers page CTA", () => {
+ // Given/When
render(
diff --git a/ui/CHANGELOG.md b/ui/CHANGELOG.md
index db38de5fd3..8b2ba973aa 100644
--- a/ui/CHANGELOG.md
+++ b/ui/CHANGELOG.md
@@ -19,6 +19,7 @@ All notable changes to the **Prowler UI** are documented in this file.
### 🐞 Fixed
- Add Provider modal now closes without reloading the providers page [(#11424)](https://github.com/prowler-cloud/prowler/pull/11424)
+- Users page now shows the "Delete User" action only on the current user's row, matching the backend rule that a user can only delete their own account [(#11447)](https://github.com/prowler-cloud/prowler/pull/11447)
### 🔐 Security
diff --git a/ui/app/(prowler)/users/page.tsx b/ui/app/(prowler)/users/page.tsx
index 4b26c0baf2..fa1e838872 100644
--- a/ui/app/(prowler)/users/page.tsx
+++ b/ui/app/(prowler)/users/page.tsx
@@ -109,6 +109,9 @@ const SSRDataTable = async ({
roles,
canBeExpelled,
currentTenantId: canBeExpelled ? currentTenantId : undefined,
+ // Users may only delete their own account; gate the delete action so the
+ // UI matches the backend rule and never offers an action that would fail.
+ isCurrentUser: user.id === currentUserId,
};
});
diff --git a/ui/components/users/table/data-table-row-actions.test.tsx b/ui/components/users/table/data-table-row-actions.test.tsx
new file mode 100644
index 0000000000..39a6c95f8b
--- /dev/null
+++ b/ui/components/users/table/data-table-row-actions.test.tsx
@@ -0,0 +1,159 @@
+import { Row } from "@tanstack/react-table";
+import { render, screen } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
+import { describe, expect, it, vi } from "vitest";
+
+// The forms pull in server actions (`@/actions/users/users`) that can't run in
+// jsdom, so stub them with identifiable markers to assert which modal opens.
+vi.mock("../forms", () => ({
+ DeleteForm: ({ userId }: { userId: string }) => (
+ delete-form:{userId}
+ ),
+ EditForm: ({ userId }: { userId: string }) => (
+ edit-form:{userId}
+ ),
+ ExpelUserForm: ({ userId }: { userId: string }) => (
+ expel-form:{userId}
+ ),
+}));
+
+import { DataTableRowActions } from "./data-table-row-actions";
+
+interface RowOptions {
+ id?: string;
+ isCurrentUser?: boolean;
+ canBeExpelled?: boolean;
+ currentTenantId?: string;
+}
+
+const createRow = ({
+ id = "user-1",
+ isCurrentUser,
+ canBeExpelled,
+ currentTenantId,
+}: RowOptions = {}) =>
+ ({
+ original: {
+ id,
+ attributes: {
+ name: "Jane Doe",
+ email: "jane@example.com",
+ company_name: "Acme",
+ role: { name: "admin" },
+ },
+ isCurrentUser,
+ canBeExpelled,
+ currentTenantId,
+ },
+ }) as unknown as Row<{ id: string }>;
+
+const openMenu = async (user: ReturnType