Introduce Judge model

app/controllers/casa_orgs_controller.rb

@@ -3,6 +3,7 @@ class CasaOrgsController < ApplicationController
   before_action :set_casa_org, only: %i[edit update]
   before_action :set_contact_type_data, only: %i[edit update]
   before_action :set_hearing_types, only: %i[edit update]
+  before_action :set_judges, only: %i[edit update]
   before_action :must_be_admin
   before_action :require_organization!
 
@@ -42,4 +43,8 @@ def set_contact_type_data
   def set_hearing_types
     @hearing_types = HearingType.for_organization(@casa_org)
   end
+
+  def set_judges
+    @judges = Judge.for_organization(@casa_org)
+  end
 end

app/controllers/judges_controller.rb

@@ -0,0 +1,43 @@
+class JudgesController < ApplicationController
+  before_action :authenticate_user!, :must_be_admin
+  before_action :set_judge, except: [:new, :create]
+
+  def new
+    @judge = Judge.new
+  end
+
+  def create
+    @judge = Judge.new(judge_params)
+
+    respond_to do |format|
+      if @judge.save
+        format.html { redirect_to edit_casa_org_path(current_organization), notice: "Judge was successfully created." }
+      else
+        format.html { render :new }
+      end
+    end
+  end
+
+  def edit
+  end
+
+  def update
+    if @judge.update(judge_params)
+      redirect_to edit_casa_org_path(current_organization), notice: "Judge was successfully updated."
+    else
+      render :edit
+    end
+  end
+
+  private
+
+  def set_judge
+    @judge = Judge.find(params[:id])
+  end
+
+  def judge_params
+    params.require(:judge).permit(:name, :active).merge(
+      casa_org: current_organization
+    )
+  end
+end

app/models/judge.rb

@@ -0,0 +1,30 @@
+class Judge < ApplicationRecord
+  has_paper_trail
+
+  belongs_to :casa_org
+
+  validates :name, presence: true, uniqueness: {scope: %i[casa_org]}
+
+  scope :for_organization, ->(org) { where(casa_org: org) }
+  scope :active, -> { where(active: true) }
+end
+
+# == Schema Information
+#
+# Table name: judges
+#
+#  id          :bigint           not null, primary key
+#  active      :boolean          default(TRUE)
+#  name        :string
+#  created_at  :datetime         not null
+#  updated_at  :datetime         not null
+#  casa_org_id :bigint           not null
+#
+# Indexes
+#
+#  index_judges_on_casa_org_id  (casa_org_id)
+#
+# Foreign Keys
+#
+#  fk_rails_...  (casa_org_id => casa_orgs.id)
+#

app/views/casa_orgs/_hearing_types.html.erb

@@ -9,7 +9,7 @@
   <thead>
     <tr>
       <th>Name</th>
-      <th>Active</th>
+      <th>Active?</th>
       <th>Actions</th>
     </tr>
   </thead>

app/views/casa_orgs/_judges.html.erb

@@ -0,0 +1,32 @@
+<div class="row">
+  <div class="col-sm-12 dashboard-table-header pt-2 pb-2">
+    <h3>Judge</h3>
+    <%= link_to "New Judge", new_judge_path, class: "btn btn-primary" %>
+  </div>
+</div>
+
+<table class="table table-striped table-bordered">
+  <thead>
+    <tr>
+      <th>Name</th>
+      <th>Active?</th>
+      <th>Actions</th>
+    </tr>
+  </thead>
+
+  <tbody>
+    <% @judges.each do |judge| %>
+      <tr id="judge-<%= judge.id %>">
+        <td scope="row">
+          <%= judge.name %>
+        </td>
+        <td scope="row">
+          <%= judge.active ? "Yes" : "No" %>
+        </td>
+        <td>
+          <%= link_to "Edit", edit_judge_path(judge) %>
+        </td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>

app/views/casa_orgs/edit.html.erb

@@ -47,5 +47,6 @@
 <div class="card card-container">
   <div class="card-body">
     <%= render "hearing_types" %>
+    <%= render "judges" %>
   </div>
 </div>

app/views/judges/_form.html.erb

