From 28830b402f30989469e33aefa1990ce3cb1ecfc5 Mon Sep 17 00:00:00 2001 From: rcourtman Date: Fri, 8 May 2026 02:01:36 +0100 Subject: [PATCH] Clarify infrastructure source picker copy Use broader platform, host, and service search copy while keeping the address-detection action clear in the Add infrastructure picker. --- .../src/components/Settings/InfrastructureSourcePicker.tsx | 6 +++--- .../Settings/__tests__/InfrastructureSourcePicker.test.tsx | 6 ++---- .../Settings/__tests__/InfrastructureWorkspace.test.tsx | 6 +++--- .../Settings/__tests__/settingsArchitecture.test.ts | 2 +- 4 files changed, 9 insertions(+), 11 deletions(-) diff --git a/frontend-modern/src/components/Settings/InfrastructureSourcePicker.tsx b/frontend-modern/src/components/Settings/InfrastructureSourcePicker.tsx index f57add5ef..736362aea 100644 --- a/frontend-modern/src/components/Settings/InfrastructureSourcePicker.tsx +++ b/frontend-modern/src/components/Settings/InfrastructureSourcePicker.tsx @@ -79,7 +79,7 @@ export const InfrastructureSourcePicker: Component setQuery(event.currentTarget.value)} class={searchInputClass} - placeholder="Search Unraid, TrueNAS, Proxmox, Docker..." + placeholder="Search platforms, hosts, services..." /> @@ -89,7 +89,7 @@ export const InfrastructureSourcePicker: Component - Detect from address + Detect API platform @@ -108,7 +108,7 @@ export const InfrastructureSourcePicker: Component 0} fallback={
- No matching source yet. Try a platform name, a host type, or use detect from address. + No matching source yet. Try a platform, host type, service, or API endpoint address.
} > diff --git a/frontend-modern/src/components/Settings/__tests__/InfrastructureSourcePicker.test.tsx b/frontend-modern/src/components/Settings/__tests__/InfrastructureSourcePicker.test.tsx index 4828dfeba..46ab3ef5a 100644 --- a/frontend-modern/src/components/Settings/__tests__/InfrastructureSourcePicker.test.tsx +++ b/frontend-modern/src/components/Settings/__tests__/InfrastructureSourcePicker.test.tsx @@ -10,9 +10,7 @@ describe('InfrastructureSourcePicker', () => { it('labels admitted VMware as first-lab-ready and keeps Pulse Agent on the host profile path', () => { render(() => ); - expect( - screen.getByPlaceholderText('Search Unraid, TrueNAS, Proxmox, Docker...'), - ).toBeInTheDocument(); + expect(screen.getByPlaceholderText('Search platforms, hosts, services...')).toBeInTheDocument(); expect(screen.getByText('Common choices')).toBeInTheDocument(); expect(screen.getByText('First lab ready')).toBeInTheDocument(); expect(screen.queryByText('Available now')).toBeNull(); @@ -40,7 +38,7 @@ describe('InfrastructureSourcePicker', () => { it('filters the catalog by user-recognizable names and aliases', () => { render(() => ); - fireEvent.input(screen.getByPlaceholderText('Search Unraid, TrueNAS, Proxmox, Docker...'), { + fireEvent.input(screen.getByPlaceholderText('Search platforms, hosts, services...'), { target: { value: 'nas' }, }); diff --git a/frontend-modern/src/components/Settings/__tests__/InfrastructureWorkspace.test.tsx b/frontend-modern/src/components/Settings/__tests__/InfrastructureWorkspace.test.tsx index 868690613..ebfeb417e 100644 --- a/frontend-modern/src/components/Settings/__tests__/InfrastructureWorkspace.test.tsx +++ b/frontend-modern/src/components/Settings/__tests__/InfrastructureWorkspace.test.tsx @@ -624,9 +624,9 @@ describe('InfrastructureWorkspace', () => { screen.getByText('Choose the system, platform, host, or service you want Pulse to monitor.'), ).toBeInTheDocument(); expect( - within(dialog).getByPlaceholderText('Search Unraid, TrueNAS, Proxmox, Docker...'), + within(dialog).getByPlaceholderText('Search platforms, hosts, services...'), ).toBeInTheDocument(); - expect(within(dialog).getByRole('button', { name: 'Detect from address' })).toBeInTheDocument(); + expect(within(dialog).getByRole('button', { name: 'Detect API platform' })).toBeInTheDocument(); expect(within(dialog).getByText('Common choices')).toBeInTheDocument(); expect(within(dialog).queryByText('Agent telemetry')).toBeNull(); expect(within(dialog).queryByText('API inventory')).toBeNull(); @@ -640,7 +640,7 @@ describe('InfrastructureWorkspace', () => { await waitFor(() => expect(screen.getByRole('dialog')).toBeInTheDocument()); fireEvent.click( - within(screen.getByRole('dialog')).getByRole('button', { name: 'Detect from address' }), + within(screen.getByRole('dialog')).getByRole('button', { name: 'Detect API platform' }), ); expect(navigateSpy).toHaveBeenCalledWith('/settings/infrastructure?add=detect', { diff --git a/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts b/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts index 604cbda98..0a9be1c0f 100644 --- a/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts +++ b/frontend-modern/src/components/Settings/__tests__/settingsArchitecture.test.ts @@ -509,7 +509,7 @@ describe('settings architecture guardrails', () => { expect(infrastructureSourceManagerSource).not.toContain('Infrastructure coverage'); expect(infrastructureSourceManagerSource).not.toContain('Fleet governance'); expect(infrastructureSourceManagerSource).not.toContain('Connection types'); - expect(infrastructureSourcePickerSource).toContain('Detect from address'); + expect(infrastructureSourcePickerSource).toContain('Detect API platform'); expect(infrastructureSourcePickerSource).toContain('getInfrastructureSourcePickerItems'); expect(infrastructureSourcePickerSource).toContain('itemMatchesQuery'); expect(infrastructureSourcePickerSource).toContain('catalogDescription');