cats_core 1.5.2 → 1.5.4

data/app/models/cats/core/rhn_request.rb

@@ -3,13 +3,13 @@ module Cats
     class RhnRequest < ApplicationRecord
       include Dispatchable
 
-      # DRAFT = 'Draft'.freeze
-      # APPROVED = 'Approved'.freeze
-      # RESERVED = 'Reserved'.freeze
+      before_validation :set_unit
+
       ALLOCATED = 'Allocated'.freeze
       STATUSES << ALLOCATED
 
       belongs_to :commodity
+      belongs_to :unit, class_name: 'Cats::Core::UnitOfMeasure'
 
       validates :reference_no, :request_date, :quantity, presence: true
       validates :reference_no, uniqueness: true
@@ -18,6 +18,15 @@ module Cats
       validate :validate_commodity_status, :validate_quantity
 
       delegate(:batch_no, to: :commodity, prefix: true)
+      delegate(:abbreviation, to: :unit, prefix: true)
+
+      def set_unit
+        return if unit
+
+        return unless commodity
+
+        self.unit = commodity.unit
+      end
 
       def request_reference
         reference_no
@@ -34,13 +43,21 @@ module Cats
       end
 
       def validate_quantity
-        return unless commodity && quantity
+        return unless commodity && quantity && unit
 
-        requested = RhnRequest.where(commodity: commodity).sum(:quantity)
-        remaining = commodity.quantity - requested
+        requested = commodity.requested_quantity
 
-        remaining += quantity_was if quantity_was
-        errors.add(:quantity, "exceeds commodity quantity. Maximum allowed is #{remaining}") if quantity > remaining
+        remaining = commodity.quantity - requested
+        if quantity_was
+          additional = UnitConversion.convert(unit, commodity.unit_of_measure, quantity_was)
+          remaining += additional
+        end
+        new_quantity = UnitConversion.convert(unit, commodity.unit_of_measure, quantity)
+        return unless new_quantity > remaining
+
+        remaining = UnitConversion.convert(commodity.unit_of_measure, unit, remaining)
+        message = "exceeds commodity quantity. Maximum allowed is #{remaining} #{unit.abbreviation}"
+        errors.add(:quantity, message)
       end
 
       def approve

data/app/models/cats/core/stack.rb

@@ -9,6 +9,8 @@ module Cats
 
       belongs_to :commodity
       belongs_to :store
+      belongs_to :unit, class_name: 'Cats::Core::UnitOfMeasure'
+
       has_many :dispatch_transactions, foreign_key: :source_id
       has_many :receipt_transactions, foreign_key: :destination_id
 
@@ -25,6 +27,7 @@ module Cats
                unless: -> { store && (store.code == 'SUP-STORE' || store.code.start_with?('FDP-ST')) }
 
       delegate :batch_no, to: :commodity, prefix: true
+      delegate :abbreviation, to: :unit, prefix: true
 
       after_save :update_store_space
 

data/app/models/cats/core/stack_transaction.rb

@@ -3,6 +3,7 @@ module Cats
     class StackTransaction < Transaction
       belongs_to :source, class_name: 'Cats::Core::Stack'
       belongs_to :destination, class_name: 'Cats::Core::Stack'
+      belongs_to :unit, class_name: 'Cats::Core::UnitOfMeasure'
 
       def validate_quantity
         return unless quantity.present? && source.present?

data/app/models/cats/core/transaction.rb

@@ -9,10 +9,14 @@ module Cats
       COMMITTED = 'Committed'.freeze
       STATUSES = [DRAFT, COMMITTED].freeze
 
+      belongs_to :unit, class_name: 'Cats::Core::UnitOfMeasure'
+
       validates :transaction_date, :quantity, :status, presence: true
       validates :quantity, numericality: { greater_than: 0 }
       validates :status, inclusion: { in: STATUSES }
 
+      delegate(:abbreviation, to: :unit, prefix: true)
+
       def commit
         raise(NotImplementedError, 'Method should be implemented in child classes.')
       end

data/app/models/cats/core/unit_conversion.rb

@@ -15,10 +15,28 @@ module Cats
         return value if from == to
 
         conversion = find_by(from: from, to: to)
