Adjustments to scan fetching/in progress/results/error state management

projects/plugins/protect/src/js/routes/scan/index.jsx

@@ -199,12 +199,16 @@ const ScanPage = () => {
 			return <ScanningSection currentProgress={ status.currentProgress } />;
 		}
 
+		if ( statusIsFetching ) {
+			return <Spinner />;
+		}
+
 		if ( scanError ) {
 			return <ErrorSection errorMessage={ scanError.message } errorCode={ scanError.code } />;
 		}
 
 		return <DefaultSection />;
-	}, [ scanInProgress, status.currentProgress, scanError ] );
+	}, [ scanInProgress, status.currentProgress, statusIsFetching, scanError ] );
 
 	return (
 		<OnboardingContext.Provider value={ onboardingSteps }>

projects/plugins/protect/src/js/state/selectors.js

@@ -8,16 +8,16 @@ import { SCAN_IN_PROGRESS_STATUSES, SCAN_STATUS_OPTIMISTICALLY_SCANNING } from '
  * @returns {boolean} Whether a scan is in progress.
  */
 const scanInProgress = state => {
-	const { status, error, lastChecked } = selectors.getStatus( state );
+	const { status, lastChecked } = selectors.getStatus( state );
 	const unavailable = selectors.getScanIsUnavailable( state );
 
 	// When "optimistically" scanning, ignore any other status or error.
 	if ( SCAN_STATUS_OPTIMISTICALLY_SCANNING === status ) {
 		return true;
 	}
 
-	// If there is an error or the scan is unavailable, scanning is not in progress.
-	if ( error || unavailable ) {
+	// If the scanner is unavailable, scanning is not in progress.
+	if ( unavailable ) {
 		return false;
 	}
 
@@ -41,7 +41,7 @@ const scanInProgress = state => {
  * @returns {ScanError|null} The error object or null.
  */
 const scanError = state => {
-	const { error, errorCode, errorMessage } = selectors.getStatus( state );
+	const { status, error, errorCode, errorMessage } = selectors.getStatus( state );
 	const unavailable = selectors.getScanIsUnavailable( state );
 
 	// If the scan results include an error, return it.
@@ -50,7 +50,7 @@ const scanError = state => {
 	}
 
 	// If the scan is unavailable, return an error.
-	if ( unavailable ) {
+	if ( unavailable || ! status ) {
 		return {
 			code: 'scan_unavailable',
 			message: __( 'We are having problems scanning your site.', 'jetpack-protect' ),