Fix/sidekiq notifications

app/models/user_skill.rb

@@ -7,15 +7,17 @@
 #  id                   :bigint           not null, primary key
 #  last_applied_in_year :integer
 #  level                :string
+#  years_of_experience  :float
 #  created_at           :datetime         not null
 #  updated_at           :datetime         not null
 #  skill_id             :bigint           not null
 #  user_id              :bigint           not null
 #
 # Indexes
 #
-#  index_user_skills_on_skill_id  (skill_id)
-#  index_user_skills_on_user_id   (user_id)
+#  index_user_skills_on_skill_id              (skill_id)
+#  index_user_skills_on_user_id               (user_id)
+#  index_user_skills_on_user_id_and_skill_id  (user_id,skill_id) UNIQUE
 #
 # Foreign Keys
 #
@@ -29,4 +31,7 @@ class UserSkill < ApplicationRecord
   VALID_LEVELS = %w[beginner intermediate advanced].freeze
   validates :level, presence: true, inclusion: { in: VALID_LEVELS }
   validates :last_applied_in_year, presence: true
+  validates :years_of_experience, presence: true
+
+  validates :user_id, uniqueness: { scope: :skill_id }
 end

app/views/user_skills/_user_skill.json.jbuilder

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+# _user_service_identifier.json.jbuilder
+
+json.extract! user_skill, :id, :last_applied_in_year, :level, :years_of_experience

app/views/users/_user.json.jbuilder

@@ -15,3 +15,5 @@ json.permissions user.permissions, partial: 'permissions/permission', as: :permi
 # Render the user_service_identifiers partial for each identifier
 json.user_service_identifiers user.user_service_identifiers,
                               partial: 'user_service_identifiers/user_service_identifier', as: :user_service_identifier
+
+json.user_skills user.user_skills, partial: 'user_skills/user_skill', as: :user_skill

config/initializers/sidekiq.rb

@@ -13,11 +13,13 @@
   end
 
   config.error_handlers << proc do |ex, ctx_hash, _cfg|
-    job = ctx_hash[:job]['class']
-    args = ctx_hash[:job]['args']
+    if Rails.env.production?
+      job = ctx_hash[:job]['class']
+      args = ctx_hash[:job]['args']
 
-    message = "There was an error with the job: #{job} with arguments: #{args.join(',')}.\nThe error message is: #{ex}"
-    RubyNotificationsClient::Channel.new.send(message, ENV.fetch('ERROR_NOTIFICATION_CHANNEL', nil))
+      message = "There was an error with the job: #{job} with arguments: #{args.join(',')}.\nThe error message is: #{ex}"
+      RubyNotificationsClient::Channel.new.send(message, ENV.fetch('ERROR_NOTIFICATION_CHANNEL', nil))
+    end
   end
 end
 

db/migrate/20240221191037_add_years_of_experience_to_user_skill.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddYearsOfExperienceToUserSkill < ActiveRecord::Migration[7.0]
+  def change
+    add_column :user_skills, :years_of_experience, :float
+  end
+end

db/migrate/20240221192743_add_unique_index_to_user_skills_on_user_id_and_skill_id.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddUniqueIndexToUserSkillsOnUserIdAndSkillId < ActiveRecord::Migration[7.0]
+  def change
+    add_index :user_skills, %i[user_id skill_id], unique: true
+  end
+end

spec/factories/user_skills.rb

@@ -7,15 +7,17 @@
 #  id                   :bigint           not null, primary key
 #  last_applied_in_year :integer
 #  level                :string
+#  years_of_experience  :float
 #  created_at           :datetime         not null
 #  updated_at           :datetime         not null
 #  skill_id             :bigint           not null
 #  user_id              :bigint           not null
 #
 # Indexes
 #
-#  index_user_skills_on_skill_id  (skill_id)
-#  index_user_skills_on_user_id   (user_id)
+#  index_user_skills_on_skill_id              (skill_id)
+#  index_user_skills_on_user_id               (user_id)
+#  index_user_skills_on_user_id_and_skill_id  (user_id,skill_id) UNIQUE
 #
 # Foreign Keys
 #
@@ -28,5 +30,6 @@
     skill
     level { UserSkill::VALID_LEVELS.sample }
     last_applied_in_year { 2020 }
+    years_of_experience { 1.5 }
   end
 end

spec/models/user_skill_spec.rb

@@ -7,15 +7,17 @@
 #  id                   :bigint           not null, primary key
 #  last_applied_in_year :integer
 #  level                :string
+#  years_of_experience  :float
 #  created_at           :datetime         not null
 #  updated_at           :datetime         not null
 #  skill_id             :bigint           not null
 #  user_id              :bigint           not null
 #
 # Indexes
 #
-#  index_user_skills_on_skill_id  (skill_id)
-#  index_user_skills_on_user_id   (user_id)
+#  index_user_skills_on_skill_id              (skill_id)
+#  index_user_skills_on_user_id               (user_id)
+#  index_user_skills_on_user_id_and_skill_id  (user_id,skill_id) UNIQUE
 #
 # Foreign Keys
 #
@@ -28,11 +30,20 @@
   context 'validations' do
     it { should validate_presence_of(:level) }
     it { should validate_presence_of(:last_applied_in_year) }
+    it { should validate_presence_of(:years_of_experience) }
     it { should validate_inclusion_of(:level).in_array(UserSkill::VALID_LEVELS) }
   end
 
   context 'associations' do
     it { should belong_to(:user) }
     it { should belong_to(:skill) }
+
+    it 'validates that :user_id is case-sensitively unique within the scope of :skill_id' do
+      user = create(:user)
+      skill = create(:skill)
+      create(:user_skill, user:, skill:)
+      new_user_skill = UserSkill.new(user_id: user.id, skill_id: skill.id)
+      expect(new_user_skill).to validate_uniqueness_of(:user_id).scoped_to(:skill_id).case_insensitive
+    end
   end
 end