Tables for export-, refresh- and forwarding configs to support pagination and automatic refresh.

apps/exports/models.py

@@ -29,14 +29,6 @@ class ExportConfigBase(ScheduleMixin, BaseModel):
     class Meta:
         abstract = True
 
-    @property
-    def last_run(self):
-        return (
-            self.runs.exclude(status=ExportRun.Status.QUEUED)
-            .order_by('-created_at')
-            .first()
-        )
-
     @property
     def latest_version(self):
         return Version.objects.get_for_object(self).first()
@@ -67,6 +59,18 @@ def details_url(self):
     def __str__(self):
         return f'{self.name} - {self.project}'
 
+    @property
+    def edit_url(self):
+        return reverse('exports:edit_export_config', args=[self.id])
+
+    @property
+    def last_run_log_url(self):
+        run = self.last_run
+        return None if run is None else reverse(
+            'exports:run_log',
+            args=[run.id],
+        )
+
 
 @reversion.register()
 class MultiProjectExportConfig(ExportConfigBase):
@@ -102,6 +106,18 @@ def get_projects_display_short(self):
                 '<br>'.join(p.domain for p in self.projects.all())
             )
 
+    @property
+    def edit_url(self):
+        return reverse('exports:edit_multi_export_config', args=[self.id])
+
+    @property
+    def last_run_log_url(self):
+        run = self.last_run
+        return None if run is None else reverse(
+            'exports:multi_run_log',
+            args=[run.id],
+        )
+
 
 class ExportRunBase(RunBaseModel):
     class Status(models.TextChoices):

apps/exports/tests/fixtures.py

@@ -6,7 +6,12 @@
 
 from apps.commcare.models import CommCareAccount, CommCareProject, CommCareServer
 from apps.db.models import Database
-from apps.exports.models import ExportConfig
+from apps.exports.models import (
+    ExportConfig,
+    ExportRun,
+    MultiProjectExportConfig,
+    MultiProjectExportRun,
+)
 from tests.fixtures import (
     commcare_account,
     commcare_project,
@@ -131,3 +136,42 @@ def export_config_db_fixture():
     )
     config_file.close()
     yield export_config
+
+
+@fixture
+@use('db')
+def export_config():
+    yield ExportConfig.objects.create(
+        name='Test Export Config',
+        project=commcare_project(),
+        account=commcare_account(),
+        database=database(),
+    )
+
+
+@fixture
+@use('db')
+def multi_export_config():
+    yield MultiProjectExportConfig.objects.create(
+        name='Multi Export Config',
+        account=commcare_account(),
+        database=database(),
+    )
+
+
+@fixture
+@use('db')
+def export_run():
+    yield ExportRun.objects.create(
+        base_export_config=export_config(),
+        status=ExportRun.Status.COMPLETED,
+    )
+
+
+@fixture
+@use('db')
+def multi_export_run():
+    yield MultiProjectExportRun.objects.create(
+        base_export_config=multi_export_config(),
+        status=MultiProjectExportRun.Status.COMPLETED,
+    )

apps/exports/tests/test_details_view.py