@@ -0,0 +1,22 @@
+<%= link_to 'Back', :back %>
+
+<h1><%= title %></h1>
+
+<div class="card card-container">
+  <div class="card-body">
+    <%= form_with(model: judge, local: true) do |form| %>
+      <%= render "/shared/error_messages", resource: judge %>
+      <div class="field form-group">
+        <%= form.label :name %>
+        <%= form.text_field :name, class: "form-control" %>
+      </div>
+      <div class="field form-group">
+        <%= form.check_box :active %>
+        <%= form.label :active %>
+      </div>
+      <div class="actions">
+        <%= form.submit "Submit", class: "btn btn-primary" %>
+      </div>
+    <% end %>
+  </div>
+</div>

app/views/judges/edit.html.erb

@@ -0,0 +1 @@
+<%= render partial: "form", locals: { title: "Edit Judge", judge: @judge } %>

app/views/judges/new.html.erb

@@ -0,0 +1 @@
+<%= render partial: "form", locals: { title: "New Judge", judge: @judge } %>

config/routes.rb

@@ -46,6 +46,7 @@
   resources :contact_type_groups, only: %i[new create edit update]
   resources :contact_types, only: %i[new create edit update]
   resources :hearing_types, only: %i[new create edit update]
+  resources :judges, only: %i[new create edit update]
 
   resources :supervisors, except: %i[destroy]
   resources :supervisor_volunteers, only: %i[create] do

db/migrate/20201023233638_create_judges.rb

@@ -0,0 +1,9 @@
+class CreateJudges < ActiveRecord::Migration[6.0]
+  def change
+    create_table :judges do |t|
+      t.references :casa_org, null: false, foreign_key: true
+
+      t.timestamps
+    end
+  end
+end

db/migrate/20201023234325_add_active_to_judge.rb

@@ -0,0 +1,5 @@
+class AddActiveToJudge < ActiveRecord::Migration[6.0]
+  def change
+    add_column :judges, :active, :boolean, default: true
+  end
+end

db/migrate/20201024003821_add_name_to_judge.rb

@@ -0,0 +1,5 @@
+class AddNameToJudge < ActiveRecord::Migration[6.0]
+  def change
+    add_column :judges, :name, :string
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2020_10_20_095451) do
+ActiveRecord::Schema.define(version: 2020_10_24_003821) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -153,6 +153,15 @@
     t.index ["casa_org_id"], name: "index_hearing_types_on_casa_org_id"
   end
 
+  create_table "judges", force: :cascade do |t|
+    t.bigint "casa_org_id", null: false
+    t.datetime "created_at", precision: 6, null: false
+    t.datetime "updated_at", precision: 6, null: false
+    t.boolean "active", default: true
+    t.string "name"
+    t.index ["casa_org_id"], name: "index_judges_on_casa_org_id"
+  end
+
   create_table "supervisor_volunteers", force: :cascade do |t|
     t.bigint "supervisor_id", null: false
     t.bigint "volunteer_id", null: false
@@ -213,6 +222,7 @@
   add_foreign_key "case_assignments", "users", column: "volunteer_id"
   add_foreign_key "case_contacts", "casa_cases"
   add_foreign_key "case_contacts", "users", column: "creator_id"
+  add_foreign_key "judges", "casa_orgs"
   add_foreign_key "supervisor_volunteers", "users", column: "supervisor_id"
   add_foreign_key "supervisor_volunteers", "users", column: "volunteer_id"
   add_foreign_key "users", "casa_orgs"

spec/factories/judges.rb

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :judge do
+    casa_org { create(:casa_org) }
+    name { Faker::Name.name }
+    active { true }
+  end
+end

spec/models/judge_spec.rb

@@ -0,0 +1,11 @@
+require "rails_helper"
+
+RSpec.describe Judge, type: :model do
+  it { is_expected.to belong_to(:casa_org) }
+  it { is_expected.to validate_presence_of(:name) }
+
+  it "has a valid factory" do
+    judge = build(:judge)
+    expect(judge.valid?).to be true
+  end
+end

spec/requests/judges_spec.rb

