Fix OrganizationPage slug regeneration on name changes (#3002)

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: jkachel <[email protected]>
Co-authored-by: James Kachel <[email protected]>

Author: Copilot

b2b/models.py

@@ -98,14 +98,15 @@ class OrganizationPage(Page):
         FieldPanel("sso_organization_id"),
     ]
 
-    promote_panels = []
+    # Use default promote_panels from Page to allow manual slug editing
 
     def save(self, clean=True, user=None, log_action=False, **kwargs):  # noqa: FBT002
         """Save the page, and update the slug and title appropriately."""
 
         self.title = str(self.name)
 
-        self.slug = slugify(f"org-{self.name}")
+        if not self.slug:
+            self.slug = slugify(f"org-{self.name}")
         Page.save(self, clean=clean, user=user, log_action=log_action, **kwargs)
 
     def get_learners(self):

b2b/models_test.py

@@ -3,11 +3,8 @@
 import faker
 import pytest
 
-from b2b.factories import ContractPageFactory
-from courses.factories import (
-    CourseRunFactory,
-    ProgramFactory,
-)
+from b2b.factories import ContractPageFactory, OrganizationPageFactory
+from courses.factories import CourseRunFactory, ProgramFactory
 
 pytestmark = [pytest.mark.django_db]
 FAKE = faker.Faker()
@@ -53,3 +50,45 @@ def test_add_program_courses_to_contract(mocker):
 
     assert contract.programs.count() == 1
     assert contract.get_course_runs().count() == 4
+
+
+def test_organization_page_slug_preserved_on_name_change():
+    """Test that the slug is not regenerated when only the name changes."""
+    org = OrganizationPageFactory.create(name="MIT")
+    original_slug = org.slug
+
+    # Change the name
+    org.name = "MIT - Universal AI"
+    org.save()
+    org.refresh_from_db()
+
+    # The slug should not have changed
+    assert org.slug == original_slug
+    # But the title should reflect the new name
+    assert org.title == "MIT - Universal AI"
+
+
+def test_organization_page_slug_generated_on_create():
+    """Test that the slug is generated when creating a new organization."""
+    org = OrganizationPageFactory.create(name="Test Organization", slug="")
+
+    # The slug should have been generated
+    assert org.slug == "org-test-organization"
+    assert org.title == "Test Organization"
+
+
+def test_organization_page_slug_not_overwritten_if_set():
+    """Test that a manually set slug is not overwritten."""
+    org = OrganizationPageFactory.create(name="Test Org", slug="custom-slug")
+
+    # The slug should be the custom one
+    assert org.slug == "custom-slug"
+
+    # Change the name
+    org.name = "Test Org Updated"
+    org.save()
+    org.refresh_from_db()
+
+    # The slug should still be the custom one
+    assert org.slug == "custom-slug"
+    assert org.title == "Test Org Updated"