awspec 1.25.1 → 1.25.2

data/lib/awspec/stub/nlb.rb

@@ -1,4 +1,6 @@
-# rubocop:disable Metrics/LineLength
+# frozen_string_literal: true
+
+# rubocop:disable Layout/LineLength
 Aws.config[:elasticloadbalancingv2] = {
   stub_responses: {
     describe_load_balancers: {

data/lib/awspec/stub/nlb_listener.rb

@@ -1,4 +1,6 @@
-# rubocop:disable Metrics/LineLength
+# frozen_string_literal: true
+
+# rubocop:disable Layout/LineLength
 Aws.config[:elasticloadbalancingv2] = {
   stub_responses: {
     describe_load_balancers: {
@@ -85,8 +87,7 @@ Aws.config[:elasticloadbalancingv2] = {
               type: 'forward'
             }
           ],
-          conditions: [
-          ],
+          conditions: [],
           is_default: true,
           priority: 'default',
           rule_arn: 'arn:aws:elasticloadbalancing:ap-northeast-1:1234567890:listener-rule/app/my-nlb/1aa1bb1cc1ddee11/f2f7dc8efc522ab2/defaaaaaaaultbbbb'

data/lib/awspec/stub/nlb_target_group.rb

@@ -1,4 +1,6 @@
-# rubocop:disable Metrics/LineLength
+# frozen_string_literal: true
+
+# rubocop:disable Layout/LineLength
 Aws.config[:elasticloadbalancingv2] = {
   stub_responses: {
     describe_load_balancers: {

data/lib/awspec/stub/rds.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:rds] = {
   stub_responses: {
     list_tags_for_resource: {

data/lib/awspec/stub/rds_db_cluster_parameter_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:rds] = {
   stub_responses: {
     describe_db_cluster_parameters: {

data/lib/awspec/stub/rds_db_parameter_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:rds] = {
   stub_responses: {
     describe_db_parameters: {

data/lib/awspec/stub/redshift.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:redshift] = {
   stub_responses: {
     describe_clusters: {

data/lib/awspec/stub/redshift_cluster_parameter_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:redshift] = {
   stub_responses: {
     describe_cluster_parameters: {

data/lib/awspec/stub/route53_hosted_zone.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:route53] = {
   stub_responses: {
     list_hosted_zones: {

data/lib/awspec/stub/route_table.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ec2] = {
   stub_responses: {
     describe_instances: {

data/lib/awspec/stub/s3_bucket.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:s3] = {
   stub_responses: {
     list_buckets: {

data/lib/awspec/stub/secretsmanager.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:secretsmanager] = {
   stub_responses: {
     describe_secret: {

data/lib/awspec/stub/security_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ec2] = {
   stub_responses: {
     describe_security_groups: {

data/lib/awspec/stub/ses_identity.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ses] = {
   stub_responses: {
     list_identities: {

data/lib/awspec/stub/sns_topic.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 OWNER = '123456789'
 REGION = 'us-east-1'
 TOPIC_ARN = "arn:aws:sns:#{REGION}:#{OWNER}:foobar"
@@ -9,7 +11,7 @@ Aws.config[:sns] = {
   stub_responses: {
     get_topic_attributes: {
       attributes: {
-        # rubocop:disable LineLength
+        # rubocop:disable Layout/LineLength
         'Policy'                  => "{\"Version\":\"2008-10-17\",\"Id\":\"__default_policy_ID\",\"Statement\":[{\"Sid\":\"__default_statement_ID\",\"Effect\":\"Allow\",\"Principal\":{\"AWS\":\"*\"},\"Action\":[\"SNS:GetTopicAttributes\",\"SNS:SetTopicAttributes\",\"SNS:AddPermission\",\"SNS:RemovePermission\",\"SNS:DeleteTopic\",\"SNS:Subscribe\",\"SNS:ListSubscriptionsByTopic\",\"SNS:Publish\",\"SNS:Receive\"],\"Resource\":\"arn:aws:sns:#{REGION}:#{OWNER}:foobar-lambda-sample\",\"Condition\":{\"StringEquals\":{\"AWS:SourceOwner\":\"#{OWNER}\"}}}]}",
         'Owner'                   => OWNER,
         'SubscriptionsPending'    => '0',

data/lib/awspec/stub/sns_topic_error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 OWNER = '123456789'
 REGION = 'us-east-1'
 TOPIC_ARN = "arn:aws:sns:#{REGION}:#{OWNER}:invalid"
@@ -6,8 +8,10 @@ TOPIC_SUBS_ARN = "arn:aws:sns:us-east-1:#{OWNER}:Foobar:3dbf4999-b3e2-4345-bd11-
 Aws.config[:sns] = {
   stub_responses: {
     get_topic_attributes: Aws::SNS::Errors::NotFound.new(
-      TOPIC_ARN, 'no such topic'),
+      TOPIC_ARN, 'no such topic'
+    ),
     list_subscriptions_by_topic: Aws::SNS::Errors::NotFound.new(
-      TOPIC_SUBS_ARN, 'no such topic')
+      TOPIC_SUBS_ARN, 'no such topic'
+    )
   }
 }

data/lib/awspec/stub/sqs.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:sqs] = {
   stub_responses: {
     list_queues: {

data/lib/awspec/stub/ssm_parameter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ssm] = {
   stub_responses: {
     describe_parameters: {

data/lib/awspec/stub/subnet.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ec2] = {
   stub_responses: {
     describe_vpcs: {

data/lib/awspec/stub/transit_gateway.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ec2] = {
   stub_responses: {
     describe_transit_gateways: {

data/lib/awspec/stub/vpc.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ec2] = {
   stub_responses: {
     describe_vpcs: {

data/lib/awspec/stub/vpc_endpoints.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:ec2] = {
   stub_responses: {
     describe_vpc_endpoints: {

data/lib/awspec/stub/waf_web_acl.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:waf] = {
   stub_responses: {
     list_web_acls: {

data/lib/awspec/stub/wafregional_web_acl.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:wafregional] = {
   stub_responses: {
     list_web_acls: {

data/lib/awspec/stub.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module Awspec
   class Stub
     def self.load(type)
-      require File.dirname(__FILE__) + '/stub/' + type
+      require "#{File.dirname(__FILE__)}/stub/#{type}"
     end
   end
 end

data/lib/awspec/toolbox.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec
   class Toolbox < Thor
     desc 'docgen > doc/resource_types.md', 'Generate doc/resource_types.md document'

data/lib/awspec/type/account.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Account < Base
     REMOVE_SUFFIX_RE = /
@@ -10,7 +12,7 @@ module Awspec::Type
         key = type.gsub(REMOVE_SUFFIX_RE, '').to_sym
         if key == 'ses'
           # https://docs.aws.amazon.com/ses/latest/DeveloperGuide/regions.html
-          next unless ['us-east-1', 'us-west-2', 'eu-west-1'].include?(Aws.config[:region])
+          next unless %w[us-east-1 us-west-2 eu-west-1].include?(Aws.config[:region])
         end
         eval "attributes[key] = Awspec::Type::#{type.camelize}.new.resource_via_client"
       end
@@ -30,7 +32,7 @@ module Awspec::Type
     end
 
     def method_missing(name)
-      name = name.to_s if name.class == Symbol
+      name = name.to_s if name.instance_of?(Symbol)
       describe = name.tr('-', '_').to_sym
       super unless resource_via_client.members.include?(describe)
       resource_via_client[describe]

data/lib/awspec/type/account_attribute.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class AccountAttribute < Base
     def initialize(key = nil)
@@ -9,9 +11,10 @@ module Awspec::Type
       Awspec::Helper::Type::ACCOUNT_ATTRIBUTES.each do |type|
         key = type.gsub(Awspec::Type::Account::REMOVE_SUFFIX_RE, '').to_sym
         next unless key == @key
+
         if key == 'ses'
           # https://docs.aws.amazon.com/ses/latest/DeveloperGuide/regions.html
-          next unless ['us-east-1', 'us-west-2', 'eu-west-1'].include?(Aws.config[:region])
+          next unless %w[us-east-1 us-west-2 eu-west-1].include?(Aws.config[:region])
         end
         eval "attributes = Awspec::Type::#{type.camelize}.new.resource_via_client"
       end
@@ -19,7 +22,7 @@ module Awspec::Type
     end
 
     def method_missing(name)
-      name = name.to_s if name.class == Symbol
+      name = name.to_s if name.instance_of?(Symbol)
       describe = name.tr('-', '_').to_sym
       super unless resource_via_client.members.include?(describe)
       resource_via_client[describe]

data/lib/awspec/type/account_attribute_base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class AccountAttributeBase < Base
   end

data/lib/awspec/type/acm.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Acm < ResourceBase
     def resource_via_client
@@ -8,7 +10,7 @@ module Awspec::Type
       @id ||= resource_via_client.certificate_arn if resource_via_client
     end
 
-    STATUSES = %w(
+    STATUSES = %w[
       PENDING_VALIDATION
       ISSUED
       INACTIVE
@@ -16,25 +18,25 @@ module Awspec::Type
       VALIDATION_TIMED_OUT
       REVOKED
       FAILED
-    )
+    ]
 
     STATUSES.each do |status|
-      define_method status.downcase + '?' do
+      define_method "#{status.downcase}?" do
         resource_via_client.status == status
       end
     end
 
     def has_domain_name?(domain)
       resource_via_client.domain_name == domain && \
-      resource_via_client.certificate_arn == id
+        resource_via_client.certificate_arn == id
     end
 
     def has_domain_validation_option?(domain_name:, validation_method:, validation_status: nil)
       resource_via_client.domain_validation_options.find do |domain_validation_option|
         validation_status ||= domain_validation_option.validation_status
         domain_validation_option.domain_name == domain_name && \
-        domain_validation_option.validation_method == validation_method && \
-        domain_validation_option.validation_status == validation_status
+          domain_validation_option.validation_method == validation_method && \
+          domain_validation_option.validation_status == validation_status
       end
     end
   end

data/lib/awspec/type/alb.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Alb < ResourceBase
     tags_allowed
@@ -9,12 +11,12 @@ module Awspec::Type
       @id ||= resource_via_client.load_balancer_name if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       active provisioning failed
-    )
+    ]
 
     STATES.each do |state|
-      define_method state + '?' do
+      define_method "#{state}?" do
         check_existence
         resource_via_client.state.code == state
       end
@@ -27,8 +29,10 @@ module Awspec::Type
         sg == sg_id
       end
       return true if ret
+
       sg2 = find_security_group(sg_id)
       return true if sgs.include? sg2.group_id
+
       false
     end
 
@@ -39,6 +43,7 @@ module Awspec::Type
         az.subnet_id == subnet_id
       end
       return true if ret
+
       subnet2 = find_subnet(subnet_id)
       subnet2.subnet_id = subnet_id
     end

data/lib/awspec/type/alb_listener.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class AlbListener < ResourceBase
     def resource_via_client
@@ -15,11 +17,13 @@ module Awspec::Type
         actions = [actions] if actions.is_a?(Hash)
         next false if !rule_id.nil? && rule.rule_arn != rule_id
         next false if !priority.nil? && rule.priority != priority
+
         unless conditions.nil?
           next false unless conditions.map do |condition|
             rule_conds = rule.conditions.select { |rule_cond| rule_cond[:field] == condition[:field] }
             next if rule_conds.empty?
             next unless rule_conds.reject { |rule_cond| condition[:values] == rule_cond[:values] }.empty?
+
             condition
           end.compact.size == conditions.size
         end

data/lib/awspec/type/alb_target_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class AlbTargetGroup < ResourceBase
     def resource_via_client
@@ -11,6 +13,7 @@ module Awspec::Type
     def has_ec2?(id)
       ec2 = find_ec2(id)
       return nil unless ec2
+
       descriptions = elbv2_client.describe_target_health(
         target_group_arn: resource_via_client.target_group_arn
       ).target_health_descriptions

data/lib/awspec/type/ami.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Ami < ResourceBase
     aws_resource Aws::EC2::Image
@@ -11,13 +13,13 @@ module Awspec::Type
       @id ||= resource_via_client.image_id if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       pending available invalid deregistered
       transient failed error
-    )
+    ]
 
     STATES.each do |state|
-      define_method state + '?' do
+      define_method "#{state}?" do
         resource_via_client.state == state
       end
     end

data/lib/awspec/type/apigateway.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'addressable/uri'
 
 module Awspec::Type
@@ -7,6 +9,7 @@ module Awspec::Type
     def resource_via_client
       @resource_via_client ||= find_apigateway_by_id(@display_name)
       return @resource_via_client if @resource_via_client
+
       @resource_via_client = find_apigateway_by_name(@display_name)
     end
 
@@ -20,7 +23,7 @@ module Awspec::Type
 
     def has_path?(path)
       check_existence
-      self.api_resources.each do |resource|
+      api_resources.each do |resource|
         return resource if resource.path == path
       end
       nil
@@ -28,11 +31,12 @@ module Awspec::Type
 
     def has_integration_path?(path)
       check_existence
-      self.api_resources.each do |resource|
+      api_resources.each do |resource|
         next if resource.resource_methods.nil?
+
         resource.resource_methods.each do |_, method|
           if method.method_integration.http_method == 'AWS'
-            aws_path = method.method_integration.uri.match(%r{(\/[^\?]+)\??.*$}).captures[0] # Matches for ARN type path
+            aws_path = method.method_integration.uri.match(%r{(/[^?]+)\??.*$}).captures[0] # Matches for ARN type path
             return resource if aws_path == path
           end
           uri = Addressable::URI.parse(method.method_integration.uri)
@@ -46,6 +50,7 @@ module Awspec::Type
       check_existence
       resource_to_check = has_path?(path)
       return nil if resource_to_check.nil?
+
       resource_to_check.resource_methods.each do |_, method|
         return resource_to_check if method.http_method == http_method
       end
@@ -56,6 +61,7 @@ module Awspec::Type
       check_existence
       integration_resource_to_check = has_integration_path?(integration_path)
       return nil if integration_resource_to_check.nil?
+
       integration_resource_to_check.resource_methods.each do |_, method|
         return integration_resource_to_check if method.method_integration.http_method == http_method
       end

data/lib/awspec/type/autoscaling_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class AutoscalingGroup < ResourceBase
     aws_resource Aws::AutoScaling::AutoScalingGroup
@@ -20,6 +22,7 @@ module Awspec::Type
     def has_ec2?(id)
       ec2 = find_ec2(id)
       return nil unless ec2
+
       resource_via_client.instances.one? do |instance|
         instance.instance_id == ec2.instance_id
       end

data/lib/awspec/type/base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'aws-sdk'
 require 'awspec/resource_reader'
 require 'awspec/helper/finder'
@@ -34,12 +36,13 @@ module Awspec::Type
           tags = resource_via_client.tag_set
         end
         return false unless tags
+
         tags.any? { |t| t['key'] == key && t['value'] == value }
       end
     end
 
     def method_missing(name)
-      name_str = name.to_s if name.class == Symbol
+      name_str = name.to_s if name.instance_of?(Symbol)
       describe = name_str.tr('-', '_').to_sym
 
       if !resource_via_client.nil? && resource_via_client.members.include?(describe)
@@ -47,12 +50,12 @@ module Awspec::Type
       elsif resource_via_client.nil?
         raise Awspec::NoExistingResource.new(self.class, @display_name)
       else
-        super unless self.respond_to?(:resource)
+        super unless respond_to?(:resource)
         method_missing_via_black_list(name, delegate_to: resource)
       end
     end
 
-    undef :timeout
+    undef :timeout if Gem::Version.create(RUBY_VERSION) < Gem::Version.create('3.1.0')
 
     private
 

data/lib/awspec/type/batch_compute_environment.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class BatchComputeEnvironment < ResourceBase
     def initialize(compenv)
@@ -13,18 +15,18 @@ module Awspec::Type
       @id ||= resource_via_client.compute_environment_name if resource_via_client
     end
 
-    STATES = %w(ENABLED DISABLED)
+    STATES = %w[ENABLED DISABLED]
 
     STATES.each do |state|
-      define_method state.downcase + '?' do
+      define_method "#{state.downcase}?" do
         resource_via_client.state == state
       end
     end
 
-    TYPES = %w(MANAGED UNMANAGED)
+    TYPES = %w[MANAGED UNMANAGED]
 
     TYPES.each do |type|
-      define_method type.downcase + '?' do
+      define_method "#{type.downcase}?" do
         resource_via_client.type == type
       end
     end

data/lib/awspec/type/batch_job_definition.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class BatchJobDefinition < ResourceBase
     def initialize(jobdef)

data/lib/awspec/type/batch_job_queue.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class BatchJobQueue < ResourceBase
     def initialize(jobqueue)
@@ -13,10 +15,10 @@ module Awspec::Type
       @id ||= resource_via_client.job_queue_name if resource_via_client
     end
 
-    STATES = %w(ENABLED DISABLED)
+    STATES = %w[ENABLED DISABLED]
 
     STATES.each do |state|
-      define_method state.downcase + '?' do
+      define_method "#{state.downcase}?" do
         resource_via_client.state == state
       end
     end

data/lib/awspec/type/cloudformation_stack.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class CloudformationStack < ResourceBase
     def resource_via_client

data/lib/awspec/type/cloudfront_distribution.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class CloudfrontDistribution < ResourceBase
     def resource_via_client
@@ -8,12 +10,12 @@ module Awspec::Type
       @id ||= resource_via_client.id if resource_via_client
     end
 
-    STATUSES = %w(
+    STATUSES = %w[
       InProgress Deployed
-    )
+    ]
 
     STATUSES.each do |status|
-      define_method status.underscore + '?' do
+      define_method "#{status.underscore}?" do
         resource_via_client.status == status
       end
     end
@@ -23,12 +25,14 @@ module Awspec::Type
                     origin_path: nil,
                     origin_access_identity: nil)
       return false unless [origin_id, domain_name].any?
+
       resource_via_client.origins.items.find do |origin|
         next false if !origin_id.nil? && origin.id != origin_id
         next false if !domain_name.nil? && origin.domain_name != domain_name
         next false if !origin_path.nil? && origin.origin_path != origin_path
         next false if !origin_access_identity.nil? && \
                       origin.s3_origin_config.origin_access_identity != origin_access_identity
+
         true
       end
     end
@@ -38,8 +42,8 @@ module Awspec::Type
     end
 
     def has_origin_domain_name_and_path?(domain_name_and_path)
-      domain_name = domain_name_and_path.gsub(%r(/.*\z), '')
-      origin_path = domain_name_and_path.gsub(%r(\A[^/]*), '')
+      domain_name = domain_name_and_path.gsub(%r{/.*\z}, '')
+      origin_path = domain_name_and_path.gsub(%r{\A[^/]*}, '')
       has_origin?(nil, domain_name: domain_name, origin_path: origin_path)
     end
 
@@ -48,11 +52,13 @@ module Awspec::Type
                                         response_code: nil,
                                         error_caching_min_ttl: nil)
       return false unless [error_code, domain_name].any?
+
       resource_via_client.custom_error_responses.items.find do |error|
         next false if !error_code.nil? && error.error_code != error_code
         next false if !response_page_path.nil? && error.response_page_path != response_page_path
         next false if !response_code.nil? && error.response_code != response_code.to_s
         next false if !error_caching_min_ttl.nil? && error.error_caching_min_ttl != error_caching_min_ttl
+
         true
       end
     end