diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2024-01-19 21:09:33 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2024-01-19 21:09:33 +0300 |
commit | d1be3e6f776e1c77976537548c1daa9af2fb2650 (patch) | |
tree | 387d3c8f06e18bbfa24a4b0b015a7245e166927c /spec/requests | |
parent | 8f3a9dbb94b5a9ae4570a22bbc2a75e7572407c8 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/requests')
-rw-r--r-- | spec/requests/api/integrations_spec.rb | 2 | ||||
-rw-r--r-- | spec/requests/projects/ml/models_controller_spec.rb | 69 |
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 |