awspec 1.25.1 → 1.25.2

data/lib/awspec/type/iam_user.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class IamUser < ResourceBase
     aws_resource Aws::IAM::User
@@ -25,6 +27,7 @@ module Awspec::Type
                                          policy_name: policy_name
                                        })
       return JSON.parse(URI.decode_www_form_component(res.policy_document)) == JSON.parse(document) if document
+
       res
     end
 

data/lib/awspec/type/internet_gateway.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class InternetGateway < ResourceBase
     aws_resource Aws::EC2::InternetGateway

data/lib/awspec/type/kinesis.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Kinesis < ResourceBase
     aws_resource Aws::Kinesis::Client
@@ -17,6 +19,7 @@ module Awspec::Type
 
     def resource_via_client
       return @resource_via_client unless @resource_via_client.nil?
+
       @resource_via_client = find_kinesis_by_stream_name(@display_name)
     end
 

data/lib/awspec/type/kms.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Kms < ResourceBase
     def resource_via_client
@@ -15,6 +17,7 @@ module Awspec::Type
     def has_key_policy?(policy_name, document = nil)
       res = kms_client.get_key_policy(key_id: id, policy_name: policy_name)
       return JSON.parse(URI.decode_www_form_component(res.policy)) == JSON.parse(document) if document
+
       res
     end
   end

data/lib/awspec/type/lambda.rb

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

data/lib/awspec/type/lambda_account_settings.rb

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

data/lib/awspec/type/launch_configuration.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class LaunchConfiguration < ResourceBase
     def resource_via_client
@@ -14,6 +16,7 @@ module Awspec::Type
         sg == sg_id
       end
       return true if ret
+
       sg2 = find_security_group(sg_id)
       sgs.find do |sg|
         sg == sg2.group_id

data/lib/awspec/type/launch_template.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class LaunchTemplate < ResourceBase
     attr_accessor :version
+
     tags_allowed
 
     def resource_via_client

data/lib/awspec/type/mq.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Mq < ResourceBase
     def resource_via_client
@@ -8,13 +10,13 @@ module Awspec::Type
       @id ||= resource_via_client.broker_id if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       running reboot-in-progress creation-in-progress
       creation-failed deletion-in-progress
-    )
+    ]
 
     STATES.each do |state|
-      define_method state.tr('-', '_') + '?' do
+      define_method "#{state.tr('-', '_')}?" do
         resource_via_client.broker_state.downcase == state
       end
     end
@@ -44,6 +46,7 @@ module Awspec::Type
       res = select_security_group_by_group_name([sg_id])
 
       return false unless res.count == 1
+
       has_vpc_security_group_id?(res.first.group_id)
     end
 
@@ -51,6 +54,7 @@ module Awspec::Type
       res = select_security_group_by_tag_name([sg_id])
 
       return false unless res.count == 1
+
       has_vpc_security_group_id?(res.first.group_id)
     end
   end

data/lib/awspec/type/msk.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Msk < ResourceBase
     def initialize(name)
@@ -13,13 +15,13 @@ module Awspec::Type
       @id ||= resource_via_client.cluster_arn if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       active creating updating
       deleting failed
-    )
+    ]
 
     STATES.each do |state|
-      define_method state + '?' do
+      define_method "#{state}?" do
         resource_via_client.state == state.upcase
       end
     end

data/lib/awspec/type/nat_gateway.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class NatGateway < ResourceBase
     tags_allowed
@@ -10,12 +12,12 @@ module Awspec::Type
       @id ||= resource_via_client.nat_gateway_id if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       pending failed available deleting deleted
-    )
+    ]
 
     STATES.each do |state|
-      define_method state.tr('-', '_') + '?' do
+      define_method "#{state.tr('-', '_')}?" do
         resource_via_client.state == state
       end
     end

data/lib/awspec/type/network_acl.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class NetworkAcl < ResourceBase
     aws_resource Aws::EC2::NetworkAcl
@@ -14,8 +16,10 @@ module Awspec::Type
     def has_subnet?(subnet_id)
       resource_via_client.associations.find do |a|
         next true if a.subnet_id == subnet_id
+
         subnet = find_subnet(subnet_id)
         next false unless subnet
+
         next a.subnet_id == subnet.subnet_id
       end
     end