@@ -0,0 +1,132 @@
+from django.urls import reverse
+from unmagic import use
+
+from apps.exports.models import ExportRun
+from apps.exports.tests.fixtures import export_config_db_fixture
+from tests.fixtures import authed_client
+
+
+class TestExportDetailsSmoke:
+    @use(authed_client, export_config_db_fixture)
+    def test_returns_200(self):
+        response = authed_client().get(
+            reverse(
+                'exports:export_details', args=[export_config_db_fixture().id]
+            )
+        )
+        assert response.status_code == 200
+
+    @use(authed_client, export_config_db_fixture)
+    def test_no_details_suffix_in_heading(self):
+        response = authed_client().get(
+            reverse(
+                'exports:export_details', args=[export_config_db_fixture().id]
+            )
+        )
+        assert '- Details' not in response.content.decode()
+
+    @use(authed_client, export_config_db_fixture)
+    def test_run_history_section_present(self):
+        response = authed_client().get(
+            reverse(
+                'exports:export_details', args=[export_config_db_fixture().id]
+            )
+        )
+        content = response.content.decode()
+        assert 'Run History' in content
+        assert 'id="run-table"' in content
+
+    @use(authed_client, export_config_db_fixture)
+    def test_schedule_column_present(self):
+        response = authed_client().get(
+            reverse(
+                'exports:export_details', args=[export_config_db_fixture().id]
+            )
+        )
+        assert 'Schedule' in response.content.decode()
+
+    @use(authed_client, export_config_db_fixture)
+    def test_status_filter_dropdown_present(self):
+        response = authed_client().get(
+            reverse(
+                'exports:export_details', args=[export_config_db_fixture().id]
+            )
+        )
+        content = response.content.decode()
+        assert 'status-filter-form' in content
+        assert 'has_status_filter' in content
+
+
+class TestExportRunHistoryTableEndpoint:
+    @use(authed_client, export_config_db_fixture)
+    def test_returns_200(self):
+        url = reverse(
+            'exports:run_history_table', args=[export_config_db_fixture().id]
+        )
+        assert authed_client().get(url).status_code == 200
+
+    @use(authed_client, export_config_db_fixture)
+    def test_no_filter_shows_all_statuses(self):
+        config = export_config_db_fixture()
+        completed_run = ExportRun.objects.create(
+            base_export_config=config, status=ExportRun.Status.COMPLETED
+        )
+        failed_run = ExportRun.objects.create(
+            base_export_config=config, status=ExportRun.Status.FAILED
+        )
+        url = reverse('exports:run_history_table', args=[config.id])
+        content = authed_client().get(url).content.decode()
+        # Use log-{id} marker which only appears in rendered run rows
+        assert f'log-{completed_run.id}' in content
+        assert f'log-{failed_run.id}' in content
+
+    @use(authed_client, export_config_db_fixture)
+    def test_status_filter_excludes_unchecked(self):
+        config = export_config_db_fixture()
+        completed_run = ExportRun.objects.create(
+            base_export_config=config, status=ExportRun.Status.COMPLETED
+        )
+        failed_run = ExportRun.objects.create(
+            base_export_config=config, status=ExportRun.Status.FAILED
+        )
+        url = reverse('exports:run_history_table', args=[config.id])
+        content = (
+            authed_client()
+            .get(
+                url,
+                QUERY_STRING='has_status_filter=1&status_filter=completed',
+            )
+            .content.decode()
+        )
+        # Use log-{id} marker which only appears in rendered run rows
+        assert f'log-{completed_run.id}' in content
+        assert f'log-{failed_run.id}' not in content
+
+    @use(authed_client, export_config_db_fixture)
+    def test_empty_filter_shows_nothing(self):
+        config = export_config_db_fixture()
+        run = ExportRun.objects.create(
+            base_export_config=config, status=ExportRun.Status.COMPLETED
+        )
+        url = reverse('exports:run_history_table', args=[config.id])
+        content = (
+            authed_client()
+            .get(url, QUERY_STRING='has_status_filter=1')
+            .content.decode()
+        )
+        # No status values sent → no runs visible; use log-{id} marker which only
+        # appears when a run row is rendered, not in URL paths.
+        assert f'log-{run.id}' not in content
+
+    @use(authed_client, export_config_db_fixture)
+    def test_pagination_default_10(self):
+        config = export_config_db_fixture()
+        for _ in range(15):
+            ExportRun.objects.create(
+                base_export_config=config, status=ExportRun.Status.COMPLETED
+            )
+        url = reverse('exports:run_history_table', args=[config.id])
+        response = authed_client().get(url)
+        assert response.status_code == 200
+        # Pagination controls should appear when there are > 10 runs
+        assert 'pagination' in response.content.decode()