Allow non-unique nil pxe_image_types during seed

https://bugzilla.redhat.com/show_bug.cgi?id=1588417

We want to disallow the same name template in different regions with the
same pxe_image_type.  Normally, region id sequences prevent this from
happening but we were treating nil pxe_image_types as the same.
Therefore you could have the "same" pxe_image_type in different regions
for the same name template.

app/models/customization_template.rb

@@ -5,11 +5,16 @@ class CustomizationTemplate < ApplicationRecord
   has_many   :pxe_images, :through => :pxe_image_type
 
   validates :pxe_image_type, :presence => true, :unless => :system?
-  validates :name,           :unique_within_region => true
+  validates :name, :unique_within_region => true
+  validates :name, :uniqueness => {:scope => :pxe_image_type } unless :system_and_no_pxe_image_type?
 
   scope :with_pxe_image_type_id, ->(pxe_image_type_id) { where(:pxe_image_type_id => pxe_image_type_id) }
   scope :with_system,            ->(bool = true)       { where(:system => bool) }
 
+  def system_and_no_pxe_image_type?
+    system? && pxe_image_type_id.nil?
+  end
+
   def self.seed_file_name
     @seed_file_name ||= Rails.root.join("db", "fixtures", "#{table_name}.yml")
   end

spec/models/customization_template_spec.rb

@@ -0,0 +1,30 @@
+describe CustomizationTemplate do
+  context "unique name validation" do
+    let(:pxe_image_type)                { FactoryGirl.create(:pxe_image_type) }
+    let(:customization_template)        { FactoryGirl.create(:customization_template, :pxe_image_type => pxe_image_type) }
+    let(:system_customization_template) { described_class.create!(:system => true, :name => "nick") }
+
+    it "doesn't allow same name in same region" do
+      FactoryGirl.create(:customization_template, :name => "fred")
+      expect { FactoryGirl.create(:customization_template, :name => "fred") }.to raise_error(ActiveRecord::RecordInvalid)
+    end
+
+    it "allows different names with the same pxe_image_type in same region" do
+      FactoryGirl.create(:customization_template, :name => "fred", :pxe_image_type => pxe_image_type)
+      FactoryGirl.create(:customization_template, :name => "nick", :pxe_image_type => pxe_image_type)
+      expect(described_class.count).to eq(2)
+    end
+
+    it "allow same name, nil pxe_image_types in different regions (for system seeding)" do
+      ctid = ApplicationRecord.id_in_region(system_customization_template.id, ApplicationRecord.my_region_number + 1)
+      expect { described_class.create!(:system => true, :name => "nick", :id => ctid) }.to change(described_class, :count).by(1)
+    end
+
+    it "allows same name, different pxe_image_types in different regions" do
+      pid =  ApplicationRecord.id_in_region(pxe_image_type.id, MiqRegion.my_region_number + 1)
+      ctid = ApplicationRecord.id_in_region(customization_template.id, MiqRegion.my_region_number + 1)
+      pt2 =  FactoryGirl.create(:pxe_image_type, :id => pid)
+      expect { FactoryGirl.create(:customization_template, :pxe_image_type => pt2, :id => ctid) }.to change(described_class, :count).by(1)
+    end
+  end
+end