Skip to content

Commit a2798c6

Browse files
committed
[API] Addresses minimal RuboCop offenses in generated code
1 parent 1e5b576 commit a2798c6

File tree

488 files changed

+1282
-1659
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

488 files changed

+1282
-1659
lines changed

elasticsearch-api/lib/elasticsearch/api/actions/async_search/delete.rb

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,10 @@ module Actions
3030
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
3131
#
3232
def delete(arguments = {})
33-
request_opts = { endpoint: arguments[:endpoint] || "async_search.delete" }
33+
request_opts = { endpoint: arguments[:endpoint] || 'async_search.delete' }
3434

35-
defined_params = [:id].inject({}) do |set_variables, variable|
35+
defined_params = [:id].each_with_object({}) do |variable, set_variables|
3636
set_variables[variable] = arguments[variable] if arguments.key?(variable)
37-
set_variables
3837
end
3938
request_opts[:defined_params] = defined_params unless defined_params.empty?
4039

elasticsearch-api/lib/elasticsearch/api/actions/async_search/get.rb

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,10 @@ module Actions
3333
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
3434
#
3535
def get(arguments = {})
36-
request_opts = { endpoint: arguments[:endpoint] || "async_search.get" }
36+
request_opts = { endpoint: arguments[:endpoint] || 'async_search.get' }
3737

38-
defined_params = [:id].inject({}) do |set_variables, variable|
38+
defined_params = [:id].each_with_object({}) do |variable, set_variables|
3939
set_variables[variable] = arguments[variable] if arguments.key?(variable)
40-
set_variables
4140
end
4241
request_opts[:defined_params] = defined_params unless defined_params.empty?
4342

elasticsearch-api/lib/elasticsearch/api/actions/async_search/status.rb

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,10 @@ module Actions
3131
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
3232
#
3333
def status(arguments = {})
34-
request_opts = { endpoint: arguments[:endpoint] || "async_search.status" }
34+
request_opts = { endpoint: arguments[:endpoint] || 'async_search.status' }
3535

36-
defined_params = [:id].inject({}) do |set_variables, variable|
36+
defined_params = [:id].each_with_object({}) do |variable, set_variables|
3737
set_variables[variable] = arguments[variable] if arguments.key?(variable)
38-
set_variables
3938
end
4039
request_opts[:defined_params] = defined_params unless defined_params.empty?
4140

elasticsearch-api/lib/elasticsearch/api/actions/async_search/submit.rb

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -72,11 +72,10 @@ module Actions
7272
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/async-search.html
7373
#
7474
def submit(arguments = {})
75-
request_opts = { endpoint: arguments[:endpoint] || "async_search.submit" }
75+
request_opts = { endpoint: arguments[:endpoint] || 'async_search.submit' }
7676

77-
defined_params = [:index].inject({}) do |set_variables, variable|
77+
defined_params = [:index].each_with_object({}) do |variable, set_variables|
7878
set_variables[variable] = arguments[variable] if arguments.key?(variable)
79-
set_variables
8079
end
8180
request_opts[:defined_params] = defined_params unless defined_params.empty?
8281

@@ -91,7 +90,7 @@ def submit(arguments = {})
9190
path = if _index
9291
"#{Utils.__listify(_index)}/_async_search"
9392
else
94-
"_async_search"
93+
'_async_search'
9594
end
9695
params = Utils.process_params(arguments)
9796

elasticsearch-api/lib/elasticsearch/api/actions/autoscaling/delete_autoscaling_policy.rb

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,10 @@ module Actions
3030
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-delete-autoscaling-policy.html
3131
#
3232
def delete_autoscaling_policy(arguments = {})
33-
request_opts = { endpoint: arguments[:endpoint] || "autoscaling.delete_autoscaling_policy" }
33+
request_opts = { endpoint: arguments[:endpoint] || 'autoscaling.delete_autoscaling_policy' }
3434

35-
defined_params = [:name].inject({}) do |set_variables, variable|
35+
defined_params = [:name].each_with_object({}) do |variable, set_variables|
3636
set_variables[variable] = arguments[variable] if arguments.key?(variable)
37-
set_variables
3837
end
3938
request_opts[:defined_params] = defined_params unless defined_params.empty?
4039

elasticsearch-api/lib/elasticsearch/api/actions/autoscaling/get_autoscaling_capacity.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,15 +29,15 @@ module Actions
2929
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-get-autoscaling-capacity.html
3030
#
3131
def get_autoscaling_capacity(arguments = {})
32-
request_opts = { endpoint: arguments[:endpoint] || "autoscaling.get_autoscaling_capacity" }
32+
request_opts = { endpoint: arguments[:endpoint] || 'autoscaling.get_autoscaling_capacity' }
3333

