Update dependency faker to v3.4.1

Reviewed-on: #4

.github/workflows/tests.yml

@@ -0,0 +1,34 @@
+name: Run unit tests
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+jobs:
+  unit_tests:
+    runs-on: ubuntu-latest
+    services:
+      postgres:
+        image: postgres
+        env:
+          POSTGRES_USER: postgres
+          POSTGRES_PASSWORD: postgres
+        options: >-
+          --health-cmd pg_isready
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+        ports:
+          - 5432
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}
+      - uses: ruby/setup-ruby@v1
+      - run: bundle install
+      - run: |
+          bundle exec rake db:create db:schema:load
+          bundle exec rspec
+        env:
+          RAILS_ENV: test
+          DATABASE_URL: postgres://postgres:postgres@postgres:5432/postgres

app/controllers/guests_controller.rb

@@ -1,14 +1,16 @@
 class GuestsController < ApplicationController
-  before_action :set_guest, only: %i[ show edit update destroy ]
+  before_action :set_guest, only: %i[show edit update destroy]
 
   # GET /guests or /guests.json
   def index
-    @guests = Guest.all.includes(:affinity_groups)
+    @guests = Guest.all
+                   .left_outer_joins(:affinity_groups)
+                   .order('tags.name' => :asc)
+                   .includes(:affinity_groups, :unbreakable_bonds)
   end
 
   # GET /guests/1 or /guests/1.json
-  def show
+  def show; end
-  end
 
   # GET /guests/new
   def new
@@ -16,8 +18,7 @@ class GuestsController < ApplicationController
   end
 
   # GET /guests/1/edit
-  def edit
+  def edit; end
-  end
 
   # POST /guests or /guests.json
   def create
@@ -25,7 +26,7 @@ class GuestsController < ApplicationController
 
     respond_to do |format|
       if @guest.save
-        format.html { redirect_to guest_url(@guest), notice: "Guest was successfully created." }
+        format.html { redirect_to guest_url(@guest), notice: 'Guest was successfully created.' }
         format.json { render :show, status: :created, location: @guest }
       else
         format.html { render :new, status: :unprocessable_entity }
@@ -38,7 +39,7 @@ class GuestsController < ApplicationController
   def update
     respond_to do |format|
       if @guest.update(guest_params)
-        format.html { redirect_to guest_url(@guest), notice: "Guest was successfully updated." }
+        format.html { redirect_to guest_url(@guest), notice: 'Guest was successfully updated.' }
         format.json { render :show, status: :ok, location: @guest }
       else
         format.html { render :edit, status: :unprocessable_entity }
@@ -52,19 +53,20 @@ class GuestsController < ApplicationController
     @guest.destroy!
 
     respond_to do |format|
-      format.html { redirect_to guests_url, notice: "Guest was successfully destroyed." }
+      format.html { redirect_to guests_url, notice: 'Guest was successfully destroyed.' }
       format.json { head :no_content }
     end
   end
 
   private
-    # Use callbacks to share common setup or constraints between actions.
-    def set_guest
-      @guest = Guest.find(params[:id])
-    end
 
-    # Only allow a list of trusted parameters through.
+  # Use callbacks to share common setup or constraints between actions.
-    def guest_params
+  def set_guest
-      params.require(:guest).permit(:first_name, :last_name, :email, :phone)
+    @guest = Guest.find(params[:id])
-    end
+  end
+
+  # Only allow a list of trusted parameters through.
+  def guest_params
+    params.require(:guest).permit(:first_name, :last_name, :email, :phone)
+  end
 end

app/views/guests/index.html.erb

@@ -10,6 +10,7 @@
       <th>Email</th>
       <th>Phone</th>
       <th>Affinity groups</th>
+      <th>Unbreakable bonds</th>
       <th colspan="2"></th>
     </tr>
   <% @guests.each_with_index do |guest, i| %>
@@ -19,6 +20,7 @@
       <td><%= guest.email %></td>
       <td><%= guest.phone %></td>
       <td><%= guest.affinity_groups.pluck(:name).join(", ") %></td>
+      <td><%= guest.unbreakable_bonds.pluck(:name).join(", ") %></td>
       <td><%= link_to "Show", guest %></td>
       <td><%= link_to "Edit", edit_guest_path(guest) %></td>
     </tr>

db/seeds.rb

@@ -28,7 +28,10 @@ Expense.create!(name: 'Transportation', amount: 3000, pricing_type: 'fixed')
 Expense.create!(name: 'Invitations', amount: 200, pricing_type: 'fixed')
 Expense.create!(name: 'Cake', amount: 500, pricing_type: 'fixed')
 
+<<<<<<< HEAD
 
+=======
+>>>>>>> 8fd0b7c (Modify seeds file to make sure every guest is part of a group)
 samples = {
   close_family: 10,
   family_1_group_a: 5,
@@ -49,10 +52,25 @@ end
 
 300.times do
   guest = Guest.create!(first_name: Faker::Name.first_name,
-                last_name: Faker::Name.last_name,
+                        last_name: Faker::Name.last_name,
-                email: Faker::Internet.email,
+                        email: Faker::Internet.email,
-                phone: Faker::PhoneNumber.cell_phone)
+                        phone: Faker::PhoneNumber.cell_phone)
 
   guest.affinity_group_list.add(samples.sample)
   guest.save!
 end
+
+Guest.affinity_group_counts.each do |group|
+  couples = (group.taggings_count / 4).floor
+
+  guests_involved = Guest.tagged_with(group.name).limit(couples * 2)
+  guests_involved.each_slice(2) do |a, b|
+    bond_name = "#{a.full_name} & #{b.full_name}"
+
+    a.unbreakable_bond_list.add(bond_name)
+    b.unbreakable_bond_list.add(bond_name)
+
+    a.save!
+    b.save!
+  end
+end