Add tests

feedvault/migrations/0002_alter_feed_status.py

@@ -0,0 +1,18 @@
+# Generated by Django 5.0.3 on 2024-03-15 13:21
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('feedvault', '0001_initial'),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name='feed',
+            name='status',
+            field=models.IntegerField(null=True),
+        ),
+    ]

feedvault/models.py

@@ -150,7 +150,7 @@ class Feed(models.Model):
     href = models.TextField(blank=True)
     modified = models.DateTimeField(null=True, blank=True)
     namespaces = JSONField(null=True, blank=True)
-    status = models.IntegerField()
+    status = models.IntegerField(null=True)
     version = models.CharField(max_length=255, blank=True)
 
     # Feed data

feedvault/tests.py

@@ -1 +1,175 @@
-# Create your tests here.
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+from django.contrib.auth.models import User
+from django.test import TestCase
+from django.urls import reverse
+
+from feedvault.models import Domain, Feed
+
+if TYPE_CHECKING:
+    from django.http import HttpResponse
+
+
+class TestIndexPage(TestCase):
+    def test_index_page(self) -> None:
+        """Test if the index page is accessible."""
+        response: HttpResponse = self.client.get(reverse("index"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+        response: HttpResponse = self.client.get("/")
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestFeedPage(TestCase):
+    def setUp(self) -> None:
+        """Create a test feed."""
+        self.domain: Domain = Domain.objects.create(
+            name="feedvault",
+            url="feedvault.se",
+        )
+
+        self.user: User = User.objects.create_user(
+            username="testuser",
+            email="hello@feedvault.se",
+            password="testpassword",  # noqa: S106
+        )
+        self.feed: Feed = Feed.objects.create(
+            user=self.user,
+            bozo=False,
+            feed_url="https://feedvault.se/feed.xml",
+            domain=self.domain,
+        )
+
+    def test_feed_page(self) -> None:
+        """Test if the feed page is accessible."""
+        response: HttpResponse = self.client.get(reverse("feed", kwargs={"feed_id": 1}))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+    def test_feed_page_not_found(self) -> None:
+        """Test if the feed page is accessible."""
+        response: HttpResponse = self.client.get(reverse("feed", kwargs={"feed_id": 2}))
+        assert response.status_code == 404, f"Expected 404, got {response.status_code}"
+
+
+class TestFeedsPage(TestCase):
+    def test_feeds_page(self) -> None:
+        """Test if the feeds page is accessible."""
+        response: HttpResponse = self.client.get(reverse("feeds"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestAddPage(TestCase):
+    def test_add_page(self) -> None:
+        """Test if the add page is accessible."""
+        response: HttpResponse = self.client.get(reverse("add"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestUploadPage(TestCase):
+    def test_upload_page(self) -> None:
+        """Test if the upload page is accessible."""
+        response: HttpResponse = self.client.get(reverse("upload"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestRobotsPage(TestCase):
+    def test_robots_page(self) -> None:
+        """Test if the robots page is accessible."""
+        response: HttpResponse = self.client.get(reverse("robots"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+    def test_robots_page_content(self) -> None:
+        """Test if the robots page contains the expected content."""
+        response: HttpResponse = self.client.get(reverse("robots"))
+        assert (
+            response.content == b"User-agent: *\nDisallow: /add\nDisallow: /upload\nDisallow: /accounts/"
+        ), f"Expected b'User-agent: *\nDisallow: /add\nDisallow: /upload\nDisallow: /accounts/', got {response.content}"
+
+
+class TestDomains(TestCase):
+    def test_domains_page(self) -> None:
+        """Test if the domains page is accessible."""
+        response: HttpResponse = self.client.get(reverse("domains"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestAPI(TestCase):
+    def test_api_page(self) -> None:
+        """Test if the API page is accessible."""
+        response: HttpResponse = self.client.get(reverse("api"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestAPIFeeds(TestCase):
+    def test_api_feeds_page(self) -> None:
+        """Test if the API feeds page is accessible."""
+        response: HttpResponse = self.client.get(reverse("api_feeds"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestAPIFeed(TestCase):
+    def setUp(self) -> None:
+        """Create a test feed."""
+        self.domain: Domain = Domain.objects.create(
+            name="feedvault",
+            url="feedvault.se",
+        )
+
+        self.user: User = User.objects.create_user(
+            username="testuser",
+            email="hello@feedvault.se",
+            password="testpassword",  # noqa: S106
+        )
+
+        self.feed: Feed = Feed.objects.create(
+            user=self.user,
+            bozo=False,
+            feed_url="https://feedvault.se/feed.xml",
+            domain=self.domain,
+        )
+
+    def test_api_feed_page(self) -> None:
+        """Test if the API feed page is accessible."""
+        response: HttpResponse = self.client.get(reverse("api_feeds_id", kwargs={"feed_id": 1}))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+    def test_api_feed_page_not_found(self) -> None:
+        """Test if the API feed page is accessible."""
+        response: HttpResponse = self.client.get(reverse("api_feeds_id", kwargs={"feed_id": 2}))
+        assert response.status_code == 404, f"Expected 404, got {response.status_code}"
+
+
+class TestAccount(TestCase):
+    def test_login_page(self) -> None:
+        """Test if the login page is accessible."""
+        response: HttpResponse = self.client.get(reverse("login"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+    def test_register_page(self) -> None:
+        """Test if the register page is accessible."""
+        response: HttpResponse = self.client.get(reverse("register"))
+        assert response.status_code == 200, f"Expected 200, got {response.status_code}"
+
+
+class TestLogoutPage(TestCase):
+    def setUp(self) -> None:
+        """Create a test user."""
+        self.user: User = User.objects.create_user(
+            username="testuser",
+            email="hello@feedvault.se",
+            password="testpassword",  # noqa: S106
+        )
+
+        self.client.force_login(user=self.user)
+
+    def test_logout_page(self) -> None:
+        """Test if the logout page is accessible."""
+        response: HttpResponse = self.client.post(reverse("logout"))
+        assert response.status_code == 302, f"Expected 300, got {response.status_code}"
+
+        # Check if the user is logged out
+        response: HttpResponse = self.client.get(reverse("index"))
+        assert response.status_code == 200
+        assert "testuser" not in response.content.decode("utf-8")

feedvault/views.py

@@ -269,7 +269,7 @@ class RobotsView(View):
     def get(self, request: HttpRequest) -> HttpResponse:  # noqa: ARG002
         """Load the robots.txt file."""
         return HttpResponse(
-            content="""User-agent: *\nDisallow: /add\nDisallow: /upload\nDisallow: /accounts/""",
+            content="User-agent: *\nDisallow: /add\nDisallow: /upload\nDisallow: /accounts/",
             content_type="text/plain",
         )