3434
arguments = arguments.clone
3535
headers = arguments.delete(:headers) || {}
3636

3737
body = nil
3838

3939
method = Elasticsearch::API::HTTP_GET
40-
path = "_autoscaling/capacity"
40+
path = '_autoscaling/capacity'
4141
params = {}
4242

4343
Elasticsearch::API::Response.new(

elasticsearch-api/lib/elasticsearch/api/actions/autoscaling/get_autoscaling_policy.rb

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,10 @@ module Actions
3030
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-get-autoscaling-policy.html
3131
#
3232
def get_autoscaling_policy(arguments = {})
33-
request_opts = { endpoint: arguments[:endpoint] || "autoscaling.get_autoscaling_policy" }
33+
request_opts = { endpoint: arguments[:endpoint] || 'autoscaling.get_autoscaling_policy' }
3434

35-
defined_params = [:name].inject({}) do |set_variables, variable|
35+
defined_params = [:name].each_with_object({}) do |variable, set_variables|
3636
set_variables[variable] = arguments[variable] if arguments.key?(variable)
37-
set_variables
3837
end
3938
request_opts[:defined_params] = defined_params unless defined_params.empty?
4039

elasticsearch-api/lib/elasticsearch/api/actions/autoscaling/put_autoscaling_policy.rb

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,10 @@ module Actions
3131
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/autoscaling-put-autoscaling-policy.html
3232
#
3333
def put_autoscaling_policy(arguments = {})
34-
request_opts = { endpoint: arguments[:endpoint] || "autoscaling.put_autoscaling_policy" }
34+
request_opts = { endpoint: arguments[:endpoint] || 'autoscaling.put_autoscaling_policy' }
3535

36-
defined_params = [:name].inject({}) do |set_variables, variable|
36+
defined_params = [:name].each_with_object({}) do |variable, set_variables|
3737
set_variables[variable] = arguments[variable] if arguments.key?(variable)
38-
set_variables
3938
end
4039
request_opts[:defined_params] = defined_params unless defined_params.empty?
4140

elasticsearch-api/lib/elasticsearch/api/actions/bulk.rb

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,10 @@ module Actions
4343
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-bulk.html
4444
#
4545
def bulk(arguments = {})
46-
request_opts = { endpoint: arguments[:endpoint] || "bulk" }
46+
request_opts = { endpoint: arguments[:endpoint] || 'bulk' }
4747

48-
defined_params = [:index].inject({}) do |set_variables, variable|
48+
defined_params = [:index].each_with_object({}) do |variable, set_variables|
4949
set_variables[variable] = arguments[variable] if arguments.key?(variable)
50-
set_variables
5150
end
5251
request_opts[:defined_params] = defined_params unless defined_params.empty?
5352

@@ -64,17 +63,17 @@ def bulk(arguments = {})
6463
path = if _index
6564
"#{Utils.__listify(_index)}/_bulk"
6665
else
67-
"_bulk"
66+
'_bulk'
6867
end
6968
params = Utils.process_params(arguments)
7069

71-
if body.is_a? Array
72-
payload = Elasticsearch::API::Utils.__bulkify(body)
73-
else
74-
payload = body
75-
end
70+
payload = if body.is_a? Array
71+
Elasticsearch::API::Utils.__bulkify(body)
72+
else
73+
body
74+
end
7675

77-
headers.merge!("Content-Type" => "application/x-ndjson")
76+
headers.merge!('Content-Type' => 'application/x-ndjson')
7877
Elasticsearch::API::Response.new(
7978
perform_request(method, path, params, payload, headers, request_opts)
8079
)

elasticsearch-api/lib/elasticsearch/api/actions/cat/aliases.rb

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,10 @@ module Actions
3737
# @see https://www.elastic.co/guide/en/elasticsearch/reference/current/cat-alias.html
3838
#
3939
def aliases(arguments = {})
40-
request_opts = { endpoint: arguments[:endpoint] || "cat.aliases" }
40+
request_opts = { endpoint: arguments[:endpoint] || 'cat.aliases' }
4141

42-
defined_params = [:name].inject({}) do |set_variables, variable|
42+
defined_params = [:name].each_with_object({}) do |variable, set_variables|
4343
set_variables[variable] = arguments[variable] if arguments.key?(variable)
44-
set_variables
4544
end
4645
request_opts[:defined_params] = defined_params unless defined_params.empty?
4746

@@ -56,7 +55,7 @@ def aliases(arguments = {})
5655
path = if _name
5756
"_cat/aliases/#{Utils.__listify(_name)}"
5857
else
59-
"_cat/aliases"
58+
'_cat/aliases'
6059
end
6160
params = Utils.process_params(arguments)
6261
params[:h] = Utils.__listify(params[:h]) if params[:h]

0 commit comments

Comments
 (0)