diff options
Diffstat (limited to 'spec/models/namespace/root_storage_statistics_spec.rb')
-rw-r--r-- | spec/models/namespace/root_storage_statistics_spec.rb | 94 |
1 files changed, 78 insertions, 16 deletions
diff --git a/spec/models/namespace/root_storage_statistics_spec.rb b/spec/models/namespace/root_storage_statistics_spec.rb index 51c191069ec..11852828eab 100644 --- a/spec/models/namespace/root_storage_statistics_spec.rb +++ b/spec/models/namespace/root_storage_statistics_spec.rb @@ -28,24 +28,24 @@ RSpec.describe Namespace::RootStorageStatistics, type: :model do let(:project1) { create(:project, namespace: namespace) } let(:project2) { create(:project, namespace: namespace) } - let!(:stat1) { create(:project_statistics, project: project1, with_data: true, size_multiplier: 100) } - let!(:stat2) { create(:project_statistics, project: project2, with_data: true, size_multiplier: 200) } + let!(:project_stat1) { create(:project_statistics, project: project1, with_data: true, size_multiplier: 100) } + let!(:project_stat2) { create(:project_statistics, project: project2, with_data: true, size_multiplier: 200) } - shared_examples 'data refresh' do + shared_examples 'project data refresh' do it 'aggregates project statistics' do root_storage_statistics.recalculate! root_storage_statistics.reload - total_repository_size = stat1.repository_size + stat2.repository_size - total_wiki_size = stat1.wiki_size + stat2.wiki_size - total_lfs_objects_size = stat1.lfs_objects_size + stat2.lfs_objects_size - total_build_artifacts_size = stat1.build_artifacts_size + stat2.build_artifacts_size - total_packages_size = stat1.packages_size + stat2.packages_size - total_storage_size = stat1.storage_size + stat2.storage_size - total_snippets_size = stat1.snippets_size + stat2.snippets_size - total_pipeline_artifacts_size = stat1.pipeline_artifacts_size + stat2.pipeline_artifacts_size - total_uploads_size = stat1.uploads_size + stat2.uploads_size + total_repository_size = project_stat1.repository_size + project_stat2.repository_size + total_wiki_size = project_stat1.wiki_size + project_stat2.wiki_size + total_lfs_objects_size = project_stat1.lfs_objects_size + project_stat2.lfs_objects_size + total_build_artifacts_size = project_stat1.build_artifacts_size + project_stat2.build_artifacts_size + total_packages_size = project_stat1.packages_size + project_stat2.packages_size + total_storage_size = project_stat1.storage_size + project_stat2.storage_size + total_snippets_size = project_stat1.snippets_size + project_stat2.snippets_size + total_pipeline_artifacts_size = project_stat1.pipeline_artifacts_size + project_stat2.pipeline_artifacts_size + total_uploads_size = project_stat1.uploads_size + project_stat2.uploads_size expect(root_storage_statistics.repository_size).to eq(total_repository_size) expect(root_storage_statistics.wiki_size).to eq(total_wiki_size) @@ -83,7 +83,7 @@ RSpec.describe Namespace::RootStorageStatistics, type: :model do end end - it_behaves_like 'data refresh' + it_behaves_like 'project data refresh' it_behaves_like 'does not include personal snippets' context 'with subgroups' do @@ -93,19 +93,81 @@ RSpec.describe Namespace::RootStorageStatistics, type: :model do let(:project1) { create(:project, namespace: subgroup1) } let(:project2) { create(:project, namespace: subgroup2) } - it_behaves_like 'data refresh' + it_behaves_like 'project data refresh' it_behaves_like 'does not include personal snippets' end + context 'with a group namespace' do + let_it_be(:root_group) { create(:group) } + let_it_be(:group1) { create(:group, parent: root_group) } + let_it_be(:subgroup1) { create(:group, parent: group1) } + let_it_be(:group2) { create(:group, parent: root_group) } + let_it_be(:root_namespace_stat) { create(:namespace_statistics, namespace: root_group, storage_size: 100, dependency_proxy_size: 100) } + let_it_be(:group1_namespace_stat) { create(:namespace_statistics, namespace: group1, storage_size: 200, dependency_proxy_size: 200) } + let_it_be(:group2_namespace_stat) { create(:namespace_statistics, namespace: group2, storage_size: 300, dependency_proxy_size: 300) } + let_it_be(:subgroup1_namespace_stat) { create(:namespace_statistics, namespace: subgroup1, storage_size: 300, dependency_proxy_size: 100) } + + let(:namespace) { root_group } + + it 'aggregates namespace statistics' do + # This group is not a descendant of the root_group so it shouldn't be included in the final stats. + other_group = create(:group) + create(:namespace_statistics, namespace: other_group, storage_size: 500, dependency_proxy_size: 500) + + root_storage_statistics.recalculate! + + total_repository_size = project_stat1.repository_size + project_stat2.repository_size + total_lfs_objects_size = project_stat1.lfs_objects_size + project_stat2.lfs_objects_size + total_build_artifacts_size = project_stat1.build_artifacts_size + project_stat2.build_artifacts_size + total_packages_size = project_stat1.packages_size + project_stat2.packages_size + total_snippets_size = project_stat1.snippets_size + project_stat2.snippets_size + total_pipeline_artifacts_size = project_stat1.pipeline_artifacts_size + project_stat2.pipeline_artifacts_size + total_uploads_size = project_stat1.uploads_size + project_stat2.uploads_size + total_wiki_size = project_stat1.wiki_size + project_stat2.wiki_size + total_dependency_proxy_size = root_namespace_stat.dependency_proxy_size + group1_namespace_stat.dependency_proxy_size + group2_namespace_stat.dependency_proxy_size + subgroup1_namespace_stat.dependency_proxy_size + total_storage_size = project_stat1.storage_size + project_stat2.storage_size + root_namespace_stat.storage_size + group1_namespace_stat.storage_size + group2_namespace_stat.storage_size + subgroup1_namespace_stat.storage_size + + expect(root_storage_statistics.repository_size).to eq(total_repository_size) + expect(root_storage_statistics.lfs_objects_size).to eq(total_lfs_objects_size) + expect(root_storage_statistics.build_artifacts_size).to eq(total_build_artifacts_size) + expect(root_storage_statistics.packages_size).to eq(total_packages_size) + expect(root_storage_statistics.snippets_size).to eq(total_snippets_size) + expect(root_storage_statistics.pipeline_artifacts_size).to eq(total_pipeline_artifacts_size) + expect(root_storage_statistics.uploads_size).to eq(total_uploads_size) + expect(root_storage_statistics.dependency_proxy_size).to eq(total_dependency_proxy_size) + expect(root_storage_statistics.wiki_size).to eq(total_wiki_size) + expect(root_storage_statistics.storage_size).to eq(total_storage_size) + end + + it 'works when there are no namespace statistics' do + NamespaceStatistics.delete_all + + root_storage_statistics.recalculate! + + total_storage_size = project_stat1.storage_size + project_stat2.storage_size + + expect(root_storage_statistics.storage_size).to eq(total_storage_size) + end + end + context 'with a personal namespace' do let_it_be(:user) { create(:user) } let(:namespace) { user.namespace } - it_behaves_like 'data refresh' + it_behaves_like 'project data refresh' + + it 'does not aggregate namespace statistics' do + create(:namespace_statistics, namespace: user.namespace, storage_size: 200, dependency_proxy_size: 200) + + root_storage_statistics.recalculate! + + expect(root_storage_statistics.storage_size).to eq(project_stat1.storage_size + project_stat2.storage_size) + expect(root_storage_statistics.dependency_proxy_size).to eq(0) + end context 'when user has personal snippets' do - let(:total_project_snippets_size) { stat1.snippets_size + stat2.snippets_size } + let(:total_project_snippets_size) { project_stat1.snippets_size + project_stat2.snippets_size } it 'aggregates personal and project snippets size' do # This is just a a snippet authored by other user |