diff options
Diffstat (limited to 'spec/graphql/resolvers/ci')
7 files changed, 83 insertions, 13 deletions
diff --git a/spec/graphql/resolvers/ci/config_resolver_spec.rb b/spec/graphql/resolvers/ci/config_resolver_spec.rb index dc030b1313b..692bdf58784 100644 --- a/spec/graphql/resolvers/ci/config_resolver_spec.rb +++ b/spec/graphql/resolvers/ci/config_resolver_spec.rb @@ -17,7 +17,7 @@ RSpec.describe Resolvers::Ci::ConfigResolver do subject(:response) do resolve(described_class, args: { project_path: project.full_path, content: content, sha: sha }, - ctx: { current_user: user }) + ctx: { current_user: user }) end shared_examples 'a valid config file' do diff --git a/spec/graphql/resolvers/ci/group_runners_resolver_spec.rb b/spec/graphql/resolvers/ci/group_runners_resolver_spec.rb index f99f48f5b07..57b2fcbea63 100644 --- a/spec/graphql/resolvers/ci/group_runners_resolver_spec.rb +++ b/spec/graphql/resolvers/ci/group_runners_resolver_spec.rb @@ -8,7 +8,7 @@ RSpec.describe Resolvers::Ci::GroupRunnersResolver do describe '#resolve' do subject do resolve(described_class, obj: obj, ctx: { current_user: user }, args: args, - arg_style: :internal) + arg_style: :internal) end include_context 'runners resolver setup' diff --git a/spec/graphql/resolvers/ci/job_token_scope_resolver_spec.rb b/spec/graphql/resolvers/ci/job_token_scope_resolver_spec.rb index ac7cef20df4..1bfd6fbf6b9 100644 --- a/spec/graphql/resolvers/ci/job_token_scope_resolver_spec.rb +++ b/spec/graphql/resolvers/ci/job_token_scope_resolver_spec.rb @@ -20,10 +20,10 @@ RSpec.describe Resolvers::Ci::JobTokenScopeResolver do project.add_member(current_user, :maintainer) end - it 'returns nil when scope is not enabled' do + it 'returns the same project in the allow list of projects for the Ci Job Token when scope is not enabled' do allow(project).to receive(:ci_job_token_scope_enabled?).and_return(false) - expect(resolve_scope).to eq(nil) + expect(resolve_scope.all_projects).to contain_exactly(project) end it 'returns the same project in the allow list of projects for the Ci Job Token' do @@ -43,8 +43,8 @@ RSpec.describe Resolvers::Ci::JobTokenScopeResolver do project.update!(ci_job_token_scope_enabled: false) end - it 'returns nil' do - expect(resolve_scope).to be_nil + it 'resolves projects' do + expect(resolve_scope.all_projects).to contain_exactly(project) end end end diff --git a/spec/graphql/resolvers/ci/jobs_resolver_spec.rb b/spec/graphql/resolvers/ci/jobs_resolver_spec.rb index 6c228861ddf..80a70938dc4 100644 --- a/spec/graphql/resolvers/ci/jobs_resolver_spec.rb +++ b/spec/graphql/resolvers/ci/jobs_resolver_spec.rb @@ -38,7 +38,7 @@ RSpec.describe Resolvers::Ci::JobsResolver do ::Types::Security::ReportTypeEnum.values['DAST'].value ] jobs = resolve(described_class, obj: pipeline, args: { security_report_types: report_types }, - arg_style: :internal) + arg_style: :internal) expect(jobs).to contain_exactly( have_attributes(name: 'DAST job'), diff --git a/spec/graphql/resolvers/ci/runner_projects_resolver_spec.rb b/spec/graphql/resolvers/ci/runner_projects_resolver_spec.rb new file mode 100644 index 00000000000..952c7337d65 --- /dev/null +++ b/spec/graphql/resolvers/ci/runner_projects_resolver_spec.rb @@ -0,0 +1,69 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Resolvers::Ci::RunnerProjectsResolver do + include GraphqlHelpers + + let_it_be(:project1) { create(:project, description: 'Project1.1') } + let_it_be(:project2) { create(:project, description: 'Project1.2') } + let_it_be(:project3) { create(:project, description: 'Project2.1') } + let_it_be(:runner) { create(:ci_runner, :project, projects: [project1, project2, project3]) } + + let(:args) { {} } + + subject { resolve_projects(args) } + + describe '#resolve' do + context 'with authorized user', :enable_admin_mode do + let(:current_user) { create(:user, :admin) } + + context 'with search argument' do + let(:args) { { search: 'Project1.' } } + + it 'returns a lazy value with projects containing the specified prefix' do + expect(subject).to be_a(GraphQL::Execution::Lazy) + expect(subject.value).to contain_exactly(project1, project2) + end + end + + context 'with supported arguments' do + let(:args) { { membership: true, search_namespaces: true, topics: %w[xyz] } } + + it 'creates ProjectsFinder with expected arguments' do + expect(ProjectsFinder).to receive(:new).with( + a_hash_including( + params: a_hash_including( + non_public: true, + search_namespaces: true, + topic: %w[xyz] + ) + ) + ).and_call_original + + expect(subject).to be_a(GraphQL::Execution::Lazy) + subject.value + end + end + + context 'without arguments' do + it 'returns a lazy value with all projects' do + expect(subject).to be_a(GraphQL::Execution::Lazy) + expect(subject.value).to contain_exactly(project1, project2, project3) + end + end + end + + context 'with unauthorized user' do + let(:current_user) { create(:user) } + + it { is_expected.to be_nil } + end + end + + private + + def resolve_projects(args = {}, context = { current_user: current_user }) + resolve(described_class, obj: runner, args: args, ctx: context) + end +end diff --git a/spec/graphql/resolvers/ci/runners_resolver_spec.rb b/spec/graphql/resolvers/ci/runners_resolver_spec.rb index 8586d359336..4038192a68a 100644 --- a/spec/graphql/resolvers/ci/runners_resolver_spec.rb +++ b/spec/graphql/resolvers/ci/runners_resolver_spec.rb @@ -11,7 +11,7 @@ RSpec.describe Resolvers::Ci::RunnersResolver do subject do resolve(described_class, obj: obj, ctx: { current_user: user }, args: args, - arg_style: :internal) + arg_style: :internal) end include_context 'runners resolver setup' diff --git a/spec/graphql/resolvers/ci/test_suite_resolver_spec.rb b/spec/graphql/resolvers/ci/test_suite_resolver_spec.rb index 606c6eb03a3..4083e77a38f 100644 --- a/spec/graphql/resolvers/ci/test_suite_resolver_spec.rb +++ b/spec/graphql/resolvers/ci/test_suite_resolver_spec.rb @@ -32,11 +32,12 @@ RSpec.describe Resolvers::Ci::TestSuiteResolver do # Each test failure in this pipeline has a matching failure in the default branch recent_failures = test_suite[:test_cases].map { |tc| tc[:recent_failures] } - expect(recent_failures).to eq([ - { count: 1, base_branch: 'master' }, - { count: 1, base_branch: 'master' }, - { count: 1, base_branch: 'master' } - ]) + expect(recent_failures).to eq( + [ + { count: 1, base_branch: 'master' }, + { count: 1, base_branch: 'master' }, + { count: 1, base_branch: 'master' } + ]) end end |