cats_core 1.2.30 → 1.2.34

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7d71e458593e0bfa2d57ddcba1e5ea4f1392df4223cff0c2b7ef5d1150859463
-  data.tar.gz: 4d2798fd1957265c3493ee8dca89b759d674d2e029489dc7382e7a6d5f62b0c4
+  metadata.gz: 28997d39a875dc6d5747f18de4f4803264aa98e8a49ee6fd09d79da78bf6a959
+  data.tar.gz: 2ce0c99ef80307ad76a4a070174898edda15ecc7a26b088f279109b9af065a8c
 SHA512:
-  metadata.gz: 739d0d54055f96059cfb2818f86026ac46518e3c64eaa81762e984ea803e95650c56f28f11ee738fe29254deeb9b5e76a668b037c0454c40c3e4a308db6001d0
-  data.tar.gz: e64a2e770bd662c26218e6d67a1c76d32ddf881deae9e77faecf3aa52628b1f19ac84f71fad4b202dcb200f8cc23aed3c2274f03aa8ff4513fba24b36ee7b254
+  metadata.gz: 6ce37b77dd8690b49307d4748a45ac60294e978d8ebb825e5f81324141ad464d0e021ba005dfa149553449f82e5e8fac68dbea2d679e13741d67b87cde1b75c0
+  data.tar.gz: 03fe91c921eb78c722bf959b5ac6f6c6368a86da82100f4238e8b854d4ad4b87b30d9ce5a24c2289a864eeba0588e4b5fb7f8bb76c500540ccdce5a51f16a69e

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

@@ -0,0 +1,26 @@
+module Cats
+  module Core
+    class OfferItem < ApplicationRecord
+      belongs_to :transport_offer
+      belongs_to :transport_bid_item
+
+      validates :price, presence: true, numericality: { greater_than: 0 }
+      validates :rank, numericality: { greater_than: 0 }, allow_nil: true
+      validate :validate_rank_is_set_for_winner
+
+      def mark_as_winner(rank)
+        raise(StandardError, 'Offer item already marked as winner.') if winner
+
+        self.rank = rank
+        self.winner = true
+        save!
+      end
+
+      def validate_rank_is_set_for_winner
+        return if rank
+
+        errors.add(:winner, 'cannot be set for a non-ranked offer item.') if winner
+      end
+    end
+  end
+end

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

@@ -8,13 +8,23 @@ module Cats
       WINNERS_DECLARED = 'Winner Declared'.freeze
       STATUSES = [NEW, OPEN, CLOSED, RANKED, WINNERS_DECLARED].freeze
 
+      # Bid types
+      REGIONAL = 'Regional'.freeze
+      NON_REGIONAL = 'Non Regional'.freeze
+      BID_TYPES = [REGIONAL, NON_REGIONAL].freeze
+
       has_many :transport_bid_items
-      has_many :transport_offers, through: :transport_bid_items
+      has_many :transport_offers
+      has_many :offer_items, through: :transport_offers
+      belongs_to :region, class_name: 'Cats::Core::Location', optional: true
 
       validates :reference_no, presence: true, uniqueness: true
-      validates :start_date, :end_date, :status, presence: true
+      validates :start_date, :end_date, :status, :bid_type, :bid_bond_amount, presence: true
+      validates :bid_bond_amount, numericality: { greater_than_or_equal_to: 0 }
       validates :status, inclusion: { in: STATUSES }
+      validates :bid_type, inclusion: { in: BID_TYPES }
       validate :validate_start_date_against_end_date
+      validate :validate_region
 
       def validate_start_date_against_end_date
         return unless start_date && end_date
@@ -22,6 +32,16 @@ module Cats
         errors.add(:start_date, 'should be before end date.') if start_date >= end_date
       end
 
+      def validate_region
+        return unless bid_type
+
+        errors.add(:region, 'should be null.') if bid_type == NON_REGIONAL && region
+
+        errors.add(:region, 'should not be null.') if bid_type == REGIONAL && !region
+
+        errors.add(:region, 'is not valid.') if region && region.location_type != Location::REGION
+      end
+
       def open
         raise(StandardError, 'Bid is already open.') if status == OPEN
 

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