@@ -52,7 +56,7 @@ module Awspec::Type
       end
     end
 
-    # rubocop:disable Metrics/LineLength
+    # rubocop:disable Layout/LineLength
     PROTOCOLS = { 'ALL' => -1, 'HOPOPT' => 0, 'ICMP' => 1, 'IGMP' => 2, 'GGP' => 3, 'IPv4' => 4, 'ST' => 5, 'TCP' => 6, 'CBT' => 7, 'EGP' => 8, 'IGP' => 9, 'BBN-RCC-MON' => 10,
                   'NVP-II' => 11, 'PUP' => 12, 'ARGUS' => 13, 'EMCON' => 14, 'XNET' => 15, 'CHAOS' => 16, 'UDP' => 17, 'MUX' => 18, 'DCN-MEAS' => 19, 'HMP' => 20,
                   'PRM' => 21, 'XNS-IDP' => 22, 'TRUNK-1' => 23, 'TRUNK-2' => 24, 'LEAF-1' => 25, 'LEAF-2' => 26, 'RDP' => 27, 'IRTP' => 28, 'ISO-TP4' => 29, 'NETBLT' => 30,
@@ -68,7 +72,7 @@ module Awspec::Type
                   'SMP' => 121, 'SM' => 122, 'PTP' => 123, 'ISIS over IPv4' => 124, 'FIRE' => 125, 'CRTP' => 126, 'CRUDP' => 127, 'SSCOPMCE' => 128, 'IPLT' => 129, 'SPS' => 130,
                   'PIPE' => 131, 'SCTP' => 132, 'FC' => 133, 'RSVP-E2E-IGNORE' => 134, 'Mobility Header' => 135, 'UDPLite' => 136, 'MPLS-in-IP' => 137, 'manet' => 138, 'HIP' => 139, 'Shim6' => 140,
                   'WESP' => 141, 'ROHC' => 142, '253' => 253, '254' => 254 }
-    # rubocop:enable Metrics/LineLength
+    # rubocop:enable Layout/LineLength
 
     private
 
@@ -77,15 +81,18 @@ module Awspec::Type
         # egress rule_action
         next false if entry.egress != @egress
         next false if entry.rule_action != rule_action
+
         # protocol
         unless protocol.nil?
           next false unless protocol_match?(protocol, entry.protocol)
         end
         # cidr
         next false if !cidr.nil? && entry.cidr_block != cidr
+
         # rule_number
         rule_number = 32_767 if rule_number == '*'
         next false if !rule_number.nil? && entry.rule_number != rule_number
+
         # port
         unless entry.port_range.nil?
           next false unless port_between?(port, entry.port_range.from, entry.port_range.to)

data/lib/awspec/type/network_interface.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class NetworkInterface < ResourceBase
     tags_allowed
@@ -10,12 +12,12 @@ module Awspec::Type
       @id ||= resource_via_client.network_interface_id if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       available attaching in-use detaching
-    )
+    ]
 
     STATES.each do |state|
-      define_method state.tr('-', '_') + '?' do
+      define_method "#{state.tr('-', '_')}?" do
         resource_via_client.status == state
       end
     end
@@ -25,6 +27,7 @@ module Awspec::Type
       return false unless instance
       return false unless resource_via_client.attachment
       return false if device_index && resource_via_client.attachment.device_index != device_index
+
       resource_via_client.attachment.instance_id == instance.instance_id && \
         resource_via_client.attachment.status == 'attached'
     end
@@ -32,6 +35,7 @@ module Awspec::Type
     def has_private_ip_address?(ip_address, primary = nil)
       resource_via_client.private_ip_addresses.find do |i|
         next false if !primary.nil? && i.primary != primary
+
         i.private_ip_address == ip_address
       end
     end
@@ -42,8 +46,10 @@ module Awspec::Type
         sg.group_id == sg_id || sg.group_name == sg_id
       end
       return true if ret
+
       sg2 = find_security_group(sg_id)
       return false unless sg2.tag_name == sg_id
+
       sgs.find do |sg|
         sg.group_id == sg2.group_id
       end

data/lib/awspec/type/nlb.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Nlb < ResourceBase
     def resource_via_client
@@ -8,12 +10,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
         resource_via_client.state.code == state
       end
     end
