<%= form_with(model: @case_contact, url: wizard_path(nil, case_contact_id: @case_contact.id), id: "casa-contact-form", class: "component-validated-form", data: { "turbo-action": "advance", "autosave-target": "form" }) do |form| %>
- <%= render "/shared/error_messages", resource: @case_contact %>
-
+ <%= render "contact_topic_notes", form: %>
-
-
<%= form.label :notes, "1. Additional notes (optional)" %>
+
+
<%= form.label :notes, "#{@case_contact.contact_topic_answers.count + 1}. Additional notes (optional)" %>
@@ -23,7 +25,7 @@
Include any additional comments or notes that may assist in future case tracking or reporting.
- <%= form.text_area :notes, :rows => 5, placeholder: "Additional notes", class: "form-control", data: { action: "keyup->autosave#save" } %>
+ <%= form.text_area :notes, :rows => 5, placeholder: "Additional notes", class: "form-control", data: { action: "input->autosave#save" } %>
diff --git a/app/views/contact_topics/_form.html.erb b/app/views/contact_topics/_form.html.erb
new file mode 100644
index 0000000000..8d93894422
--- /dev/null
+++ b/app/views/contact_topics/_form.html.erb
@@ -0,0 +1,39 @@
+
+
+
+
+ <%= form_with(model: contact_topic, local: true) do |form| %>
+ <%= form.hidden_field :casa_org_id %>
+
+ <%= render "/shared/error_messages", resource: contact_topic %>
+
+
+ <%= form.label :question, "Question" %>
+ <%= form.text_field :question, class: "form-control", required: true %>
+
+
+ <%= form.label :details, "Details?" %>
+ <%= form.text_area :details, rows: 5, class: "form-control", required: true %>
+
+
+ <%= form.check_box :active, class: 'form-check-input' %>
+ <%= form.label :active, "Active?", class: 'form-check-label' %>
+
+
+ <%= button_tag(type: "submit", class: "btn-sm main-btn primary-btn btn-hover") do %>
+
+ <% end %>
+
+
diff --git a/app/views/contact_topics/edit.html.erb b/app/views/contact_topics/edit.html.erb
new file mode 100644
index 0000000000..a7a887e793
--- /dev/null
+++ b/app/views/contact_topics/edit.html.erb
@@ -0,0 +1 @@
+<%= render partial: "form", locals: {title: "Contact Topic", contact_topic: @contact_topic} %>
diff --git a/app/views/contact_topics/new.html.erb b/app/views/contact_topics/new.html.erb
new file mode 100644
index 0000000000..6ab25614b7
--- /dev/null
+++ b/app/views/contact_topics/new.html.erb
@@ -0,0 +1 @@
+<%= render partial: "form", locals: {title: "New Contact Topic", contact_topic: @contact_topic} %>
diff --git a/config/routes.rb b/config/routes.rb
index f4be75cf06..e0c8161f37 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -116,6 +116,11 @@
resources :learning_hours_reports, only: %i[index]
resources :learning_hour_types, only: %i[new create edit update]
resources :learning_hour_topics, only: %i[new create edit update]
+
+ resources :contact_topics, except: %i[index show delete] do
+ delete "soft_delete", on: :member
+ end
+
resources :followup_reports, only: :index
resources :placement_reports, only: :index
resources :banners, except: %i[show] do
diff --git a/db/migrate/20240216013254_add_contact_topics.rb b/db/migrate/20240216013254_add_contact_topics.rb
new file mode 100644
index 0000000000..94f0cff052
--- /dev/null
+++ b/db/migrate/20240216013254_add_contact_topics.rb
@@ -0,0 +1,22 @@
+class AddContactTopics < ActiveRecord::Migration[7.1]
+ def change
+ create_table :contact_topics do |t|
+ t.references :casa_org, null: false, foreign_key: true
+ t.boolean :active, null: false, default: true
+ t.boolean :soft_delete, null: false, default: false
+ t.text :details
+ t.string :question
+
+ t.timestamps
+ end
+
+ create_table :contact_topic_answers do |t|
+ t.text :value
+ t.references :case_contact, null: false, foreign_key: true
+ t.references :contact_topic, null: false, foreign_key: true
+ t.boolean :selected, null: false, default: false
+
+ t.timestamps
+ end
+ end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 40340f3789..eab57fb9aa 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.
-ActiveRecord::Schema[7.0].define(version: 2023_11_25_150721) do
+ActiveRecord::Schema[7.1].define(version: 2024_02_16_013254) do
# These are extensions that must be enabled in order to support this database
enable_extension "plpgsql"
@@ -245,6 +245,28 @@
t.index ["hearing_type_id"], name: "index_checklist_items_on_hearing_type_id"
end
+ create_table "contact_topic_answers", force: :cascade do |t|
+ t.text "value"
+ t.bigint "case_contact_id", null: false
+ t.bigint "contact_topic_id", null: false
+ t.boolean "selected", default: false, null: false
+ t.datetime "created_at", null: false
+ t.datetime "updated_at", null: false
+ t.index ["case_contact_id"], name: "index_contact_topic_answers_on_case_contact_id"
+ t.index ["contact_topic_id"], name: "index_contact_topic_answers_on_contact_topic_id"
+ end
+
+ create_table "contact_topics", force: :cascade do |t|
+ t.bigint "casa_org_id", null: false
+ t.boolean "active", default: true, null: false
+ t.boolean "soft_delete", default: false, null: false
+ t.text "details"
+ t.string "question"
+ t.datetime "created_at", null: false
+ t.datetime "updated_at", null: false
+ t.index ["casa_org_id"], name: "index_contact_topics_on_casa_org_id"
+ end
+
create_table "contact_type_groups", force: :cascade do |t|
t.bigint "casa_org_id", null: false
t.string "name", null: false
@@ -366,12 +388,6 @@
t.index ["casa_org_id"], name: "index_judges_on_casa_org_id"
end
- create_table "jwt_denylist", force: :cascade do |t|
- t.string "jti", null: false
- t.datetime "exp", null: false
- t.index ["jti"], name: "index_jwt_denylist_on_jti"
- end
-
create_table "languages", force: :cascade do |t|
t.string "name"
t.bigint "casa_org_id", null: false
@@ -637,6 +653,9 @@
add_foreign_key "case_group_memberships", "casa_cases"
add_foreign_key "case_group_memberships", "case_groups"
add_foreign_key "case_groups", "casa_orgs"
+ add_foreign_key "contact_topic_answers", "case_contacts"
+ add_foreign_key "contact_topic_answers", "contact_topics"
+ add_foreign_key "contact_topics", "casa_orgs"
add_foreign_key "court_dates", "casa_cases"
add_foreign_key "emancipation_options", "emancipation_categories"
add_foreign_key "followups", "users", column: "creator_id"
@@ -646,14 +665,14 @@
add_foreign_key "learning_hour_types", "casa_orgs"
add_foreign_key "learning_hours", "learning_hour_types"
add_foreign_key "learning_hours", "users"
- add_foreign_key "mileage_rates", "casa_orgs"
+ add_foreign_key "mileage_rates", "casa_orgs", validate: false
add_foreign_key "mileage_rates", "users"
- add_foreign_key "notes", "users", column: "creator_id"
+ add_foreign_key "notes", "users", column: "creator_id", validate: false
add_foreign_key "other_duties", "users", column: "creator_id"
add_foreign_key "patch_notes", "patch_note_groups"
add_foreign_key "patch_notes", "patch_note_types"
add_foreign_key "placement_types", "casa_orgs"
- add_foreign_key "placements", "casa_cases"
+ add_foreign_key "placements", "casa_cases", validate: false
add_foreign_key "placements", "placement_types"
add_foreign_key "placements", "users", column: "creator_id"
add_foreign_key "preference_sets", "users"
diff --git a/db/seeds.rb b/db/seeds.rb
index 1751534412..106c853b9f 100644
--- a/db/seeds.rb
+++ b/db/seeds.rb
@@ -12,6 +12,7 @@
require_relative "../lib/tasks/data_post_processors/case_contact_populator"
require_relative "../lib/tasks/data_post_processors/contact_type_populator"
require_relative "../lib/tasks/data_post_processors/sms_notification_event_populator"
+require_relative "../lib/tasks/data_post_processors/contact_topic_populator"
class SeederMain
attr_reader :db_populator, :rng
@@ -76,6 +77,7 @@ def active_record_classes
def post_process_data
ContactTypePopulator.populate
CaseContactPopulator.populate
+ ContactTopicPopulator.populate
end
def get_seed_specification
diff --git a/db/seeds/default_contact_topics.yml b/db/seeds/default_contact_topics.yml
new file mode 100644
index 0000000000..f1b83c0831
--- /dev/null
+++ b/db/seeds/default_contact_topics.yml
@@ -0,0 +1,32 @@
+- question: "Background information"
+ details: |-
+ a) When did the family first come into contact with the Department of Social Services or Department of Juvenile Justice – how many times?
+ b) Tell the history of their involvement with the department and any facts about their life that could help determine the need for placement and/or services.
+ c) Discuss the child’s history – behavior problems, educational history, medical history, psychological history (any hospitalizations, previous counseling, etc.)
+ d) If child has been placed previously give a history of the child’s placements (placed with different parents, relatives, DSS, etc).
+- question: "Current situation"
+ details: |-
+ a) Where is the child placed?
+ b) How is the child adjusting to the placement?
+ c) Are there any issues or concerns about the placement? If so, describe these concerns and specify the actions being taken to address them.
+- question: "Education, vocation, or daycare"
+ details: |-
+ a) Where is the child placed for education (daycare, public school, non-public school, GED, Job Corps, etc)?
+ b) How is the child adjusting to the educational placement? Are there any education-related concerns at this point? If yes, detail them and mention the steps taken to address them.
+ c) Does the child have an IEP? If not, is there a need for one?
+ d) Is the child employed? If not, are they looking for a job?
+ e) Does the child have vocational/life skills? Are they attending life skill classes?
+ f) Are there any other life skill needs? (Driver’s education, state ID, transportation assistance, etc.)
+ g) What is the feedback from professionals providing these services about the child's progress? Include strengths and not just needs.
+- question: "Health and mental health"
+ details: |-
+ a) Is the child up to date with medical exams?
+ b) Are there any other medical concerns?
+ c) Is the child receiving therapy, medication monitoring, mentoring, or other services? If so, specify with whom these services are being received.
+- question: "Family and community connections"
+ details: |-
+ a) Is this child seeing parents, siblings, other relatives? If so, who is the child visiting, and how often? Does the child desire a different arrangement?
+ b) Detail the steps parents have taken to address court orders. Address any barriers and highlight positive steps.
+- question: "Child’s strengths"
+ details: |-
+ a) Describe the child’s strengths, interests, and hobbies to provide a well-rounded perspective.
diff --git a/lib/tasks/data_post_processors/contact_topic_populator.rb b/lib/tasks/data_post_processors/contact_topic_populator.rb
new file mode 100644
index 0000000000..b433b8cadd
--- /dev/null
+++ b/lib/tasks/data_post_processors/contact_topic_populator.rb
@@ -0,0 +1,13 @@
+module ContactTopicPopulator
+ def self.populate
+ CasaOrg.all.each do |casa_org|
+ ContactTopic.generate_for_org!(casa_org)
+ topics = casa_org.contact_topics
+ topics.each do |topic|
+ CaseContact.all.each do |contact|
+ FactoryBot.create(:contact_topic_answer, case_contact: contact, contact_topic: topic)
+ end
+ end
+ end
+ end
+end
diff --git a/spec/components/dropdown_menu_component_spec.rb b/spec/components/dropdown_menu_component_spec.rb
new file mode 100644
index 0000000000..64f6ad7636
--- /dev/null
+++ b/spec/components/dropdown_menu_component_spec.rb
@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe DropdownMenuComponent, type: :component do
+ it "renders the dropdown menu with an icon and label" do
+ render_inline(DropdownMenuComponent.new(menu_title: "Example", icon_name: "example-icon")) { "Example Content" }
+
+ expect(page).to have_css("div.dropdown")
+ expect(page).to have_css("button.btn.btn-secondary.dropdown-toggle span", text: "Example")
+ expect(page).to have_css("button.btn.btn-secondary.dropdown-toggle i.lni.mr-10.lni-example-icon")
+ expect(page).to have_css(".dropdown-menu", text: "Example Content")
+ end
+
+ it "renders the dropdown menu with a hidden label" do
+ render_inline(DropdownMenuComponent.new(menu_title: "Example", icon_name: "example-icon", hide_label: true)) { "Example Content" }
+
+ expect(page).to have_css("div.dropdown")
+ expect(page).to have_css("button.btn.btn-secondary.dropdown-toggle span.sr-only", text: "Example")
+ expect(page).to have_css("button.btn.btn-secondary.dropdown-toggle i.lni.mr-10.lni-example-icon")
+ expect(page).to have_css(".dropdown-menu", text: "Example Content")
+ end
+
+ it "renders the dropdown menu with only a label and content" do
+ render_inline(DropdownMenuComponent.new(menu_title: "Example Title")) { "Example Item" }
+
+ expect(page).to have_css("div.dropdown")
+ expect(page).to have_css("button.btn.btn-secondary.dropdown-toggle svg")
+ expect(page).to have_css("svg title", text: "Example Title")
+ expect(page).to have_css(".dropdown-menu", text: "Example Item")
+ end
+
+ it "doesn't render anything if no content provided" do
+ render_inline(DropdownMenuComponent.new(menu_title: nil))
+
+ expect(page).not_to have_css("div.dropdown")
+ end
+
+ it "renders the dropdown menu with additional classes" do
+ render_inline(DropdownMenuComponent.new(menu_title: "Example", klass: "example-class")) { "Example Content" }
+
+ expect(page).to have_css("div.dropdown.example-class")
+ end
+
+ it "doesn't render if render_check is false" do
+ render_inline(DropdownMenuComponent.new(menu_title: "Example", render_check: false))
+
+ expect(page).not_to have_css("div.dropdown")
+ end
+end
diff --git a/spec/components/modal/body_component_spec.rb b/spec/components/modal/body_component_spec.rb
new file mode 100644
index 0000000000..91d59e3d3c
--- /dev/null
+++ b/spec/components/modal/body_component_spec.rb
@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe Modal::BodyComponent, type: :component do
+ it "renders the body with text" do
+ render_inline(Modal::BodyComponent.new(text: "Example Body", klass: "example-class"))
+
+ expect(page).to have_css("div.modal-body.example-class")
+ expect(page).to have_css("div.modal-body p", text: "Example Body")
+ end
+
+ it "renders the body with multiple paragraphs" do
+ render_inline(Modal::BodyComponent.new(text: ["Paragraph 1", "Paragraph 2"]))
+
+ expect(page).to have_css("div.modal-body p", text: "Paragraph 1")
+ expect(page).to have_css("div.modal-body p", text: "Paragraph 2")
+ end
+
+ it "renders the body with content" do
+ render_inline(Modal::BodyComponent.new) do
+ "Content Override"
+ end
+
+ expect(page).to have_css("div.modal-body", text: "Content Override")
+ end
+
+ it "renders the body with content and overrides text" do
+ render_inline(Modal::BodyComponent.new(text: "Example Body")) do
+ "Content Override"
+ end
+
+ expect(page).to have_css("div.modal-body", text: "Content Override")
+ expect(page).not_to have_css("div.modal-body", text: "Example Body")
+ end
+
+ it "does not render if text and content missing" do
+ render_inline(Modal::BodyComponent.new)
+
+ expect(page).not_to have_css("div.modal-body")
+ end
+
+ it "doesn't render if render_check is false" do
+ render_inline(Modal::BodyComponent.new(text: "Example Body", render_check: false))
+
+ expect(page).not_to have_css("div.modal-body")
+ end
+end
diff --git a/spec/components/modal/footer_component_spec.rb b/spec/components/modal/footer_component_spec.rb
new file mode 100644
index 0000000000..d50488755d
--- /dev/null
+++ b/spec/components/modal/footer_component_spec.rb
@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe Modal::FooterComponent, type: :component do
+ it "renders the footer with content" do
+ render_inline(Modal::FooterComponent.new(klass: "example-class")) do
+ "Footer Content"
+ end
+
+ expect(page).to have_css("div.modal-footer.example-class")
+ expect(page).to have_css("div.modal-footer button.btn.btn-secondary", text: "Close")
+ expect(page).to have_text("Footer Content")
+ end
+
+ it "does not render the footer if content missing" do
+ render_inline(Modal::FooterComponent.new)
+
+ expect(page).not_to have_css("div.modal-footer")
+ end
+
+ it "doesn't render if render_check is false" do
+ render_inline(Modal::FooterComponent.new(render_check: false)) do
+ "Footer Content"
+ end
+
+ expect(page).not_to have_css("div.modal-footer")
+ end
+end
diff --git a/spec/components/modal/group_component_spec.rb b/spec/components/modal/group_component_spec.rb
new file mode 100644
index 0000000000..6d681fdac5
--- /dev/null
+++ b/spec/components/modal/group_component_spec.rb
@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe Modal::GroupComponent, type: :component do
+ before do
+ @component = Modal::GroupComponent.new(id: "exampleModal", klass: "example-class")
+ end
+
+ it "renders the modal group with header, body, and footer" do
+ @component.with_header(id: "header-id") { "Example Header" }
+ @component.with_body { "Example Body" }
+ @component.with_footer { "Example Footer" }
+ render_inline(@component)
+
+ expect(page).to have_css("div.modal.fade.example-class#exampleModal")
+ expect(page).to have_css("div.modal-dialog.modal-dialog-centered")
+ expect(page).to have_css("div.modal-content")
+ expect(page).to have_text("Example Header")
+ expect(page).to have_text("Example Body")
+ expect(page).to have_text("Example Footer")
+ end
+
+ it "renders the modal group with only a header" do
+ @component.with_header(id: "header-id") { "Example Header" }
+ render_inline(@component)
+
+ expect(page).to have_css("div.modal.fade.example-class#exampleModal")
+ expect(page).to have_css("div.modal-dialog.modal-dialog-centered")
+ expect(page).to have_css("div.modal-content")
+ expect(page).to have_text("Example Header")
+ expect(page).not_to have_css("div.modal-body")
+ expect(page).not_to have_css("div.modal-footer")
+ end
+
+ it "renders the modal group with only a body" do
+ @component.with_body { "Example Body" }
+ render_inline(@component)
+
+ expect(page).to have_css("div.modal.fade.example-class#exampleModal")
+ expect(page).to have_css("div.modal-dialog.modal-dialog-centered")
+ expect(page).to have_css("div.modal-content")
+ expect(page).to have_text("Example Body")
+ expect(page).not_to have_css("div.modal-header")
+ expect(page).not_to have_css("div.modal-footer")
+ end
+
+ it "doesn't render anything if no content provided" do
+ render_inline(@component)
+
+ expect(page).not_to have_css("div.modal.fade.example-class#exampleModal")
+ expect(page).not_to have_css("div.modal-dialog.modal-dialog-centered")
+ expect(page).not_to have_css("div.modal-content")
+ end
+
+ it "doesn't render if render_check is false" do
+ @component = Modal::GroupComponent.new(id: "exampleModal", klass: "example-class", render_check: false)
+ @component.with_header(id: "header-id") { "Example Header" }
+ @component.with_body { "Example Body" }
+ @component.with_footer { "Example Footer" }
+ render_inline(@component)
+
+ expect(page).not_to have_css("div.modal.fade.example-class#exampleModal")
+ expect(page).not_to have_css("div.modal-dialog.modal-dialog-centered")
+ expect(page).not_to have_css("div.modal-content")
+ end
+end
diff --git a/spec/components/modal/header_component_spec.rb b/spec/components/modal/header_component_spec.rb
new file mode 100644
index 0000000000..6b2bb8af47
--- /dev/null
+++ b/spec/components/modal/header_component_spec.rb
@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe Modal::HeaderComponent, type: :component do
+ it "renders the header with text and icon" do
+ render_inline(Modal::HeaderComponent.new(text: "Example Header", id: "modalHeader", icon: "example-icon", klass: "example-class"))
+
+ expect(page).to have_css("div.modal-header.example-class")
+ expect(page).to have_css("div.modal-header h1#modalHeader-label.modal-title.fs-5")
+ expect(page).to have_css("div.modal-header h1#modalHeader-label i.lni.mr-10.lni-example-icon")
+ expect(page).to have_text("Example Header")
+ expect(page).to have_css("div.modal-header button.btn-close")
+ end
+
+ it "renders the header with only text" do
+ render_inline(Modal::HeaderComponent.new(text: "Example Header", id: "modalHeader"))
+
+ expect(page).to have_css("div.modal-header")
+ expect(page).to have_css("div.modal-header h1#modalHeader-label.modal-title.fs-5")
+ expect(page).not_to have_css("div.modal-header i")
+ expect(page).to have_text("Example Header")
+ expect(page).to have_css("div.modal-header button.btn-close")
+ end
+
+ it "renders the header with content" do
+ render_inline(Modal::HeaderComponent.new(id: "modalHeader")) do
+ "Header Content"
+ end
+
+ expect(page).to have_css("div.modal-header")
+ expect(page).to have_text("Header Content")
+ expect(page).to have_css("div.modal-header button.btn-close")
+ end
+
+ it "content overrides text" do
+ render_inline(Modal::HeaderComponent.new(id: "modalHeader", text: "Missing")) do
+ "Header Content"
+ end
+
+ expect(page).to have_css("div.modal-header")
+ expect(page).to have_text("Header Content")
+ expect(page).to_not have_text("Missing")
+ expect(page).to have_css("div.modal-header button.btn-close")
+ end
+
+ it "doesn't render anything if both text and content are absent" do
+ render_inline(Modal::HeaderComponent.new(id: "modalHeader"))
+
+ expect(page).not_to have_css("div.modal-header")
+ end
+
+ it "doesn't render if render_check is false" do
+ render_inline(Modal::HeaderComponent.new(text: "Example Header", id: "modalHeader", render_check: false))
+
+ expect(page).not_to have_css("div.modal-header")
+ end
+end
diff --git a/spec/components/modal/open_button_component_spec.rb b/spec/components/modal/open_button_component_spec.rb
new file mode 100644
index 0000000000..6052e9f70c
--- /dev/null
+++ b/spec/components/modal/open_button_component_spec.rb
@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+# spec/components/modal/open_button_component_spec.rb
+
+require "rails_helper"
+
+RSpec.describe Modal::OpenButtonComponent, type: :component do
+ it "renders the button with text and icon" do
+ render_inline(Modal::OpenButtonComponent.new(target: "myModal", text: "Example Text", icon: "example-icon", klass: "example-class"))
+
+ expect(page).to have_css("button[type='button'][class='btn example-class'][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).to have_css("button i.lni.mr-10.lni-example-icon")
+ expect(page).to have_text("Example Text")
+ end
+
+ it "renders the button with only text" do
+ render_inline(Modal::OpenButtonComponent.new(target: "myModal", text: "Example Text"))
+
+ expect(page).to have_css("button[type='button'][class='btn '][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).not_to have_css("button i")
+ expect(page).to have_text("Example Text")
+ end
+
+ it "renders the button with content" do
+ render_inline(Modal::OpenButtonComponent.new(target: "myModal")) do
+ "Example Text"
+ end
+
+ expect(page).to have_css("button[type='button'][class='btn '][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).not_to have_css("button i")
+ expect(page).to have_text("Example Text")
+ end
+
+ it "content overrides text" do
+ render_inline(Modal::OpenButtonComponent.new(target: "myModal", text: "Overwritten")) do
+ "Example Text"
+ end
+
+ expect(page).to have_css("button[type='button'][class='btn '][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).not_to have_css("button i")
+ expect(page).to have_text("Example Text")
+ expect(page).to_not have_text("Overwritten")
+ end
+
+ it "doesn't render anything if both text and content are absent" do
+ render_inline(Modal::OpenButtonComponent.new(target: "myModal"))
+
+ expect(page).not_to have_css("button")
+ end
+
+ it "doesn't render if render_check is false" do
+ render_inline(Modal::OpenButtonComponent.new(target: "myModal", text: "Example Text", render_check: false))
+
+ expect(page).not_to have_css("button")
+ end
+end
diff --git a/spec/components/modal/open_link_component_spec.rb b/spec/components/modal/open_link_component_spec.rb
new file mode 100644
index 0000000000..76784ef930
--- /dev/null
+++ b/spec/components/modal/open_link_component_spec.rb
@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe Modal::OpenLinkComponent, type: :component do
+ it "renders the link with text and icon" do
+ render_inline(Modal::OpenLinkComponent.new(target: "myModal", text: "Example Text", icon: "example-icon", klass: "example-class"))
+
+ expect(page).to have_css("a[href='#'][role='button'][class='btn example-class'][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).to have_css("a i.lni.mr-10.lni-example-icon")
+ expect(page).to have_text("Example Text")
+ end
+
+ it "renders the link with only text" do
+ render_inline(Modal::OpenLinkComponent.new(target: "myModal", text: "Example Text"))
+
+ expect(page).to have_css("a[href='#'][role='button'][class='btn '][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).not_to have_css("a i")
+ expect(page).to have_text("Example Text")
+ end
+
+ it "renders the link with content" do
+ render_inline(Modal::OpenLinkComponent.new(target: "myModal")) do
+ "Example Text"
+ end
+
+ expect(page).to have_css("a[href='#'][role='button'][class='btn '][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).not_to have_css("a i")
+ expect(page).to have_text("Example Text")
+ end
+
+ it "content overrides text" do
+ render_inline(Modal::OpenLinkComponent.new(target: "myModal", text: "Override")) do
+ "Example Text"
+ end
+
+ expect(page).to have_css("a[href='#'][role='button'][class='btn '][data-bs-toggle='modal'][data-bs-target='#myModal']")
+ expect(page).not_to have_css("a i")
+ expect(page).to have_text("Example Text")
+ expect(page).to_not have_text("Override")
+ end
+
+ it "doesn't render anything if both text and content are absent" do
+ render_inline(Modal::OpenLinkComponent.new(target: "myModal"))
+
+ expect(page).not_to have_css("a")
+ end
+
+ it "doesn't render if render_check is false" do
+ render_inline(Modal::OpenLinkComponent.new(target: "myModal", text: "Example Text", render_check: false))
+
+ expect(page).not_to have_css("a")
+ end
+end
diff --git a/spec/factories/case_contacts.rb b/spec/factories/case_contacts.rb
index 37eb7eefb0..1de4fbe618 100644
--- a/spec/factories/case_contacts.rb
+++ b/spec/factories/case_contacts.rb
@@ -61,5 +61,16 @@
casa_case { nil }
draft_case_ids { [1] }
end
+
+ trait :with_org_topics do
+ after(:create) do |case_contact, _|
+ return if case_contact.casa_case.nil?
+
+ casa_org = case_contact.casa_case.casa_org
+ casa_org.contact_topics.active.each do |contact_topic|
+ case_contact.contact_topic_answers << build(:contact_topic_answer, contact_topic: contact_topic)
+ end
+ end
+ end
end
end
diff --git a/spec/factories/contact_topic_answers.rb b/spec/factories/contact_topic_answers.rb
new file mode 100644
index 0000000000..b7d202e5f9
--- /dev/null
+++ b/spec/factories/contact_topic_answers.rb
@@ -0,0 +1,8 @@
+FactoryBot.define do
+ factory :contact_topic_answer do
+ case_contact
+ contact_topic
+ selected { false }
+ value { Faker::Lorem.paragraph_by_chars(number: 300) }
+ end
+end
diff --git a/spec/factories/contact_topics.rb b/spec/factories/contact_topics.rb
new file mode 100644
index 0000000000..8281db92d8
--- /dev/null
+++ b/spec/factories/contact_topics.rb
@@ -0,0 +1,8 @@
+FactoryBot.define do
+ factory :contact_topic do
+ casa_org
+ active { true }
+ question { Faker::Lorem.sentence }
+ details { Faker::Lorem.paragraph_by_chars(number: 300) }
+ end
+end
diff --git a/spec/lib/tasks/data_post_processors/contact_topic_populator.rb b/spec/lib/tasks/data_post_processors/contact_topic_populator.rb
new file mode 100644
index 0000000000..d4f7d810be
--- /dev/null
+++ b/spec/lib/tasks/data_post_processors/contact_topic_populator.rb
@@ -0,0 +1,74 @@
+require "rails_helper"
+require "./lib/tasks/data_post_processors/contact_topic_populator"
+
+RSpec.describe "populates each existing organization with contact groups and types" do
+ let(:fake_topics) { [{"question" => "Test Title", "details" => "Test details"}] }
+ let(:org_one) { create(:casa_org) }
+ let(:org_two) { create(:casa_org) }
+
+ before do
+ Rake::Task.clear
+ Casa::Application.load_tasks
+
+ allow(ContactTopic).to receive(:default_contact_topics).and_return(fake_topics)
+ ContactTopicPopulator.populate
+ end
+
+ it "does nothing on an empty database" do
+ ContactTopicPopulator.populate
+
+ expect(ContactTopic.count).to eq(0)
+ expect(ContactTopicAnswer.count).to eq(0)
+ end
+
+ context "there are orgs" do
+ before {
+ org_one
+ org_two
+ }
+
+ it "populates contact_topics" do
+ expect {
+ ContactTopicPopulator.populate
+ }.to change(ContactTopic, :count).from(0).to(2)
+
+ questions = ContactTopic.all.map(&:question)
+ details = ContactTopic.all.map(&:details)
+ active = ContactTopic.all.map(&:active)
+
+ expect(questions).to be_all("Test Title")
+ expect(details).to be_all("Test details")
+ expect(active).to be_all(true)
+ end
+
+ context "there are case_contacts" do
+ let(:case_one) { create(:casa_case, casa_org: org_one) }
+ let(:case_two) { create(:casa_case, casa_org: org_two) }
+
+ before {
+ create_list(:case_contact, 3, casa_case: case_one)
+ create_list(:case_contact, 3, casa_case: case_two)
+ }
+
+ it "populates contact_topics_answers for each case_contact" do
+ ContactTopicPopulator.populate
+
+ case_contacts = CaseContact.all
+ case_contacts.each do |case_contact|
+ expect(case_contact.contact_topic_answers).to_not be_empty
+ end
+
+ answers = case_contacts.map(&:contact_topic_answers).flatten
+ answers.each do |answer|
+ expect(answer.contact_topic).to_not be_nil
+ end
+
+ contact_topics = answers.map(&:contact_topic)
+ case_questions = contact_topics.map(&:question)
+ case_details = contact_topics.map(&:details)
+ expect(case_questions).to be_all("Test Title")
+ expect(case_details).to be_all("Test details")
+ end
+ end
+ end
+end
diff --git a/spec/models/casa_org_spec.rb b/spec/models/casa_org_spec.rb
index fce9d87da2..839aafafdf 100644
--- a/spec/models/casa_org_spec.rb
+++ b/spec/models/casa_org_spec.rb
@@ -10,6 +10,7 @@
it { is_expected.to have_many(:case_assignments).through(:users) }
it { is_expected.to have_one_attached(:logo) }
it { is_expected.to have_one_attached(:court_report_template) }
+ it { is_expected.to have_many(:contact_topics) }
it "has unique name" do
org = create(:casa_org)
@@ -86,10 +87,14 @@
it { is_expected.to eq 15 }
end
- describe "generate_contact_types_and_hearing_types" do
+ describe "generate_defaults" do
let(:org) { create(:casa_org) }
+ let(:fake_topics) { [{"question" => "Test Title", "details" => "Test details"}] }
- before { org.generate_contact_types_and_hearing_types }
+ before do
+ allow(ContactTopic).to receive(:default_contact_topics).and_return(fake_topics)
+ org.generate_defaults
+ end
describe "generates default contact type groups" do
let(:groups) { ContactTypeGroup.where(casa_org: org).joins(:contact_types).pluck(:name, "contact_types.name").sort }
@@ -128,52 +133,61 @@
end
end
- describe "mileage rate for a given date" do
- let(:casa_org) { build(:casa_org) }
+ describe "generates default contact topics" do
+ let(:contact_topics) { ContactTopic.where(casa_org: org).map(&:question) }
- describe "with a casa org with no rates" do
- it "is nil" do
- expect(casa_org.mileage_rate_for_given_date(Date.today)).to be_nil
- end
+ it "matches default contact topics" do
+ expected = fake_topics.map { |topic| topic["question"] }
+ expect(contact_topics).to include(*expected)
end
+ end
+ end
- describe "with a casa org with inactive dates" do
- let!(:mileage_rates) do
- [
- create(:mileage_rate, casa_org: casa_org, effective_date: 10.days.ago, is_active: false),
- create(:mileage_rate, casa_org: casa_org, effective_date: 3.days.ago, is_active: false)
- ]
- end
-
- it "is nil" do
- expect(casa_org.mileage_rates.count).to eq 2
- expect(casa_org.mileage_rate_for_given_date(Date.today)).to be_nil
- end
+ describe "mileage rate for a given date" do
+ let(:casa_org) { build(:casa_org) }
+
+ describe "with a casa org with no rates" do
+ it "is nil" do
+ expect(casa_org.mileage_rate_for_given_date(Date.today)).to be_nil
end
+ end
- describe "with active dates in the future" do
- let!(:mileage_rate) { create(:mileage_rate, casa_org: casa_org, effective_date: 3.days.from_now) }
+ describe "with a casa org with inactive dates" do
+ let!(:mileage_rates) do
+ [
+ create(:mileage_rate, casa_org: casa_org, effective_date: 10.days.ago, is_active: false),
+ create(:mileage_rate, casa_org: casa_org, effective_date: 3.days.ago, is_active: false)
+ ]
+ end
+
+ it "is nil" do
+ expect(casa_org.mileage_rates.count).to eq 2
+ expect(casa_org.mileage_rate_for_given_date(Date.today)).to be_nil
+ end
+ end
+
+ describe "with active dates in the future" do
+ let!(:mileage_rate) { create(:mileage_rate, casa_org: casa_org, effective_date: 3.days.from_now) }
+
+ it "is nil" do
+ expect(casa_org.mileage_rates.count).to eq 1
+ expect(casa_org.mileage_rate_for_given_date(Date.today)).to be_nil
+ end
+ end
- it "is nil" do
- expect(casa_org.mileage_rates.count).to eq 1
- expect(casa_org.mileage_rate_for_given_date(Date.today)).to be_nil
- end
+ describe "with active dates in the past" do
+ let!(:mileage_rates) do
+ [
+ create(:mileage_rate, casa_org: casa_org, amount: 4.50, effective_date: 20.days.ago),
+ create(:mileage_rate, casa_org: casa_org, amount: 5.50, effective_date: 10.days.ago),
+ create(:mileage_rate, casa_org: casa_org, amount: 6.50, effective_date: 3.days.ago)
+ ]
end
- describe "with active dates in the past" do
- let!(:mileage_rates) do
- [
- create(:mileage_rate, casa_org: casa_org, amount: 4.50, effective_date: 20.days.ago),
- create(:mileage_rate, casa_org: casa_org, amount: 5.50, effective_date: 10.days.ago),
- create(:mileage_rate, casa_org: casa_org, amount: 6.50, effective_date: 3.days.ago)
- ]
- end
-
- it "uses the most recent date" do
- expect(casa_org.mileage_rate_for_given_date(12.days.ago.to_date)).to eq 4.50
- expect(casa_org.mileage_rate_for_given_date(5.days.ago.to_date)).to eq 5.50
- expect(casa_org.mileage_rate_for_given_date(Date.today)).to eq 6.50
- end
+ it "uses the most recent date" do
+ expect(casa_org.mileage_rate_for_given_date(12.days.ago.to_date)).to eq 4.50
+ expect(casa_org.mileage_rate_for_given_date(5.days.ago.to_date)).to eq 5.50
+ expect(casa_org.mileage_rate_for_given_date(Date.today)).to eq 6.50
end
end
end
diff --git a/spec/models/case_contact_spec.rb b/spec/models/case_contact_spec.rb
index 8e1508ce8b..233c882ec3 100644
--- a/spec/models/case_contact_spec.rb
+++ b/spec/models/case_contact_spec.rb
@@ -1,6 +1,7 @@
require "rails_helper"
RSpec.describe CaseContact, type: :model do
+ it { should have_many(:contact_topic_answers).dependent(:destroy) }
it { is_expected.to validate_numericality_of(:miles_driven).is_less_than 10_000 }
it { is_expected.to validate_numericality_of(:miles_driven).is_greater_than_or_equal_to 0 }
@@ -140,6 +141,55 @@
end
end
+ describe "#create_with_answers" do
+ let(:contact_topics) {
+ [
+ build(:contact_topic, active: true, soft_delete: false),
+ build(:contact_topic, active: false, soft_delete: false),
+ build(:contact_topic, active: true, soft_delete: true),
+ build(:contact_topic, active: false, soft_delete: true)
+ ]
+ }
+ let(:org) { create(:casa_org, contact_topics:) }
+ let(:admin) { create(:casa_admin, casa_org: org) }
+ let(:casa_case) { create(:casa_case, casa_org: org) }
+
+ context "when creation is successful" do
+ it "create a case_contact" do
+ org
+ expect {
+ CaseContact.create_with_answers(org, creator: admin)
+ }.to change(CaseContact, :count).from(0).to(1)
+ end
+
+ it "creates only active and non-deleted contact_topic_answers" do
+ org
+ expect {
+ CaseContact.create_with_answers(org, creator: admin)
+ }.to change(ContactTopicAnswer, :count).from(0).to(1)
+
+ case_contact = CaseContact.last
+ topics = case_contact.contact_topic_answers.map(&:contact_topic)
+
+ expect(topics).to include(contact_topics.first)
+ end
+ end
+
+ context "when a topic answer creation fails" do
+ it "does not create a case contact" do
+ expect {
+ CaseContact.create_with_answers(org)
+ }.to_not change(CaseContact, :count)
+ end
+
+ it "adds errors from contact_topic_answers" do
+ allow(org.contact_topics).to receive(:active).and_return([nil])
+ result = CaseContact.create_with_answers(org, creator: admin)
+ expect(result.errors[:contact_topic_answers]).to include("could not create topic nil")
+ end
+ end
+ end
+
describe "scopes" do
describe "date related scopes" do
let!(:case_contacts) do
diff --git a/spec/models/contact_topic_answer_spec.rb b/spec/models/contact_topic_answer_spec.rb
new file mode 100644
index 0000000000..df65741ed7
--- /dev/null
+++ b/spec/models/contact_topic_answer_spec.rb
@@ -0,0 +1,12 @@
+require "rails_helper"
+
+RSpec.describe ContactTopicAnswer, type: :model do
+ it { should belong_to(:case_contact) }
+ it { should belong_to(:contact_topic) }
+
+ it "can hold more than 255 characters" do
+ expect {
+ create(:contact_topic_answer, value: Faker::Lorem.characters(number: 300))
+ }.not_to raise_error
+ end
+end
diff --git a/spec/models/contact_topic_spec.rb b/spec/models/contact_topic_spec.rb
new file mode 100644
index 0000000000..4066055e27
--- /dev/null
+++ b/spec/models/contact_topic_spec.rb
@@ -0,0 +1,76 @@
+require "rails_helper"
+
+RSpec.describe ContactTopic, type: :model do
+ it { should belong_to(:casa_org) }
+ it { should have_many(:contact_topic_answers) }
+
+ it { should validate_presence_of(:question) }
+ it { should validate_presence_of(:details) }
+
+ describe "scopes" do
+ describe ".active" do
+ it "returns only active and non-soft deleted contact topics" do
+ active_contact_topic = create(:contact_topic, active: true, soft_delete: false)
+ inactive_contact_topic = create(:contact_topic, active: false, soft_delete: false)
+ soft_deleted_contact_topic = create(:contact_topic, active: true, soft_delete: true)
+
+ expect(ContactTopic.active).to include(active_contact_topic)
+ expect(ContactTopic.active).not_to include(inactive_contact_topic)
+ expect(ContactTopic.active).not_to include(soft_deleted_contact_topic)
+ end
+ end
+ end
+
+ describe "generate for org" do
+ let(:org) { create(:casa_org) }
+ let(:fake_topics) { [{"question" => "Test Title", "details" => "Test details"}] }
+
+ describe "generate_contact_topics" do
+ before do
+ allow(ContactTopic).to receive(:default_contact_topics).and_return(fake_topics)
+ end
+
+ it "creates contact topics" do
+ expect { ContactTopic.generate_for_org!(org) }.to change { org.contact_topics.count }.by(1)
+
+ created_topic = org.contact_topics.first
+ expect(created_topic.question).to eq(fake_topics.first["question"])
+ expect(created_topic.details).to eq(fake_topics.first["details"])
+ end
+
+ context "there are no default topics" do
+ let(:fake_topics) { [] }
+
+ it { expect { ContactTopic.generate_for_org!(org) }.not_to(change { org.contact_topics.count }) }
+ end
+
+ it "generates from parameter" do
+ topics = fake_topics.push({"question" => "a", "details" => "a"})
+ expect { ContactTopic.generate_for_org!(org) }.to change { org.contact_topics.count }.by(2)
+
+ questions = org.contact_topics.map(&:question)
+ details = org.contact_topics.map(&:details)
+ expect(questions).to match_array(topics.map { |t| t["question"] })
+ expect(details).to match_array(topics.map { |t| t["details"] })
+ end
+
+ it "fails if not all required attrs are present " do
+ fake_topics.first["question"] = nil
+
+ expect { ContactTopic.generate_for_org!(org) }.to raise_error(ActiveRecord::RecordInvalid)
+ end
+
+ it "creates if needed fields all present" do
+ fake_topics.first["invalid_field"] = "invalid"
+ expect { ContactTopic.generate_for_org!(org) }.to change { org.contact_topics.count }.by(1)
+ end
+ end
+ end
+
+ describe "details" do
+ it "can hold more than 255 characters" do
+ contact_topic_details = build(:contact_topic, details: Faker::Lorem.characters(number: 300))
+ expect { contact_topic_details.save! }.not_to raise_error
+ end
+ end
+end
diff --git a/spec/policies/contact_topic_policy_spec.rb b/spec/policies/contact_topic_policy_spec.rb
new file mode 100644
index 0000000000..2a25dc71d0
--- /dev/null
+++ b/spec/policies/contact_topic_policy_spec.rb
@@ -0,0 +1,29 @@
+require "rails_helper"
+
+RSpec.describe ContactTopicPolicy, type: :policy do
+ subject { described_class }
+ let(:contact_topic) { build(:contact_topic, casa_org: organization) }
+
+ let(:organization) { build(:casa_org) }
+ let(:casa_admin) { create(:casa_admin, casa_org: organization) }
+ let(:other_org_admin) { create(:casa_admin) }
+ let(:volunteer) { build(:volunteer, casa_org: organization) }
+ let(:supervisor) { build(:supervisor, casa_org: organization) }
+
+ permissions :create?, :edit?, :new?, :show?, :soft_delete?, :update? do
+ it "allows same org casa_admins" do
+ is_expected.to permit(casa_admin, contact_topic)
+ end
+
+ it "allows does not allow different org casa_admins" do
+ is_expected.to_not permit(other_org_admin, contact_topic)
+ end
+ it "does not permit supervisor" do
+ is_expected.to_not permit(supervisor, contact_topic)
+ end
+
+ it "does not permit volunteer" do
+ is_expected.to_not permit(volunteer, contact_topic)
+ end
+ end
+end
diff --git a/spec/requests/all_casa_admins/casa_orgs_spec.rb b/spec/requests/all_casa_admins/casa_orgs_spec.rb
index b5fbca00d5..c5e1397b12 100644
--- a/spec/requests/all_casa_admins/casa_orgs_spec.rb
+++ b/spec/requests/all_casa_admins/casa_orgs_spec.rb
@@ -47,11 +47,25 @@
{casa_org: {name: "New Org", display_name: "New org display",
address: "29207 Weimann Canyon, New Andrew, PA 40510-7416"}}
end
+ let(:contact_topics) { [{"question" => "Title 1", "details" => "details 1"}, {"question" => "Title 2", "details" => "details 2"}] }
+
+ before do
+ allow(ContactTopic).to receive(:default_contact_topics).and_return(contact_topics)
+ end
it "creates a new CASA org" do
expect { post_create }.to change(CasaOrg, :count).by(1)
end
+ it "generates correct defaults during creation" do
+ expect { post_create }.to change(ContactTopic, :count).by(2)
+
+ casa_org = CasaOrg.last
+ expect(casa_org.contact_topics.map(&:question)).to eq contact_topics.map { |t| t["question"] }
+ expect(casa_org.contact_topics.map(&:details)).to eq contact_topics.map { |t| t["details"] }
+ expect(casa_org.contact_topics.pluck(:active)).to be_all true
+ end
+
it "redirects to CASA org show page, with notice flash", :aggregate_failures do
post_create
diff --git a/spec/requests/case_contacts/form_spec.rb b/spec/requests/case_contacts/form_spec.rb
index c3d0d99065..afb89b961b 100644
--- a/spec/requests/case_contacts/form_spec.rb
+++ b/spec/requests/case_contacts/form_spec.rb
@@ -57,11 +57,33 @@
expect(page).not_to include(*contact_types_b.pluck(:name))
end
end
+
+ context "when the org has topics assigned" do
+ let(:contact_topics) {
+ [
+ build(:contact_topic, active: true, soft_delete: false),
+ build(:contact_topic, active: false, soft_delete: false),
+ build(:contact_topic, active: true, soft_delete: true),
+ build(:contact_topic, active: false, soft_delete: true)
+ ]
+ }
+ let(:organization) { create(:casa_org, contact_topics:) }
+ let!(:case_contact) { create(:case_contact, :details_status, :with_org_topics, casa_case: casa_case) }
+
+ it "shows contact topics" do
+ page = request.parsed_body.to_html
+ expect(page).to include(contact_topics[0].question)
+ expect(page).to_not include(contact_topics[1].question)
+ expect(page).to_not include(contact_topics[2].question)
+ expect(page).to_not include(contact_topics[3].question)
+ end
+ end
end
end
describe "PATCH /update" do
let!(:casa_case) { create(:casa_case, casa_org: organization) }
+ let!(:case_contact) { create(:case_contact, :details_status, casa_case:) }
let(:advance_form) { true }
let(:params) { {case_contact: attributes} }
@@ -72,7 +94,8 @@
end
context "submitting details step" do
- let!(:case_contact) { create(:case_contact, :started_status, creator: creator) }
+ let!(:case_contact) { create(:case_contact, :started_status, creator: creator, contact_topic_answers: topic_answers) }
+ let(:topic_answers) { build_list(:contact_topic_answer, 3) }
let(:step) { :details }
let!(:contact_type_group_b) { create(:contact_type_group, casa_org: organization, name: "B") }
let!(:contact_types_b) do
@@ -98,7 +121,8 @@
duration_minutes: 50,
contact_made: true,
medium_type: CaseContact::CONTACT_MEDIUMS.second,
- case_contact_contact_type_attributes: contact_type_attributes
+ case_contact_contact_type_attributes: contact_type_attributes,
+ contact_topic_answers_attributes: topic_answers_attributes
}
end
let(:contact_type_attributes) do
@@ -108,6 +132,14 @@
}
end
+ let(:topic_answers_attributes) do
+ {
+ "0" => {id: topic_answers.first.id, value: "test", selected: true},
+ "1" => {id: topic_answers.second.id, value: "test", selected: true},
+ "2" => {id: topic_answers.third.id, value: "test", selected: true}
+ }
+ end
+
it "with valid attributes updates the requested case_contact" do
request
case_contact.reload
@@ -117,6 +149,14 @@
expect(case_contact.medium_type).to eq(CaseContact::CONTACT_MEDIUMS.second)
end
+ it "updates only answer field for contact topics" do
+ request
+ case_contact.reload
+
+ expect(case_contact.contact_topic_answers.pluck(:value)).to be_all "test"
+ expect(case_contact.contact_topic_answers.pluck(:selected)).to be_all true
+ end
+
context "contact types" do
it "attaches contact types" do
request
@@ -163,6 +203,52 @@
end
end
+ context "submitting notes step: contact topics" do
+ let!(:case_contact) { create(:case_contact, :details_status, creator: creator, contact_topic_answers: topic_answers) }
+ let(:topic_answers) { build_list(:contact_topic_answer, 3) }
+ let(:topic_answers_attributes) do
+ {
+ "0" => {id: topic_answers.first.id, value: "test", selected: true},
+ "1" => {id: topic_answers.second.id, value: "test", selected: true},
+ "2" => {id: topic_answers.third.id, value: "test", selected: true}
+ }
+ end
+ let(:step) { :notes }
+ let(:attributes) do
+ {contact_topic_answers_attributes: topic_answers_attributes}
+ end
+
+ context "with valid contact topic answers" do
+ context "when submitting via button" do
+ it "updates the requested case_contact" do
+ request
+ case_contact.reload
+
+ expect(case_contact.contact_topic_answers.pluck(:value)).to be_all "test"
+ expect(case_contact.contact_topic_answers.pluck(:selected)).to be_all true
+ end
+ end
+
+ context "when autosaving" do
+ subject(:request) do
+ patch "/case_contacts/#{case_contact.id}/form/#{step}", params:, as: :json
+
+ response
+ end
+
+ it "updates the requested case_contact" do
+ request
+ case_contact.reload
+
+ expect(case_contact.contact_topic_answers.pluck(:value)).to be_all "test"
+ expect(case_contact.contact_topic_answers.pluck(:selected)).to be_all true
+ end
+
+ it { is_expected.to have_http_status(:success) }
+ end
+ end
+ end
+
context "submitting notes step" do
let!(:case_contact) { create(:case_contact, :details_status, creator: creator) }
let(:step) { :notes }
@@ -215,7 +301,16 @@
end
context "submitting expenses step" do
- let!(:case_contact) { create(:case_contact, :notes_status, draft_case_ids: [casa_case.id], creator: creator) }
+ let!(:case_contact) { create(:case_contact, :notes_status, draft_case_ids: [casa_case.id], creator: creator, contact_topic_answers: topic_answers) }
+ let(:case_contact_topics) { build_list(:contact_topic_answer, 3) }
+ let(:topic_answers) { build_list(:contact_topic_answer, 3) }
+ let(:topic_answers_attributes) do
+ {
+ "0" => {id: topic_answers.first.id, value: "test", selected: true},
+ "1" => {id: topic_answers.second.id, value: "test", selected: true},
+ "2" => {id: topic_answers.third.id, value: "test", selected: true}
+ }
+ end
let(:additional_expenses) do
{
"0" => {other_expense_amount: 50, other_expenses_describe: "meal"},
@@ -230,7 +325,8 @@
want_driving_reimbursement: true,
miles_driven: 60,
volunteer_address: "123 str",
- additional_expenses_attributes: additional_expenses
+ additional_expenses_attributes: additional_expenses,
+ contact_topic_answers_attributes: topic_answers_attributes
}
end
@@ -245,6 +341,8 @@
expect(case_contact.additional_expenses.first.other_expenses_describe).to eq "meal"
expect(case_contact.additional_expenses.last.other_expense_amount).to eq 100
expect(case_contact.additional_expenses.last.other_expenses_describe).to eq "hotel"
+ expect(case_contact.contact_topic_answers.pluck(:value)).to be_all "test"
+ expect(case_contact.contact_topic_answers.pluck(:selected)).to be_all true
end
it "sets the case_contact's status to active" do
@@ -288,7 +386,10 @@
context "with multiple cases selected" do
let!(:other_casa_case) { create(:casa_case, casa_org: organization) }
- let!(:case_contact) { create(:case_contact, :notes_status, draft_case_ids: [casa_case.id, other_casa_case.id], creator: admin) }
+ let!(:case_contact) {
+ create(:case_contact, :notes_status, draft_case_ids: [casa_case.id, other_casa_case.id],
+ creator: admin, contact_topic_answers: topic_answers)
+ }
it "creates a copy of the draft for each case" do
expect {
@@ -299,6 +400,12 @@
expect(CaseContact.last.status).to eq "active"
end
+ it "sets contact_topics for all cases" do
+ expect { request }.to change(ContactTopicAnswer, :count).by(3)
+ expect(CaseContact.last.contact_topic_answers.pluck(:value)).to be_all("test")
+ expect(CaseContact.last.contact_topic_answers.pluck(:selected)).to be_all(true)
+ end
+
it "sets the draft_case_ids of the draft to only the first case" do
expect(case_contact.draft_case_ids.count).to eq 2
request
diff --git a/spec/requests/case_contacts_spec.rb b/spec/requests/case_contacts_spec.rb
index e72425a6e0..edde42bd47 100644
--- a/spec/requests/case_contacts_spec.rb
+++ b/spec/requests/case_contacts_spec.rb
@@ -51,6 +51,27 @@
request
}.to change(CaseContact, :count).by(1)
end
+
+ context "when current org has contact topics" do
+ let(:contact_topics) {
+ [
+ build(:contact_topic, active: true, soft_delete: false),
+ build(:contact_topic, active: false, soft_delete: false),
+ build(:contact_topic, active: true, soft_delete: true),
+ build(:contact_topic, active: false, soft_delete: true)
+ ]
+ }
+ let(:organization) { create(:casa_org, contact_topics:) }
+
+ it "should set empty contact topic answers for new case contact to active/non-softdelet org topics" do
+ expect { request }.to change(ContactTopicAnswer, :count).by(1)
+
+ got = CaseContact.last.contact_topic_answers.first.contact_topic.question
+ expect(got).to eq(contact_topics[0].question)
+
+ expect(CaseContact.last.contact_topic_answers.first.value).to be_nil
+ end
+ end
end
describe "GET /edit" do
diff --git a/spec/requests/contact_topics_spec.rb b/spec/requests/contact_topics_spec.rb
new file mode 100644
index 0000000000..01ee5fd4d0
--- /dev/null
+++ b/spec/requests/contact_topics_spec.rb
@@ -0,0 +1,135 @@
+require "rails_helper"
+
+RSpec.describe "/contact_topics", type: :request do
+ # This should return the minimal set of attributes required to create a valid
+ # ContactTopic. As you add validations to ContactTopic, be sure to
+ # adjust the attributes here as well.
+ let(:casa_org) { create(:casa_org) }
+ let(:is_active) { nil }
+ let(:contact_topic) { create(:contact_topic, casa_org:) }
+ let(:attributes) { {casa_org_id: casa_org.id} }
+ let(:admin) { create(:casa_admin, casa_org: casa_org) }
+
+ before { sign_in admin }
+
+ describe "GET /new" do
+ it "renders a successful response" do
+ get new_contact_topic_url
+ expect(response).to be_successful
+ end
+ end
+
+ describe "GET /edit" do
+ it "renders a successful response" do
+ get edit_contact_topic_url(contact_topic)
+ expect(response).to be_successful
+ expect(response.body).to include(contact_topic.question)
+ expect(response.body).to include(contact_topic.details)
+ end
+ end
+
+ describe "POST /create" do
+ context "with valid parameters" do
+ let(:attributes) do
+ {
+ casa_org_id: casa_org.id,
+ question: "test question",
+ details: "test details"
+ }
+ end
+
+ it "creates a new ContactTopic" do
+ expect do
+ post contact_topics_url, params: {contact_topic: attributes}
+ end.to change(ContactTopic, :count).by(1)
+
+ topic = ContactTopic.last
+
+ expect(topic.question).to eq("test question")
+ expect(topic.details).to eq("test details")
+ end
+
+ it "redirects to the edit casa_org" do
+ post contact_topics_url, params: {contact_topic: attributes}
+ expect(response).to redirect_to(edit_casa_org_path(casa_org))
+ end
+ end
+
+ context "with invalid parameters" do
+ let(:attributes) { {casa_org_id: 0} }
+
+ it "does not create a new ContactTopic" do
+ expect do
+ post contact_topics_url, params: {contact_topic: attributes}
+ end.to change(ContactTopic, :count).by(0)
+ end
+
+ it "renders a response with 422 status (i.e. to display the 'new' template)" do
+ post contact_topics_url, params: {contact_topic: attributes}
+ expect(response).to have_http_status(:unprocessable_entity)
+ end
+ end
+ end
+
+ describe "PATCH /update" do
+ context "with valid parameters" do
+ let!(:contact_topic) { create(:contact_topic, casa_org:) }
+
+ let(:new_attributes) do
+ {
+ casa_org_id: casa_org.id,
+ active: false,
+ question: "test question",
+ details: "test details",
+ soft_delete: true
+ }
+ end
+
+ it "updates only active, details, question contact_topic" do
+ expect(contact_topic.soft_delete).to eq(false)
+
+ patch contact_topic_url(contact_topic), params: {contact_topic: new_attributes}
+ contact_topic.reload
+
+ expect(contact_topic.soft_delete).to eq(false)
+ expect(contact_topic.active).to eq(false)
+ expect(contact_topic.details).to eq("test details")
+ expect(contact_topic.question).to eq("test question")
+ end
+
+ it "redirects to the casa_org edit" do
+ patch contact_topic_url(contact_topic), params: {contact_topic: new_attributes}
+ expect(response).to redirect_to(edit_casa_org_path(casa_org))
+ end
+ end
+
+ context "with invalid parameters" do
+ let(:attributes) { {casa_org_id: 0} }
+
+ it "renders a response with 422 status (i.e. to display the 'edit' template)" do
+ patch contact_topic_url(contact_topic), params: {contact_topic: attributes}
+ expect(response).to have_http_status(:unprocessable_entity)
+ end
+ end
+ end
+
+ describe "DELETE /soft_delete" do
+ let!(:contact_topic) { create(:contact_topic, casa_org: casa_org) }
+ it "does not destroy the requested contact_topic" do
+ expect do
+ delete soft_delete_contact_topic_url(contact_topic)
+ end.to_not change(ContactTopic, :count)
+ end
+
+ it "set the requested contact_topic to soft_deleted" do
+ delete soft_delete_contact_topic_url(contact_topic)
+ contact_topic.reload
+ expect(contact_topic.soft_delete).to be true
+ end
+
+ it "redirects to edit casa_org" do
+ delete soft_delete_contact_topic_url(contact_topic)
+ expect(response).to redirect_to(edit_casa_org_path(casa_org))
+ end
+ end
+end
diff --git a/spec/support/fill_in_case_contact_fields.rb b/spec/support/fill_in_case_contact_fields.rb
index e9506488c5..485fe0350e 100644
--- a/spec/support/fill_in_case_contact_fields.rb
+++ b/spec/support/fill_in_case_contact_fields.rb
@@ -6,7 +6,7 @@ module FillInCaseContactFields
# @param occurred_on [String], date in the format MM/dd/YYYY
# @param hours [Integer]
# @param minutes [Integer]
- def complete_details_page(contact_made:, medium: nil, occurred_on: nil, hours: nil, minutes: nil, case_numbers: [], contact_types: [])
+ def complete_details_page(contact_made:, medium: nil, occurred_on: nil, hours: nil, minutes: nil, case_numbers: [], contact_types: [], contact_topics: [])
case_numbers.each do |case_number|
check case_number
end
@@ -24,6 +24,10 @@ def complete_details_page(contact_made:, medium: nil, occurred_on: nil, hours: n
fill_in "case_contact_duration_hours", with: hours if hours
fill_in "case_contact_duration_minutes", with: minutes if minutes
+ contact_topics.each do |contact_topic|
+ check contact_topic
+ end
+
click_on "Save and continue"
end
diff --git a/spec/system/case_contacts/create_spec.rb b/spec/system/case_contacts/create_spec.rb
index 6a3819352b..5ad959fc5e 100644
--- a/spec/system/case_contacts/create_spec.rb
+++ b/spec/system/case_contacts/create_spec.rb
@@ -1,7 +1,9 @@
require "rails_helper"
RSpec.describe "case_contacts/create", type: :system, js: true do
- let(:volunteer) { create(:volunteer, :with_cases_and_contacts, :with_assigned_supervisor) }
+ let(:contact_topics) { [build(:contact_topic, question: "q1"), build(:contact_topic, question: "q2")] }
+ let(:org) { create(:casa_org, contact_topics: contact_topics) }
+ let(:volunteer) { create(:volunteer, :with_cases_and_contacts, :with_assigned_supervisor, casa_org: org) }
let(:casa_case) { volunteer.casa_cases.first }
context "redirects to where new case contact started from" do
@@ -45,4 +47,82 @@
expect(page).to have_text "Case contact successfully created"
end
end
+
+ describe "notes page", js: true do
+ before(:each) do
+ sign_in volunteer
+ visit case_contacts_path
+ click_on "New Case Contact"
+
+ complete_details_page(
+ case_numbers: [casa_case.case_number],
+ medium: "In Person",
+ contact_made: true,
+ hours: 1,
+ minutes: 45,
+ contact_topics: [contact_topics.first.question]
+ )
+ end
+
+ it "has selected topics expanded but no details expanded" do
+ topic_one_id = contact_topics.first.question.parameterize.underscore
+ topic_two_id = contact_topics.last.question.parameterize.underscore
+
+ expect(page).to have_text contact_topics.first.question
+ expect(page).to_not have_text contact_topics.first.details
+
+ within("##{topic_one_id}") do
+ expect(page).to have_text("read more")
+ expect(page).to have_selector("##{topic_one_id} textarea")
+ end
+
+ expect(page).to have_text contact_topics.last.question
+ expect(page).to_not have_text contact_topics.last.details
+ expect(page).to_not have_selector("##{topic_two_id}")
+ end
+
+ it "expands to show and hide the text field and details", js: true do
+ click_on "read more"
+ topic_id = contact_topics.first.question.parameterize.underscore
+
+ expect(page).to have_text(contact_topics.first.question)
+ expect(page).to have_text(contact_topics.first.details)
+ expect(page).to have_selector("##{topic_id} textarea")
+
+ find("##{topic_id}_button").click
+
+ expect(page).to have_text(contact_topics.first.question)
+ expect(page).to_not have_text(contact_topics.first.details)
+ expect(page).to_not have_selector("##{topic_id} textarea")
+
+ sleep 0.4 # BUG: have to wait for the animation to finish
+ find("##{topic_id}_button").click
+
+ expect(page).to have_text(contact_topics.first.question)
+ expect(page).to have_text(contact_topics.first.details)
+ expect(page).to have_selector("##{topic_id} textarea")
+ end
+
+ it "expands to show/hide details", js: true do
+ topic_id = contact_topics.first.question.parameterize.underscore
+
+ expect(page).to have_text(contact_topics.first.question)
+
+ within("##{topic_id}") do
+ expect(page).to_not have_text(contact_topics.first.details)
+ expect(page).to have_selector("##{topic_id} textarea")
+
+ click_on "read more"
+
+ expect(page).to have_text(contact_topics.first.details)
+ expect(page).to have_selector("##{topic_id} textarea")
+
+ sleep 0.4 # BUG: have to wait for the animation to finish
+ click_on "read less"
+
+ expect(page).to_not have_text(contact_topics.first.details)
+ expect(page).to have_selector("##{topic_id} textarea")
+ end
+ end
+ end
end
diff --git a/spec/system/volunteers/index_spec.rb b/spec/system/volunteers/index_spec.rb
index 648fbf0047..ce4fffbf20 100644
--- a/spec/system/volunteers/index_spec.rb
+++ b/spec/system/volunteers/index_spec.rb
@@ -314,7 +314,7 @@
end
context "when none is selected" do
- it "is enabled" do
+ xit "is enabled" do # TODO: Flaky. Fix this test
visit volunteers_path
find("#supervisor_volunteer_volunteer_ids_#{volunteer.id}", wait: 3).click
find("[data-select-all-target='button']", wait: 3).click
diff --git a/spec/values/case_contact_parameters_spec.rb b/spec/values/case_contact_parameters_spec.rb
index 9a25c59520..139f17ba65 100644
--- a/spec/values/case_contact_parameters_spec.rb
+++ b/spec/values/case_contact_parameters_spec.rb
@@ -2,6 +2,7 @@
RSpec.describe CaseContactParameters do
subject { described_class.new(params) }
+
let(:params) {
ActionController::Parameters.new(
case_contact: ActionController::Parameters.new(
@@ -13,10 +14,15 @@
miles_driven: "123",
want_driving_reimbursement: "want_driving_reimbursement",
notes: "notes",
- case_contact_contact_type_attributes: [:contact_type_id]
+ case_contact_contact_type_attributes: [:contact_type_id],
+ contact_topic_answers_attributes:
)
)
}
+ let(:contact_topic_answers_attributes) do
+ {"0" => {"id" => 1, "value" => "test",
+ "question" => "question", "selected" => true}}
+ end
it "returns data" do
expect(subject["duration_minutes"]).to eq(62)
@@ -27,5 +33,8 @@
expect(subject["want_driving_reimbursement"]).to eq("want_driving_reimbursement")
expect(subject["notes"]).to eq("notes")
expect(subject["case_contact_contact_type_attributes"]).to eq([])
+
+ expected_attrs = contact_topic_answers_attributes["0"].except("question")
+ expect(subject["contact_topic_answers_attributes"]["0"].to_h).to eq(expected_attrs)
end
end
diff --git a/spec/views/casa_orgs/edit.html.erb_spec.rb b/spec/views/casa_orgs/edit.html.erb_spec.rb
index ec89074bfe..fccc63d40e 100644
--- a/spec/views/casa_orgs/edit.html.erb_spec.rb
+++ b/spec/views/casa_orgs/edit.html.erb_spec.rb
@@ -9,6 +9,7 @@
assign(:learning_hour_types, [])
assign(:learning_hour_topics, [])
assign(:sent_emails, [])
+ assign(:contact_topics, [])
sign_in build_stubbed(:casa_admin)
end
@@ -24,6 +25,23 @@
expect(rendered).to have_selector("input[required=required]", id: "casa_org_name")
end
+ it "has contact topic content" do
+ organization = build_stubbed(:casa_org)
+ allow(view).to receive(:current_organization).and_return(organization)
+ contact_topic = build_stubbed(:contact_topic, question: "Test Question", details: "Test details")
+ assign(:contact_topics, [contact_topic])
+
+ render template: "casa_org/edit"
+
+ expect(rendered).to have_text("Test Question")
+ expect(rendered).to have_text("Test details")
+ expect(rendered).to have_table("contact-topics",
+ with_rows:
+ [
+ ["Test Question", "Test details", "Edit"]
+ ])
+ end
+
it "has contact types content" do
organization = build_stubbed(:casa_org)
allow(view).to receive(:current_organization).and_return(organization)