-        error = "Conversion factor from #{from.abbreviation} to #{to.abbreviation} not found."
-        raise(StandardError, error) unless conversion
+        return value * conversion.factor if conversion
 
-        value * conversion.factor
+        # Check if there is reverse conversion entry defined
+        conversion = find_by(from: to, to: from)
+        unless conversion
+          raise(StandardError, "Conversion factor between #{from.abbreviation} and #{to.abbreviation} not found.")
+        end
+
+        value / conversion.factor
+      end
+
+      ##
+      # This function calculates the sum of quantities of a list of objects with quantities and different
+      # units into one. The list can be a list of hub authorizations, RHN requests, dispatch plan items,
+      # etc ... This method assumes that quantity is represented by the attribute <tt>quantity</tt> and unit of
+      # measure is represented by the attribute <tt>unit</tt> for each object in the object list. If that is not
+      # true then one should create those two attributes as aliases in the respective models.
+      #
+      def self.harmonized_total(list, unit)
+        list.inject(0) do |res, obj|
+          res + UnitConversion.convert(obj.unit, unit, obj.quantity)
+        end
       end
     end
   end

data/app/notifications/cats/core/dispatch_notification.rb

@@ -12,11 +12,13 @@ module Cats
         date = Date.today
         body = <<~BODY
           Commodity with the following details has been dispatched to you:
+              Authorization no. = #{dispatch.dispatch_plan_item.reference_no}
               Dispatch Ref. = #{dispatch.reference_no}
               Batch No. = #{commodity.batch_no}
               Commodity = #{commodity.name}
               Allocated Quantity = #{dispatch.dispatch_plan_item.quantity}
               Quantity = #{dispatch.quantity}
+              Unit = #{dispatch.unit.abbreviation}
               Truck Plate No. = #{dispatch.plate_no}
               Driver Name = #{dispatch.driver_name}
               Driver Phone = #{dispatch.driver_phone}

data/app/serializers/cats/core/commodity_serializer.rb

@@ -3,7 +3,7 @@ module Cats
     class CommoditySerializer < ActiveModel::Serializer
       attributes :id, :name, :batch_no, :description, :unit_of_measure_id, :unit_abbreviation, :project_id,
                  :project_code, :quantity, :best_use_before, :volume_per_metric_ton, :arrival_status, :status,
-                 :shipping_reference, :commodity_grade
+                 :shipping_reference, :commodity_grade, :package_unit_id, :package_unit_abbreviation
     end
   end
 end

data/app/serializers/cats/core/dispatch_authorization_serializer.rb

@@ -2,7 +2,8 @@ module Cats
   module Core
     class DispatchAuthorizationSerializer < ActiveModel::Serializer
       attributes :id, :dispatch_id, :dispatch_reference_no, :store_id, :store_name, :quantity, :authorized_by_id,
-                 :authorizer_full_name, :dispatch_status, :plate_no, :driver_name, :status
+                 :authorizer_full_name, :dispatch_status, :plate_no, :driver_name, :status, :unit_id,
+                 :unit_abbreviation
     end
   end
 end

data/app/serializers/cats/core/dispatch_plan_item_serializer.rb

@@ -4,7 +4,7 @@ module Cats
       attributes :id, :reference_no, :dispatch_plan_id, :plan_reference_no, :source_id, :source_name, :destination_id,
                  :destination_name, :quantity, :source_location_type, :destination_location_type, :commodity_status,
                  :status, :commodity_id, :commodity_name, :commodity_batch_no, :unit_abbreviation,
-                 :commodity_shipping_reference
+                 :commodity_shipping_reference, :unit_id
     end
   end
 end

data/app/serializers/cats/core/dispatch_transaction_serializer.rb

@@ -1,7 +1,8 @@
 module Cats
   module Core
     class DispatchTransactionSerializer < ActiveModel::Serializer
-      attributes :id, :source_id, :source_code, :dispatch_authorization_id, :quantity, :transaction_date, :status
+      attributes :id, :source_id, :source_code, :dispatch_authorization_id, :quantity, :transaction_date, :status,
+                 :unit_id, :unit_abbreviation
     end
   end
 end

data/app/serializers/cats/core/lost_commodity_serializer.rb

