Add contributor association between Users and Providers

Introduced a many-to-many relationship between Users and Providers via Contributors. Updated views and forms to manage contributors, restricted to admin users. Enhanced Providers with a unique name validation and adjusted tests to cover new behavior.

Author: hydrognomik

app/controllers/providers_controller.rb

@@ -51,6 +51,6 @@ def set_provider
     end
 
     def provider_params
-      params.expect(provider: [ :name, :provider_type, region_ids: [] ])
+      params.expect(provider: [ :name, :provider_type, region_ids: [], user_ids: [] ])
     end
 end

app/models/contributor.rb

@@ -0,0 +1,4 @@
+class Contributor < ApplicationRecord
+  belongs_to :provider
+  belongs_to :user
+end

app/models/provider.rb

@@ -1,6 +1,9 @@
 class Provider < ApplicationRecord
   has_many :branches
   has_many :regions, through: :branches
+  has_many :contributors
+  has_many :users, through: :contributors
 
   validates :name, :provider_type, presence: true
+  validates :name, uniqueness: true
 end

app/models/user.rb

@@ -1,6 +1,8 @@
 class User < ApplicationRecord
   has_secure_password
   has_many :sessions, dependent: :destroy
+  has_many :contributors
+  has_many :providers, through: :contributors
 
   normalizes :email, with: ->(e) { e.strip.downcase }
 

app/views/providers/_form.html.erb

@@ -32,6 +32,15 @@
     </div>
   </div>
 
+  <% if Current.user.is_admin %>
+    <div class="col-md-4">
+      <div class="form-group">
+        <%= form.label :contributors, style: "display: block" %>
+        <%= form.collection_select :user_ids, User.order(:email), :id, :email, { include_hidden: false }, class: "form-select", multiple: true %>
+      </div>
+    </div>
+  <% end %>
+
   <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

@@ -21,4 +21,17 @@
       <% end %>
     </ul>
   </p>
+
+  <p>
+    <strong>Provider contributors:</strong>
+    <ul>
+      <% provider.users.each do |user| %>
+        <li>
+          <%= link_to user_path(user) do %>
+            <%= user.email %>
+          <% end %>
+        </li>
+      <% end %>
+    </ul>
+  </p>
 </div>

db/migrate/20250204141359_create_contributor.rb

@@ -0,0 +1,10 @@
+class CreateContributor < ActiveRecord::Migration[8.0]
+  def change
+    create_table :contributors do |t|
+      t.belongs_to :provider
+      t.belongs_to :user
+
+      t.timestamps
+    end
+  end
+end

db/schema.rb

@@ -8,4 +8,10 @@
       is_admin { true }
     end
   end
+
+  factory :admin do
+    email { "john@doe.com" }
+    password { "password" }
+    is_admin { true }
+  end
 end

spec/factories/users.rb

@@ -1,12 +1,13 @@
 require "rails_helper"
 
 RSpec.describe "providers/edit", type: :view do
-  let(:provider) {
-    Provider.create!(
-      name: "MyString",
-      provider_type: "MyString",
-    )
-  }
+  let(:provider) { create(:provider) }
+  let(:admin) { create(:admin) }
+  let(:user) { create(:user) }
+
+  before do
+    allow(Current).to receive(:user).and_return(user)
+  end
 
   before(:each) do
     assign(:provider, provider)
@@ -17,8 +18,29 @@
 
     assert_select "form[action=?][method=?]", provider_path(provider), "post" do
       assert_select "input[name=?]", "provider[name]"
-
       assert_select "input[name=?]", "provider[provider_type]"
     end
   end
+
+  it "does not render the contributor form group for common user" do
+    render
+
+    assert_select "form[action=?][method=?]", provider_path(provider), "post" do
+      assert_select "select[id=?]", "provider_user_ids", false
+    end
+  end
+
+  describe "for admin" do
+    before do
+      allow(Current).to receive(:user).and_return(admin)
+    end
+
+    it "renders the contributor form group" do
+      render
+
+      assert_select "form[action=?][method=?]", provider_path(provider), "post" do
+        assert_select "select[id=?]", "provider_user_ids"
+      end
+    end
+  end
 end