Sync: add a new method, do_only_first_initial_sync

projects/packages/sync/changelog/update-add_method_for_initial_sync_check

@@ -0,0 +1,4 @@
+Significance: patch
+Type: changed
+
+Actions: add the do_only_first_initial_sync method which starts an initial sync only when one hasn't already been done

projects/packages/sync/src/class-actions.php

@@ -485,7 +485,7 @@ public static function send_data( $data, $codec_name, $sent_timestamp, $queue_id
 	 * @return bool|null False if sync is not allowed.
 	 */
 	public static function do_initial_sync() {
-		// Lets not sync if we are not suppose to.
+		// Let's not sync if we are not supposed to.
 		if ( ! self::sync_allowed() ) {
 			return false;
 		}
@@ -507,6 +507,20 @@ public static function do_initial_sync() {
 		self::do_full_sync( $initial_sync_config );
 	}
 
+	/**
+	 * Do an initial full sync only if one has not already been started.
+	 *
+	 * @return bool|null False if the initial full sync was already started, otherwise null.
+	 */
+	public static function do_only_first_initial_sync() {
+		$full_sync_module = Modules::get_module( 'full-sync' );
+		if ( $full_sync_module && $full_sync_module->is_started() ) {
+			return false;
+		}
+
+		static::do_initial_sync();
+	}
+
 	/**
 	 * Kicks off a full sync.
 	 *

projects/packages/sync/src/class-package-version.php

@@ -12,7 +12,7 @@
  */
 class Package_Version {
 
-	const PACKAGE_VERSION = '1.27.2';
+	const PACKAGE_VERSION = '1.27.3-alpha';
 
 	const PACKAGE_SLUG = 'sync';
 

projects/packages/sync/tests/php/test-actions.php

@@ -0,0 +1,46 @@
+<?php // phpcs:ignore WordPress.Files.FileName.InvalidClassFileName
+
+namespace Automattic\Jetpack\Sync;
+
+use Automattic\Jetpack\Constants;
+use WorDBless\BaseTestCase;
+
+/**
+ * Unit tests for the Actions class.
+ *
+ * @package automattic/jetpack-sync
+ */
+class Test_Actions extends BaseTestCase {
+
+	/**
+	 * Set up before each test.
+	 *
+	 * @before
+	 */
+	public function set_up() {
+		// Don't try to get options directly from the database.
+		Constants::set_constant( 'JETPACK_DISABLE_RAW_OPTIONS', true );
+	}
+
+	/**
+	 * Tests the do_only_first_intitial_sync method when an initial sync has not been performed yet.
+	 */
+	public function test_do_only_first_intitial_sync_successful() {
+		$this->assertNull( Actions::do_only_first_initial_sync() );
+	}
+
+	/**
+	 * Tests the do_only_first_intitial_sync method when an initial sync has already been performed.
+	 */
+	public function test_do_only_first_intitial_sync_already_started() {
+		$full_sync_option = array(
+			'started'  => time(),
+			'finished' => false,
+			'progress' => array(),
+			'config'   => array(),
+		);
+		update_option( Modules\Full_Sync_Immediately::STATUS_OPTION, $full_sync_option );
+
+		$this->assertFalse( Actions::do_only_first_initial_sync() );
+	}
+}