arthexis 0.1.4__tar.gz → 0.1.6__tar.gz

{arthexis-0.1.4 → arthexis-0.1.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arthexis
-Version: 0.1.4
+Version: 0.1.6
 Summary: Django-based MESH system
 Author-email: "Rafael J. Guillén-Osorio" <tecnologia@gelectriic.com>
 License-Expression: MIT
@@ -118,7 +118,7 @@ Arthexis Constellation is a narrative-driven Django-based suite that centralizes
 - Single codebase with special features per role
 
 ## Support
-You may contact us at tecnologia at gelectriic dot com or visit our web page https://www.gelectriic.com/ for professional services and commercial support.
+You may contact us at tecnologia at gelectriic dot com or visit our [web page](https://www.gelectriic.com/) for professional services and commercial support.
 
 ## About Me
 > "What? You want to know about me too? Well, I enjoy developing software, roleplaying games, long walks on the beach and a fourth secret thing."

{arthexis-0.1.4 → arthexis-0.1.6}/README.md

@@ -10,7 +10,7 @@ Arthexis Constellation is a narrative-driven Django-based suite that centralizes
 - Single codebase with special features per role
 
 ## Support
-You may contact us at tecnologia at gelectriic dot com or visit our web page https://www.gelectriic.com/ for professional services and commercial support.
+You may contact us at tecnologia at gelectriic dot com or visit our [web page](https://www.gelectriic.com/) for professional services and commercial support.
 
 ## About Me
 > "What? You want to know about me too? Well, I enjoy developing software, roleplaying games, long walks on the beach and a fourth secret thing."

{arthexis-0.1.4 → arthexis-0.1.6}/arthexis.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: arthexis
-Version: 0.1.4
+Version: 0.1.6
 Summary: Django-based MESH system
 Author-email: "Rafael J. Guillén-Osorio" <tecnologia@gelectriic.com>
 License-Expression: MIT
@@ -118,7 +118,7 @@ Arthexis Constellation is a narrative-driven Django-based suite that centralizes
 - Single codebase with special features per role
 
 ## Support
-You may contact us at tecnologia at gelectriic dot com or visit our web page https://www.gelectriic.com/ for professional services and commercial support.
+You may contact us at tecnologia at gelectriic dot com or visit our [web page](https://www.gelectriic.com/) for professional services and commercial support.
 
 ## About Me
 > "What? You want to know about me too? Well, I enjoy developing software, roleplaying games, long walks on the beach and a fourth secret thing."

{arthexis-0.1.4 → arthexis-0.1.6}/arthexis.egg-info/SOURCES.txt

@@ -102,6 +102,7 @@ tests/test_odoo_profile.py
 tests/test_odoo_profile_admin.py
 tests/test_offline.py
 tests/test_package_release_admin_actions.py
+tests/test_post_office_admin_group.py
 tests/test_pypi_token.py
 tests/test_readme_language.py
 tests/test_reference_qr_code.py

{arthexis-0.1.4 → arthexis-0.1.6}/core/admin.py

@@ -3,7 +3,7 @@ from django.contrib import admin
 from django.contrib.admin.widgets import RelatedFieldWidgetWrapper
 from django.urls import path, reverse
 from django.shortcuts import redirect, render
-from django.http import JsonResponse
+from django.http import JsonResponse, HttpResponseBase
 from django.template.response import TemplateResponse
 from django.views.decorators.csrf import csrf_exempt
 from django.core.exceptions import ValidationError
@@ -17,6 +17,7 @@ from import_export import resources, fields
 from import_export.admin import ImportExportModelAdmin
 from import_export.widgets import ForeignKeyWidget
 from django.contrib.auth.models import Group
+from django.templatetags.static import static
 from django.utils.html import format_html
 import json
 import uuid
@@ -37,7 +38,7 @@ from .models import (
     Reference,
     OdooProfile,
     FediverseProfile,
-    EmailInbox,
+    EmailInbox as CoreEmailInbox,
     Package,
     PackageRelease,
     ReleaseManager,
@@ -49,6 +50,19 @@ from .user_data import UserDatumAdminMixin
 admin.site.unregister(Group)
 
 
+class SaveBeforeChangeAction(DjangoObjectActions):
+    def response_change(self, request, obj):
+        action = request.POST.get("_action")
+        if action:
+            allowed = self.get_change_actions(request, str(obj.pk), None)
+            if action in allowed and hasattr(self, action):
+                response = getattr(self, action)(request, obj)
+                if isinstance(response, HttpResponseBase):
+                    return response
+                return redirect(request.path)
+        return super().response_change(request, obj)
+
+
 @admin.register(Reference)
 class ReferenceAdmin(admin.ModelAdmin):
     list_display = (
@@ -131,7 +145,7 @@ class ReleaseManagerAdmin(admin.ModelAdmin):
 
 
 @admin.register(Package)
-class PackageAdmin(DjangoObjectActions, admin.ModelAdmin):
+class PackageAdmin(SaveBeforeChangeAction, admin.ModelAdmin):
     list_display = ("name", "description", "homepage_url", "release_manager")
     actions = ["prepare_next_release"]
     change_actions = ["prepare_next_release_action"]
@@ -366,6 +380,14 @@ class FediverseProfileAdmin(admin.ModelAdmin):
                 self.message_user(request, f"{profile}: {exc}", level=messages.ERROR)
 
 
+class EmailInbox(CoreEmailInbox):
+    class Meta:
+        proxy = True
+        app_label = "post_office"
+        verbose_name = CoreEmailInbox._meta.verbose_name
+        verbose_name_plural = CoreEmailInbox._meta.verbose_name_plural
+
+
 class EmailInboxAdminForm(forms.ModelForm):
     """Admin form for :class:`core.models.EmailInbox` with hidden password."""
 
@@ -376,7 +398,7 @@ class EmailInboxAdminForm(forms.ModelForm):
     )
 
     class Meta:
-        model = EmailInbox
+        model = CoreEmailInbox
         fields = "__all__"
 
     def __init__(self, *args, **kwargs):
@@ -431,6 +453,10 @@ class EmailInboxAdmin(admin.ModelAdmin):
         ),
     )
 
+    def save_model(self, request, obj, form, change):
+        super().save_model(request, obj, form, change)
+        obj.__class__ = EmailInbox
+
     @admin.action(description="Test selected inboxes")
     def test_connection(self, request, queryset):
         for inbox in queryset:
@@ -755,7 +781,7 @@ class RFIDAdmin(ImportExportModelAdmin):
 
 
 @admin.register(PackageRelease)
-class PackageReleaseAdmin(DjangoObjectActions, admin.ModelAdmin):
+class PackageReleaseAdmin(SaveBeforeChangeAction, admin.ModelAdmin):
     list_display = (
         "version",
         "package",
@@ -808,18 +834,18 @@ class PackageReleaseAdmin(DjangoObjectActions, admin.ModelAdmin):
     publish_release_action.short_description = "Publish this release"
 
     @staticmethod
-    def _checkbox(value: bool) -> str:
-        return format_html(
-            '<input type="checkbox"{} disabled>', " checked" if value else ""
-        )
+    def _boolean_icon(value: bool) -> str:
+        icon = static("admin/img/icon-yes.svg" if value else "admin/img/icon-no.svg")
+        alt = "True" if value else "False"
+        return format_html('<img src="{}" alt="{}">', icon, alt)
 
     @admin.display(description="Published")
     def published_status(self, obj):
-        return self._checkbox(obj.is_published)
+        return self._boolean_icon(obj.is_published)
 
     @admin.display(description="Is current")
     def is_current(self, obj):
-        return self._checkbox(obj.is_current)
+        return self._boolean_icon(obj.is_current)
 
     def pr_link(self, obj):
         if obj.pr_url:

{arthexis-0.1.4 → arthexis-0.1.6}/core/models.py

@@ -10,7 +10,7 @@ from django.utils.translation import gettext_lazy as _
 from django.core.validators import RegexValidator
 from django.core.exceptions import ValidationError
 from django.apps import apps
-from django.db.models.signals import m2m_changed, post_delete
+from django.db.models.signals import m2m_changed, post_delete, post_save
 from django.dispatch import receiver
 from datetime import timedelta
 from django.contrib.contenttypes.models import ContentType
@@ -1257,8 +1257,9 @@ class PackageRelease(Entity):
         return self.revision[-6:] if self.revision else ""
 
 
-@receiver(post_delete, sender=PackageRelease)
-def _delete_release_fixture(sender, instance, **kwargs) -> None:
+@receiver([post_save, post_delete], sender=PackageRelease)
+def _update_release_fixture(sender, instance, **kwargs) -> None:
+    """Keep the release fixture in sync with the database."""
     PackageRelease.dump_fixture()
 
 # Ensure each RFID can only be linked to one energy account

{arthexis-0.1.4 → arthexis-0.1.6}/core/views.py

@@ -399,7 +399,10 @@ def release_progress(request, pk: int, action: str):
     identifier = f"{release.package.name}-{release.version}"
     if release.revision:
         identifier = f"{identifier}-{release.revision[:7]}"
-    log_name = ctx.get("log") or f"{identifier}.log"
+    log_name = f"{identifier}.log"
+    if ctx.get("log") != log_name:
+        ctx = {"step": 0, "log": log_name}
+        step_count = 0
     log_path = Path("logs") / log_name
     ctx.setdefault("log", log_name)
 

{arthexis-0.1.4 → arthexis-0.1.6}/nodes/admin.py

@@ -18,7 +18,7 @@ from .actions import NodeAction
 
 from .models import (
     Node,
-    EmailOutbox,
+    EmailOutbox as NodeEmailOutbox,
     NodeRole,
     ContentSample,
     NodeTask,
@@ -181,10 +181,22 @@ class NodeAdmin(admin.ModelAdmin):
         return redirect(reverse("admin:nodes_node_change", args=[node_id]))
 
 
+class EmailOutbox(NodeEmailOutbox):
+    class Meta:
+        proxy = True
+        app_label = "post_office"
+        verbose_name = NodeEmailOutbox._meta.verbose_name
+        verbose_name_plural = NodeEmailOutbox._meta.verbose_name_plural
+
+
 @admin.register(EmailOutbox)
 class EmailOutboxAdmin(admin.ModelAdmin):
     list_display = ("node", "host", "port", "username", "use_tls", "use_ssl")
 
+    def save_model(self, request, obj, form, change):
+        super().save_model(request, obj, form, change)
+        obj.__class__ = EmailOutbox
+
 
 class NodeRoleAdminForm(forms.ModelForm):
     nodes = forms.ModelMultipleChoiceField(

{arthexis-0.1.4 → arthexis-0.1.6}/nodes/tests.py

@@ -560,11 +560,6 @@ class NetMessageAdminTests(TransactionTestCase):
 class LastNetMessageViewTests(TestCase):
     def setUp(self):
         self.client = Client()
-        User = get_user_model()
-        self.user = User.objects.create_user(
-            username="lastmsg", password="pwd"
-        )
-        self.client.force_login(self.user)
         NodeRole.objects.get_or_create(name="Terminal")
 
     def test_returns_latest_message(self):

{arthexis-0.1.4 → arthexis-0.1.6}/nodes/views.py

@@ -252,7 +252,6 @@ def net_message(request):
     return JsonResponse({"status": "propagated", "complete": msg.complete})
 
 
-@api_login_required
 def last_net_message(request):
     """Return the most recent :class:`NetMessage`."""
 

{arthexis-0.1.4 → arthexis-0.1.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "arthexis"
-version = "0.1.4"
+version = "0.1.6"
 description = "Django-based MESH system"
 requires-python = ">=3.10"
 license = "MIT"

{arthexis-0.1.4 → arthexis-0.1.6}/tests/test_email_inbox_search_action.py

@@ -4,7 +4,7 @@ from unittest.mock import patch
 from django.contrib.admin.sites import AdminSite
 from django.test import TestCase, RequestFactory
 
-from core.admin import EmailInboxAdmin
+from core.admin import EmailInboxAdmin, EmailInbox as AdminEmailInbox
 from core.models import EmailInbox, User
 
 
@@ -93,11 +93,13 @@ class EmailInboxSearchTests(TestCase):
             use_ssl=True,
         )
         site = AdminSite()
-        ma = EmailInboxAdmin(EmailInbox, site)
+        ma = EmailInboxAdmin(AdminEmailInbox, site)
         factory = RequestFactory()
         request = factory.post("/", {"apply": "yes", "subject": "S"})
         request.user = admin
-        response = ma.search_inbox(request, EmailInbox.objects.filter(id=inbox.id))
+        response = ma.search_inbox(
+            request, AdminEmailInbox.objects.filter(id=inbox.id)
+        )
         self.assertEqual(response.status_code, 200)
         content = response.render().content.decode()
         self.assertIn("S", content)

{arthexis-0.1.4 → arthexis-0.1.6}/tests/test_package_release_admin_actions.py

@@ -28,12 +28,10 @@ class PackageReleaseAdminActionsTests(TestCase):
 
     def test_change_page_contains_publish_action(self):
         change_url = reverse("admin:core_packagerelease_change", args=[self.release.pk])
-        action_url = reverse(
-            "admin:core_packagerelease_actions",
-            args=[self.release.pk, "publish_release_action"],
-        )
         resp = self.client.get(change_url)
-        self.assertContains(resp, action_url)
+        self.assertContains(
+            resp, 'name="_action" value="publish_release_action"'
+        )
 
     def test_publish_action_redirects(self):
         url = reverse(
@@ -45,6 +43,21 @@ class PackageReleaseAdminActionsTests(TestCase):
             resp, reverse("release-progress", args=[self.release.pk, "publish"])
         )
 
+    def test_publish_action_saves_changes_before_execution(self):
+        change_url = reverse("admin:core_packagerelease_change", args=[self.release.pk])
+        data = {
+            "package": self.package.pk,
+            "release_manager": "",
+            "version": "1.0.1",
+            "_action": "publish_release_action",
+        }
+        resp = self.client.post(change_url, data)
+        self.assertRedirects(
+            resp, reverse("release-progress", args=[self.release.pk, "publish"])
+        )
+        self.release.refresh_from_db()
+        self.assertEqual(self.release.version, "1.0.1")
+
     def test_change_page_pypi_url_readonly(self):
         change_url = reverse("admin:core_packagerelease_change", args=[self.release.pk])
         resp = self.client.get(change_url)
@@ -77,7 +90,7 @@ class PackageReleaseAdminActionsTests(TestCase):
             resp = self.client.get(list_url)
         content = resp.content.decode()
         self.assertIn("Is current", content)
-        self.assertIn('<input type="checkbox" checked disabled>', content)
+        self.assertIn('icon-yes.svg', content)
 
     def test_release_revision_defaults_to_repo_revision(self):
         expected = revision_utils.get_revision()

arthexis-0.1.6/tests/test_post_office_admin_group.py

@@ -0,0 +1,33 @@
+from django.test import TestCase
+from django.contrib.auth import get_user_model
+from django.urls import reverse
+from django.contrib.admin.sites import site
+
+from core.admin import EmailInbox as PostOfficeEmailInbox
+from nodes.admin import EmailOutbox as PostOfficeEmailOutbox
+
+
+class PostOfficeAdminGroupTests(TestCase):
+    def setUp(self):
+        User = get_user_model()
+        self.admin = User.objects.create_superuser(
+            username="po-admin", password="pwd", email="admin@example.com"
+        )
+        self.client.force_login(self.admin)
+
+    def test_models_registered(self):
+        registry = site._registry
+        self.assertIn(PostOfficeEmailInbox, registry)
+        self.assertIn(PostOfficeEmailOutbox, registry)
+        self.assertEqual(
+            registry[PostOfficeEmailInbox].model._meta.app_label, "post_office"
+        )
+        self.assertEqual(
+            registry[PostOfficeEmailOutbox].model._meta.app_label, "post_office"
+        )
+
+    def test_admin_index_shows_post_office_group(self):
+        response = self.client.get(reverse("admin:index"))
+        self.assertContains(response, "Post Office")
+        self.assertContains(response, "Email Inboxes")
+        self.assertContains(response, "Email Outboxes")

{arthexis-0.1.4 → arthexis-0.1.6}/tests/test_release_fixture_cleanup.py

@@ -30,3 +30,8 @@ class ReleaseFixtureCleanupTests(TestCase):
         data = self.fixture_path.read_text()
         self.assertNotIn("1.0.0", data)
 
+    def test_create_updates_fixture(self):
+        PackageRelease.objects.create(package=self.package, version="2.0.0")
+        data = self.fixture_path.read_text()
+        self.assertIn("2.0.0", data)
+

{arthexis-0.1.4 → arthexis-0.1.6}/tests/test_release_progress.py

@@ -62,7 +62,12 @@ class ReleaseProgressTests(TestCase):
         self.assertContains(resp, "All steps completed")
         self.assertContains(
             resp,
-            '<a href="http://example.com/pr/1" target="_blank">http://example.com/pr/1</a>',
+            '<a href="http://example.com/pr/1" target="_blank" rel="noopener">http://example.com/pr/1</a>',
+            html=True,
+        )
+        self.assertContains(
+            resp,
+            '<a href="https://pypi.org/project/pkg/1.0.0/" target="_blank" rel="noopener">https://pypi.org/project/pkg/1.0.0/</a>',
             html=True,
         )
         release.refresh_from_db()
@@ -94,6 +99,11 @@ class ReleaseProgressTests(TestCase):
         self.assertEqual(resp.status_code, 200)
         self.assertContains(resp, "All steps completed")
         self.assertIsNone(resp.context["pr_url"])
+        self.assertContains(
+            resp,
+            '<a href="https://pypi.org/project/pkg/1.1.0/" target="_blank" rel="noopener">https://pypi.org/project/pkg/1.1.0/</a>',
+            html=True,
+        )
         release.refresh_from_db()
         self.assertTrue(release.is_published)
         pub.assert_called_once()
@@ -131,3 +141,25 @@ class ReleaseProgressTests(TestCase):
             resp = self.client.get(f"{url}?step=0")
         self.assertEqual(resp.context["error"], "Version 2.0.0 is older than existing 3.0.0")
         self.assertEqual(self.version_path.read_text().strip(), "3.0.0")
+
+    def test_session_resets_when_version_changes(self):
+        release = PackageRelease.objects.create(package=self.package, version="1.2.0")
+        url = reverse("release-progress", args=[release.pk, "publish"])
+        self.client.get(url)
+        with patch("core.views.release_utils.network_available", return_value=False):
+            self.client.get(f"{url}?step=0")
+
+        release.version = "1.3.0"
+        release.save()
+
+        resp = self.client.get(url)
+        expected_log = f"logs/pkg-1.3.0-{release.revision[:7]}.log"
+        self.assertEqual(resp.context["log_path"], expected_log)
+        self.assertEqual(resp.context["current_step"], 0)
+
+        with patch("core.views.release_utils.network_available", return_value=False):
+            self.client.get(f"{url}?step=0")
+
+        log_content = Path(expected_log).read_text()
+        self.assertIn("Checking if version 1.3.0 exists on PyPI", log_content)
+        self.assertNotIn("1.2.0", log_content)