@@ -1,7 +1,7 @@
 module Cats
   module Core
     class LostCommoditySerializer < ActiveModel::Serializer
-      attributes :id, :receipt_authorization_id, :quantity, :remark
+      attributes :id, :receipt_authorization_id, :quantity, :remark, :unit_id, :unit_abbreviation
     end
   end
 end

data/app/serializers/cats/core/receipt_authorization_serializer.rb

@@ -3,7 +3,7 @@ module Cats
     class ReceiptAuthorizationSerializer < ActiveModel::Serializer
       attributes :id, :dispatch_id, :dispatch_reference_no, :store_id, :store_name, :quantity, :received_quantity,
                  :remark, :status, :authorized_by_id, :authorizer_full_name, :auth_details, :plate_no, :driver_name,
-                 :dispatch_status
+                 :dispatch_status, :unit_id, :unit_abbreviation
     end
   end
 end

data/app/serializers/cats/core/receipt_serializer.rb

@@ -1,7 +1,8 @@
 module Cats
   module Core
     class ReceiptSerializer < ActiveModel::Serializer
-      attributes :id, :receipt_authorization_id, :commodity_status, :commodity_grade, :quantity, :remark
+      attributes :id, :receipt_authorization_id, :commodity_status, :commodity_grade, :quantity, :remark,
+                 :unit_id, :unit_abbreviation
     end
   end
 end

data/app/serializers/cats/core/receipt_transaction_serializer.rb

@@ -2,7 +2,7 @@ module Cats
   module Core
     class ReceiptTransactionSerializer < ActiveModel::Serializer
       attributes :id, :receipt_authorization_id, :dispatch_reference_no, :destination_id, :destination_code, :quantity,
-                 :transaction_date, :status
+                 :transaction_date, :status, :unit_id, :unit_abbreviation
     end
   end
 end

data/app/serializers/cats/core/stack_serializer.rb

@@ -2,7 +2,7 @@ module Cats
   module Core
     class StackSerializer < ActiveModel::Serializer
       attributes :id, :code, :length, :width, :height, :start_x, :start_y, :commodity_id, :store_id, :commodity_status,
-                 :stack_status, :quantity, :commodity_batch_no
+                 :stack_status, :quantity, :commodity_batch_no, :unit_id, :unit_abbreviation
     end
   end
 end

data/app/services/cats/core/authorization_service.rb

@@ -46,6 +46,7 @@ module Cats
           start_x: 2,
           start_y: 2,
           commodity: commodity,
+          unit: commodity.package_unit,
           store: authorization.store,
           commodity_status: Commodity::GOOD,
           stack_status: Stack::ALLOCATED
@@ -54,6 +55,7 @@ module Cats
           receipt_authorization: authorization,
           destination: stack,
           quantity: authorization.received_quantity,
+          unit: authorization.unit,
           status: Transaction::DRAFT,
           transaction_date: Date.today
         )

data/app/services/cats/core/dispatch_plan_service.rb

@@ -15,6 +15,7 @@ module Cats
               dispatch_plan_item_id: item.id,
               store_id: store.id,
               quantity: item.quantity,
+              unit_id: item.unit_id,
               authorization_type: HubAuthorization::SOURCE,
               authorized_by_id: keeper.id,
               created_at: Date.today,

data/app/utils/cats/core/util.rb

@@ -0,0 +1,15 @@
+module Cats
+  module Core
+    class Util
+      def self.send_chain(obj, arr)
+        arr.inject(obj) { |o, a| o.send(a) }
+      end
+
+      def self.generate_pin(obj)
+        pin = SecureRandom.hex(10)
+        obj.auth_details = { pin: pin, active: true, expires_at: DateTime.now + 24.hour }
+        obj.save!
+      end
+    end
+  end
+end

data/app/utils/cats/core/validation_util.rb

