Add test environment default mailer port configuration and update lin…

…k helpers tests to consider an optional port in the URL

.rubocop.yml

@@ -66,6 +66,7 @@ Metrics/BlockLength:
     - lib/github_oauthable.rb
     - app/models/concerns/**/*.rb
     - config/routes.rb
+    - config/environments/development.rb
 
 Metrics/ClassLength:
   Max: 356 # TODO: Lower to 100

config/environments/test.rb

@@ -39,6 +39,7 @@
   # ActionMailer::Base.deliveries array.
   config.action_mailer.delivery_method = :test
   config.action_mailer.default_url_options = { host: Gemcutter::HOST,
+                                               port: "31337",
                                                protocol: Gemcutter::PROTOCOL }
 
   # Print deprecation notices to the stderr.

test/helpers/email_helpers.rb

@@ -15,7 +15,7 @@ def mails_count
   def confirmation_link
     refute_empty ActionMailer::Base.deliveries
     body = last_email.parts[1].body.decoded.to_s
-    link = %r{http://localhost/email_confirmations([^";]*)}.match(body)
+    link = %r{http://localhost(?::\d+)?/email_confirmations([^";]*)}.match(body)
     link[0]
   end
 end

test/integration/password_reset_test.rb

@@ -5,7 +5,7 @@ class PasswordResetTest < SystemTest
 
   def password_reset_link
     body = ActionMailer::Base.deliveries.last.parts[1].body.decoded.to_s
-    link = %r{http://localhost/users([^";]*)}.match(body)
+    link = %r{http://localhost(?::\d+)?/users([^";]*)}.match(body)
     link[0]
   end
 

test/test_helper.rb

@@ -48,6 +48,7 @@
 Capybara.default_max_wait_time = 2
 Capybara.app_host = "#{Gemcutter::PROTOCOL}://#{Gemcutter::HOST}"
 Capybara.always_include_port = true
+Capybara.server_port = 31_337
 Capybara.server = :puma
 
 GoodJob::Execution.delete_all