@@ -28,6 +30,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/nlb_listener.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class NlbListener < 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/nlb_target_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class NlbTargetGroup < 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/rds.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Rds < ResourceBase
     aws_resource Aws::RDS::DBInstance
@@ -10,7 +12,7 @@ module Awspec::Type
       @id ||= resource_via_client.db_instance_identifier if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       available backing-up creating deleting
       failed inaccessible-encryption-credentials
       incompatible-credentials incompatible-network
@@ -18,10 +20,10 @@ module Awspec::Type
       incompatible-restore maintenance
       modifying rebooting renaming resetting-master-credentials
       restore-error storage-full upgrading
-    )
+    ]
 
     STATES.each do |state|
-      define_method state.tr('-', '_') + '?' do
+      define_method "#{state.tr('-', '_')}?" do
         resource_via_client.db_instance_status == state
       end
     end
@@ -83,6 +85,7 @@ module Awspec::Type
                                                   filters: [{ name: 'group-name', values: [sg_id] }]
                                                 })
       return false unless res.security_groups.count == 1
+
       sgs.find do |sg|
         sg.vpc_security_group_id == res.security_groups.first.group_id
       end
@@ -94,6 +97,7 @@ module Awspec::Type
                                                   filters: [{ name: 'tag:Name', values: [sg_id] }]
                                                 })
       return false unless res.security_groups.count == 1
+
       sgs.find do |sg|
         sg.vpc_security_group_id == res.security_groups.first.group_id
       end

data/lib/awspec/type/rds_account_attributes.rb

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

data/lib/awspec/type/rds_db_cluster_parameter_group.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class RdsDbClusterParameterGroup < ResourceBase
     def resource_via_client
       return @resource_via_client if @resource_via_client
+
       @resource_via_client ||= select_all_rds_db_cluster_parameters(@display_name)
     end
 

data/lib/awspec/type/rds_db_parameter_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class InvalidRdsDbParameter < StandardError
     ##
@@ -24,12 +26,13 @@ module Awspec::Type
     end
 
     def to_s
-      return "RdsDBParameters = #{@params}"
+      "RdsDBParameters = #{@params}"
     end
 
     def method_missing(name)
       param_name = name.to_sym
       return @params[param_name] if @params.include?(param_name)
+
       raise InvalidRdsDbParameter, name
     end
   end
@@ -37,6 +40,7 @@ module Awspec::Type
   class RdsDbParameterGroup < ResourceBase
     def resource_via_client
       return @resource_via_client if @resource_via_client
+
       @resource_via_client ||= select_all_rds_db_parameters(@display_name)
     end
 

data/lib/awspec/type/redshift.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Redshift < ResourceBase
     def resource_via_client
@@ -8,16 +10,16 @@ module Awspec::Type
       @id ||= resource_via_client.cluster_identifier if resource_via_client
     end
 
-    STATES = %w(
+    STATES = %w[
       available cancelling-resize creating deleting
       final-snapshot hardware-failure incompatible-hsm
       incompatible-network incompatible-parameters incompatible-restore
       modifying rebooting renaming resizing rotating-keys
       storage-full updating-hsm
-    )
+    ]
 
     STATES.each do |state|
-      define_method state.tr('-', '_') + '?' do
+      define_method "#{state.tr('-', '_')}?" do
         resource_via_client.cluster_status == state
       end
     end
@@ -62,6 +64,7 @@ module Awspec::Type
                                                   filters: [{ name: 'group-name', values: [sg_id] }]
                                                 })
       return false unless res.security_groups.count == 1
+
       sgs.find do |sg|
         sg.vpc_security_group_id == res.security_groups.first.group_id
       end
@@ -73,6 +76,7 @@ module Awspec::Type
                                                   filters: [{ name: 'tag:Name', values: [sg_id] }]
                                                 })
       return false unless res.security_groups.count == 1
+
       sgs.find do |sg|
         sg.vpc_security_group_id == res.security_groups.first.group_id
       end

data/lib/awspec/type/redshift_cluster_parameter_group.rb

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

data/lib/awspec/type/resource_base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class ResourceBase < Base
     attr_reader :id, :resource_via_client

data/lib/awspec/type/route53_hosted_zone.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Route53HostedZone < ResourceBase
     def resource_via_client
