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

apps/exports/models.py

@@ -41,6 +41,25 @@ def last_run(self):
     def latest_version(self):
         return Version.objects.get_for_object(self).first()
 
+    @property
+    def edit_url(self):
+        if isinstance(self, ExportConfig):
+            return reverse('exports:edit_export_config', args=[self.id])
+        elif isinstance(self, MultiProjectExportConfig):
+            return reverse('exports:edit_multi_export_config', args=[self.id])
+        raise ValueError(f"Unknown config type: {type(self)}")
+
+    @property
+    def last_run_log_url(self):
+        run = self.last_run
+        if run is None:
+            return None
+        if isinstance(self, ExportConfig):
+            return reverse('exports:run_log', args=[run.id])
+        elif isinstance(self, MultiProjectExportConfig):
+            return reverse('exports:multi_run_log', args=[run.id])
+        raise ValueError(f"Unknown config type: {type(self)}")
+
     @property
     def details_url(self):
         raise NotImplementedError

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_list_view.py

@@ -1,24 +1,30 @@
 from django.urls import reverse
-from unmagic import fixture, use
-
-from apps.exports.models import ExportConfig
-from tests.fixtures import (
-    authed_client,
-    commcare_account,
-    commcare_project,
-    database,
-)
-
-
-@fixture
-@use('db')
-def export_config():
-    yield ExportConfig.objects.create(
-        name='Test Export Config',
-        project=commcare_project(),
-        account=commcare_account(),
-        database=database(),
-    )
+from unmagic import use
+
+from apps.exports.tests.fixtures import export_config, multi_export_config
+from tests.fixtures import authed_client
+
+
+class TestExportConfigBaseProperties:
+    @use(export_config)
+    def test_export_config_edit_url(self):
+        config = export_config()
+        expected = reverse('exports:edit_export_config', args=[config.id])
+        assert config.edit_url == expected
+
+    @use(multi_export_config)
+    def test_multi_export_config_edit_url(self):
+        config = multi_export_config()
+        expected = reverse('exports:edit_multi_export_config', args=[config.id])
+        assert config.edit_url == expected
+
+    @use(export_config)
+    def test_last_run_log_url_none_when_no_run(self):
+        assert export_config().last_run_log_url is None
+
+    @use(multi_export_config)
+    def test_last_run_log_url_none_when_no_run_multi(self):
+        assert multi_export_config().last_run_log_url is None
 
 
 class TestExportsHomeView:

commcare_sync/tests/test_views.py

@@ -0,0 +1,46 @@
+import pytest
+from django.test import RequestFactory
+
+from commcare_sync.views import get_config_page_size, get_page_from_request
+
+VALID_PAGE_SIZES = [10, 20, 50]
+
+
+class TestGetConfigPageSize:
+    def _req(self, params=''):
+        rf = RequestFactory()
+        return rf.get(f'/?{params}')
+
+    def test_default_is_10(self):
+        assert get_config_page_size(self._req()) == 10
+
+    def test_valid_sizes_accepted(self):
+        for size in VALID_PAGE_SIZES:
+            assert get_config_page_size(self._req(f'page_size={size}')) == size
+
+    def test_invalid_size_falls_back_to_default(self):
+        assert get_config_page_size(self._req('page_size=7')) == 10
+
+    def test_non_integer_falls_back_to_default(self):
+        assert get_config_page_size(self._req('page_size=abc')) == 10
+
+
+class TestGetPageFromRequest:
+    def _req(self, params=''):
+        rf = RequestFactory()
+        return rf.get(f'/?{params}')
+
+    def test_default_is_1(self):
+        assert get_page_from_request(self._req()) == 1
+
+    def test_valid_page_returned(self):
+        assert get_page_from_request(self._req('page=3')) == 3
+
+    def test_zero_clamped_to_1(self):
+        assert get_page_from_request(self._req('page=0')) == 1
+
+    def test_negative_clamped_to_1(self):
+        assert get_page_from_request(self._req('page=-5')) == 1
+
+    def test_non_integer_returns_1(self):
+        assert get_page_from_request(self._req('page=abc')) == 1

commcare_sync/views.py

@@ -1,5 +1,7 @@
 from django.conf import settings
 
+_VALID_CONFIG_PAGE_SIZES = (10, 20, 50)
+
 
 def get_ui_page_size(request):
     limit = settings.COMMCARE_SYNC_UI_PAGE_SIZE
@@ -15,3 +17,21 @@ def get_hide_skipped_from_request(request):
     if 'hide_skipped' in request.GET:
         return request.GET['hide_skipped'] == 'y'
     return False
+
+
+def get_config_page_size(request):
+    if 'page_size' in request.GET:
+        try:
+            size = int(request.GET['page_size'])
+            if size in _VALID_CONFIG_PAGE_SIZES:
+                return size
+        except ValueError:
+            pass
+    return _VALID_CONFIG_PAGE_SIZES[0]
+
+
+def get_page_from_request(request):
+    try:
+        return max(int(request.GET.get('page', 1)), 1)
+    except ValueError:
+        return 1