Commit 51ad6723 authored by Dmytro Zaporozhets (DZ)'s avatar Dmytro Zaporozhets (DZ)

Merge branch 'id-fix-cloud-platform-warnings' into 'master'

Fix Ruby 2.7 warnings for Cloud Platform

See merge request gitlab-org/gitlab!44456
parents 36e252d1 e36dea90
...@@ -66,7 +66,7 @@ module GoogleApi ...@@ -66,7 +66,7 @@ module GoogleApi
cluster_options = make_cluster_options(cluster_name, cluster_size, machine_type, legacy_abac, enable_addons) cluster_options = make_cluster_options(cluster_name, cluster_size, machine_type, legacy_abac, enable_addons)
request_body = Google::Apis::ContainerV1beta1::CreateClusterRequest.new(cluster_options) request_body = Google::Apis::ContainerV1beta1::CreateClusterRequest.new(**cluster_options)
service.create_cluster(project_id, zone, request_body, options: user_agent_header) service.create_cluster(project_id, zone, request_body, options: user_agent_header)
end end
......
...@@ -83,10 +83,7 @@ RSpec.describe Clusters::Gcp::FinalizeCreationService, '#execute' do ...@@ -83,10 +83,7 @@ RSpec.describe Clusters::Gcp::FinalizeCreationService, '#execute' do
shared_context 'kubernetes information successfully fetched' do shared_context 'kubernetes information successfully fetched' do
before do before do
stub_cloud_platform_get_zone_cluster( stub_cloud_platform_get_zone_cluster(
provider.gcp_project_id, provider.zone, cluster.name, provider.gcp_project_id, provider.zone, cluster.name, { endpoint: endpoint, username: username, password: password }
endpoint: endpoint,
username: username,
password: password
) )
stub_kubeclient_discover(api_url) stub_kubeclient_discover(api_url)
......
...@@ -22,9 +22,9 @@ module GoogleApi ...@@ -22,9 +22,9 @@ module GoogleApi
.to_return(cloud_platform_response(cloud_platform_projects_billing_info_body(project_id, billing_enabled))) .to_return(cloud_platform_response(cloud_platform_projects_billing_info_body(project_id, billing_enabled)))
end end
def stub_cloud_platform_get_zone_cluster(project_id, zone, cluster_id, **options) def stub_cloud_platform_get_zone_cluster(project_id, zone, cluster_id, options = {})
WebMock.stub_request(:get, cloud_platform_get_zone_cluster_url(project_id, zone, cluster_id)) WebMock.stub_request(:get, cloud_platform_get_zone_cluster_url(project_id, zone, cluster_id))
.to_return(cloud_platform_response(cloud_platform_cluster_body(**options))) .to_return(cloud_platform_response(cloud_platform_cluster_body(options)))
end end
def stub_cloud_platform_get_zone_cluster_error(project_id, zone, cluster_id) def stub_cloud_platform_get_zone_cluster_error(project_id, zone, cluster_id)
...@@ -32,7 +32,7 @@ module GoogleApi ...@@ -32,7 +32,7 @@ module GoogleApi
.to_return(status: [500, "Internal Server Error"]) .to_return(status: [500, "Internal Server Error"])
end end
def stub_cloud_platform_create_cluster(project_id, zone, **options) def stub_cloud_platform_create_cluster(project_id, zone, options = {})
WebMock.stub_request(:post, cloud_platform_create_cluster_url(project_id, zone)) WebMock.stub_request(:post, cloud_platform_create_cluster_url(project_id, zone))
.to_return(cloud_platform_response(cloud_platform_operation_body(options))) .to_return(cloud_platform_response(cloud_platform_operation_body(options)))
end end
...@@ -42,7 +42,7 @@ module GoogleApi ...@@ -42,7 +42,7 @@ module GoogleApi
.to_return(status: [500, "Internal Server Error"]) .to_return(status: [500, "Internal Server Error"])
end end
def stub_cloud_platform_get_zone_operation(project_id, zone, operation_id, **options) def stub_cloud_platform_get_zone_operation(project_id, zone, operation_id, options = {})
WebMock.stub_request(:get, cloud_platform_get_zone_operation_url(project_id, zone, operation_id)) WebMock.stub_request(:get, cloud_platform_get_zone_operation_url(project_id, zone, operation_id))
.to_return(cloud_platform_response(cloud_platform_operation_body(options))) .to_return(cloud_platform_response(cloud_platform_operation_body(options)))
end end
...@@ -86,7 +86,7 @@ module GoogleApi ...@@ -86,7 +86,7 @@ module GoogleApi
# https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1/projects.zones.clusters/create # https://cloud.google.com/kubernetes-engine/docs/reference/rest/v1/projects.zones.clusters/create
# rubocop:disable Metrics/CyclomaticComplexity # rubocop:disable Metrics/CyclomaticComplexity
# rubocop:disable Metrics/PerceivedComplexity # rubocop:disable Metrics/PerceivedComplexity
def cloud_platform_cluster_body(**options) def cloud_platform_cluster_body(options)
{ {
"name": options[:name] || 'string', "name": options[:name] || 'string',
"description": options[:description] || 'string', "description": options[:description] || 'string',
...@@ -121,7 +121,7 @@ module GoogleApi ...@@ -121,7 +121,7 @@ module GoogleApi
} }
end end
def cloud_platform_operation_body(**options) def cloud_platform_operation_body(options)
{ {
"name": options[:name] || 'operation-1234567891234-1234567', "name": options[:name] || 'operation-1234567891234-1234567',
"zone": options[:zone] || 'us-central1-a', "zone": options[:zone] || 'us-central1-a',
...@@ -136,7 +136,7 @@ module GoogleApi ...@@ -136,7 +136,7 @@ module GoogleApi
} }
end end
def cloud_platform_projects_body(**options) def cloud_platform_projects_body(options)
{ {
"projects": [ "projects": [
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment