Helper functions to test generic backup flags (#4602)
Follow up on #4557 [comment](https://github.com/alcionai/corso/pull/4557#pullrequestreview-1701971318) to consolidate test code for ease of use --- #### Does this PR need a docs update or release note? - [ ] ✅ Yes, it's included - [ ] 🕐 Yes, but in a later PR - [x] ⛔ No #### Type of change - [ ] 🌻 Feature - [ ] 🐛 Bugfix - [ ] 🗺️ Documentation - [x] 🤖 Supportability/Tests - [ ] 💻 CI/Deployment - [x] 🧹 Tech Debt/Cleanup #### Test Plan - [ ] 💪 Manual - [x] ⚡ Unit test - [ ] 💚 E2E
This commit is contained in:
parent
f7b817e8ef
commit
56e09a1a69
@ -106,13 +106,11 @@ func (suite *ExchangeUnitSuite) TestBackupCreateFlags() {
|
|||||||
"--" + flags.DeltaPageSizeFN, flagsTD.DeltaPageSize,
|
"--" + flags.DeltaPageSizeFN, flagsTD.DeltaPageSize,
|
||||||
|
|
||||||
// bool flags
|
// bool flags
|
||||||
"--" + flags.FailFastFN,
|
|
||||||
"--" + flags.DisableIncrementalsFN,
|
|
||||||
"--" + flags.ForceItemDataDownloadFN,
|
|
||||||
"--" + flags.DisableDeltaFN,
|
"--" + flags.DisableDeltaFN,
|
||||||
"--" + flags.EnableImmutableIDFN,
|
"--" + flags.EnableImmutableIDFN,
|
||||||
"--" + flags.DisableConcurrencyLimiterFN,
|
"--" + flags.DisableConcurrencyLimiterFN,
|
||||||
},
|
},
|
||||||
|
flagsTD.PreparedGenericBackupFlags(),
|
||||||
flagsTD.PreparedProviderFlags(),
|
flagsTD.PreparedProviderFlags(),
|
||||||
flagsTD.PreparedStorageFlags()))
|
flagsTD.PreparedStorageFlags()))
|
||||||
|
|
||||||
@ -128,6 +126,7 @@ func (suite *ExchangeUnitSuite) TestBackupCreateFlags() {
|
|||||||
assert.True(t, co.ToggleFeatures.DisableDelta)
|
assert.True(t, co.ToggleFeatures.DisableDelta)
|
||||||
assert.True(t, co.ToggleFeatures.ExchangeImmutableIDs)
|
assert.True(t, co.ToggleFeatures.ExchangeImmutableIDs)
|
||||||
assert.True(t, co.ToggleFeatures.DisableConcurrencyLimiter)
|
assert.True(t, co.ToggleFeatures.DisableConcurrencyLimiter)
|
||||||
|
flagsTD.AssertGenericBackupFlags(t, cmd)
|
||||||
flagsTD.AssertProviderFlags(t, cmd)
|
flagsTD.AssertProviderFlags(t, cmd)
|
||||||
flagsTD.AssertStorageFlags(t, cmd)
|
flagsTD.AssertStorageFlags(t, cmd)
|
||||||
}
|
}
|
||||||
|
|||||||
@ -143,11 +143,9 @@ func (suite *GroupsUnitSuite) TestBackupCreateFlags() {
|
|||||||
"--" + flags.GroupFN, flagsTD.FlgInputs(flagsTD.GroupsInput),
|
"--" + flags.GroupFN, flagsTD.FlgInputs(flagsTD.GroupsInput),
|
||||||
"--" + flags.CategoryDataFN, flagsTD.FlgInputs(flagsTD.GroupsCategoryDataInput),
|
"--" + flags.CategoryDataFN, flagsTD.FlgInputs(flagsTD.GroupsCategoryDataInput),
|
||||||
"--" + flags.FetchParallelismFN, flagsTD.FetchParallelism,
|
"--" + flags.FetchParallelismFN, flagsTD.FetchParallelism,
|
||||||
"--" + flags.FailFastFN,
|
|
||||||
"--" + flags.DisableIncrementalsFN,
|
|
||||||
"--" + flags.ForceItemDataDownloadFN,
|
|
||||||
"--" + flags.DisableDeltaFN,
|
"--" + flags.DisableDeltaFN,
|
||||||
},
|
},
|
||||||
|
flagsTD.PreparedGenericBackupFlags(),
|
||||||
flagsTD.PreparedProviderFlags(),
|
flagsTD.PreparedProviderFlags(),
|
||||||
flagsTD.PreparedStorageFlags()))
|
flagsTD.PreparedStorageFlags()))
|
||||||
|
|
||||||
@ -160,6 +158,7 @@ func (suite *GroupsUnitSuite) TestBackupCreateFlags() {
|
|||||||
assert.True(t, co.ToggleFeatures.DisableIncrementals)
|
assert.True(t, co.ToggleFeatures.DisableIncrementals)
|
||||||
assert.True(t, co.ToggleFeatures.ForceItemDataDownload)
|
assert.True(t, co.ToggleFeatures.ForceItemDataDownload)
|
||||||
assert.True(t, co.ToggleFeatures.DisableDelta)
|
assert.True(t, co.ToggleFeatures.DisableDelta)
|
||||||
|
flagsTD.AssertGenericBackupFlags(t, cmd)
|
||||||
flagsTD.AssertProviderFlags(t, cmd)
|
flagsTD.AssertProviderFlags(t, cmd)
|
||||||
flagsTD.AssertStorageFlags(t, cmd)
|
flagsTD.AssertStorageFlags(t, cmd)
|
||||||
}
|
}
|
||||||
|
|||||||
@ -101,10 +101,8 @@ func (suite *OneDriveUnitSuite) TestBackupCreateFlags() {
|
|||||||
[]string{
|
[]string{
|
||||||
"--" + flags.RunModeFN, flags.RunModeFlagTest,
|
"--" + flags.RunModeFN, flags.RunModeFlagTest,
|
||||||
"--" + flags.UserFN, flagsTD.FlgInputs(flagsTD.UsersInput),
|
"--" + flags.UserFN, flagsTD.FlgInputs(flagsTD.UsersInput),
|
||||||
"--" + flags.FailFastFN,
|
|
||||||
"--" + flags.DisableIncrementalsFN,
|
|
||||||
"--" + flags.ForceItemDataDownloadFN,
|
|
||||||
},
|
},
|
||||||
|
flagsTD.PreparedGenericBackupFlags(),
|
||||||
flagsTD.PreparedProviderFlags(),
|
flagsTD.PreparedProviderFlags(),
|
||||||
flagsTD.PreparedStorageFlags()))
|
flagsTD.PreparedStorageFlags()))
|
||||||
|
|
||||||
@ -115,6 +113,7 @@ func (suite *OneDriveUnitSuite) TestBackupCreateFlags() {
|
|||||||
assert.Equal(t, control.FailFast, co.FailureHandling)
|
assert.Equal(t, control.FailFast, co.FailureHandling)
|
||||||
assert.True(t, co.ToggleFeatures.DisableIncrementals)
|
assert.True(t, co.ToggleFeatures.DisableIncrementals)
|
||||||
assert.True(t, co.ToggleFeatures.ForceItemDataDownload)
|
assert.True(t, co.ToggleFeatures.ForceItemDataDownload)
|
||||||
|
flagsTD.AssertGenericBackupFlags(t, cmd)
|
||||||
flagsTD.AssertProviderFlags(t, cmd)
|
flagsTD.AssertProviderFlags(t, cmd)
|
||||||
flagsTD.AssertStorageFlags(t, cmd)
|
flagsTD.AssertStorageFlags(t, cmd)
|
||||||
}
|
}
|
||||||
|
|||||||
@ -105,10 +105,8 @@ func (suite *SharePointUnitSuite) TestBackupCreateFlags() {
|
|||||||
"--" + flags.SiteIDFN, flagsTD.FlgInputs(flagsTD.SiteIDInput),
|
"--" + flags.SiteIDFN, flagsTD.FlgInputs(flagsTD.SiteIDInput),
|
||||||
"--" + flags.SiteFN, flagsTD.FlgInputs(flagsTD.WebURLInput),
|
"--" + flags.SiteFN, flagsTD.FlgInputs(flagsTD.WebURLInput),
|
||||||
"--" + flags.CategoryDataFN, flagsTD.FlgInputs(flagsTD.SharepointCategoryDataInput),
|
"--" + flags.CategoryDataFN, flagsTD.FlgInputs(flagsTD.SharepointCategoryDataInput),
|
||||||
"--" + flags.FailFastFN,
|
|
||||||
"--" + flags.DisableIncrementalsFN,
|
|
||||||
"--" + flags.ForceItemDataDownloadFN,
|
|
||||||
},
|
},
|
||||||
|
flagsTD.PreparedGenericBackupFlags(),
|
||||||
flagsTD.PreparedProviderFlags(),
|
flagsTD.PreparedProviderFlags(),
|
||||||
flagsTD.PreparedStorageFlags()))
|
flagsTD.PreparedStorageFlags()))
|
||||||
|
|
||||||
@ -120,6 +118,7 @@ func (suite *SharePointUnitSuite) TestBackupCreateFlags() {
|
|||||||
assert.Equal(t, control.FailFast, co.FailureHandling)
|
assert.Equal(t, control.FailFast, co.FailureHandling)
|
||||||
assert.True(t, co.ToggleFeatures.DisableIncrementals)
|
assert.True(t, co.ToggleFeatures.DisableIncrementals)
|
||||||
assert.True(t, co.ToggleFeatures.ForceItemDataDownload)
|
assert.True(t, co.ToggleFeatures.ForceItemDataDownload)
|
||||||
|
flagsTD.AssertGenericBackupFlags(t, cmd)
|
||||||
flagsTD.AssertProviderFlags(t, cmd)
|
flagsTD.AssertProviderFlags(t, cmd)
|
||||||
flagsTD.AssertStorageFlags(t, cmd)
|
flagsTD.AssertStorageFlags(t, cmd)
|
||||||
}
|
}
|
||||||
|
|||||||
16
src/cli/flags/testdata/repo.go
vendored
16
src/cli/flags/testdata/repo.go
vendored
@ -4,7 +4,7 @@ import (
|
|||||||
"testing"
|
"testing"
|
||||||
|
|
||||||
"github.com/spf13/cobra"
|
"github.com/spf13/cobra"
|
||||||
"gotest.tools/v3/assert"
|
"github.com/stretchr/testify/assert"
|
||||||
|
|
||||||
"github.com/alcionai/corso/src/cli/flags"
|
"github.com/alcionai/corso/src/cli/flags"
|
||||||
)
|
)
|
||||||
@ -40,3 +40,17 @@ func AssertProviderFlags(t *testing.T, cmd *cobra.Command) {
|
|||||||
assert.Equal(t, AzureTenantID, flags.AzureClientTenantFV)
|
assert.Equal(t, AzureTenantID, flags.AzureClientTenantFV)
|
||||||
assert.Equal(t, AzureClientSecret, flags.AzureClientSecretFV)
|
assert.Equal(t, AzureClientSecret, flags.AzureClientSecretFV)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func PreparedGenericBackupFlags() []string {
|
||||||
|
return []string{
|
||||||
|
"--" + flags.FailFastFN,
|
||||||
|
"--" + flags.DisableIncrementalsFN,
|
||||||
|
"--" + flags.ForceItemDataDownloadFN,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
func AssertGenericBackupFlags(t *testing.T, cmd *cobra.Command) {
|
||||||
|
assert.True(t, flags.FailFastFV, "fail fast flag")
|
||||||
|
assert.True(t, flags.DisableIncrementalsFV, "disable incrementals flag")
|
||||||
|
assert.True(t, flags.ForceItemDataDownloadFV, "force item data download flag")
|
||||||
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user