awspec 1.24.3 → 1.25.2

data/lib/awspec/stub/kinesis.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 Aws.config[:kinesis] = {
   stub_responses: {
     list_streams: {
-      stream_names: [
-        'my-kinesis',
-        'second-stream',
-        'third-stream'
+      stream_names: %w[
+        my-kinesis
+        second-stream
+        third-stream
       ],
       has_more_streams: false
     },
@@ -17,7 +19,7 @@ Aws.config[:kinesis] = {
         stream_creation_timestamp: Time.at(1_529_001_577),
         enhanced_monitoring: [
           {
-            shard_level_metrics: %w(
+            shard_level_metrics: %w[
               IncomingBytes
               OutgoingRecords
               IteratorAgeMilliseconds
@@ -25,7 +27,7 @@ Aws.config[:kinesis] = {
               ReadProvisionedThroughputExceeded
               WriteProvisionedThroughputExceeded
               OutgoingBytes
-            )
+            ]
           }
         ],
         encryption_type: 'KMS',

data/lib/awspec/stub/kms.rb

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

data/lib/awspec/stub/lambda.rb

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

data/lib/awspec/stub/launch_configuration.rb

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

data/lib/awspec/stub/launch_template.rb

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

data/lib/awspec/stub/mq.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:mq] = {
   stub_responses: {
     describe_broker: {
@@ -66,9 +68,9 @@ Aws.config[:mq] = {
       tags: {
         'Name' => 'my-mq'
       },
-      subnet_ids: [
-        'subnet-c102731fbf7b7abb9',
-        'subnet-4efe21eb33f86ef91'
+      subnet_ids: %w[
+        subnet-c102731fbf7b7abb9
+        subnet-4efe21eb33f86ef91
       ],
       host_instance_type: 'mq.m4.large',
       security_groups: [

data/lib/awspec/stub/msk.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Aws.config[:kafka] = {
   stub_responses: {
     list_clusters: {
@@ -6,9 +8,9 @@ Aws.config[:kafka] = {
           active_operation_arn: nil,
           broker_node_group_info: {
             broker_az_distribution: 'DEFAULT',
-            client_subnets: [
-              'subnet-12345678909876543',
-              'subnet-90876543212345678'
+            client_subnets: %w[
+              subnet-12345678909876543
+              subnet-90876543212345678
             ],
             instance_type: 'kafka.t3.small',
             security_groups: [

data/lib/awspec/stub/nat_gateway.rb

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

data/lib/awspec/stub/network_acl.rb

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

data/lib/awspec/stub/network_interface.rb

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

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: {
@@ -13,6 +15,14 @@ Aws.config[:rds] = {
         {
           parameter_name: 'max_allowed_packet',
           parameter_value: '16777216'
+        },
+        {
+          parameter_name: 'rds.logical_replication',
+          parameter_value: '1'
+        },
+        {
+          parameter_name: 'rds.accepted_password_auth_method',
+          parameter_value: 'md5+scram'
         }
       ]
     }

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,26 +1,35 @@
+# frozen_string_literal: true
+
+OWNER = '123456789'
+REGION = 'us-east-1'
+TOPIC_ARN = "arn:aws:sns:#{REGION}:#{OWNER}:foobar"
+DISPLAY_NAME = 'Useless'
+SUBSCRIBED = "arn:aws:sns:#{REGION}:#{OWNER}:Foobar:3dbf4999-b3e2-4345-bd11-c34c9784ecca"
+ENDPOINT = "arn:aws:lambda:#{REGION}:#{OWNER}:function:foobar"
+
 Aws.config[:sns] = {
   stub_responses: {
     get_topic_attributes: {
       attributes: {
-        # rubocop:disable 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:us-east-1:123456789:foobar-lambda-sample\",\"Condition\":{\"StringEquals\":{\"AWS:SourceOwner\":\"123456789\"}}}]}',
-        'Owner'                   => '123456789',
+        # 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',
-        'TopicArn'                => 'arn:aws:sns:us-east-1:123456789:foobar',
+        'TopicArn'                => TOPIC_ARN,
         'EffectiveDeliveryPolicy' => '{\"http\":{\"defaultHealthyRetryPolicy\":{\"minDelayTarget\":20,\"maxDelayTarget\":20,\"numRetries\":3,\"numMaxDelayRetries\":0,\"numNoDelayRetries\":0,\"numMinDelayRetries\":0,\"backoffFunction\":\"linear\"},\"disableSubscriptionOverrides\":false}}',
         'SubscriptionsConfirmed'  => '1',
-        'DisplayName'             => 'Useless',
+        'DisplayName'             => DISPLAY_NAME,
         'SubscriptionsDeleted'    => '0'
       }
     },
     list_subscriptions_by_topic: {
       subscriptions: [
         {
-          subscription_arn: 'arn:aws:sns:us-east-1:123456789:Foobar:3dbf4999-b3e2-4345-bd11-c34c9784ecca',
-          owner: '123456789',
+          subscription_arn: SUBSCRIBED,
+          owner: OWNER,
           protocol: 'lambda',
-          endpoint: 'arn:aws:lambda:us-east-1:123456789:function:foobar',
-          topic_arn: 'arn:aws:sns:us-east-1:123456789:foobar'
+          endpoint: ENDPOINT,
+          topic_arn: TOPIC_ARN
         }
       ],
       next_token: nil

data/lib/awspec/stub/sns_topic_error.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+OWNER = '123456789'
+REGION = 'us-east-1'
+TOPIC_ARN = "arn:aws:sns:#{REGION}:#{OWNER}:invalid"
+TOPIC_SUBS_ARN = "arn:aws:sns:us-east-1:#{OWNER}:Foobar:3dbf4999-b3e2-4345-bd11-c34c9784ecca"
+
+Aws.config[:sns] = {
+  stub_responses: {
+    get_topic_attributes: Aws::SNS::Errors::NotFound.new(
+      TOPIC_ARN, 'no such topic'
+    ),
+    list_subscriptions_by_topic: Aws::SNS::Errors::NotFound.new(
+      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