@@ -0,0 +1,147 @@
+require "rails_helper"
+
+RSpec.describe "/judges", type: :request do
+  describe "GET /judges/new" do
+    context "logged in as admin user" do
+      it "can successfully access a judge create page" do
+        sign_in_as_admin
+
+        get new_judge_path
+
+        expect(response).to be_successful
+      end
+    end
+
+    context "logged in as a non-admin user" do
+      it "cannot access a judge create page" do
+        sign_in_as_volunteer
+
+        get new_judge_path
+
+        expect(response).to redirect_to root_path
+        expect(response.request.flash[:notice]).to eq "Sorry, you are not authorized to perform this action."
+      end
+    end
+
+    context "unauthenticated request" do
+      it "cannot access a judge create page" do
+        get new_judge_path
+
+        expect(response).to redirect_to new_user_session_path
+      end
+    end
+  end
+
+  describe "POST /judges" do
+    let(:params) { {judge: {name: "Joe Judge", active: true}} }
+    context "logged in as admin user" do
+      it "can successfully create a judge" do
+        casa_org = create(:casa_org)
+        sign_in create(:casa_admin, casa_org: casa_org)
+
+        expect {
+          post judges_path, params: params
+        }.to change(Judge, :count).by(1)
+
+        judge = Judge.last
+
+        expect(judge.name).to eql "Joe Judge"
+        expect(judge.casa_org).to eql casa_org
+        expect(judge.active).to be_truthy
+        expect(response).to redirect_to edit_casa_org_path(casa_org)
+        expect(response.request.flash[:notice]).to eq "Judge was successfully created."
+      end
+    end
+
+    context "logged in as a non-admin user" do
+      it "cannot create a judge" do
+        sign_in_as_volunteer
+
+        post judges_path, params: params
+
+        expect(response).to redirect_to root_path
+        expect(response.request.flash[:notice]).to eq "Sorry, you are not authorized to perform this action."
+      end
+    end
+
+    context "unauthenticated request" do
+      it "cannot create a judge" do
+        post judges_path, params: params
+
+        expect(response).to redirect_to new_user_session_path
+      end
+    end
+  end
+
+  describe "GET /judges/:id/edit" do
+    let(:judge) { create(:judge) }
+
+    context "logged in as admin user" do
+      it "can successfully access a judge edit page" do
+        sign_in_as_admin
+
+        get edit_judge_path(judge)
+
+        expect(response).to be_successful
+      end
+    end
+
+    context "logged in as a non-admin user" do
+      it "cannot access a judge edit page" do
+        sign_in_as_volunteer
+
+        get edit_judge_path(judge)
+
+        expect(response).to redirect_to root_path
+        expect(response.request.flash[:notice]).to eq "Sorry, you are not authorized to perform this action."
+      end
+    end
+
+    context "unauthenticated request" do
+      it "cannot access a judge edit page" do
+        get edit_judge_path(judge)
+
+        expect(response).to redirect_to new_user_session_path
+      end
+    end
+  end
+
+  describe "PUT /judges/:id" do
+    let(:judge) { create(:judge) }
+    let(:params) { {judge: {name: "New Name", judge_id: judge.id, active: false}} }
+    context "logged in as admin user" do
+      it "can successfully update a judge" do
+        casa_org = create(:casa_org)
+        sign_in create(:casa_admin, casa_org: casa_org)
+
+        put judge_path(judge), params: params
+
+        judge.reload
+        expect(judge.name).to eq "New Name"
+        expect(judge.active).to be_falsey
+
+        expect(response).to redirect_to edit_casa_org_path(casa_org)
+        expect(response.request.flash[:notice]).to eq "Judge was successfully updated."
+      end
+    end
+
+    context "logged in as a non-admin user" do
+      it "cannot update a judge" do
+        sign_in_as_volunteer
+
+        put judge_path(judge), params: params
+
+        expect(response).to redirect_to root_path
+        expect(response.request.flash[:notice]).to eq "Sorry, you are not authorized to perform this action."
+      end
+    end
+
+    context "unauthenticated request" do
+      it "cannot update a judge" do
+        put judge_path(judge), params: params
+
+        expect(response).to redirect_to new_user_session_path
+      end
+    end
+  end
+end