Handle linking to unregistered hosts

assets/js/components/ClusterDetails/AscsErsClusterDetails.jsx

@@ -6,7 +6,7 @@ import Table from '@components/Table';
 import ListView from '@components/ListView';
 import ProviderLabel from '@components/ProviderLabel';
 import DottedPagination from '@components/DottedPagination';
-import HostLink from '@components/HostLink';
+import ClusterNodeLink from '@components/ClusterDetails/ClusterNodeLink';
 import SapSystemLink from '@components/SapSystemLink';
 import { renderEnsaVersion } from '@components/SapSystemDetails';
 
@@ -23,7 +23,7 @@ const nodeDetailsConfig = {
     {
       title: 'Hostname',
       key: '',
-      render: (_, { id, name }) => <HostLink hostId={id}>{name}</HostLink>,
+      render: (_, { id, name }) => <ClusterNodeLink name={name} hostId={id} />,
     },
     {
       title: 'Role',

assets/js/components/ClusterDetails/AscsErsClusterDetails.stories.jsx

@@ -76,6 +76,18 @@ export const Single = {
   ),
 };
 
+export const WithUnregisteredHost = {
+  args: {
+    ...Single.args,
+    hosts: [buildHostsFromAscsErsClusterDetails(details).pop()],
+  },
+  render: (args) => (
+    <ContainerWrapper>
+      <AscsErsClusterDetails {...args} />
+    </ContainerWrapper>
+  ),
+};
+
 export const MultiSID = {
   args: {
     ...Single.args,

assets/js/components/ClusterDetails/AscsErsClusterDetails.test.jsx

@@ -200,4 +200,31 @@ describe('ClusterDetails AscsErsClusterDetails component', () => {
     expect(sidContainer).toHaveTextContent(sid);
     expect(sidContainer.querySelector('a')).toBeNull();
   });
+
+  it('should display a host link for registered hosts', () => {
+    const {
+      name,
+      cib_last_written: cibLastWritten,
+      provider,
+      details,
+    } = clusterFactory.build({ type: 'ascs_ers' });
+
+    const hosts = buildHostsFromAscsErsClusterDetails(details);
+    renderWithRouter(
+      <AscsErsClusterDetails
+        clusterName={name}
+        hosts={hosts}
+        cibLastWritten={cibLastWritten}
+        provider={provider}
+        sapSystems={[]}
+        details={details}
+      />
+    );
+    const registeredHostContainer = screen.getByText(hosts[0].hostname);
+
+    expect(registeredHostContainer).toHaveAttribute(
+      'href',
+      `/hosts/${hosts[0].id}`
+    );
+  });
 });

assets/js/components/ClusterDetails/ClusterNodeLink.jsx

@@ -0,0 +1,23 @@
+import React from 'react';
+import { EOS_WARNING_OUTLINED } from 'eos-icons-react';
+
+import Tooltip from '@components/Tooltip';
+import HostLink from '@components/HostLink';
+
+function ClusterNodeLink({ name, hostId }) {
+  if (hostId) {
+    return <HostLink hostId={hostId}>{name}</HostLink>;
+  }
+  return (
+    <span className="group flex items-center relative">
+      <EOS_WARNING_OUTLINED size="base" className="centered fill-yellow-500" />
+      <span className="ml-1 truncate max-w-[100px]">{name}</span>
+      <Tooltip
+        tooltipText="Host currently not registered."
+        width="w-52 -translate-x-1/3"
+      />
+    </span>
+  );
+}
+
+export default ClusterNodeLink;

assets/js/components/ClusterDetails/ClusterNodeLink.test.jsx

@@ -0,0 +1,32 @@
+import React from 'react';
+import { screen, render } from '@testing-library/react';
+import '@testing-library/jest-dom';
+import { renderWithRouterMatch } from '@lib/test-utils';
+import ClusterNodeLink from '@components/ClusterDetails/ClusterNodeLink';
+
+describe('ClusterNodeLink', () => {
+  it('renders HostLink when hostId is provided', () => {
+    const hostId = '12345678-abcd-1234-abcd-1234567890ab';
+    const name = 'host01';
+    const path = '/hosts/:hostId';
+
+    renderWithRouterMatch(<ClusterNodeLink hostId={hostId} name={name} />, {
+      path,
+      route: `/hosts/${hostId}`,
+    });
+
+    const hostLinkElement = screen.getByRole('link', { name });
+
+    expect(hostLinkElement).toBeInTheDocument();
+    expect(hostLinkElement).toHaveAttribute('href', `/hosts/${hostId}`);
+  });
+
+  it('renders warning span when hostId is not provided', () => {
+    const name = 'host02';
+
+    render(<ClusterNodeLink name={name} />);
+    expect(
+      screen.getByText('Host currently not registered.')
+    ).toBeInTheDocument();
+  });
+});

assets/js/components/ClusterDetails/HanaClusterDetails.jsx

@@ -10,7 +10,7 @@ import ListView from '@components/ListView';
 import Table from '@components/Table';
 import Tooltip from '@components/Tooltip';
 import TriggerChecksExecutionRequest from '@components/TriggerChecksExecutionRequest';
-import HostLink from '@components/HostLink';
+import ClusterNodeLink from '@components/ClusterDetails/ClusterNodeLink';
 import ChecksResultOverview from '@components/ClusterDetails/ChecksResultOverview';
 import ProviderLabel from '@components/ProviderLabel';
 import SapSystemLink from '@components/SapSystemLink';
@@ -34,7 +34,7 @@ const siteDetailsConfig = {
       title: 'Hostname',
       key: '',
       render: (_, hostData) => (
-        <HostLink hostId={hostData.id}>{hostData.name}</HostLink>
+        <ClusterNodeLink name={hostData.name} hostId={hostData.id} />
       ),
     },
     { title: 'Role', key: 'hana_status' },

assets/js/components/ClusterDetails/HanaClusterDetails.stories.jsx

@@ -80,6 +80,13 @@ export const Hana = {
   },
 };
 
