awspec 1.25.1 → 1.25.2

data/lib/awspec/helper/type.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec
   module Helper
     module Type
@@ -7,7 +9,7 @@ module Awspec
       require 'awspec/type/account'
       require 'awspec/type/account_attribute'
 
-      TYPES = %w(
+      TYPES = %w[
         alb alb_listener alb_target_group ami apigateway autoscaling_group
         batch_compute_environment batch_job_definition batch_job_queue cloudtrail
         cloudwatch_alarm cloudwatch_event directconnect_virtual_interface
@@ -21,11 +23,11 @@ module Awspec
         internet_gateway acm cloudwatch_logs dynamodb_table eip sqs ssm_parameter cloudformation_stack
         codebuild sns_topic redshift redshift_cluster_parameter_group codedeploy codedeploy_deployment_group
         secretsmanager msk transit_gateway cognito_identity_pool cognito_user_pool vpc_endpoints
-      )
+      ]
 
-      ACCOUNT_ATTRIBUTES = %w(
+      ACCOUNT_ATTRIBUTES = %w[
         ec2_account_attributes rds_account_attributes lambda_account_settings ses_send_quota
-      )
+      ]
 
       TYPES.each do |type|
         require "awspec/type/#{type}"
@@ -33,6 +35,7 @@ module Awspec
           unless Object.const_get("Awspec::Type::#{type.camelize}").superclass.to_s == 'Awspec::Type::ResourceBase'
             raise "Awspec::Type::#{type.camelize} should extend Awspec::Type::ResourceBase"
           end
+
           name = args.first
           eval "Awspec::Type::#{type.camelize}.new(name)"
         end
@@ -45,6 +48,7 @@ module Awspec
                  == 'Awspec::Type::AccountAttributeBase'
             raise "Awspec::Type::#{type.camelize} should extend Awspec::Type::AccountAttributeBase"
           end
+
           eval "Awspec::Type::#{type.camelize}.new"
         end
       end

data/lib/awspec/helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'awspec/helper/color'
 require 'awspec/helper/type'
 require 'awspec/helper/finder'

data/lib/awspec/matcher/be_allowed.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :be_allowed do |port|
   match do |type|
     type.allowed?(port, @protocol, @cidr, @rule_number)

data/lib/awspec/matcher/be_allowed_action.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :be_allowed_action do |action_name|
   match do |type|
     results = type.select_policy_evaluation_results(type.resource_via_client[:arn],

data/lib/awspec/matcher/be_attached_to.rb

@@ -1,9 +1,12 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :be_attached_to do |instance_id|
   match do |type|
     # NetworkInterface
     if type.instance_of?(Awspec::Type::NetworkInterface)
       return type.attached_to?(instance_id, @device_index)
     end
+
     type.attached_to?(instance_id)
   end
 

data/lib/awspec/matcher/be_connected_to_vpc.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :be_connected_to_vpc do |vpc_id|
   match do |vpc|
     vpc.connected_to_vpc?(vpc_id, @accepter_or_requester)

data/lib/awspec/matcher/be_denied.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :be_denied do |port|
   match do |type|
     type.denied?(port, @protocol, @cidr, @rule_number)

data/lib/awspec/matcher/be_opened.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :be_opened do |port|
   match do |sg|
     sg.opened?(port, @protocol, @cidr)

data/lib/awspec/matcher/be_opened_only.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :be_opened_only do |port|
   match do |sg|
     sg.opened_only?(port, @protocol, @cidr)

data/lib/awspec/matcher/belong_to_alb.rb

@@ -1,8 +1,12 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_alb do |alb_arn|
   match do |type|
     return true if type.load_balancer_arns.include?(alb_arn)
+
     ret = type.find_alb(alb_arn)
     return false unless ret
+
     type.load_balancer_arns.include?(ret.load_balancer_arn)
   end
 end

data/lib/awspec/matcher/belong_to_cache_subnet_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_cache_subnet_group do |cache_subnet_group_name|
   match do |type|
     return true if type.resource_via_client[:cache_subnet_group_name] == cache_subnet_group_name

data/lib/awspec/matcher/belong_to_cluster_subnet_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_cluster_subnet_group do |cluster_subnet_group_name|
   match do |type|
     return true if type.resource_via_client[:cluster_subnet_group_name] == cluster_subnet_group_name

data/lib/awspec/matcher/belong_to_db_subnet_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_db_subnet_group do |db_subnet_group_name|
   match do |type|
     return true if type.resource_via_client[:db_subnet_group][:db_subnet_group_name] == db_subnet_group_name

data/lib/awspec/matcher/belong_to_domain.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_domain do |domain|
   match do |type|
     return true if type.resource_via_client.last[:domain] == domain

data/lib/awspec/matcher/belong_to_iam_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_iam_group do |group_id|
   match do |type|
     groups = type.select_iam_group_by_user_name(type.resource_via_client[:user_name])

data/lib/awspec/matcher/belong_to_metric.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_metric do |name|
   match do |type|
     if @namespace

data/lib/awspec/matcher/belong_to_nlb.rb

@@ -1,8 +1,12 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_nlb do |nlb_arn|
   match do |type|
     return true if type.load_balancer_arns.include?(nlb_arn)
+
     ret = type.find_nlb(nlb_arn)
     return false unless ret
+
     type.load_balancer_arns.include?(ret.load_balancer_arn)
   end
 end

data/lib/awspec/matcher/belong_to_replication_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_replication_group do |group_id|
   match do |type|
     # ElastiCache

data/lib/awspec/matcher/belong_to_subnet.rb

@@ -1,10 +1,14 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_subnet do |subnet_id|
   match do |type|
     # EC2, NetworkInterface
     if type.instance_of?(Awspec::Type::Ec2) || type.instance_of?(Awspec::Type::NetworkInterface)
       return true if type.subnet_id == subnet_id
+
       subnet = type.find_subnet(subnet_id)
       return false unless subnet
+
       return subnet[:subnet_id] == type.subnet_id
     end
 
@@ -21,6 +25,7 @@ RSpec::Matchers.define :belong_to_subnet do |subnet_id|
                                                filters: [{ name: 'tag:Name', values: [subnet_id] }]
                                              })
       return false unless res