@@ -28,6 +30,7 @@ module Awspec::Type
 
       record_sets.each do |record_set|
         return match_record(record_set, name, value, options) unless record_set.resource_records.empty?
+
         return match_alias_record(record_set, name, options)
       end
     end
@@ -37,8 +40,10 @@ module Awspec::Type
     def match_failover_record(record_sets, name, value, options)
       record_sets.select! { |record_set| record_set.failover == options[:record_type] }
       return false if record_sets.empty?
+
       record_sets.each do |record_set|
         return match_alias_record(record_set, name, options) if value.nil?
+
         return match_record(record_set, name, value, options)
       end
     end
@@ -58,8 +63,8 @@ module Awspec::Type
 
     def match_alias_record(record_set, name, options)
       record_set.name == name && \
-      record_set.alias_target.dns_name == options[:alias_dns_name] && \
-      record_set.alias_target.hosted_zone_id == options[:alias_hosted_zone_id]
+        record_set.alias_target.dns_name == options[:alias_dns_name] && \
+        record_set.alias_target.hosted_zone_id == options[:alias_hosted_zone_id]
     end
   end
 end

data/lib/awspec/type/route_table.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class RouteTable < ResourceBase
     aws_resource Aws::EC2::RouteTable
@@ -30,6 +32,7 @@ module Awspec::Type
     def has_subnet?(subnet_id)
       subnet = find_subnet(subnet_id)
       return false unless subnet
+
       resource_via_client.associations.find do |a|
         a.subnet_id == subnet.subnet_id
       end
@@ -44,26 +47,33 @@ module Awspec::Type
     def target_gateway?(route, gateway_id)
       # * gateway
       return true if route.gateway_id == gateway_id
+
       # internet gateway
       igw = find_internet_gateway(gateway_id)
       return true if igw && igw.internet_gateway_id == route.gateway_id
+
       # vpn gateway
       vgw = find_vpn_gateway(gateway_id)
       return true if vgw && vgw.vpn_gateway_id == route.gateway_id
+
       # customer gateway
       cgw = find_customer_gateway(gateway_id)
       return true if cgw && cgw.customer_gateway_id == route.gateway_id
+
       # nat gateway
       nat = find_nat_gateway(gateway_id)
       return true if nat.nat_gateway_id == route.nat_gateway_id
+
       false
     end
 
     def target_instance?(route, instance_id)
       # instance
       return true if route.instance_id == instance_id
+
       instance = find_ec2(instance_id)
       return true if instance && instance.instance_id == route.instance_id
+
       false
     end
 
@@ -76,8 +86,10 @@ module Awspec::Type
     def target_vpc_peering_connection?(route, vpc_peering_connection_id)
       # vpc_peering_connection_id
       return true if route.vpc_peering_connection_id == vpc_peering_connection_id
+
       connection = find_vpc_peering_connection(vpc_peering_connection_id)
       return true if connection && connection.vpc_peering_connection_id == route.vpc_peering_connection_id
+
       false
     end
   end

data/lib/awspec/type/s3_bucket.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class S3Bucket < ResourceBase
     aws_resource Aws::S3::Bucket
@@ -41,9 +43,7 @@ module Awspec::Type
       @acl.grants.count
     end
 
-    def has_cors_rule?(allowed_headers: [],
-                       allowed_methods:,
-                       allowed_origins:,
+    def has_cors_rule?(allowed_methods:, allowed_origins:, allowed_headers: [],
                        expose_headers: [],
                        max_age_seconds: nil)
       cors_rules.any? do |rule|
@@ -74,6 +74,7 @@ module Awspec::Type
       check_existence
       tag = find_bucket_tag(id, key)
       return nil if tag.value != value
+
       tag
     end
 
@@ -85,6 +86,7 @@ module Awspec::Type
       return false if le.nil?
       return false if target_bucket && target_bucket != le.target_bucket
       return false if target_prefix && target_prefix != le.target_prefix
+
       true
     end
 
@@ -102,6 +104,7 @@ module Awspec::Type
       rule.all? do |key, value|
         lc_rule.each do |r|
           return false if value.is_a?(String) && r[key] != value
+
           if value.is_a?(Hash)
             return false if r[key].to_h != value
           end

data/lib/awspec/type/secretsmanager.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec::Type
   class Secretsmanager < ResourceBase
     aws_resource Aws::SecretsManager