Technical/Update gem dependencies (#269)

* Updated gemspecs
* Updated codeclimate config
* Changed rspec helpers namespaces
* Updated reek config

Author: bestwebua

.circleci/gemspecs/compatible

@@ -16,7 +16,7 @@ Gem::Specification.new do |spec|
   spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   spec.require_paths = %w[lib]
 
-  spec.add_runtime_dependency 'net-smtp', '~> 0.3.3' if ::Gem::Version.new(::RUBY_VERSION) >= ::Gem::Version.new('3.1.0')
+  spec.add_runtime_dependency 'net-smtp', '~> 0.4.0' if ::Gem::Version.new(::RUBY_VERSION) >= ::Gem::Version.new('3.1.0')
   spec.add_runtime_dependency 'simpleidn', '~> 0.2.1'
 
   spec.add_development_dependency 'dns_mock'

.circleci/gemspecs/latest

@@ -16,23 +16,23 @@ Gem::Specification.new do |spec|
   spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   spec.require_paths = %w[lib]
 
-  spec.add_runtime_dependency 'net-smtp', '~> 0.3.3'
+  spec.add_runtime_dependency 'net-smtp', '~> 0.4.0'
   spec.add_runtime_dependency 'simpleidn', '~> 0.2.1'
 
   spec.add_development_dependency 'bundler-audit', '~> 0.9.1'
-  spec.add_development_dependency 'dns_mock', '~> 1.5', '>= 1.5.15'
+  spec.add_development_dependency 'dns_mock', '~> 1.5', '>= 1.5.16'
   spec.add_development_dependency 'fasterer', '~> 0.10.1'
-  spec.add_development_dependency 'ffaker', '~> 2.21'
+  spec.add_development_dependency 'ffaker', '~> 2.23'
   spec.add_development_dependency 'json_matchers', '~> 0.11.1'
   spec.add_development_dependency 'pry-byebug', '~> 3.10', '>= 3.10.1'
-  spec.add_development_dependency 'rake', '~> 13.0', '>= 13.0.6'
+  spec.add_development_dependency 'rake', '~> 13.1'
   spec.add_development_dependency 'reek', '~> 6.1', '>= 6.1.4'
   spec.add_development_dependency 'rspec', '~> 3.12'
-  spec.add_development_dependency 'rubocop', '~> 1.56'
-  spec.add_development_dependency 'rubocop-performance', '~> 1.19'
-  spec.add_development_dependency 'rubocop-rspec', '~> 2.23', '>= 2.23.2'
+  spec.add_development_dependency 'rubocop', '~> 1.57', '>= 1.57.2'
+  spec.add_development_dependency 'rubocop-performance', '~> 1.19', '>= 1.19.1'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.25'
   spec.add_development_dependency 'simplecov', '~> 0.22.0'
-  spec.add_development_dependency 'smtp_mock', '~> 1.3', '>= 1.3.4'
-  spec.add_development_dependency 'truemail-rspec', '~> 1.1'
-  spec.add_development_dependency 'webmock', '~> 3.18', '>= 3.18.1'
+  spec.add_development_dependency 'smtp_mock', '~> 1.3', '>= 1.3.5'
+  spec.add_development_dependency 'truemail-rspec', '~> 1.2'
+  spec.add_development_dependency 'webmock', '~> 3.19', '>= 3.19.1'
 end

.codeclimate.yml

@@ -9,7 +9,7 @@ checks:
 plugins:
   rubocop:
     enabled: true
-    channel: rubocop-1-56
+    channel: rubocop-1-57
     config:
       file: .circleci/linter_configs/.rubocop.yml
 

.reek.yml

@@ -56,7 +56,7 @@ detectors:
 
   ControlParameter:
     exclude:
-      - Truemail::GenerateEmailHelper#calculate_email_size
+      - Truemail::RspecHelper::GenerateEmail#calculate_email_size
       - Truemail::Worker#success
       - Truemail#raise_unless
       - Truemail::Configuration#raise_unless
@@ -65,7 +65,7 @@ detectors:
   FeatureEnvy:
     exclude:
       - Truemail::Validate::Smtp#not_includes_user_not_found_errors
-      - Truemail::GenerateEmailHelper#prepare_user_name
+      - Truemail::RspecHelper::GenerateEmail#prepare_user_name
       - Truemail::ConfigurationHelper#create_configuration
       - Truemail::Log::Serializer::Base#smtp_debug
       - Truemail::Log::Serializer::Text#data_composer

CHANGELOG.md

@@ -356,8 +356,8 @@ Optimized DNS (MX) validation flow. Removed needless DNS request for case when c
 
 ### Updated
 
-- Updated gem codebase, refactored `Truemail::ContextHelper`
-- Updated tests with `Truemail::DnsHelper#dns_mock_gateway`
+- Updated gem codebase, refactored `Truemail::RspecHelper::Context`
+- Updated tests with `Truemail::RspecHelper::Dns#dns_mock_gateway`
 - Updated gem development dependencies
 - Updated gem version
 

spec/spec_helper.rb

@@ -9,9 +9,9 @@
 
 RSpec::Mocks.configuration.allow_message_expectations_on_nil = true
 RSpec.configure do |config|
-  config.include Truemail::ContextHelper
-  config.include Truemail::DnsHelper
-  config.include Truemail::IpifyHelper
+  config.include Truemail::RspecHelper::Context
+  config.include Truemail::RspecHelper::Dns
+  config.include Truemail::RspecHelper::Ipify
   config.order = :random
   config.example_status_persistence_file_path = '.rspec_status'
   config.disable_monkey_patching!

spec/support/helpers/context.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module Truemail
+  module RspecHelper
+    module Context
+      NON_ASCII_WORDS = %w[mañana ĉapelo dấu παράδειγμα 屋企].freeze
+
+      def random_email
+        ffaker.email
+      end
+
+      def random_uniq_email
+        ffaker.unique.email
+      end
+
+      def random_internationalized_email
+        "#{ffaker.user_name}@#{Truemail::RspecHelper::Context::NON_ASCII_WORDS.sample}.#{ffaker.domain_suffix}"
+      end
+
+      def random_ip_address
+        ffaker.ip_v4_address
+      end
+
+      def random_domain_name
+        ffaker.domain_name
+      end
+
+      def random_uniq_domain_name
+        ffaker.unique.domain_name
+      end
+
+      def rdns_lookup_host_address(host_address)
+        DnsMock::Representer::RdnsLookup.call(host_address)
+      end
+
+      def domain_from_email(email)
+        email[Truemail::RegexConstant::REGEX_DOMAIN_FROM_EMAIL, 1]
+      end
+
+      def email_punycode_domain(email)
+        DnsMock::Representer::Punycode.call(domain_from_email(email))
+      end
+
+      def attempts_getter
+        ->(smtp_request_instance) { smtp_request_instance.send(:attempts) }
+      end
+
+      private
+
+      def ffaker
+        FFaker::Internet
+      end
+    end
+  end
+end

spec/support/helpers/context_helper.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module Truemail
+  module RspecHelper
+    module Dns
+      LOCALHOST_IP_ADDRESS = '127.0.0.1'
+
+      def dns_mock_gateway
+        ["#{Truemail::RspecHelper::Dns::LOCALHOST_IP_ADDRESS}:#{dns_mock_server.port}"]
+      end
+
+      def dns_mock_records_by_email(email, dimension: 1)
+        mx_records = ::Array.new(dimension) { random_domain_name }
+        a_records = ::Array.new(dimension) { [Truemail::RspecHelper::Dns::LOCALHOST_IP_ADDRESS] }
+        mx_records_dns_mock = mx_records.zip(a_records).to_h.transform_values { |value| { a: value } }
+        { domain_from_email(email) => { mx: mx_records } }.merge(mx_records_dns_mock)
+      end
+    end
+  end
+end