Studio: Fix incorrect offline message displayed alongside "delete site" link

src/components/delete-site.tsx

@@ -19,6 +19,11 @@ const DeleteSite = () => {
 		'This site has active demo sites that cannot be deleted without an internet connection.'
 	);
 
+	const { snapshots } = useSiteDetails();
+	const snapshotsOnSite = snapshots.filter(
+		( snapshot ) => snapshot.localSiteId === selectedSite?.id
+	);
+
 	const handleDeleteSite = async () => {
 		if ( ! selectedSite ) {
 			return;
@@ -65,12 +70,17 @@ const DeleteSite = () => {
 			? `${ name.substring( 0, MAX_LENGTH_SITE_TITLE - 3 ) }...`
 			: name;
 
-	const isSiteDeletionDisabled = ! selectedSite || isOffline || isDeleting;
+	const isSiteDeletionDisabled =
+		! selectedSite || ( isOffline && snapshotsOnSite.length > 0 ) || isDeleting;
 
 	return (
-		<Tooltip disabled={ ! isOffline } icon={ offlineIcon } text={ offlineMessage }>
+		<Tooltip
+			disabled={ ! ( isOffline && snapshotsOnSite.length > 0 ) }
+			icon={ offlineIcon }
+			text={ offlineMessage }
+		>
 			<Button
-				aria-description={ isOffline ? offlineMessage : '' }
+				aria-description={ isOffline && snapshotsOnSite.length > 0 ? offlineMessage : '' }
 				aria-disabled={ isSiteDeletionDisabled }
 				onClick={ () => {
 					if ( isSiteDeletionDisabled ) {

src/components/tests/content-tab-settings.test.tsx

@@ -3,10 +3,12 @@ import { fireEvent, render, screen } from '@testing-library/react';
 import { userEvent } from '@testing-library/user-event';
 import { useGetWpVersion } from '../../hooks/use-get-wp-version';
 import { useOffline } from '../../hooks/use-offline';
+import { useSiteDetails } from '../../hooks/use-site-details';
 import { getIpcApi } from '../../lib/get-ipc-api';
 import { ContentTabSettings } from '../content-tab-settings';
 
 jest.mock( '../../hooks/use-get-wp-version' );
+jest.mock( '../../hooks/use-site-details' );
 jest.mock( '../../lib/get-ipc-api' );
 
 const selectedSite: SiteDetails = {
@@ -30,6 +32,15 @@ describe( 'ContentTabSettings', () => {
 			openLocalPath,
 			generateProposedSitePath,
 		} );
+
+		( useSiteDetails as jest.Mock ).mockReturnValue( {
+			selectedSite,
+			snapshots: [],
+			uploadingSites: {},
+			deleteSite: jest.fn(),
+			isDeleting: false,
+			updateSite: jest.fn(),
+		} );
 	} );
 
 	test( 'renders site details correctly', () => {
@@ -101,8 +112,16 @@ describe( 'ContentTabSettings', () => {
 		expect( copyText ).toHaveBeenCalledWith( 'test-password' );
 	} );
 
-	it( 'disables delete site button when offline', async () => {
+	it( 'disables delete site button when offline and there is at least one snapshot present for the site', async () => {
 		( useOffline as jest.Mock ).mockReturnValue( true );
+
+		// Mock snapshots to include a snapshot for the selected site
+		( useSiteDetails as jest.Mock ).mockReturnValue( {
+			selectedSite: selectedSite,
+			snapshots: [ { localSiteId: selectedSite.id } ],
+			deleteSite: jest.fn(),
+			isDeleting: false,
+		} );
 		render( <ContentTabSettings selectedSite={ selectedSite } /> );
 		const deleteSiteButton = await screen.findByRole( 'button', { name: 'Delete site' } );
 		expect( deleteSiteButton ).toHaveAttribute( 'aria-disabled', 'true' );