+
       ret = subnets.find do |s|
         s[:subnet_identifier] == res[:subnets][0][:subnet_id]
       end

data/lib/awspec/matcher/belong_to_subnets.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_subnets do |*subnets|
   match do |nodegroup|
     superset = Set.new(subnets)

data/lib/awspec/matcher/belong_to_vpc.rb

@@ -1,8 +1,12 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :belong_to_vpc do |vpc_id|
   match do |type|
     return true if type.vpc_id == vpc_id
+
     ret = type.find_vpc(vpc_id)
     return false unless ret
+
     ret[:vpc_id] == type.vpc_id
   end
 end

data/lib/awspec/matcher/have_attribute_definition.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_attribute_definition do |attribute_name|
   match do |attribute|
     attribute.has_attribute_definition?(attribute_name, @attribute_type)

data/lib/awspec/matcher/have_cluster_parameter_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_cluster_parameter_group do |name|
   match do |cluster_identifier|
     cluster_identifier.has_cluster_parameter_group?(name, @parameter_apply_status)

data/lib/awspec/matcher/have_custom_response_error_code.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_custom_response_error_code do |error_code|
   match do |cloudfront_distribution|
     cloudfront_distribution.has_custom_response_error_code?(error_code,
                                                             response_page_path: @response_page_path,
                                                             response_code: @response_code,
-                                                            error_caching_min_ttl: @error_caching_min_ttl
-                                                           )
+                                                            error_caching_min_ttl: @error_caching_min_ttl)
   end
 
   chain :response_page_path do |response_page_path|

data/lib/awspec/matcher/have_db_parameter_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_db_parameter_group do |name|
   match do |db_instance_identifier|
     db_instance_identifier.has_db_parameter_group?(name, @parameter_apply_status)

data/lib/awspec/matcher/have_env_var.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_env_var do |env_var|
   match do |lambda_function|
     lambda_function.environment.variables.key?(env_var)

data/lib/awspec/matcher/have_env_var_value.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_env_var_value do |env_var, value|
   match do |lambda_function|
     if lambda_function.environment.variables.key?(env_var)

data/lib/awspec/matcher/have_env_vars.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_env_vars do
   match do |lambda_function|
     !lambda_function.environment.variables.empty?

data/lib/awspec/matcher/have_inline_policy.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_inline_policy do |policy_name|
   match do |iam_type|
     iam_type.has_inline_policy?(policy_name, @document)

