Increase coverage of project

Gemfile.lock

@@ -192,12 +192,12 @@ GEM
       railties (>= 5.0.0)
     faker (3.5.1)
       i18n (>= 1.8.11, < 2)
-    faraday (2.12.0)
-      faraday-net_http (>= 2.0, < 3.4)
+    faraday (2.12.2)
+      faraday-net_http (>= 2.0, < 3.5)
       json
       logger
-    faraday-net_http (3.3.0)
-      net-http
+    faraday-net_http (3.4.0)
+      net-http (>= 0.5.0)
     ffi (1.17.0-x86_64-linux-gnu)
     flipper (1.3.1)
       concurrent-ruby (< 2)
@@ -237,7 +237,7 @@ GEM
     jbuilder (2.13.0)
       actionview (>= 5.0.0)
       activesupport (>= 5.0.0)
-    json (2.7.5)
+    json (2.9.0)
     jsonapi-renderer (0.2.2)
     jwt (2.9.3)
       base64
@@ -263,7 +263,7 @@ GEM
     listen (3.9.0)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
-    logger (1.6.1)
+    logger (1.6.3)
     loofah (2.23.1)
       crass (~> 1.0.2)
       nokogiri (>= 1.12.0)
@@ -283,7 +283,7 @@ GEM
     msgpack (1.7.3)
     multi_xml (0.6.0)
     mutex_m (0.1.2)
-    net-http (0.4.1)
+    net-http (0.6.0)
       uri
     net-imap (0.5.0)
       date
@@ -315,7 +315,7 @@ GEM
     omniauth-facebook (10.0.0)
       bigdecimal
       omniauth-oauth2 (>= 1.2, < 3)
-    omniauth-google-oauth2 (0.8.2)
+    omniauth-google-oauth2 (0.8.0)
       jwt (>= 2.0)
       omniauth (>= 1.1.1)
       omniauth-oauth2 (>= 1.6)
@@ -557,7 +557,7 @@ GEM
       concurrent-ruby (~> 1.0)
     unaccent (0.4.0)
     unicode-display_width (2.6.0)
-    uri (0.13.1)
+    uri (1.0.2)
     useragent (0.16.10)
     version_gem (1.1.4)
     warden (1.2.9)

app/controllers/account/products_controller.rb

@@ -46,11 +46,11 @@ def update
   def destroy
     @product = resource
 
-    if @product.destroy
-      redirect_to account_products_path, notice: t(".deleted")
-    else
-      redirect_to account_products_path, status: :unprocessable_entity
-    end
+    notice = @product.destroy ? t(".deleted") : t(".not_deleted")
+
+    status = @product.destroy ? :moved_permanently : :unprocessable_entity
+
+    redirect_to account_products_path, notice: notice, status: status
   end
 
   private

app/models/authorization.rb

@@ -4,7 +4,7 @@ class Authorization < ApplicationRecord
   # include Encryptable
   # attr_encrypted :token, :secret, :refresh_token
 
-  belongs_to :admin, optional: true
+  belongs_to :admin, class_name: "User", foreign_key: "uid", optional: true, inverse_of: :authorizations
 
   validates :uid, uniqueness: { scope: :provider }
 end

app/models/user.rb

@@ -44,6 +44,8 @@ class User < ApplicationRecord
   scope :ordered_by_first_name, -> { order(:first_name) }
   scope :ordered_by_last_name, -> { order(:last_name) }
 
+  has_many :authorizations, foreign_key: "uid", inverse_of: :admin, dependent: :destroy
+
   has_paper_trail ignore: [
     :current_sign_in_at, :last_sign_in_at, :confirmation_token,
     :encrypted_password

config/locales/en/en.yml

@@ -311,6 +311,7 @@ en:
       created: "A product was successfully created"
       updated: "A product was successfully updated"
       deleted: "A product was successfully deleted"
+      not_deleted: "Can't delete product."
     users:
       index:
         meta-title: "Users"

db/migrate/20241114131028_add_index_to_authorizations_on_uid_and_provider.rb

@@ -0,0 +1,5 @@
+class AddIndexToAuthorizationsOnUidAndProvider < ActiveRecord::Migration[7.1]
+  def change
+    add_index :authorizations, [:uid, :provider], unique: true
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.2].define(version: 2024_10_31_011638) do
+ActiveRecord::Schema[7.2].define(version: 2024_11_14_131028) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "pgcrypto"
   enable_extension "plpgsql"