@@ -0,0 +1,20 @@
+module Cats
+  module Core
+    class ValidationUtil
+      def self.validate_quantity(obj, parent_attr, relation, error_key, quantity_attribs = nil, unit_attribs = nil)
+        parent = obj.send(parent_attr)
+        quantity = quantity_attribs ? Util.send_chain(parent, quantity_attribs) : parent.quantity
+        unit = unit_attribs ? Util.send_chain(parent, unit_attribs) : parent.unit
+        total = UnitConversion.convert(unit, obj.unit, quantity)
+        used = UnitConversion.harmonized_total(relation, obj.unit)
+        used -= obj.quantity_was if obj.quantity_was
+
+        remaining = total - used
+        return unless obj.quantity > remaining
+
+        error = "exceeds #{error_key} quantity. Maximum allowed is #{remaining} #{obj.unit.abbreviation}"
+        obj.errors.add(:quantity, error)
+      end
+    end
+  end
+end

data/db/migrate/20210717033223_create_cats_core_commodities.rb

@@ -10,6 +10,11 @@ class CreateCatsCoreCommodities < ActiveRecord::Migration[6.1]
                    null: false,
                    index: { name: 'project_on_commodity_indx' },
                    foreign_key: { to_table: :cats_core_projects }
+      t.references :package_unit,
+                   null: true,
+                   index: { name: 'pu_on_commodity_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
+
       t.float :quantity, null: false
       t.string :description
       t.date :best_use_before, null: false

data/db/migrate/20210717171101_create_cats_core_stacks.rb

@@ -18,6 +18,10 @@ class CreateCatsCoreStacks < ActiveRecord::Migration[6.1]
       t.string :commodity_status, null: false, default: 'Good'
       t.string :stack_status, null: false, default: 'Reserved'
       t.float :quantity, null: false, default: 0
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_stack_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
 
       t.timestamps
     end

data/db/migrate/20210718042755_create_cats_core_rhn_requests.rb

@@ -7,6 +7,10 @@ class CreateCatsCoreRhnRequests < ActiveRecord::Migration[6.1]
                    index: { name: 'commodity_on_rhn_indx' },
                    foreign_key: { to_table: :cats_core_commodities }
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_rhn_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.date :request_date, null: false
       t.string :requested_by
 

data/db/migrate/20210718045516_create_cats_core_dispatches.rb

@@ -17,7 +17,7 @@ class CreateCatsCoreDispatches < ActiveRecord::Migration[6.1]
       t.references :unit,
                    null: false,
                    index: { name: 'unit_on_dispatches_indx' },
-                   foreign_key: { to_table: :cats_core_unit_of_measures } 
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :commodity_status, null: false, default: 'Good'
       t.string :remark
       t.references :prepared_by,

data/db/migrate/20210718055414_create_cats_core_dispatch_authorizations.rb

@@ -10,6 +10,10 @@ class CreateCatsCoreDispatchAuthorizations < ActiveRecord::Migration[7.0]
                    index: { name: 'store_on_da_indx' },
                    foreign_key: { to_table: :cats_core_stores }
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_da_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :status, null: false, default: 'Authorized'
       t.references :authorized_by,
                    null: false,

data/db/migrate/20210718202957_create_cats_core_dispatch_transactions.rb

@@ -11,6 +11,10 @@ class CreateCatsCoreDispatchTransactions < ActiveRecord::Migration[6.1]
                    foreign_key: { to_table: :cats_core_dispatch_authorizations }
       t.date :transaction_date, null: false
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_dt_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :status, null: false, default: 'Draft'
 
       t.timestamps

data/db/migrate/20210727074646_create_cats_core_receipt_authorizations.rb

@@ -10,6 +10,10 @@ class CreateCatsCoreReceiptAuthorizations < ActiveRecord::Migration[6.1]
                    index: { name: 'store_on_ra_indx' },
                    foreign_key: { to_table: :cats_core_stores } 
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'r_unit_on_da_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.float :received_quantity, null: false, default: 0
       t.string :status, null: false, default: 'Authorized'
       t.string :remark

data/db/migrate/20210727105834_create_cats_core_receipts.rb

@@ -8,6 +8,10 @@ class CreateCatsCoreReceipts < ActiveRecord::Migration[7.0]
       t.string :commodity_status, null: false
       t.string :commodity_grade
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_receipts_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :remark
 
       t.timestamps

data/db/migrate/20210728041505_create_cats_core_lost_commodities.rb

@@ -6,6 +6,10 @@ class CreateCatsCoreLostCommodities < ActiveRecord::Migration[7.0]
                    index: { name: 'ra_on_lc_indx' },
                    foreign_key: { to_table: :cats_core_receipt_authorizations }
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_lc_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :remark
 
       t.timestamps

data/db/migrate/20210814160628_create_cats_core_receipt_transactions.rb

@@ -11,6 +11,10 @@ class CreateCatsCoreReceiptTransactions < ActiveRecord::Migration[6.1]
                    foreign_key: { to_table: :cats_core_stacks }
       t.date :transaction_date, null: false
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'r_unit_on_dt_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :status, null: false, default: 'Draft'
 
       t.timestamps

data/db/migrate/20210814175406_create_cats_core_stack_transactions.rb

@@ -11,6 +11,10 @@ class CreateCatsCoreStackTransactions < ActiveRecord::Migration[6.1]
                    foreign_key: { to_table: :cats_core_stacks }
       t.date :transaction_date, null: false
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_st_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :status, null: false, default: 'Draft'
 
       t.timestamps

data/db/migrate/20220209083928_create_cats_core_hub_authorizations.rb

@@ -10,6 +10,10 @@ class CreateCatsCoreHubAuthorizations < ActiveRecord::Migration[7.0]
                    index: { name: 'store_on_ha_indx' },
                    foreign_key: { to_table: :cats_core_stores }
       t.float :quantity, null: false
+      t.references :unit,
+                   null: false,
+                   index: { name: 'unit_on_ha_indx' },
+                   foreign_key: { to_table: :cats_core_unit_of_measures }
       t.string :authorization_type, null: false
       t.references :authorized_by,
                    null: false,

data/db/migrate/20220923190857_create_cats_core_application_settings.rb

@@ -0,0 +1,14 @@
+class CreateCatsCoreApplicationSettings < ActiveRecord::Migration[7.0]
+  def change
+    create_table :cats_core_application_settings do |t|
+      t.string :key, unique: true
+      t.string :value, null: false
+      t.references :application_module,
+                   null: false,
+                   index: { name: 'am_on_as_indx' },
+                   foreign_key: { to_table: :cats_core_application_modules }
+
+      t.timestamps
+    end
+  end
+end

data/lib/cats/core/version.rb

@@ -1,5 +1,5 @@
 module Cats
   module Core
-    VERSION = '1.5.2'.freeze
+    VERSION = '1.5.4'.freeze
   end
 end

data/spec/factories/cats/core/application_settings.rb

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory :application_setting, class: 'Cats::Core::ApplicationSetting' do
+    key { FFaker::Name.name }
+    value { FFaker::Name.name }
+    application_module
+  end
+end

data/spec/factories/cats/core/commodities.rb

@@ -3,6 +3,7 @@ FactoryBot.define do
     batch_no { FFaker::Name.name }
     description { FFaker::Name.name }
     unit_of_measure
+    package_unit factory: :unit_of_measure
     project
     quantity { 100 }
     best_use_before { Date.today + 2.month }

data/spec/factories/cats/core/dispatch_authorizations.rb

@@ -7,6 +7,7 @@ FactoryBot.define do
     dispatch
     store { s }
     quantity { 100 }
+    unit { dispatch.unit }
     status { Cats::Core::ReceiptAuthorization::AUTHORIZED }
     authorized_by factory: :user
 

data/spec/factories/cats/core/dispatch_transactions.rb

@@ -1,12 +1,13 @@
 FactoryBot.define do
   factory :dispatch_transaction, class: 'Cats::Core::DispatchTransaction' do
-    transient do
-      stack { create(:stack) }
-    end
-    source { stack }
     association :dispatch_authorization, :approved
+    source do
+      create(:stack, unit: dispatch_authorization.unit)
+    end
+
     transaction_date { Date.today }
     quantity { 100 }
+    unit { dispatch_authorization.unit }
     status { Cats::Core::Transaction::DRAFT }
   end
 end

data/spec/factories/cats/core/dispatches.rb

@@ -10,7 +10,7 @@ FactoryBot.define do
       create(:hub_authorization, dispatch_plan_item: plan_item, quantity: 100, store: stack.store)
       plan_item
     end
-    unit factory: :unit_of_measure
+    unit { dispatch_plan_item.unit }
     transporter
     plate_no { FFaker::Name.name }
     driver_name { FFaker::Name.name }
@@ -29,6 +29,8 @@ FactoryBot.define do
       after(:create) do |dispatch, options|
         authorization = create(:dispatch_authorization, dispatch: dispatch, store: options.stack.store)
         dispatch.approve
+        options.stack.unit = dispatch.unit
+        options.stack.save!
         create(:dispatch_transaction, dispatch_authorization: authorization, source: options.stack, quantity: 100)
       end
     end
@@ -37,6 +39,8 @@ FactoryBot.define do
       after(:create) do |dispatch, options|
         authorization = create(:dispatch_authorization, dispatch: dispatch, store: options.stack.store)
         dispatch.approve
+        options.stack.unit = dispatch.unit
+        options.stack.save!
         create(:dispatch_transaction, dispatch_authorization: authorization, source: options.stack, quantity: 100)
         authorization.confirm
         dispatch.quantity = 100
@@ -49,6 +53,8 @@ FactoryBot.define do
       after(:create) do |dispatch, options|
         authorization = create(:dispatch_authorization, dispatch: dispatch, store: options.stack.store)
         dispatch.approve
+        options.stack.unit = dispatch.unit
+        options.stack.save!
         create(:dispatch_transaction, dispatch_authorization: authorization, source: options.stack, quantity: 100)
         authorization.confirm
         dispatch.quantity = 100

data/spec/factories/cats/core/hub_authorizations.rb

@@ -3,6 +3,7 @@ FactoryBot.define do
     dispatch_plan_item
     store
     quantity { 100 }
+    unit { dispatch_plan_item.unit }
     authorization_type { Cats::Core::HubAuthorization::SOURCE }
     authorized_by factory: :user
   end

data/spec/factories/cats/core/lost_commodities.rb

@@ -2,6 +2,7 @@ FactoryBot.define do
   factory :lost_commodity, class: 'Cats::Core::LostCommodity' do
     receipt_authorization
     quantity { 100 }
+    unit { receipt_authorization.unit }
     remark { FFaker::Name.name }
   end
 end

data/spec/factories/cats/core/receipt_authorizations.rb

@@ -3,6 +3,7 @@ FactoryBot.define do
     association :dispatch, :started
     store
     quantity { 100 }
+    unit { dispatch.unit }
     received_quantity { 0 }
     remark { FFaker::Name.name }
     status { Cats::Core::ReceiptAuthorization::AUTHORIZED }

data/spec/factories/cats/core/receipt_transactions.rb

@@ -1,9 +1,13 @@
 FactoryBot.define do
   factory :receipt_transaction, class: 'Cats::Core::ReceiptTransaction' do
     association :receipt_authorization, :confirmed
-    destination factory: :stack
+    destination do
+      create(:stack, unit: receipt_authorization.unit)
+    end
+
     transaction_date { Date.today }
     quantity { 100 }
+    unit { receipt_authorization.unit }
     status { Cats::Core::Transaction::DRAFT }
   end
 end

data/spec/factories/cats/core/receipts.rb

@@ -3,6 +3,7 @@ FactoryBot.define do
     receipt_authorization
     commodity_status { Cats::Core::Commodity::GOOD }
     quantity { 50 }
+    unit { receipt_authorization.unit }
     remark { FFaker::Name.name }
   end
 end

data/spec/factories/cats/core/rhn_requests.rb

@@ -6,6 +6,7 @@ FactoryBot.define do
       c.approve
       c
     end
+    unit { commodity.unit_of_measure }
     quantity { 100 }
     request_date { Date.today }
     requested_by { FFaker::Name.name }

data/spec/factories/cats/core/stack_transactions.rb

@@ -4,6 +4,7 @@ FactoryBot.define do
     destination factory: :stack
     transaction_date { Date.today }
     quantity { 25 }
+    unit { source.unit }
     status { Cats::Core::Transaction::DRAFT }
   end
 end

data/spec/factories/cats/core/stacks.rb

@@ -11,5 +11,6 @@ FactoryBot.define do
     commodity_status { Cats::Core::Commodity::GOOD }
     stack_status { Cats::Core::Stack::RESERVED }
     quantity { 100 }
+    unit { commodity.package_unit }
   end
 end