data/lib/awspec/matcher/have_key_policy.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_key_policy do |policy_name|
   match do |key_type|
     key_type.has_key_policy?(policy_name, @document)

data/lib/awspec/matcher/have_key_schema.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_key_schema do |attribute_name|
   match do |attribute|
     attribute.has_key_schema?(attribute_name, @key_type)

data/lib/awspec/matcher/have_metric_filter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_metric_filter do |filter_name|
   match do |log_group_name|
     log_group_name.has_metric_filter?(filter_name, @pattern)

data/lib/awspec/matcher/have_network_interface.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_network_interface do |network_interface_id|
   match do |type|
     type.has_network_interface?(network_interface_id, @device_index)

data/lib/awspec/matcher/have_option_group.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_option_group do |name|
   match do |db_instance_identifier|
     db_instance_identifier.has_option_group?(name, @status)

data/lib/awspec/matcher/have_origin.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_origin do |origin_id|
   match do |cloudfront_distribution|
     cloudfront_distribution.has_origin?(origin_id,
                                         domain_name: @domain_name,
                                         origin_path: @origin_path,
-                                        origin_access_identity: @origin_access_identity
-                                       )
+                                        origin_access_identity: @origin_access_identity)
   end
 
   chain :domain_name do |domain_name|

data/lib/awspec/matcher/have_private_ip_address.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_private_ip_address do |ip_address|
   match do |eni|
     eni.has_private_ip_address?(ip_address, @primary)

data/lib/awspec/matcher/have_record_set.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_record_set do |name|
   match do |hosted_zone|
     hosted_zone.has_record_set?(name, @type, @value, @options)
   end
 
-  %w(soa a txt ns cname mx ptr srv spf aaaa caa).each do |type|
+  %w[soa a txt ns cname mx ptr srv spf aaaa caa].each do |type|
     chain type do |value|
       @type = type
       @value = value

data/lib/awspec/matcher/have_route.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_route do |destination|
   match do |route_table|
     if !is_ipaddr?(destination) && @use_destination # for deprecated usage
@@ -16,10 +18,10 @@ RSpec::Matchers.define :have_route do |destination|
   end
 
   chain :destination do |dest|
-    # rubocop:disable Metrics/LineLength
+    # rubocop:disable Layout/LineLength
     puts ''
     warn Color.on_red(Color.white('!!! route_table destination is deprecated. see https://github.com/k1LoW/awspec/pull/65 !!!'))
-    # rubocop:enable Metrics/LineLength
+    # rubocop:enable Layout/LineLength
     @use_destination = true
     @destination = dest
   end
@@ -28,7 +30,7 @@ RSpec::Matchers.define :have_route do |destination|
 
   def is_ipaddr?(str)
     IPAddr.new(str)
-  rescue
+  rescue StandardError
     false
   end
 end

data/lib/awspec/matcher/have_rule.rb

@@ -1,8 +1,11 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_rule do |rule_id|
   match do |type|
     return type.has_rule?(rule_id, @priority, @action) if type.instance_of?(Awspec::Type::WafWebAcl)
     return type.has_rule?(rule_id, @priority, @action) if type.instance_of?(Awspec::Type::WafregionalWebAcl)
     return type.has_rule?(rule_id, @priority, @conditions, @actions) if type.instance_of?(Awspec::Type::AlbListener)
+
     type.has_rule?(rule_id, @priority, @conditions, @actions) if type.instance_of?(Awspec::Type::NlbListener)
   end
 

data/lib/awspec/matcher/have_subscription_attributes.rb

@@ -1,8 +1,11 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_subscription_attributes do |expected_attribs|
   match do |sns_topic|
     unless expected_attribs.key?(:subscription_arn)
       raise '"subscription_arn" is the only required attribute to validate'
     end
+
     subscribed = sns_topic.subscribed(expected_attribs[:subscription_arn])
     temp = subscribed.to_h
     to_remove = temp.keys - expected_attribs.keys

data/lib/awspec/matcher/have_subscription_filter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_subscription_filter do |filter_name|
   match do |log_group_name|
     log_group_name.has_subscription_filter?(filter_name, @pattern)

data/lib/awspec/matcher/have_tag.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_tag do |key|
   match do |type|
     type.has_tag?(key, @value)

data/lib/awspec/matcher/have_vpc_peering_connection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :have_vpc_peering_connection do |vpc_peering_connection_id|
   match do |vpc|
     vpc.has_vpc_peering_connection?(vpc_peering_connection_id, @accepter_or_requester)

