itest: turn on syncer cache by default

To make sure the syncer cache doesn't have any other hidden bugs, we
turn it on by default in the integration tests.

Author: guggero

itest/tapd_harness.go

@@ -126,6 +126,10 @@ type harnessOpts struct {
 	// sqliteDatabaseFilePath is the path to the SQLite database file to
 	// use.
 	sqliteDatabaseFilePath *string
+
+	// disableSyncCache is a flag that can be set to true to disable the
+	// universe syncer cache.
+	disableSyncCache bool
 }
 
 type harnessOption func(*harnessOpts)
@@ -283,6 +287,10 @@ func newTapdHarness(t *testing.T, ht *harnessTest, cfg tapdConfig,
 		finalCfg.Universe.SyncInterval = *opts.fedSyncTickerInterval
 	}
 
+	if !opts.disableSyncCache {
+		finalCfg.Universe.MultiverseCaches.SyncerCacheEnabled = true
+	}
+
 	return &tapdHarness{
 		cfg:       &cfg,
 		clientCfg: finalCfg,

itest/test_harness.go

@@ -378,6 +378,9 @@ type tapdHarnessParams struct {
 	// sqliteDatabaseFilePath is the path to the SQLite database file to
 	// use.
 	sqliteDatabaseFilePath *string
+
+	// disableSyncCache indicates whether the sync cache should be disabled.
+	disableSyncCache bool
 }
 
 type Option func(*tapdHarnessParams)
@@ -413,6 +416,7 @@ func setupTapdHarness(t *testing.T, ht *harnessTest,
 		ho.addrAssetSyncerDisable = params.addrAssetSyncerDisable
 		ho.fedSyncTickerInterval = params.fedSyncTickerInterval
 		ho.sqliteDatabaseFilePath = params.sqliteDatabaseFilePath
+		ho.disableSyncCache = params.disableSyncCache
 	}
 
 	tapdCfg := tapdConfig{