Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2024-01-19 21:09:33 +0300
committerGitLab Bot <gitlab-bot@gitlab.com>2024-01-19 21:09:33 +0300
commitd1be3e6f776e1c77976537548c1daa9af2fb2650 (patch)
tree387d3c8f06e18bbfa24a4b0b015a7245e166927c /spec/requests
parent8f3a9dbb94b5a9ae4570a22bbc2a75e7572407c8 (diff)
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/requests')
-rw-r--r--spec/requests/api/integrations_spec.rb2
-rw-r--r--spec/requests/projects/ml/models_controller_spec.rb69
2 files changed, 2 insertions, 69 deletions
diff --git a/spec/requests/api/integrations_spec.rb b/spec/requests/api/integrations_spec.rb
index 4696be07045..36667022fc5 100644
--- a/spec/requests/api/integrations_spec.rb
+++ b/spec/requests/api/integrations_spec.rb
@@ -52,7 +52,7 @@ RSpec.describe API::Integrations, feature_category: :integrations do
Integrations::Zentao.to_param
]
- names = Integration.available_integration_names
+ names = Integration.available_integration_names(include_instance_specific: false)
names.reject { |name| name.in?(unavailable_integration_names) }
end
diff --git a/spec/requests/projects/ml/models_controller_spec.rb b/spec/requests/projects/ml/models_controller_spec.rb
index e469ee837bc..8f68d0c8d00 100644
--- a/spec/requests/projects/ml/models_controller_spec.rb
+++ b/spec/requests/projects/ml/models_controller_spec.rb
@@ -6,9 +6,6 @@ RSpec.describe Projects::Ml::ModelsController, feature_category: :mlops do
let_it_be(:project) { create(:project) }
let_it_be(:user) { project.first_owner }
let_it_be(:model1) { create(:ml_models, :with_versions, project: project) }
- let_it_be(:model2) { create(:ml_models, project: project) }
- let_it_be(:model3) { create(:ml_models, project: project) }
- let_it_be(:model_in_different_project) { create(:ml_models) }
let(:read_model_registry) { true }
let(:write_model_registry) { true }
@@ -37,36 +34,6 @@ RSpec.describe Projects::Ml::ModelsController, feature_category: :mlops do
expect(index_request).to render_template('projects/ml/models/index')
end
- it 'fetches the models using the finder' do
- expect(::Projects::Ml::ModelFinder).to receive(:new).with(project, {}).and_call_original
-
- index_request
- end
-
- it 'fetches the correct variables', :aggregate_failures do
- stub_const("Projects::Ml::ModelsController::MAX_MODELS_PER_PAGE", 2)
-
- index_request
-
- page_models = [model3, model2]
- all_models = [model3, model2, model1]
-
- expect(assigns(:paginator).records).to match_array(page_models)
- expect(assigns(:model_count)).to be all_models.count
- end
-
- it 'does not perform N+1 sql queries' do
- list_models
-
- control_count = ActiveRecord::QueryRecorder.new(skip_cached: false) { list_models }
-
- create_list(:ml_model_versions, 2, model: model1)
- create_list(:ml_model_versions, 2, model: model2)
- create_list(:ml_models, 4, project: project)
-
- expect { list_models }.not_to exceed_all_query_limit(control_count)
- end
-
context 'when user does not have access' do
let(:read_model_registry) { false }
@@ -74,40 +41,6 @@ RSpec.describe Projects::Ml::ModelsController, feature_category: :mlops do
is_expected.to have_gitlab_http_status(:not_found)
end
end
-
- context 'with search params' do
- let(:params) { { name: 'some_name', order_by: 'name', sort: 'asc' } }
-
- it 'passes down params to the finder' do
- expect(Projects::Ml::ModelFinder).to receive(:new).and_call_original do |_exp, params|
- expect(params.to_h).to include({
- name: 'some_name',
- order_by: 'name',
- sort: 'asc'
- })
- end
-
- index_request
- end
- end
-
- describe 'pagination' do
- before do
- stub_const("Projects::Ml::ModelsController::MAX_MODELS_PER_PAGE", 2)
- end
-
- it 'paginates', :aggregate_failures do
- list_models
-
- paginator = assigns(:paginator)
-
- expect(paginator.records).to match_array([model3, model2])
-
- list_models({ cursor: paginator.cursor_for_next_page })
-
- expect(assigns(:paginator).records.first).to eq(model1)
- end
- end
end
describe 'show' do
@@ -140,7 +73,7 @@ RSpec.describe Projects::Ml::ModelsController, feature_category: :mlops do
end
context 'when model project does not match project id' do
- let(:request_project) { model_in_different_project.project }
+ let(:request_project) { create(:project) }
it { is_expected.to have_gitlab_http_status(:not_found) }
end