data/lib/awspec/matcher/include_subscribed.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :include_subscribed do |subscribed_arn|
   match do |sns_topic|
     sns_topic.has_subscription?(subscribed_arn)

data/lib/awspec/matcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # EC2
 require 'awspec/matcher/belong_to_vpc'
 require 'awspec/matcher/belong_to_subnet'

data/lib/awspec/resource_reader.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Awspec
   module BlackListForwardable
     class CalledMethodInBlackList < StandardError
@@ -8,6 +10,7 @@ module Awspec
       if match_black_list?(name) && !match_white_list?(name)
         raise CalledMethodInBlackList, "Method call #{name.inspect} is black-listed"
       end
+
       attr = delegate_to.send(name)
       if !attr.is_a?(Struct) && attr.class.name.match(/^Aws::/)
         ResourceReader.new(attr)

data/lib/awspec/setup.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'fileutils'
 require 'yaml'
 
@@ -18,7 +20,7 @@ EOF
       dir = 'spec'
       if File.exist? dir
         unless File.directory? dir
-          $stderr.puts "!! #{dir} already exists and is not a directory"
+          warn "!! #{dir} already exists and is not a directory"
         end
       else
         FileUtils.mkdir dir
@@ -26,7 +28,7 @@ EOF
       end
 
       if File.exist? 'spec/spec_helper.rb'
-        $stderr.puts '!! spec/spec_helper.rb already exists'
+        warn '!! spec/spec_helper.rb already exists'
       else
         File.open('spec/spec_helper.rb', 'w') do |f|
           f.puts content
@@ -43,7 +45,7 @@ task :default => :spec
 EOF
 
       if File.exist? 'Rakefile'
-        $stderr.puts '!! Rakefile already exists'
+        warn '!! Rakefile already exists'
       else
         File.open('Rakefile', 'w') do |f|
           f.puts content
@@ -57,7 +59,7 @@ EOF
 secrets.yml
 EOF
       if File.exist? 'spec/.gitignore'
-        $stderr.puts '!! spec/.gitignore already exists'
+        warn '!! spec/.gitignore already exists'
       else
         File.open('spec/.gitignore', 'w') do |f|
           f.puts content
@@ -72,7 +74,7 @@ EOF
 --format documentation
 EOF
       if File.exist? '.rspec'
-        $stderr.puts '!! .rspec already exists'
+        warn '!! .rspec already exists'
       else
         File.open('.rspec', 'w') do |f|
           f.puts content

data/lib/awspec/shared_context.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 shared_context 'region', :region do
   before do |example|
     region = example.metadata[:region]
@@ -14,6 +16,7 @@ shared_context 'account', :account do
   before do |example|
     sts_client = Aws::STS::Client.new
     raise Awspec::InvalidAccountError unless sts_client.get_caller_identity.account == example.metadata[:account]
+
     example.metadata[:described_class].account = example.metadata[:account]
   end
 end

data/lib/awspec/stub/account.rb

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

data/lib/awspec/stub/acm.rb

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

data/lib/awspec/stub/alb.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/alb_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: {
@@ -129,8 +131,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-alb/1aa1bb1cc1ddee11/f2f7dc8efc522ab2/defaaaaaaaultbbbb'

data/lib/awspec/stub/alb_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/ami.rb

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

data/lib/awspec/stub/apigateway.rb

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

data/lib/awspec/stub/autoscaling_group.rb

@@ -1,4 +1,6 @@
-# rubocop:disable Metrics/LineLength
+# frozen_string_literal: true
+
+# rubocop:disable Layout/LineLength
 Aws.config[:autoscaling] = {
   stub_responses: {
     describe_auto_scaling_groups: {
@@ -11,9 +13,9 @@ Aws.config[:autoscaling] = {
           max_size: 20,
           desired_capacity: 4,
           default_cooldown: 300,
-          availability_zones: [
-            'ap-northeast-1a',
-            'ap-northeast-1c'
+          availability_zones: %w[
+            ap-northeast-1a
+            ap-northeast-1c
           ],
           load_balancer_names: [
             'my-elb'

data/lib/awspec/stub/batch_compute_environment.rb

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

data/lib/awspec/stub/batch_job_definition.rb

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

data/lib/awspec/stub/batch_job_queue.rb

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