Applied rubocop suggestions.

test/browser/integration_cti_test.rb

@@ -3,7 +3,7 @@
 require 'browser_test_helper'
 
 class IntegrationCtiTest < TestCase
-  setup do
+  before do
     if !ENV['CTI_TOKEN']
       raise "ERROR: Need CTI_TOKEN - hint CTI_TOKEN='some_token'"
     end

test/browser_test_helper.rb

@@ -24,7 +24,7 @@ class TestCase < ActiveSupport::TestCase
 
   DEBUG = true
 
-  setup do
+  before do
     # print current test case to STDOUT
     # for status reasoning and debugging purposes
     source_location  = self.class.instance_method(method_name).source_location

test/integration/elasticsearch_active_test.rb

@@ -5,7 +5,7 @@ require 'test_helper'
 class ElasticsearchActiveTest < ActiveSupport::TestCase
   include SearchindexHelper
 
-  setup do
+  before do
 
     configure_elasticsearch(required: true)
 

test/integration/elasticsearch_test.rb

@@ -5,7 +5,7 @@ require 'test_helper'
 class ElasticsearchTest < ActiveSupport::TestCase
   include SearchindexHelper
 
-  setup do
+  before do
 
     configure_elasticsearch(required: true)
 

test/integration/email_keep_on_server_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 require 'net/imap'
 
 class EmailKeepOnServerTest < ActiveSupport::TestCase
-  setup do
+  before do
 
     if ENV['KEEP_ON_MAIL_SERVER'].blank?
       raise "Need KEEP_ON_MAIL_SERVER as ENV variable like export KEEP_ON_MAIL_SERVER='mx.example.com'"

test/integration/email_postmaster_to_sender.rb

@@ -5,7 +5,7 @@ require 'net/imap'
 
 class EmailPostmasterToSender < ActiveSupport::TestCase
 
-  setup do
+  before do
     Setting.set('postmaster_max_size', 0.1)
 
     @test_id = rand(999_999_999)

test/integration/geo_location_test.rb

@@ -5,7 +5,7 @@ require 'webmock/minitest'
 
 class GeoLocationTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     @mock = true
     #WebMock.allow_net_connect!
   end

test/integration/report_test.rb

@@ -5,7 +5,7 @@ require 'integration_test_helper'
 class ReportTest < ActiveSupport::TestCase
   include SearchindexHelper
 
-  setup do
+  before do
 
     # create attribute
     ObjectManager::Attribute.add(

test/integration/user_agent_test.rb

@@ -6,7 +6,7 @@ class UserAgentTest < ActiveSupport::TestCase
   host = 'https://web-test.dc.zammad.com'
   #host = 'http://127.0.0.1:3003'
 
-  setup do
+  before do
     if ENV['ZAMMAD_PROXY_TEST'] == 'true'
       Setting.set('proxy', ENV['ZAMMAD_PROXY'])
       Setting.set('proxy_username', ENV['ZAMMAD_PROXY_USERNAME'])

test/integration_test_helper.rb

@@ -25,7 +25,7 @@ class ActiveSupport::TestCase
   # set system mode to done / to activate
   Setting.set('system_init_done', true)
 
-  setup do
+  before do
 
     # clear cache
     Cache.clear

test/test_helper.rb

@@ -22,7 +22,7 @@ class ActiveSupport::TestCase
   # set system mode to done / to activate
   Setting.set('system_init_done', true)
 
-  setup do
+  before do
 
     # exit all threads
     Thread.list.each do |thread|

test/unit/activity_stream_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class ActivityStreamTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     roles = Role.where(name: %w[Admin Agent])
     groups = Group.where(name: 'Users')
     @admin_user = User.create_or_update(

test/unit/chat_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class ChatTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     groups = Group.all
     roles  = Role.where(name: %w[Agent])
     @agent1 = User.create!(

test/unit/email_process_sender_is_system_address_or_agent_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class EmailProcessSenderIsSystemAddressOrAgent < ActiveSupport::TestCase
 
-  setup do
+  before do
     EmailAddress.create_or_update(
       channel_id:    1,
       realname:      'My System',

test/unit/integration_icinga_test.rb

@@ -9,7 +9,7 @@ class IntegrationIcingaTest < ActiveSupport::TestCase
   # http://docs.icinga.org/icinga2/latest/doc/module/icinga2/chapter/monitoring-basics#host-states
   # http://docs.icinga.org/icinga2/latest/doc/module/icinga2/chapter/monitoring-basics#service-states
 
-  setup do
+  before do
     Setting.set('icinga_integration', true)
     Setting.set('icinga_sender', 'icinga2@monitoring.example.com')
   end

test/unit/integration_monit_test.rb

@@ -7,7 +7,7 @@ class IntegrationMonitTest < ActiveSupport::TestCase
   # according
   # https://mmonit.com/monit/documentation/#ALERT-MESSAGES
 
-  setup do
+  before do
     Setting.set('monit_integration', true)
     Setting.set('monit_sender', 'monit@monitoring.example.com')
   end

test/unit/integration_nagios_test.rb

@@ -7,7 +7,7 @@ class IntegrationNagiosTest < ActiveSupport::TestCase
   # according
   # https://github.com/NagiosEnterprises/nagioscore/blob/754218e67653929a58938b99ef6b6039b6474fe4/sample-config/template-object/commands.cfg.in#L35
 
-  setup do
+  before do
     Setting.set('nagios_integration', true)
     Setting.set('nagios_sender', 'nagios2@monitoring.example.com')
   end

test/unit/online_notifiaction_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class OnlineNotificationTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     role = Role.lookup(name: 'Agent')
     @group = Group.create_or_update(
       name:          'OnlineNotificationTest',

test/unit/ticket_last_owner_update_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketLastOwnerUpdateTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     Group.create_or_update(
       name:               'LastOwnerUpdate',
       email_address:      EmailAddress.first,

test/unit/ticket_notification_test.rb

@@ -3,7 +3,7 @@
 require 'test_helper'
 
 class TicketNotificationTest < ActiveSupport::TestCase
-  setup do
+  before do
     Setting.set('timezone_default', 'Europe/Berlin')
     Trigger.create_or_update(
       name:                 'auto reply - new ticket',

test/unit/ticket_overview_out_of_office_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketOverviewOutOfOfficeTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     group = Group.create_or_update(
       name:          'OverviewReplacementTest',
       updated_at:    '2015-02-05 16:37:00',

test/unit/ticket_overview_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketOverviewTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     group = Group.create_or_update(
       name:          'OverviewTest',
       updated_at:    '2015-02-05 16:37:00',

test/unit/ticket_selector_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketSelectorTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     @group = Group.create_or_update(
       name:          'SelectorTest',
       updated_at:    '2015-02-05 16:37:00',

test/unit/ticket_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     Ticket.destroy_all
   end
 

test/unit/ticket_trigger_extended_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketTriggerExtendedTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     Setting.set('ticket_trigger_recursive', true)
   end
 

test/unit/ticket_trigger_recursive_disabled_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketTriggerRecursiveDisabledTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     Setting.set('ticket_trigger_recursive', false)
   end
 

test/unit/ticket_trigger_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 
 class TicketTriggerTest < ActiveSupport::TestCase
 
-  setup do
+  before do
     Setting.set('ticket_trigger_recursive', true)
   end