+export const WithUnregisteredHost = {
+  args: {
+    ...Hana.args,
+    hosts: [hosts.pop()],
+  },
+};
+
 export const WithNoSelectedChecks = {
   args: {
     ...Hana.args,

assets/js/components/ClusterDetails/HanaClusterDetails.test.jsx

@@ -6,6 +6,7 @@ import { renderWithRouter } from '@lib/test-utils';
 import {
   clusterFactory,
   hostFactory,
+  hanaClusterDetailsNodesFactory,
   checksExecutionCompletedFactory,
   checksExecutionRunningFactory,
   sapSystemFactory,
@@ -181,4 +182,51 @@ describe('HanaClusterDetails component', () => {
     expect(sidContainer).toHaveTextContent(sid);
     expect(sidContainer.querySelector('a')).toBeNull();
   });
+
+  it('should display a host link in the site details if the host is registered', () => {
+    const registeredClusterNode = hanaClusterDetailsNodesFactory.build({
+      name: 'registeredhost',
+    });
+
+    const {
+      clusterID,
+      clusterName,
+      cib_last_written: cibLastWritten,
+      type: clusterType,
+      sid,
+      provider,
+      details,
+    } = clusterFactory.build({ details: { nodes: [registeredClusterNode] } });
+
+    const host = hostFactory.build({
+      hostname: registeredClusterNode.name,
+      cluster_id: clusterID,
+    });
+
+    const sapSystems = sapSystemFactory.buildList(2, { tenant: sid });
+
+    renderWithRouter(
+      <HanaClusterDetails
+        clusterID={clusterID}
+        clusterName={clusterName}
+        selectedChecks={[]}
+        hasSelectedChecks={false}
+        hosts={[host]}
+        clusterType={clusterType}
+        cibLastWritten={cibLastWritten}
+        sid={sid}
+        provider={provider}
+        sapSystems={sapSystems}
+        details={details}
+        lastExecution={null}
+      />
+    );
+
+    const registeredHostContainer = screen.getByText('registeredhost');
+
+    expect(registeredHostContainer).toHaveAttribute(
+      'href',
+      `/hosts/${host.id}`
+    );
+  });
 });