123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414 |
- // Copyright 2020 The Gitea Authors. All rights reserved.
- // SPDX-License-Identifier: MIT
-
- package setting
-
- import (
- "path/filepath"
- "testing"
-
- "github.com/stretchr/testify/assert"
- )
-
- func Test_getStorageMultipleName(t *testing.T) {
- iniStr := `
- [lfs]
- MINIO_BUCKET = gitea-lfs
-
- [attachment]
- MINIO_BUCKET = gitea-attachment
-
- [storage]
- STORAGE_TYPE = minio
- MINIO_BUCKET = gitea-storage
- `
- cfg, err := NewConfigProviderFromData(iniStr)
- assert.NoError(t, err)
-
- assert.NoError(t, loadAttachmentFrom(cfg))
- assert.EqualValues(t, "gitea-attachment", Attachment.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "attachments/", Attachment.Storage.MinioConfig.BasePath)
-
- assert.NoError(t, loadLFSFrom(cfg))
- assert.EqualValues(t, "gitea-lfs", LFS.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "lfs/", LFS.Storage.MinioConfig.BasePath)
-
- assert.NoError(t, loadAvatarsFrom(cfg))
- assert.EqualValues(t, "gitea-storage", Avatar.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "avatars/", Avatar.Storage.MinioConfig.BasePath)
- }
-
- func Test_getStorageUseOtherNameAsType(t *testing.T) {
- iniStr := `
- [attachment]
- STORAGE_TYPE = lfs
-
- [storage.lfs]
- STORAGE_TYPE = minio
- MINIO_BUCKET = gitea-storage
- `
- cfg, err := NewConfigProviderFromData(iniStr)
- assert.NoError(t, err)
-
- assert.NoError(t, loadAttachmentFrom(cfg))
- assert.EqualValues(t, "gitea-storage", Attachment.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "attachments/", Attachment.Storage.MinioConfig.BasePath)
-
- assert.NoError(t, loadLFSFrom(cfg))
- assert.EqualValues(t, "gitea-storage", LFS.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "lfs/", LFS.Storage.MinioConfig.BasePath)
- }
-
- func Test_getStorageInheritStorageType(t *testing.T) {
- iniStr := `
- [storage]
- STORAGE_TYPE = minio
- `
- cfg, err := NewConfigProviderFromData(iniStr)
- assert.NoError(t, err)
-
- assert.NoError(t, loadPackagesFrom(cfg))
- assert.EqualValues(t, "minio", Packages.Storage.Type)
- assert.EqualValues(t, "gitea", Packages.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "packages/", Packages.Storage.MinioConfig.BasePath)
-
- assert.NoError(t, loadRepoArchiveFrom(cfg))
- assert.EqualValues(t, "minio", RepoArchive.Storage.Type)
- assert.EqualValues(t, "gitea", RepoArchive.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "repo-archive/", RepoArchive.Storage.MinioConfig.BasePath)
-
- assert.NoError(t, loadActionsFrom(cfg))
- assert.EqualValues(t, "minio", Actions.LogStorage.Type)
- assert.EqualValues(t, "gitea", Actions.LogStorage.MinioConfig.Bucket)
- assert.EqualValues(t, "actions_log/", Actions.LogStorage.MinioConfig.BasePath)
-
- assert.EqualValues(t, "minio", Actions.ArtifactStorage.Type)
- assert.EqualValues(t, "gitea", Actions.ArtifactStorage.MinioConfig.Bucket)
- assert.EqualValues(t, "actions_artifacts/", Actions.ArtifactStorage.MinioConfig.BasePath)
-
- assert.NoError(t, loadAvatarsFrom(cfg))
- assert.EqualValues(t, "minio", Avatar.Storage.Type)
- assert.EqualValues(t, "gitea", Avatar.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "avatars/", Avatar.Storage.MinioConfig.BasePath)
-
- assert.NoError(t, loadRepoAvatarFrom(cfg))
- assert.EqualValues(t, "minio", RepoAvatar.Storage.Type)
- assert.EqualValues(t, "gitea", RepoAvatar.Storage.MinioConfig.Bucket)
- assert.EqualValues(t, "repo-avatars/", RepoAvatar.Storage.MinioConfig.BasePath)
- }
-
- type testLocalStoragePathCase struct {
- loader func(rootCfg ConfigProvider) error
- storagePtr **Storage
- expectedPath string
- }
-
- func testLocalStoragePath(t *testing.T, appDataPath, iniStr string, cases []testLocalStoragePathCase) {
- cfg, err := NewConfigProviderFromData(iniStr)
- assert.NoError(t, err)
- AppDataPath = appDataPath
- for _, c := range cases {
- assert.NoError(t, c.loader(cfg))
- storage := *c.storagePtr
-
- assert.EqualValues(t, "local", storage.Type)
- assert.True(t, filepath.IsAbs(storage.Path))
- assert.EqualValues(t, filepath.Clean(c.expectedPath), filepath.Clean(storage.Path))
- }
- }
-
- func Test_getStorageInheritStorageTypeLocal(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage]
- STORAGE_TYPE = local
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/appdata/attachments"},
- {loadLFSFrom, &LFS.Storage, "/appdata/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/appdata/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/appdata/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/appdata/repo-archive"},
- {loadActionsFrom, &Actions.LogStorage, "/appdata/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/appdata/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/appdata/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPath(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage]
- STORAGE_TYPE = local
- PATH = /data/gitea
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/data/gitea/attachments"},
- {loadLFSFrom, &LFS.Storage, "/data/gitea/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/data/gitea/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/data/gitea/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/data/gitea/repo-archive"},
- {loadActionsFrom, &Actions.LogStorage, "/data/gitea/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/data/gitea/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/data/gitea/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalRelativePath(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage]
- STORAGE_TYPE = local
- PATH = storages
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/appdata/storages/attachments"},
- {loadLFSFrom, &LFS.Storage, "/appdata/storages/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/appdata/storages/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/appdata/storages/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/appdata/storages/repo-archive"},
- {loadActionsFrom, &Actions.LogStorage, "/appdata/storages/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/appdata/storages/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/appdata/storages/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPathOverride(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage]
- STORAGE_TYPE = local
- PATH = /data/gitea
-
- [repo-archive]
- PATH = /data/gitea/the-archives-dir
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/data/gitea/attachments"},
- {loadLFSFrom, &LFS.Storage, "/data/gitea/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/data/gitea/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/data/gitea/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/data/gitea/the-archives-dir"},
- {loadActionsFrom, &Actions.LogStorage, "/data/gitea/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/data/gitea/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/data/gitea/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPathOverrideEmpty(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage]
- STORAGE_TYPE = local
- PATH = /data/gitea
-
- [repo-archive]
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/data/gitea/attachments"},
- {loadLFSFrom, &LFS.Storage, "/data/gitea/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/data/gitea/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/data/gitea/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/data/gitea/repo-archive"},
- {loadActionsFrom, &Actions.LogStorage, "/data/gitea/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/data/gitea/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/data/gitea/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalRelativePathOverride(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage]
- STORAGE_TYPE = local
- PATH = /data/gitea
-
- [repo-archive]
- PATH = the-archives-dir
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/data/gitea/attachments"},
- {loadLFSFrom, &LFS.Storage, "/data/gitea/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/data/gitea/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/data/gitea/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/data/gitea/the-archives-dir"},
- {loadActionsFrom, &Actions.LogStorage, "/data/gitea/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/data/gitea/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/data/gitea/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPathOverride3(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage.repo-archive]
- STORAGE_TYPE = local
- PATH = /data/gitea/archives
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/appdata/attachments"},
- {loadLFSFrom, &LFS.Storage, "/appdata/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/appdata/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/appdata/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/data/gitea/archives"},
- {loadActionsFrom, &Actions.LogStorage, "/appdata/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/appdata/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/appdata/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPathOverride3_5(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage.repo-archive]
- STORAGE_TYPE = local
- PATH = a-relative-path
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/appdata/attachments"},
- {loadLFSFrom, &LFS.Storage, "/appdata/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/appdata/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/appdata/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/appdata/a-relative-path"},
- {loadActionsFrom, &Actions.LogStorage, "/appdata/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/appdata/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/appdata/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPathOverride4(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage.repo-archive]
- STORAGE_TYPE = local
- PATH = /data/gitea/archives
-
- [repo-archive]
- PATH = /tmp/gitea/archives
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/appdata/attachments"},
- {loadLFSFrom, &LFS.Storage, "/appdata/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/appdata/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/appdata/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/tmp/gitea/archives"},
- {loadActionsFrom, &Actions.LogStorage, "/appdata/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/appdata/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/appdata/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPathOverride5(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage.repo-archive]
- STORAGE_TYPE = local
- PATH = /data/gitea/archives
-
- [repo-archive]
- `, []testLocalStoragePathCase{
- {loadAttachmentFrom, &Attachment.Storage, "/appdata/attachments"},
- {loadLFSFrom, &LFS.Storage, "/appdata/lfs"},
- {loadActionsFrom, &Actions.ArtifactStorage, "/appdata/actions_artifacts"},
- {loadPackagesFrom, &Packages.Storage, "/appdata/packages"},
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/data/gitea/archives"},
- {loadActionsFrom, &Actions.LogStorage, "/appdata/actions_log"},
- {loadAvatarsFrom, &Avatar.Storage, "/appdata/avatars"},
- {loadRepoAvatarFrom, &RepoAvatar.Storage, "/appdata/repo-avatars"},
- })
- }
-
- func Test_getStorageInheritStorageTypeLocalPathOverride72(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [repo-archive]
- STORAGE_TYPE = local
- PATH = archives
- `, []testLocalStoragePathCase{
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/appdata/archives"},
- })
- }
-
- func Test_getStorageConfiguration20(t *testing.T) {
- cfg, err := NewConfigProviderFromData(`
- [repo-archive]
- STORAGE_TYPE = my_storage
- PATH = archives
- `)
- assert.NoError(t, err)
-
- assert.Error(t, loadRepoArchiveFrom(cfg))
- }
-
- func Test_getStorageConfiguration21(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage.repo-archive]
- `, []testLocalStoragePathCase{
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/appdata/repo-archive"},
- })
- }
-
- func Test_getStorageConfiguration22(t *testing.T) {
- testLocalStoragePath(t, "/appdata", `
- [storage.repo-archive]
- PATH = archives
- `, []testLocalStoragePathCase{
- {loadRepoArchiveFrom, &RepoArchive.Storage, "/appdata/archives"},
- })
- }
-
- func Test_getStorageConfiguration23(t *testing.T) {
- cfg, err := NewConfigProviderFromData(`
- [repo-archive]
- STORAGE_TYPE = minio
- MINIO_ACCESS_KEY_ID = my_access_key
- MINIO_SECRET_ACCESS_KEY = my_secret_key
- `)
- assert.NoError(t, err)
-
- _, err = getStorage(cfg, "", "", nil)
- assert.Error(t, err)
-
- assert.NoError(t, loadRepoArchiveFrom(cfg))
- cp := RepoArchive.Storage.ToShadowCopy()
- assert.EqualValues(t, "******", cp.MinioConfig.AccessKeyID)
- assert.EqualValues(t, "******", cp.MinioConfig.SecretAccessKey)
- }
-
- func Test_getStorageConfiguration24(t *testing.T) {
- cfg, err := NewConfigProviderFromData(`
- [repo-archive]
- STORAGE_TYPE = my_archive
-
- [storage.my_archive]
- ; unsupported, storage type should be defined explicitly
- PATH = archives
- `)
- assert.NoError(t, err)
- assert.Error(t, loadRepoArchiveFrom(cfg))
- }
-
- func Test_getStorageConfiguration25(t *testing.T) {
- cfg, err := NewConfigProviderFromData(`
- [repo-archive]
- STORAGE_TYPE = my_archive
-
- [storage.my_archive]
- ; unsupported, storage type should be known type
- STORAGE_TYPE = unknown // should be local or minio
- PATH = archives
- `)
- assert.NoError(t, err)
- assert.Error(t, loadRepoArchiveFrom(cfg))
- }
-
- func Test_getStorageConfiguration26(t *testing.T) {
- cfg, err := NewConfigProviderFromData(`
- [repo-archive]
- STORAGE_TYPE = minio
- MINIO_ACCESS_KEY_ID = my_access_key
- MINIO_SECRET_ACCESS_KEY = my_secret_key
- ; wrong configuration
- MINIO_USE_SSL = abc
- `)
- assert.NoError(t, err)
- // assert.Error(t, loadRepoArchiveFrom(cfg))
- // FIXME: this should return error but now ini package's MapTo() doesn't check type
- assert.NoError(t, loadRepoArchiveFrom(cfg))
- }
-
- func Test_getStorageConfiguration27(t *testing.T) {
- cfg, err := NewConfigProviderFromData(`
- [storage.repo-archive]
- STORAGE_TYPE = minio
- MINIO_ACCESS_KEY_ID = my_access_key
- MINIO_SECRET_ACCESS_KEY = my_secret_key
- MINIO_USE_SSL = true
- `)
- assert.NoError(t, err)
- assert.NoError(t, loadRepoArchiveFrom(cfg))
- assert.EqualValues(t, "my_access_key", RepoArchive.Storage.MinioConfig.AccessKeyID)
- assert.EqualValues(t, "my_secret_key", RepoArchive.Storage.MinioConfig.SecretAccessKey)
- assert.EqualValues(t, true, RepoArchive.Storage.MinioConfig.UseSSL)
- assert.EqualValues(t, "repo-archive/", RepoArchive.Storage.MinioConfig.BasePath)
- }
|