Support Provider-region association with UI and tests.

Author: hydrognomik

app/controllers/providers_controller.rb

@@ -60,6 +60,6 @@ def set_provider
 
     # Only allow a list of trusted parameters through.
     def provider_params
-      params.expect(provider: [ :name, :provider_type ])
+      params.expect(provider: [ :name, :provider_type, region_ids: [] ])
     end
 end

app/models/branch.rb

@@ -1,4 +1,4 @@
 class Branch < ApplicationRecord
-  has_many :providers
-  has_many :regions
+  belongs_to :provider
+  belongs_to :region
 end

app/views/providers/_form.html.erb

@@ -25,6 +25,13 @@
     </div>
   </div>
 
+  <div class="col-md-4">
+    <div class="form-group">
+      <%= form.label :region, style: "display: block" %>
+      <%= form.collection_select :region_ids, Region.order(:name), :id, :name, { include_hidden: false }, class: "form-select", multiple: true %>
+    </div>
+  </div>
+
   <div class="mt-4">
     <div class="col-12 d-flex justify-content-end">
       <%= form.submit "Create Provider", class: "btn btn-primary me-1 mb-1" %>

app/views/providers/_provider.html.erb

@@ -9,4 +9,16 @@
     <%= provider.provider_type %>
   </p>
 
+  <p>
+    <strong>Provider regions:</strong>
+    <ul>
+      <% provider.regions.each do |region| %>
+        <li>
+          <%= link_to region_path(region) do %>
+            <%= region.name %>
+          <% end %>
+        </li>
+      <% end %>
+    </ul>
+  </p>
 </div>

spec/requests/providers_spec.rb

@@ -15,17 +15,17 @@
 RSpec.describe "/providers", type: :request do
   let(:user) { create(:user) }
 
-  include AuthHelper
-
   before do
     sign_in(user)
   end
 
+  let(:region_ids) { [create(:region).id, create(:region).id] }
+  let(:updated_region_ids) { [create(:region).id] }
   # This should return the minimal set of attributes required to create a valid
   # Provider. As you add validations to Provider, be sure to
   # adjust the attributes here as well.
   let(:valid_attributes) {
-    { name: "MyString", provider_type: "MyString" }
+    { name: "MyString", provider_type: "MyString", region_ids: region_ids }
   }
 
   let(:invalid_attributes) {
@@ -71,6 +71,12 @@
         }.to change(Provider, :count).by(1)
       end
 
+      it "creates two new Branches" do
+        expect {
+          post providers_url, params: { provider: valid_attributes }
+        }.to change(Branch, :count).by(2)
+      end
+
       it "redirects to the created provider" do
         post providers_url, params: { provider: valid_attributes }
         expect(response).to redirect_to(provider_url(Provider.last))
@@ -94,7 +100,7 @@
   describe "PATCH /update" do
     context "with valid parameters" do
       let(:new_attributes) {
-        { name: "MyString", provider_type: "MyType" }
+        { name: "MyString", provider_type: "MyType", region_ids: updated_region_ids }
       }
 
       it "updates the requested provider" do
@@ -103,6 +109,7 @@
         provider.reload
         expect(provider.name).to eq("MyString")
         expect(provider.provider_type).to eq("MyType")
+        expect(provider.regions.length).to eq(updated_region_ids.length)
       end
 
       it "redirects to the provider" do