Add comment to pages_access_levelgeorgekoltsov/13698-override-params

1 files changed, 14 insertions, 11 deletions

diff --git a/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb b/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb
index 37f28ce39a0..0aef4887c75 100644
--- a/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb
+++ b/spec/lib/gitlab/import_export/project_tree_restorer_spec.rb
@@ -397,20 +397,23 @@ describe Gitlab::ImportExport::ProjectTreeRestorer do
         expect(project.lfs_enabled).to be_falsey
       end
 
-      context 'when overriding feature access levels' do
-        it 'overrides project feature access levels' do
-          access_level_keys = project.project_feature.attributes.keys.select { |a| a =~ /_access_level/ }
-          access_level_keys.delete('pages_access_level')
-          disabled_access_levels = Hash[access_level_keys.collect { |item| [item, 'disabled'] }]
+      it 'overrides project feature access levels' do
+        access_level_keys = project.project_feature.attributes.keys.select { |a| a =~ /_access_level/ }
 
-          project.create_import_data(data: { override_params: disabled_access_levels })
+        # `pages_access_level` is not included, since it is not available in the public API
+        # and has a dependency on project's visibility level
+        # see ProjectFeature model
+        access_level_keys.delete('pages_access_level')
 
-          restored_project_json
+        disabled_access_levels = Hash[access_level_keys.collect { |item| [item, 'disabled'] }]
 
-          aggregate_failures do
-            access_level_keys.each do |key|
-              expect(project.public_send(key)).to eq(ProjectFeature::DISABLED)
-            end
+        project.create_import_data(data: { override_params: disabled_access_levels })
+
+        restored_project_json
+
+        aggregate_failures do
+          access_level_keys.each do |key|
+            expect(project.public_send(key)).to eq(ProjectFeature::DISABLED)
           end
         end
       end