releng: convert to pytest fixtures

Author: jelly

README.md

@@ -59,7 +59,6 @@ packages, you will probably want the following:
         ./manage.py loaddata main/fixtures/*.json
         ./manage.py loaddata devel/fixtures/*.json
         ./manage.py loaddata mirrors/fixtures/*.json
-        ./manage.py loaddata releng/fixtures/*.json
 
 6. Use the following commands to start a service instance
 

releng/fixtures/release.json

@@ -0,0 +1,18 @@
+from datetime import datetime
+
+import pytest
+
+from releng.models import Release
+
+
+VERSION = '1.0'
+KERNEL_VERSION = '4.18'
+
+
+@pytest.fixture
+def release(db):
+    release =  Release.objects.create(release_date=datetime.now(),
+                                      version=VERSION,
+                                      kernel_version=KERNEL_VERSION)
+    yield release
+    release.delete()

releng/tests/conftest.py

@@ -1,34 +1,28 @@
-from django.test import TestCase
+def test_feed(client, release):
+    response = client.get('/feeds/releases/')
+    assert response.status_code == 200
 
-from releng.models import Release
 
+def test_absolute_url(release):
+    assert release.version in release.get_absolute_url()
 
-class RelengTest(TestCase):
-    fixtures = ['releng/fixtures/release.json']
 
-    def setUp(self):
-        self.release = Release.objects.first()
+def test_iso_url(release):
+    url = release.iso_url()
+    ver = release.version
+    expected = 'iso/{}/archlinux-{}-x86_64.iso'.format(ver, ver)
+    assert url == expected
 
-    def test_feed(self):
-        response = self.client.get('/feeds/releases/')
-        self.assertEqual(response.status_code, 200)
 
-    def test_absolute_url(self):
-        self.assertIn(self.release.version, self.release.get_absolute_url())
+def test_info_html(release):
+    assert release.info in release.info_html()
 
-    def test_iso_url(self):
-        url = self.release.iso_url()
-        ver = self.release.version
-        expected = 'iso/{}/archlinux-{}-x86_64.iso'.format(ver, ver)
-        self.assertEqual(url, expected)
 
-    def test_info_html(self):
-        self.assertIn(self.release.info, self.release.info_html())
+def test_dir_path(release):
+    dir_path = u'iso/{}/'.format(release.version)
+    assert dir_path == release.dir_path()
 
-    def test_dir_path(self):
-        dir_path = u'iso/{}/'.format(self.release.version)
-        self.assertEqual(dir_path, self.release.dir_path())
 
-    def test_sitemap(self):
-        response = self.client.get('/sitemap-releases.xml')
-        self.assertEqual(response.status_code, 200)
+def test_sitemap(client, release):
+    response = client.get('/sitemap-releases.xml')
+    assert response.status_code == 200

releng/tests/test_models.py

@@ -1,40 +1,31 @@
-from django.test import TestCase
+def test_release_json(client, release):
+    version = release.version
+    response = client.get('/releng/releases/json/')
+    assert response.status_code == 200
 
-from releng.models import Release
+    data = response.json()
+    assert data['version'] == 1
+    release = data['releases'][0]
+    assert release['version'] == version
 
 
-class RelengViewTest(TestCase):
-    fixtures = ['releng/fixtures/release.json']
+def test_netboot_page(db, client):
+    response = client.get('/releng/netboot/')
+    assert response.status_code == 200
 
-    def setUp(self):
-        self.release = Release.objects.first()
 
-    def test_release_json(self):
-        version = self.release.version
-        response = self.client.get('/releng/releases/json/')
-        self.assertEqual(response.status_code, 200)
-        data = response.json()
+def test_netboot_config(db, client):
+    response = client.get('/releng/netboot/archlinux.ipxe')
+    assert response.status_code == 200
 
-        self.assertEqual(data['version'], 1)
-        release = data['releases'][0]
-        self.assertEqual(release['version'], version)
 
-    def test_netboot_page(self):
-        response = self.client.get('/releng/netboot/')
-        self.assertEqual(response.status_code, 200)
+def test_release_torrent_not_found(client, release):
+    # TODO: Add torrent data to release fixture
+    response = client.get('/releng/releases/{}/torrent/'.format(release.version))
+    assert response.status_code == 404
 
-    def test_netboot_config(self):
-        response = self.client.get('/releng/netboot/archlinux.ipxe')
-        self.assertEqual(response.status_code, 200)
 
-    def test_release_torrent_not_found(self):
-        # TODO: Add torrent data to release fixture
-        version = self.release.version
-        response = self.client.get('/releng/releases/{}/torrent/'.format(version))
-        self.assertEqual(response.status_code, 404)
-
-    def test_release_details(self):
-        version = self.release.version
-        response = self.client.get('/releng/releases/{}/'.format(version))
-        self.assertEqual(response.status_code, 200)
-        self.assertIn(version, response.content.decode('utf-8'))
+def test_release_details(client, release):
+    response = client.get('/releng/releases/{}/'.format(release.version))
+    assert response.status_code == 200
+    assert release.version in response.content.decode()