@@ -32,7 +32,7 @@
     t.text "metadata"
     t.string "service_name", null: false
     t.bigint "byte_size", null: false
-    t.string "checksum", null: false
+    t.string "checksum"
     t.datetime "created_at", precision: nil, null: false
     t.index ["key"], name: "index_active_storage_blobs_on_key", unique: true
   end
@@ -43,13 +43,23 @@
     t.index ["blob_id", "variation_digest"], name: "index_active_storage_variant_records_uniqueness", unique: true
   end
 
+  create_table "authorizations", force: :cascade do |t|
+    t.string "uid"
+    t.string "provider"
+    t.bigint "user_id", null: false
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["uid", "provider"], name: "index_authorizations_on_uid_and_provider", unique: true
+    t.index ["user_id"], name: "index_authorizations_on_user_id"
+  end
+
   create_table "calculators", force: :cascade do |t|
     t.uuid "uuid", default: -> { "gen_random_uuid()" }, null: false
     t.string "name"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
     t.string "slug"
-    t.boolean "preferable", default: false, null: false
+    t.boolean "preferable", default: false
     t.index ["name"], name: "index_calculators_on_name", unique: true
     t.index ["slug"], name: "index_calculators_on_slug", unique: true
     t.index ["uuid"], name: "index_calculators_on_uuid", unique: true
@@ -201,9 +211,9 @@
     t.string "last_sign_in_ip"
     t.string "provider"
     t.string "uid"
-    t.boolean "blocked", default: false, null: false
+    t.boolean "blocked", default: false
     t.integer "role", default: 0
-    t.boolean "receive_recomendations", default: false, null: false
+    t.boolean "receive_recomendations", default: false
     t.index ["email"], name: "index_users_on_email", unique: true
     t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
   end
@@ -222,6 +232,7 @@
 
   add_foreign_key "active_storage_attachments", "active_storage_blobs", column: "blob_id"
   add_foreign_key "active_storage_variant_records", "active_storage_blobs", column: "blob_id"
+  add_foreign_key "authorizations", "users"
   add_foreign_key "category_categoryables", "categories"
   add_foreign_key "diapers_periods", "categories"
 end

spec/factories/authorization.rb

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :authorization do
+    uid { "unique_uid" }
+    provider { "provider_name" }
+    admin factory: [:user]
+  end
+end

spec/features/sign_out_spec.rb

@@ -21,8 +21,8 @@
     end
   end
 
-  xit "signs the user out" do
+  it "signs the user out" do
     expect(page).to have_current_path("/calculators/#{calculator.slug}")
-    expect(page).to have_content("LOG IN")
+    expect(page).to have_content("Signed out successfully")
   end
 end

spec/helpers/account/application_helper_spec.rb

@@ -0,0 +1,21 @@
+require "rails_helper"
+
+RSpec.describe ApplicationHelper, type: :helper do
+  describe "#current_locale?" do
+    context "when the current locale matches the given locale" do
+      it "returns true" do
+        I18n.with_locale(:en) do
+          expect(helper.current_locale?(:en)).to be true
+        end
+      end
+    end
+
+    context "when the current locale does not match the given locale" do
+      it "returns false" do
+        I18n.with_locale(:en) do
+          expect(helper.current_locale?(:uk)).to be false
+        end
+      end
+    end
+  end
+end

spec/models/authorization_spec.rb

@@ -0,0 +1,7 @@
+require "rails_helper"
+
+RSpec.describe Authorization, type: :model do
+  describe "associations" do
+    it { is_expected.to belong_to(:admin).class_name("User").with_foreign_key("uid").optional.inverse_of(:authorizations) }
+  end
+end

spec/models/product_spec.rb

@@ -68,6 +68,26 @@
     end
   end
 
