Sree::Zone, require at least one member

Add validations and specs
This commit is contained in:
Gaetan Craig-Riou
2025-01-15 14:33:11 +11:00
parent bcb4525cdd
commit 75c0752340
2 changed files with 8 additions and 0 deletions

View File

@@ -7,6 +7,8 @@ module Spree
has_and_belongs_to_many :shipping_methods, join_table: 'spree_shipping_methods_zones'
validates :name, presence: true, uniqueness: true
validates :zone_members, presence: true
after_save :remove_defunct_members
after_save :remove_previous_default

View File

@@ -3,6 +3,12 @@
require 'spec_helper'
RSpec.describe Spree::Zone do
describe "validations" do
it { is_expected.to validate_presence_of(:name) }
it { is_expected.to validate_uniqueness_of(:name) }
it { is_expected.to validate_presence_of(:zone_members) }
end
describe "#match" do
let(:country_zone) { create(:zone, name: 'CountryZone') }
let(:country) do