cats_core 1.2.29 → 1.2.30

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 88acfd698b11129e941ca058a9fb856745acb440e3f624a6cbcdcf0f1f4304b7
-  data.tar.gz: 451c7311e7c447d36729102c1dcb7d5756491eb08bbad88266b72bd52a7f9893
+  metadata.gz: 7d71e458593e0bfa2d57ddcba1e5ea4f1392df4223cff0c2b7ef5d1150859463
+  data.tar.gz: 4d2798fd1957265c3493ee8dca89b759d674d2e029489dc7382e7a6d5f62b0c4
 SHA512:
-  metadata.gz: '096b15bd9fb62182f055d7ddbeee44d5614f0631bdc7f2f93869c26414a60dcede4b3d1971b6787e61fa7eb34ed687086964dd93978ffc6a0cb7c380c7c7b259'
-  data.tar.gz: bfc6eb7f162a838f48a4c153faffa9652b25fa951e9d0ad1711fee49044496a499a28f22ca45d5df4cd47739646195301ef118e67b6695ff0e6cfd85fae107a1
+  metadata.gz: 739d0d54055f96059cfb2818f86026ac46518e3c64eaa81762e984ea803e95650c56f28f11ee738fe29254deeb9b5e76a668b037c0454c40c3e4a308db6001d0
+  data.tar.gz: e64a2e770bd662c26218e6d67a1c76d32ddf881deae9e77faecf3aa52628b1f19ac84f71fad4b202dcb200f8cc23aed3c2274f03aa8ff4513fba24b36ee7b254

data/app/controllers/cats/core/routes_controller.rb

@@ -6,7 +6,7 @@ module Cats
       private
 
       def model_params
-        params.require(:payload).permit(:source_id, :destination_id)
+        params.require(:payload).permit(:region_id, :source_id, :destination_id)
       end
     end
   end

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

@@ -3,16 +3,36 @@ module Cats
     class Route < ApplicationRecord
       before_validation :set_name, on: %i[create update]
 
+      belongs_to :region, class_name: 'Cats::Core::Location'
       belongs_to :source, class_name: 'Cats::Core::Location'
       belongs_to :destination, class_name: 'Cats::Core::Location'
 
       validates :source_id, uniqueness: { scope: :destination_id }
+      validate :validate_region, :validate_source, :validate_destination
 
       def set_name
         return unless source && destination
 
         self.name = "#{source.name} - #{destination.name}"
       end
+
+      def validate_region
+        return unless region
+
+        errors.add(:region, 'should be a valid region.') unless region.location_type == Location::REGION
+      end
+
+      def validate_source
+        return unless source
+
+        errors.add(:source, 'cannot be a region.') if source.location_type == Location::REGION
+      end
+
+      def validate_destination
+        return unless destination
+
+        errors.add(:destination, 'cannot be a region.') if destination.location_type == Location::REGION
+      end
     end
   end
 end

data/db/migrate/20210718040129_create_cats_core_routes.rb

@@ -2,6 +2,10 @@ class CreateCatsCoreRoutes < ActiveRecord::Migration[6.1]
   def change
     create_table :cats_core_routes do |t|
       t.string :name, null: false
+      t.references :region,
+                   null: false,
+                   index: { name: 'region_on_routes_indx' },
+                   foreign_key: { to_table: :cats_core_locations }
       t.references :source,
                    null: false,
                    index: { name: 'source_on_routes_indx' },
@@ -13,7 +17,7 @@ class CreateCatsCoreRoutes < ActiveRecord::Migration[6.1]
 
       t.timestamps
 
-      t.index [:source_id, :destination_id], unique: true
+      t.index %i[source_id destination_id], unique: true
     end
   end
 end

data/lib/cats/core/version.rb

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

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

@@ -1,6 +1,7 @@
 FactoryBot.define do
   factory :route, class: 'Cats::Core::Route' do
-    source factory: :location
-    destination factory: :location
+    region factory: :location
+    source factory: :woreda
+    destination factory: :woreda
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cats_core
 version: !ruby/object:Gem::Version
-  version: 1.2.29
+  version: 1.2.30
 platform: ruby
 authors:
 - Henock L.