Improve the Seeding of sofia accounts (#1201)

* generate better seeding files

* intial commit message

* fix suggestions

* fix lint

* fix lint

Author: lodewiges

db/seeds.rb

@@ -11,16 +11,49 @@
 end
 
 p 'Seeding users...'
-users = []
-4.times do
-  users << FactoryBot.create(:user)
+# Nederlandse testnamen
+dutch_names = [
+  'Jan de Vries',
+  'Piet Bakker',
+  'Lisa Jansen',
+  'Emma van der Berg',
+  'Daan Visser',
+  'Sophie Smit',
+  'Thomas Mulder',
+  'Anna de Boer'
+]
+
+users = dutch_names.map do |name|
+  FactoryBot.create(:user, name:)
 end
 users << FactoryBot.create(:user, name: 'Benjamin Knopje', birthday: 16.years.ago)
 
 p 'Seeding activities...'
-activities = []
-4.times do
-  activities << FactoryBot.create(:activity, price_list: price_lists.sample, created_by: users.sample)
+# Recente activiteiten
+activity_names = %w[Donderdagborrel Vrijdagborrel ALV Kerstdiner Filmavond Bierproeverij]
+activities = activity_names.map do |title|
+  FactoryBot.create(:activity, title:, price_list: price_lists.sample, created_by: users.sample)
+end
+
+# Historische activiteiten (locked)
+p 'Seeding historical activities...'
+historical_activities = [
+  { title: 'Nieuwjaarsborrel', days_ago: 15 },
+  { title: 'Studiemiddag', days_ago: 30 },
+  { title: 'Kroegentocht', days_ago: 45 },
+  { title: 'Dies Natalis', days_ago: 90 }
+]
+
+historical_activities.each do |hist_act|
+  start_time = hist_act[:days_ago].days.ago.beginning_of_hour
+  end_time = (hist_act[:days_ago] - 1).days.ago.beginning_of_hour
+  activity = FactoryBot.create(:activity,
+                               title: hist_act[:title],
+                               start_time:,
+                               end_time:,
+                               price_list: price_lists.sample,
+                               created_by: users.sample)
+  activities << activity
 end
 
 p 'Seeding orders...'
@@ -32,9 +65,27 @@
 end
 
 p 'Seeding credit mutations...'
+# Realistische credit mutations met variatie
+mutation_descriptions = [
+  { description: 'Opwaardering', amount_range: 10..50 },
+  { description: 'Contant gestort', amount_range: 5..30 },
+  { description: 'iDEAL betaling', amount_range: 10..100 },
+  { description: 'Correctie admin', amount_range: -20..20 },
+  { description: 'Restitutie borrel', amount_range: 5..15 },
+  { description: 'Handmatige correctie', amount_range: -10..10 }
+]
+
 users.each do |user|
-  FactoryBot.create_list(:credit_mutation, 3, user:, created_by: users.sample,
-                                              activity: (activities + [nil]).sample)
+  3.times do
+    mutation = mutation_descriptions.sample
+    amount = rand(mutation[:amount_range])
+    FactoryBot.create(:credit_mutation,
+                      user:,
+                      created_by: users.sample,
+                      description: mutation[:description],
+                      amount:,
+                      activity: (activities + [nil]).sample)
+  end
 end
 
 p 'Seeding invoices'
@@ -47,4 +98,22 @@
 Role.create(role_type: :treasurer)
 Role.create(role_type: :renting_manager)
 Role.create(role_type: :main_bartender)
+
+p 'Seeding Sofia accounts...'
+# Use environment variable for password or fallback to default for development
+
+treasurer_user = FactoryBot.create(:user, :sofia_account, name: 'Penningmeester Test')
+SofiaAccount.create!(username: 'penningmeester', password: 'password1234', user: treasurer_user)
+treasurer_role = Role.create(role_type: :treasurer)
+treasurer_user.roles << treasurer_role
+
+main_bartender_user = FactoryBot.create(:user, :sofia_account, name: 'Hoofdtapper Test')
+SofiaAccount.create!(username: 'hoofdtapper', password: 'password1234', user: main_bartender_user)
+main_bartender_role = Role.create(role_type: :main_bartender)
+main_bartender_user.roles << main_bartender_role
+
+renting_manager_user = FactoryBot.create(:user, :sofia_account, name: 'Verhuur Test')
+SofiaAccount.create!(username: 'verhuur', password: 'password1234', user: renting_manager_user)
+renting_manager_role = Role.create(role_type: :renting_manager)
+renting_manager_user.roles << renting_manager_role
 # rubocop:enable Rails/Output

db/seeds/products.rb

@@ -1,4 +1,17 @@
 def seed_products # rubocop:disable Metrics/AbcSize, Metrics/MethodLength, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
+  # Category colors for products
+  category_colors = {
+    beer: '#FFD700', # Goud/geel - bierkleur
+    low_alcohol_beer: '#90EE90', # Lichtgroen - light/gezonder
+    craft_beer: '#CD7F32',     # Brons - premium/craft
+    non_alcoholic: '#87CEEB',  # Lichtblauw - fris/water
+    distilled: '#DC143C',      # Donkerrood - sterke drank
+    whiskey: '#8B4513',        # Bruin - whiskykleur
+    wine: '#722F37',           # Bordeaux - wijnkleur
+    food: '#FFA500',           # Oranje - eten
+    tobacco: '#808080',        # Grijs - rook
+    donation: '#9370DB'        # Paars - speciaal/donatie
+  }.freeze
   # rubocop:disable Style/WordArray
   products_beer = ['Bier (glas)', 'Bier (pul)', 'Bier (pitcher)', '12+1']
   products_low_alcohol_beer = ['Alcoholarm bier', 'Radler 0.0']
@@ -13,43 +26,43 @@ def seed_products # rubocop:disable Metrics/AbcSize, Metrics/MethodLength, Metri
   # rubocop:enable Style/WordArray
 
   products = products_beer.map do |name|
-    Product.create(name:, category: :beer)
+    Product.create(name:, category: :beer, color: category_colors[:beer])
   end
 
   products_low_alcohol_beer.each do |name|
-    products << Product.create(name:, category: :low_alcohol_beer)
+    products << Product.create(name:, category: :low_alcohol_beer, color: category_colors[:low_alcohol_beer])
   end
 
   products_craft_beer.each do |name|
-    products << Product.create(name:, category: :craft_beer)
+    products << Product.create(name:, category: :craft_beer, color: category_colors[:craft_beer])
   end
 
   products_non_alcoholic.each do |name|
-    products << Product.create(name:, category: :non_alcoholic)
+    products << Product.create(name:, category: :non_alcoholic, color: category_colors[:non_alcoholic])
   end
 
   products_distilled.each do |name|
-    products << Product.create(name:, category: :distilled)
+    products << Product.create(name:, category: :distilled, color: category_colors[:distilled])
   end
 
   products_whiskey.each do |name|
-    products << Product.create(name:, category: :whiskey)
+    products << Product.create(name:, category: :whiskey, color: category_colors[:whiskey])
   end
 
   products_wine.each do |name|
-    products << Product.create(name:, category: :wine)
+    products << Product.create(name:, category: :wine, color: category_colors[:wine])
   end
 
   products_food.each do |name|
-    products << Product.create(name:, category: :food)
+    products << Product.create(name:, category: :food, color: category_colors[:food])
   end
 
   products_tobacco.each do |name|
-    products << Product.create(name:, category: :tobacco)
+    products << Product.create(name:, category: :tobacco, color: category_colors[:tobacco])
   end
 
   products_donation.each do |name|
-    products << Product.create(name:, category: :donation)
+    products << Product.create(name:, category: :donation, color: category_colors[:donation])
   end
 
   products