refactor: Renames event_tracking to Volunteer

Author: cbaudouinjr

app/controllers/manage/application_controller.rb

@@ -1,6 +1,6 @@
 class Manage::ApplicationController < ApplicationController
   before_action :logged_in
-  before_action :require_director_or_organizer_or_event_tracking
+  before_action :require_director_or_organizer_or_volunteer
   before_action :limit_write_access_to_directors, only: ["edit", "update", "new", "create", "destroy", "convert_to_admin", "deliver", "merge", "perform_merge", "toggle_bus_captain", "duplicate", "update_acc_status", "send_update_email", "live_preview"]
   skip_before_action :verify_authenticity_token, if: :json_request?
 
@@ -16,8 +16,8 @@ def require_director_or_organizer
     return redirect_to root_path unless current_user.try(:director?) || current_user.try(:organizer?)
   end
 
-  def require_director_or_organizer_or_event_tracking
-    redirect_to root_path unless current_user.try(:director?) || current_user.try(:organizer?) || current_user.try(:event_tracking?)
+  def require_director_or_organizer_or_volunteer
+    redirect_to root_path unless current_user.try(:director?) || current_user.try(:organizer?) || current_user.try(:volunteer?)
   end
 
   def limit_write_access_to_directors

app/controllers/manage/users_controller.rb

@@ -5,7 +5,7 @@ class Manage::UsersController < Manage::ApplicationController
   respond_to :html, :json
 
   def index
-    respond_with(:manage, User.where(role: [:director, :organizer, :event_tracking]))
+    respond_with(:manage, User.where(role: [:director, :organizer, :volunteer]))
   end
 
   def user_datatable

app/datatables/staff_datatable.rb

@@ -43,6 +43,6 @@ def data
   end
 
   def get_raw_records
-    User.where(role: [:director, :organizer, :event_tracking])
+    User.where(role: [:director, :organizer, :volunteer])
   end
 end

app/models/user.rb

@@ -23,7 +23,7 @@ class User < ApplicationRecord
   after_create :queue_reminder_email
   after_initialize :set_default_role, if: :new_record?
 
-  enum role: { user: 0, event_tracking: 1, organizer: 2, director: 3 }
+  enum role: { user: 0, volunteer: 1, organizer: 2, director: 3 }
 
   def set_default_role
     self.role ||= :user

test/controllers/manage/checkins_controller_test.rb

@@ -53,7 +53,7 @@ class Manage::CheckinsControllerTest < ActionController::TestCase
   end
 
   limited_conditions = {
-    'event tracking user' => :event_tracking,
+    'volunteer' => :volunteer,
   }
 
   limited_conditions.each do |condition_name, user_role|

test/controllers/manage/trackable_events_controller_test.rb

@@ -52,7 +52,7 @@ class Manage::TrackableEventsControllerTest < ActionController::TestCase
   end
 
   limited_conditions = {
-    'event tracking user' => :event_tracking,
+    'volunteer' => :volunteer,
     'organizer' => :organizer
   }
 

test/controllers/manage/trackable_tags_controller_test.rb

@@ -51,7 +51,7 @@ class Manage::TrackableTagsControllerTest < ActionController::TestCase
   end
 
   limited_conditions = {
-    'event tracking user' => :event_tracking,
+    'volunteer' => :volunteer,
     'organizer' => :organizer
   }
 

test/models/user_test.rb

@@ -66,7 +66,7 @@ class UserTest < ActiveSupport::TestCase
 
     should "not return staff" do
       create(:questionnaire) # user, has questionnaire
-      create(:user, role: :event_tracking) # user, does not
+      create(:user, role: :volunteer) # volunteer, does not
       create(:user, role: :organizer) # organizer, does not
       create(:user, role: :director) # director, does not
       assert_equal 4, User.count