Enforce strict competition ID changes when pre-registering (#10059)

* Only allow pre-register for announced competitions

* Disable changing of competition ID if has registrations

* Stub microservice in relevant tests

* Use less strict checks for pre-registering

* Harmonize Delegate and Organizer test cases

app/models/competition.rb

@@ -612,7 +612,7 @@ def info_for(user)
   end
 
   def user_can_pre_register?(user)
-    delegates.include?(user) || trainee_delegates.include?(user) || organizers.include?(user)
+    (delegates.include?(user) || trainee_delegates.include?(user) || organizers.include?(user)) && self.confirmed_or_visible?
   end
 
   attr_accessor :being_cloned_from_id

app/views/competitions/_competition_form.html.erb

@@ -4,6 +4,7 @@
   competition: @competition.to_form_data,
   usesV2Registrations: @competition.uses_v2_registrations,
   canChangeRegistrationSystem: @competition.can_change_registration_system?,
+  hasAnyRegistrations: @competition.any_registrations?,
   storedEvents: @competition.events,
   isAdminView: @competition_admin_view,
   isPersisted: @competition.persisted?,

app/webpacker/components/CompetitionForm/FormSections/NameDetails.js

@@ -6,7 +6,7 @@ import { competitionMaxShortNameLength } from '../../../lib/wca-data.js.erb';
 import { useFormObject } from '../../wca/FormBuilder/provider/FormObjectProvider';
 
 export default function NameDetails() {
-  const { isPersisted, isAdminView } = useStore();
+  const { hasAnyRegistrations, isPersisted, isAdminView } = useStore();
 
   const { name } = useFormObject();
 
@@ -15,7 +15,7 @@ export default function NameDetails() {
 
   return (
     <>
-      {isPersisted && <InputString id="competitionId" disabled={disableIdAndShortName} />}
+      {isPersisted && <InputString id="competitionId" disabled={disableIdAndShortName || hasAnyRegistrations} />}
       <InputString id="name" required />
       {isPersisted && (
         <InputString

app/webpacker/components/CompetitionForm/index.js

@@ -77,6 +77,7 @@ export default function Wrapper({
   competition = null,
   usesV2Registrations = false,
   canChangeRegistrationSystem = false,
+  hasAnyRegistrations = false,
   storedEvents = [],
   isAdminView = false,
   isPersisted = false,
@@ -105,6 +106,7 @@ export default function Wrapper({
       initialState={{
         usesV2Registrations,
         canChangeRegistrationSystem,
+        hasAnyRegistrations,
         storedEvents,
         isAdminView,
         isPersisted,

spec/controllers/registrations_controller_spec.rb

@@ -5,7 +5,7 @@
 RSpec.describe RegistrationsController, clean_db_with_truncation: true do
   context "signed in as organizer" do
     let!(:organizer) { FactoryBot.create(:user) }
-    let(:competition) { FactoryBot.create(:competition, :registration_open, organizers: [organizer], events: Event.where(id: %w(222 333))) }
+    let(:competition) { FactoryBot.create(:competition, :registration_open, :visible, organizers: [organizer], events: Event.where(id: %w(222 333))) }
     let(:zzyzx_user) { FactoryBot.create :user, name: "Zzyzx" }
     let(:registration) { FactoryBot.create(:registration, competition: competition, user: zzyzx_user) }
 

spec/features/competition_management_spec.rb

@@ -40,6 +40,12 @@ def wca_registration_checkbox
 # Apparition browser engine does not render React state changes properly.
 # We can remove this `retry` count when we migrated to a "proper" browser engine in tests.
 RSpec.feature "Competition management", js: true, retry: 10 do
+  before(:each) do
+    # Stub microservice as an interim solution during our Microservice->Monolith V2 migration
+    #   Can safely be removed once the wca-registrations microservice is gone. (GB 2024-10-15)
+    allow(Microservices::Registrations).to receive(:competitor_count_by_competition).and_return(0)
+  end
+
   context "when signed in as admin" do
     let!(:admin) { FactoryBot.create :admin }
     before :each do