Skip to content

Commit 53cc189

Browse files
committed
backup: use more than 1 row in bank workload
The validation added in the bank workload now requires at least two rows for validation to pass. This change updates the unit tests to pass at least 2 accounts.
1 parent cc971a4 commit 53cc189

14 files changed

+109
-108
lines changed

pkg/backup/alter_backup_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ func TestAlterBackupStatement(t *testing.T) {
3737
key2 := getAWSEncryptionOption(t, "NEW_AWS_KMS_REGION", "NEW_AWS_KEY_ID")
3838

3939
const userfile = "'userfile:///a'"
40-
const numAccounts = 1
40+
const numAccounts = 2
4141

4242
tc, sqlDB, _, cleanupFn := backupRestoreTestSetup(t, singleNode, numAccounts, InitManualReplication)
4343
execCfg := tc.Server(0).ExecutorConfig().(sql.ExecutorConfig)
@@ -77,7 +77,7 @@ func TestAlterBackupRestore(t *testing.T) {
7777
newURI := getAWSEncryptionOption(t, "NEW_AWS_KMS_REGION", "NEW_AWS_KEY_ID")
7878

7979
const userfile = "'userfile:///a'"
80-
const numAccounts = 1
80+
const numAccounts = 2
8181

8282
_, sqlDB, _, cleanupFn := backupRestoreTestSetup(t, singleNode, numAccounts, InitManualReplication)
8383
defer cleanupFn()
@@ -178,7 +178,7 @@ func TestAlterBackupIncremental(t *testing.T) {
178178
newURI := getAWSEncryptionOption(t, "NEW_AWS_KMS_REGION", "NEW_AWS_KEY_ID")
179179

180180
const userfile = "'userfile:///a'"
181-
const numAccounts = 1
181+
const numAccounts = 2
182182

183183
_, sqlDB, _, cleanupFn := backupRestoreTestSetup(t, singleNode, numAccounts, InitManualReplication)
184184
defer cleanupFn()

pkg/backup/backup_cloud_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ func TestCloudBackupRestoreKMSInaccessibleMetric(t *testing.T) {
295295
uri: fmt.Sprintf("azure-kms:///non-existent-key/000?AUTH=implicit&AZURE_VAULT_NAME=%s", azureVaultName),
296296
},
297297
} {
298-
tc, sqlDB, _, cleanupFn := backupRestoreTestSetup(t, 1, 1, InitManualReplication)
298+
tc, sqlDB, _, cleanupFn := backupRestoreTestSetup(t, 1, 2, InitManualReplication)
299299
defer cleanupFn()
300300

301301
t.Run(tt.name, func(t *testing.T) {

pkg/backup/backup_tenant_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ func TestBackupSharedProcessTenantNodeDown(t *testing.T) {
4949
},
5050
}
5151
params.ServerArgs.Knobs.JobsTestingKnobs = jobs.NewTestingKnobsWithShortIntervals()
52-
tc, hostDB, _, cleanup := backupRestoreTestSetupWithParams(t, multiNode, 0, /* numAccounts */
52+
tc, hostDB, _, cleanup := backupRestoreTestSetupWithParams(t, multiNode, 2, /* numAccounts */
5353
InitManualReplication, params)
5454
defer cleanup()
5555

0 commit comments

Comments
 (0)