Replace the CI workflow with the speed.python.org one

Also, fix flake8 warnings.

Author: StanFromIreland

.github/workflows/test.yml

@@ -0,0 +1,48 @@
+name: Tests
+
+on:
+  push:
+    branches: [speed.pypy.org]
+  pull_request:
+    branches: [speed.pypy.org]
+
+permissions:
+  contents: read
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
+      - uses: actions/setup-python@6d4fd22b20465ccc0f7354476588e8df7c9ca0e0 # v7.1.0
+        with:
+          python-version: "3.13"
+      - run: pip install -r requirements-dev.txt
+      - run: flake8 codespeed
+
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ["3.12", "3.13", "3.14"]
+        django: ["5.2", "6.1"]
+    name: Python ${{ matrix.python-version }} / Django ${{ matrix.django }}
+    env:
+      DJANGO_SETTINGS_MODULE: codespeed.tests.settings
+
+    steps:
+      - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
+      - uses: actions/setup-python@6d4fd22b20465ccc0f7354476588e8df7c9ca0e0 # v7.1.0
+        with:
+          python-version: ${{ matrix.python-version }}
+      # codespeed/tests/test_commits.py drives a real Mercurial repository, and
+      # hg is no longer preinstalled on the ubuntu-latest runner image. This
+      # step is the only difference from the speed.python.org workflow; drop
+      # it together with that test if the Mercurial backend stops being tested.
+      - run: |
+          sudo apt-get update
+          sudo apt-get install -y --no-install-recommends mercurial
+          hg --version
+      - run: pip install -r requirements.txt "django==${{ matrix.django }}.*"
+      - run: python manage.py test codespeed

.github/workflows/tests.yml

@@ -1,5 +1,5 @@
 # Codespeed
-[![Build Status](https://travis-ci.org/tobami/codespeed.png?branch=master)](https://travis-ci.org/tobami/codespeed)
+[![Tests](https://github.com/python/codespeed/actions/workflows/test.yml/badge.svg?branch=speed.pypy.org)](https://github.com/python/codespeed/actions/workflows/test.yml)
 [![PyPI version](https://img.shields.io/pypi/v/codespeed.svg)](https://pypi.python.org/pypi/codespeed)
 
 Codespeed is a web application to monitor and analyze the performance of your code.

README.md

@@ -67,7 +67,7 @@ def getlogs(endrev, startrev):
     p, stdout, stderr = execute_command(cmd, working_copy)
 
     if p.returncode != 0:
-        raise CommitLogError(f"'%s' in '%s' returned %s: %s" % (
+        raise CommitLogError("'%s' in '%s' returned %s: %s" % (
                              " ".join(cmd), working_copy, p.returncode, stderr))
     logs = []
     for log in filter(None, stdout.split('\x1e')):

codespeed/commits/git.py

@@ -68,7 +68,7 @@ def save_result(data, update_repo=True):
     if error:
         return res, True
     else:
-        assert(isinstance(res, Environment))
+        assert isinstance(res, Environment)
         env = res
 
     p, created = Project.objects.get_or_create(name=data["project"])

codespeed/results.py

@@ -4,18 +4,19 @@
 import tempfile
 import os
 
-from django.test import TestCase, override_settings
+from django.test import TestCase
 from django.urls import reverse
 
 from codespeed.models import (Project, Benchmark, Revision, Branch, Executable,
-                              Environment, Result, Report)
+                              Environment)
 
 hgdir = ''
 
 # hg aborts on commit when ui.username is unset, which is the case on a CI
 # runner with no ~/.hgrc, so supply one explicitly.
 HG_USER = ['-u', 'Codespeed Tests <tests@example.com>']
 
+
 def setUpModule():
     global hgdir
     hgdir = tempfile.TemporaryDirectory()
@@ -34,9 +35,11 @@ def setUpModule():
     subprocess.check_call(['hg', 'commit', '-m', "second commit"] + HG_USER,
                           cwd=hgdir.name)
 
+
 def tearDownModule():
     hgdir.cleanup()
 
+
 class TestMercurial(TestCase):
 
     def setUp(self):
@@ -65,5 +68,5 @@ def setUp(self):
 
     def test_hg(self):
         with self.settings(REPOSITORY_BASE_PATH=os.path.dirname(self.hgdir)):
-            response = self.client.get(reverse('displaylogs'), {'revisionid':1})
+            response = self.client.get(reverse('displaylogs'), {'revisionid': 1})
         assert response.status_code == 200, 'expected 200 got %d' % response.status_code

codespeed/tests/test_commits.py

@@ -287,17 +287,17 @@ def test_github_browsing_url(self):
         # It should work with https:// as well as git:// urls
         self.github_project.save()
         self.assertEqual(self.github_project.commit_browsing_url,
-                          'https://github.com/tobami/codespeed.git/'
-                          'commit/{commitid}')
+                         'https://github.com/tobami/codespeed.git/'
+                         'commit/{commitid}')
 
         self.github_project.repo_path = 'git://github.com/tobami/codespeed.git'
         self.github_project.save()
         self.assertEqual(self.github_project.commit_browsing_url,
-                          'https://github.com/tobami/codespeed.git/'
-                          'commit/{commitid}')
+                         'https://github.com/tobami/codespeed.git/'
+                         'commit/{commitid}')
 
         # If filled in, commit browsing url should not change
         self.github_project.commit_browsing_url = 'https://example.com/{commitid}'
         self.github_project.save()
         self.assertEqual(self.github_project.commit_browsing_url,
-                          'https://example.com/{commitid}')
+                         'https://example.com/{commitid}')

codespeed/tests/test_models.py

@@ -99,7 +99,7 @@ def test_bad_environment(self):
         response = self.client.post(self.path, self.data)
         self.assertEqual(response.status_code, 400)
         self.assertEqual(response.content.decode(),
-                          "Environment " + bad_name + " not found")
+                         "Environment " + bad_name + " not found")
         self.data['environment'] = 'Dual Core'
 
     def test_empty_argument(self):
@@ -118,7 +118,7 @@ def test_missing_argument(self):
         """Should respond 400 when a POST request is missing an argument"""
         for key in list(self.data):
             backup = self.data[key]
-            del(self.data[key])
+            del self.data[key]
             response = self.client.post(self.path, self.data)
             self.assertEqual(response.status_code, 400)
             self.assertEqual(
@@ -290,7 +290,7 @@ def test_add_correct_results(self):
         # Check that we get a success response
         self.assertEqual(response.status_code, 202)
         self.assertEqual(response.content.decode(),
-                          "All result data saved successfully")
+                         "All result data saved successfully")
 
         # Check that the data was correctly saved
         e = Environment.objects.get(name='bigdog')
@@ -365,7 +365,7 @@ def test_missing_argument(self):
         data = self.data[2]
         for key in list(data):
             backup = data[key]
-            del(data[key])
+            del data[key]
             response = self.client.post(self.path,
                                         {'json': json.dumps(self.data)})
             self.assertEqual(response.status_code, 400)

codespeed/tests/test_views.py

@@ -126,7 +126,8 @@ def getdefaultexecutable():
             settings.DEF_EXECUTABLES is not None):
         try:
             def_name = settings.DEF_EXECUTABLES[0]['name']
-            def_project = Project.objects.get(name=settings.DEF_EXECUTABLES[0]['project'])
+            def_project = Project.objects.get(
+                name=settings.DEF_EXECUTABLES[0]['project'])
             default = Executable.objects.get(name=def_name, project=def_project)
         except Executable.DoesNotExist:
             pass