@@ -5,7 +5,7 @@ module Cats
       belongs_to :route
       belongs_to :unit, class_name: 'Cats::Core::UnitOfMeasure', optional: true
 
-      has_many :transport_offers
+      has_many :offer_items
 
       validates :quantity, presence: true, numericality: { greater_than: 0 }
 

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

@@ -1,14 +1,13 @@
 module Cats
   module Core
     class TransportContract < ApplicationRecord
-      belongs_to :region, class_name: 'Cats::Core::Location'
       belongs_to :transporter
       belongs_to :transport_bid
+      has_many :contract_items
 
       validates :contract_no, presence: true, uniqueness: true
       validates :contract_date, :expires_on, presence: true
 
-      delegate(:name, to: :region, prefix: true)
       delegate(:name, to: :transporter, prefix: true)
       delegate(:reference_no, to: :transport_bid, prefix: :bid)
     end

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

@@ -1,28 +1,15 @@
 module Cats
   module Core
     class TransportOffer < ApplicationRecord
-      belongs_to :transport_bid_item
+      belongs_to :transport_bid
       belongs_to :transporter
+      has_many :offer_items
 
-      validates :price, presence: true, numericality: { greater_than: 0 }
-      validates :rank, numericality: { greater_than: 0 }, allow_nil: true
-      validate :validate_rank_is_set_for_winner
+      validates :offer_date, :bid_bond_amount, presence: true
+      validates :bid_bond_amount, numericality: { greater_than_or_equal_to: 0 }
 
       delegate(:name, to: :transporter, prefix: true)
-
-      def mark_as_winner(rank)
-        raise(StandardError, 'Offer already marked as winner.') if winner
-
-        self.rank = rank
-        self.winner = true
-        save!
-      end
-
-      def validate_rank_is_set_for_winner
-        return if rank
-
-        errors.add(:winner, 'cannot be set for a non-ranked offer.') if winner
-      end
+      delegate(:reference_no, to: :transport_bid, prefix: 'bid')
     end
   end
 end

data/db/migrate/20211215121151_create_cats_core_transport_bids.rb

@@ -6,6 +6,12 @@ class CreateCatsCoreTransportBids < ActiveRecord::Migration[6.1]
       t.date :start_date, null: false
       t.date :end_date, null: false
       t.string :status, null: false, default: 'New'
+      t.string :bid_type, null: false, default: 'Regional'
+      t.references :region,
+                   null: true,
+                   index: { name: 'region_on_tb_indx' },
+                   foreign_key: { to_table: :cats_core_locations } 
+      t.float :bid_bond_amount, null: false, default: 0
 
       t.timestamps
     end

data/db/migrate/{20211229160126_create_cats_core_transport_offers.rb → 20211229160125_create_cats_core_transport_offers.rb}

@@ -1,17 +1,16 @@
 class CreateCatsCoreTransportOffers < ActiveRecord::Migration[6.1]
   def change
     create_table :cats_core_transport_offers do |t|
-      t.references :transport_bid_item,
+      t.references :transport_bid,
                    null: false,
-                   index: { name: 'to_on_tbi_indx' },
-                   foreign_key: { to_table: :cats_core_transport_bid_items }
+                   index: { name: 'to_on_tb_indx' },
+                   foreign_key: { to_table: :cats_core_transport_bids }
       t.references :transporter,
                    null: false,
                    index: { name: 'to_on_transporter_indx' },
                    foreign_key: { to_table: :cats_core_transporters }
-      t.float :price, null: false
-      t.boolean :winner, null: false, default: false
-      t.integer :rank
+      t.date :offer_date, null: false
+      t.float :bid_bond_amount, null: false, default: 0
 
       t.timestamps
     end

data/db/migrate/20211229160126_create_cats_core_offer_items.rb