+  describe "#price_by_category" do
+    let(:product) { create(:product, title: "Valid Title") }
+    let(:category) { create(:category, :medium) }
+    let(:valid_sum) { 10.0 }
+
+    context "when the product has a price for the category" do
+      let!(:existing_price) { create(:price, priceable: product, category: category, sum: valid_sum) }
+
+      it "returns the price for the specified category" do
+        expect(product.price_by_category(category)).to eq(existing_price)
+      end
+    end
+
+    context "when the product does not have a price for the category" do
+      it "returns nil" do
+        expect(product.price_by_category(category)).to be_nil
+      end
+    end
+  end
+
   describe "#find_or_build_price_for_category" do
     let(:product) { create(:product, title: "Valid Title") }
     let(:category) { create(:category, :medium) }
@@ -91,4 +111,43 @@
       end
     end
   end
+
+  describe ".diaper" do
+    let!(:diaper_product) { create(:product, title: "diaper") }
+    let!(:other_product) { create(:product, title: "shampoo") }
+
+    it "returns the product with title 'diaper'" do
+      expect(Product.diaper).to eq(diaper_product)
+    end
+
+    it "does not return a product with a different title" do
+      expect(Product.diaper).not_to eq(other_product)
+    end
+  end
+
+  describe "#blank_prices" do
+    let(:product) { create(:product, title: "title") }
+
+    context "when price sum is blank" do
+      it "rejects nested attributes for price" do
+        product.attributes = {
+          prices_attributes: [
+            { sum: nil }
+          ]
+        }
+        expect(product.prices).to be_empty
+      end
+    end
+
+    context "when price sum is present" do
+      it "accepts nested attributes for price" do
+        product.attributes = {
+          prices_attributes: [
+            { sum: 100 }
+          ]
+        }
+        expect(product.prices.size).to eq(1)
+      end
+    end
+  end
 end

spec/models/user_spec.rb

@@ -93,6 +93,24 @@
     end
   end
 
+  describe "#inactive_message" do
+    context "when the user is blocked" do
+      let(:blocked_user) { create(:user, blocked: true) }
+
+      it "returns :locked" do
+        expect(blocked_user.inactive_message).to eq(:locked)
+      end
+    end
+
+    context "when the user is active" do
+      let(:active_user) { create(:user, blocked: false) }
+
+      it "returns the default devise message" do
+        expect(active_user.inactive_message).to eq(:inactive)
+      end
+    end
+  end
+
   describe ".from_omniauth" do
     context "when the user exists" do
       let(:user) { create(:user) }

spec/requests/aplication_spec.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require "rails_helper"
+
+RSpec.describe ApplicationController, type: :controller do
+  describe "GET #set_i18n_locale_from_params" do
+    context "when an invalid locale is provided" do
+      it "sets a flash notice about the unavailable translation" do
+        get :redirection, params: { locale: "invalid_locale" }
+
+        expect(flash.now[:notice]).to eq("invalid_locale translation not available")
+      end
+    end
+
+    context "when a valid locale is provided" do
+      it "does not set a flash notice" do
+        get :redirection, params: { locale: "en" }
+
+        expect(flash.now[:notice]).to be_nil
+      end
+    end
+  end
+end

spec/requests/calculators_spec.rb

@@ -29,6 +29,7 @@
                    label: "three",
                    kind: "parameter", calculator: calculator)
   end
+  let!(:user) { create(:user) }
   let(:json_response) { response.parsed_body }
 
   describe "POST api/v2/calculators/PERMALINK/compute" do
@@ -159,4 +160,28 @@
       end
     end
   end
+
+  describe "POST #receive_recomendations" do
+    context "when user sign_in" do
+      it "does change recieve_recomendation" do
+        sign_in user
+
+        expect do
+          post receive_recomendations_path
+        end.to change(user, :receive_recomendations)
+      end
+    end
+  end
+
+  describe "POST #calculate" do
+    context "when user sign_in" do
+      it "returns a successful response" do
+        sign_in user
+
+        post calculate_calculator_path(calculator.slug)
+
+        expect(response).to be_successful
+      end
+    end
+  end
 end

spec/requests/home_spec.rb

@@ -22,6 +22,14 @@
       end
     end
 
+    context "redirect to root_url" do
+      it "is successful" do
+        get root_redirection_path
+
+        expect(response).to redirect_to(root_path)
+      end
+    end
+
     context "with locale switching" do
       it "is successful" do
         get root_path(locale: :uk)