Skip to content

Commit 66b5fea

Browse files
committed
Fix missing format field
1 parent 5b10d53 commit 66b5fea

File tree

2 files changed

+21
-2
lines changed

2 files changed

+21
-2
lines changed

astra_app/core/templates/admin/core/membership_csv_import.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -696,8 +696,8 @@ <h3 class="card-title">
696696
</div>
697697
<div class="card-body">
698698
{% for field in form %}
699-
{% if field.name == "format" %}
700-
{# Format is inferred; skip rendering. #}
699+
{% if field.is_hidden %}
700+
{{ field }}
701701
{% elif field.name == "import_file" %}
702702
<div class="form-group field-name">
703703
<div class="row">

astra_app/core/tests/test_admin_import_memberships_csv.py

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,25 @@ def test_import_formats_csv_only(self) -> None:
121121
formats = admin_instance.get_import_formats()
122122
self.assertEqual(formats, [base_formats.CSV])
123123

124+
def test_import_page_includes_format_field(self) -> None:
125+
self._login_as_freeipa_admin("alex")
126+
127+
admin_user = FreeIPAUser(
128+
"alex",
129+
{
130+
"uid": ["alex"],
131+
"mail": ["[email protected]"],
132+
"memberof_group": ["admins"],
133+
},
134+
)
135+
136+
with patch("core.backends.FreeIPAUser.get", return_value=admin_user):
137+
url = reverse("admin:core_membershipcsvimportlink_import")
138+
resp = self.client.get(url)
139+
140+
self.assertEqual(resp.status_code, 200)
141+
self.assertContains(resp, 'name="format"')
142+
124143
def test_live_import_creates_membership_and_unmatched_export(self) -> None:
125144
MembershipType.objects.update_or_create(
126145
code="individual",

0 commit comments

Comments
 (0)