@@ -0,0 +1,19 @@
+class CreateCatsCoreOfferItems < ActiveRecord::Migration[6.1]
+  def change
+    create_table :cats_core_offer_items do |t|
+      t.references :transport_offer,
+                   null: false,
+                   index: { name: 'to_on_oi_indx' },
+                   foreign_key: { to_table: :cats_core_transport_offers }
+      t.references :transport_bid_item,
+                   null: false,
+                   index: { name: 'tbi_on_to_indx' },
+                   foreign_key: { to_table: :cats_core_transport_bid_items } 
+      t.float :price, null: false
+      t.boolean :winner, null: false, default: false
+      t.integer :rank
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20211229160127_create_cats_core_transport_contracts.rb

@@ -2,10 +2,6 @@ class CreateCatsCoreTransportContracts < ActiveRecord::Migration[6.1]
   def change
     create_table :cats_core_transport_contracts do |t|
       t.string :contract_no, unique: true
-      t.references :region,
-                   null: false,
-                   index: { name: 'region_on_tc_indx' },
-                   foreign_key: { to_table: :cats_core_locations }
       t.references :transporter,
                    null: false,
                    index: { name: 'transporter_on_tc_indx' },

data/lib/cats/core/version.rb

@@ -1,5 +1,5 @@
 module Cats
   module Core
-    VERSION = '1.2.30'.freeze
+    VERSION = '1.2.34'.freeze
   end
 end

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

@@ -0,0 +1,9 @@
+FactoryBot.define do
+  factory :offer_item, class: 'Cats::Core::OfferItem' do
+    transport_offer
+    transport_bid_item
+    price { 150 }
+    rank { 2 }
+    winner { false }
+  end
+end

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

@@ -4,5 +4,8 @@ FactoryBot.define do
     description { FFaker::Name.name }
     start_date { Date.today - 1.week }
     end_date { Date.today }
+    bid_type { Cats::Core::TransportBid::REGIONAL }
+    region factory: :location
+    bid_bond_amount { 100 }
   end
 end

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

@@ -1,7 +1,6 @@
 FactoryBot.define do
   factory :transport_contract, class: 'Cats::Core::TransportContract' do
     contract_no { FFaker::Name.name }
-    region factory: :location
     transporter
     transport_bid
     contract_date { Date.today }

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

@@ -1,9 +1,8 @@
 FactoryBot.define do
   factory :transport_offer, class: 'Cats::Core::TransportOffer' do
-    transport_bid_item
+    transport_bid
     transporter
-    price { 100 }
-    winner { false }
-    rank { nil }
+    bid_bond_amount { 100 }
+    offer_date { Date.today }
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cats_core
 version: !ruby/object:Gem::Version
-  version: 1.2.30
+  version: 1.2.34
 platform: ruby
 authors:
 - Henock L.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-12-22 00:00:00.000000000 Z
+date: 2022-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_model_serializers
@@ -271,6 +271,7 @@ files:
 - app/models/cats/core/menu_item.rb
 - app/models/cats/core/notification.rb
 - app/models/cats/core/notification_rule.rb
+- app/models/cats/core/offer_item.rb
 - app/models/cats/core/operator.rb
 - app/models/cats/core/plan.rb
 - app/models/cats/core/plan_item.rb
@@ -368,7 +369,8 @@ files:
 - db/migrate/20211030133752_add_status_to_cats_core_commodities.rb
 - db/migrate/20211215121151_create_cats_core_transport_bids.rb
 - db/migrate/20211215124452_create_cats_core_transport_bid_items.rb
-- db/migrate/20211229160126_create_cats_core_transport_offers.rb
+- db/migrate/20211229160125_create_cats_core_transport_offers.rb
+- db/migrate/20211229160126_create_cats_core_offer_items.rb
 - db/migrate/20211229160127_create_cats_core_transport_contracts.rb
 - db/migrate/20211229160128_create_cats_core_contract_items.rb
 - db/migrate/20211229160129_create_cats_core_commodity_substitutions.rb
@@ -397,6 +399,7 @@ files:
 - spec/factories/cats/core/menus.rb
 - spec/factories/cats/core/notification_rules.rb
 - spec/factories/cats/core/notifications.rb
+- spec/factories/cats/core/offer_items.rb
 - spec/factories/cats/core/operators.rb
 - spec/factories/cats/core/plan_item_details.rb
 - spec/factories/cats/core/plan_items.rb