squad 1.93.2__tar.gz → 1.93.5__tar.gz

{squad-1.93.2 → squad-1.93.5}/CHANGELOG.md

@@ -1,3 +1,17 @@
+# 1.93.5
+
+This release adds a bugfix on BuildPlaceHolder.
+
+# 1.93.4
+
+This 1.93.4 release adds a hotfix for a bug added in 1.93.3.
+
+# 1.93.3
+
+This 1.93.3 release adds performance improvement when fetching test results. 
+
+Also it bumps SQUAD's Python version to 3.11.
+
 # 1.93.2
 
 This 1.93.2 release add minor bug fixes in the linux log parser plugin.
@@ -17,7 +31,7 @@ This 1.92 release avoids an extra retry when fetching errored builds/tests in Tu
 
 # 1.91
 
-This 1.91 release brings a few changes worth pointing out separately:
+his 1.91 release brings a few changes worth pointing out separately:
 
 * Tuxsuite backend: add support for changing environment name if "test_name" is present. Artificial boot tests
 when jobs fail or get canceled. Also checks if the testjob's project contains a token allowing authenticated

squad-1.93.5/Dockerfile

@@ -0,0 +1,67 @@
+FROM debian:bookworm
+
+ENV DEBIAN_FRONTEND=noninteractive
+COPY requirements.txt /requirements.txt
+
+RUN apt-get update -q=2 && \
+    apt-get -qq install --no-install-recommends iproute2 auto-apt-proxy >/dev/null && \
+    apt-get -qq install --no-install-recommends >/dev/null \
+        fail2ban \
+        gettext \
+        gcc \
+        git \
+        libdbd-pg-perl \
+        libldap2-dev \
+        libpq-dev \
+        libsasl2-dev \
+        libyaml-dev \
+        moreutils \
+        openssh-client \
+        postgresql-client \
+        python3-dev \
+        python3-pip \
+        python3-psycopg2 \
+        python3-venv \
+        unzip \
+        wget && \
+    python3 -m venv /venv/ && . /venv/bin/activate && \
+    pip install -r /requirements.txt && \
+    pip install \
+        build \
+        django-auth-ldap \
+        django-storages[s3] \
+        sentry-sdk[django] && \
+    pip install --no-dependencies squad-linaro-plugins
+
+# Make sure python virtual environment is always on
+ENV PATH="/venv/bin:$PATH"
+
+# Prepare the environment
+COPY . /squad-build/
+
+ENV SQUAD_STATIC_DIR=/app/static
+
+RUN cd /squad-build && ./scripts/git-build && \
+    pip3 install --no-dependencies ./dist/squad*.whl && \
+    mkdir -p /app/static && \
+    useradd -d /app squad && \
+    python3 -m squad.frontend && \
+    squad-admin collectstatic --noinput --verbosity 0 && \
+    chown -R squad:squad /app && \
+    cd /app
+
+# Clean environment
+RUN rm -rf /squad-build && \
+    apt-get remove -y \
+        gcc \
+        git \
+        libpq-dev \
+        libsasl2-dev \
+        libyaml-dev \
+        python3-dev \
+        libsasl2-dev && \
+    apt-get clean && \
+    apt-get autoremove -y && \
+    . /venv/bin/activate && pip uninstall -y build && pip cache purge
+
+USER squad

squad-1.93.5/PKG-INFO

@@ -0,0 +1,55 @@
+Metadata-Version: 2.2
+Name: squad
+Version: 1.93.5
+Summary: Software Quality Dashboard
+Home-page: https://github.com/Linaro/squad
+Author: Antonio Terceiro
+Author-email: antonio.terceiro@linaro.org
+License: GPLv3+
+Platform: any
+License-File: COPYING
+Requires-Dist: aiohttp
+Requires-Dist: celery
+Requires-Dist: cryptography
+Requires-Dist: coreapi
+Requires-Dist: django_crispy_forms==1.14.0
+Requires-Dist: Django<5,>=3
+Requires-Dist: django-allauth
+Requires-Dist: django-bootstrap3
+Requires-Dist: django-celery-results
+Requires-Dist: django-cors-headers
+Requires-Dist: django-debug-toolbar
+Requires-Dist: django-simple-history>3.0
+Requires-Dist: django_filter>=2.0
+Requires-Dist: djangorestframework>=3.9.2
+Requires-Dist: djangorestframework-filters>=1.0.0.dev0
+Requires-Dist: drf-extensions
+Requires-Dist: future
+Requires-Dist: gunicorn
+Requires-Dist: importlib-metadata>3
+Requires-Dist: Jinja2==3.0.3
+Requires-Dist: Markdown
+Requires-Dist: msgpack>=0.5.0
+Requires-Dist: psycopg2-binary
+Requires-Dist: python_dateutil
+Requires-Dist: PyYAML>=5.1
+Requires-Dist: PyJWT
+Requires-Dist: pyzmq
+Requires-Dist: requests
+Requires-Dist: setuptools
+Requires-Dist: sqlparse
+Requires-Dist: svgwrite
+Requires-Dist: whitenoise
+Provides-Extra: postgres
+Requires-Dist: psycopg2-binary; extra == "postgres"
+Dynamic: author
+Dynamic: author-email
+Dynamic: description
+Dynamic: home-page
+Dynamic: license
+Dynamic: platform
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: summary
+
+Software Quality Dashboard

{squad-1.93.2 → squad-1.93.5}/dev-docker

@@ -25,21 +25,16 @@ fi
 dockerfile="$datadir"/../Dockerfile.dev
 (
   sed -e '1,/# Prepare the environment/!d' "$basedir"/Dockerfile
-  echo 'RUN apt-get update -q2 && \
+  echo 'COPY requirements-dev.txt /app/requirements-dev.txt'
+  echo 'RUN sed -i "1d" /app/requirements-dev.txt && \
+            pip install -r /app/requirements-dev.txt && \
             apt-get install -q2 \
-	        tmux \
-		flake8 \
-		python3-django-extensions \
-		python3-pytest \
-		python3-pytest-django \
-		python3-requests-mock \
-		ipython3 \
-		rabbitmq-server \
-		chromium \
-		nodejs \
-		ruby-foreman \
-		procps && \'
-  echo "    pip3 install --no-dependencies django-bootstrap3"
+                tmux \
+                rabbitmq-server \
+                chromium \
+                nodejs \
+                ruby-foreman \
+                procps'
 
   if [ ! -z "${has_input}" ]
   then

squad-1.93.2/squad.egg-info/requires.txt → squad-1.93.5/requirements.txt

@@ -3,7 +3,7 @@ celery
 cryptography
 coreapi
 django_crispy_forms==1.14.0
-Django>=3
+Django<5,>=3
 django-allauth
 django-bootstrap3
 django-celery-results
@@ -20,6 +20,7 @@ importlib-metadata>3
 Jinja2==3.0.3
 Markdown
 msgpack>=0.5.0
+psycopg2-binary
 python_dateutil
 PyYAML>=5.1
 PyJWT
@@ -29,6 +30,3 @@ setuptools
 sqlparse
 svgwrite
 whitenoise
-
-[postgres]
-psycopg2-binary

{squad-1.93.2 → squad-1.93.5}/scripts/build

@@ -11,7 +11,7 @@ local_settings=$(ls -1 squad/local_settings.py 2>/dev/null || true)
 if [ -n "$local_settings" ]; then
     mv "$local_settings" "$local_settings".off
 fi
-python3 setup.py sdist bdist_wheel
+python3 -m build
 
 # test
 tar=$(mktemp tar.XXXXXXXXX)

{squad-1.93.2 → squad-1.93.5}/scripts/git-build

@@ -23,6 +23,6 @@ for po in $(find squad/*/locale -name \*.po); do
     chronic msgfmt -o ${po%.po}.mo ${po}
 done
 
-SQUAD_RELEASE=1 chronic python3 setup.py bdist_wheel
+SQUAD_RELEASE=1 chronic python3 -m build
 
 echo "$v" > dist/LASTBUILD

{squad-1.93.2 → squad-1.93.5}/squad/ci/backend/fake.py

@@ -42,7 +42,8 @@ class Backend(object):
         tests = {test: (random.randint(1, 10) <= 8) and "pass" or 'fail' for test in TESTS}
         metrics = {metric: random.random() for metric in METRICS}
         logs = "a fake log file\ndate: " + time.strftime('%c') + "\n"
-        return (status, completed, metadata, tests, metrics, logs)
+        attachments = {}
+        return (status, completed, metadata, tests, metrics, logs, attachments)
 
     def listen(self):
         max_id = 0

{squad-1.93.2 → squad-1.93.5}/squad/ci/backend/lava.py

@@ -718,7 +718,8 @@ class Backend(BaseBackend):
                     # automatically resubmit in some cases
                     if error_type in ['Infrastructure', 'Job', 'Test']:
                         self.__resubmit_job__(test_job, metadata)
-        return (data[status_key], completed, job_metadata, results, metrics, self.__parse_log__(raw_logs))
+        attachments = {}
+        return (data[status_key], completed, job_metadata, results, metrics, self.__parse_log__(raw_logs), attachments)
 
     def __resubmit_job__(self, test_job, metadata):
         infra_messages_re_list = []

{squad-1.93.2 → squad-1.93.5}/squad/ci/backend/null.py

@@ -70,8 +70,9 @@ class Backend:
         proper id.
 
         The return value must be a tuple (status, completed, metadata, tests,
-        metrics, logs), where status and logs are strings, metadata, tests and
-        metrics are dictionaries, and completed is a boolean.
+        metrics, logs, attachments), where status and logs are strings,
+        metadata, tests and metrics are dictionaries, and completed is a
+        boolean.
 
         On errors, implementations can raise two classes of exceptions:
             * squad.ci.exceptions.FetchIssue, when there is an unrecoverable

{squad-1.93.2 → squad-1.93.5}/squad/ci/backend/tuxsuite.py

@@ -17,6 +17,8 @@ from cryptography.hazmat.primitives import (
     serialization,
 )
 
+from django.core.files.base import ContentFile
+
 from squad.ci.backend.null import Backend as BaseBackend
 from squad.ci.exceptions import FetchIssue, TemporaryFetchIssue
 from squad.ci.models import TestJob
@@ -284,7 +286,14 @@ class Backend(BaseBackend):
             except KeyError:
                 raise FetchIssue('Missing duration from build results')
 
-        return status, completed, metadata, tests, metrics, logs
+        attachment_list = ["config", "tuxmake_reproducer.sh", "tux_plan.yaml"]
+        attachments = {}
+        for name in attachment_list:
+            response = self.fetch_url(results['download_url'], name)
+            if response.ok:
+                attachments[name] = ContentFile(response.content)
+
+        return status, completed, metadata, tests, metrics, logs, attachments
 
     def parse_oebuild_results(self, test_job, job_url, results, settings):
         required_keys = ['download_url', 'result']
@@ -301,6 +310,7 @@ class Backend(BaseBackend):
             metadata['sources'] = sources
 
         # Create tests and metrics
+        attachments = {}
         tests = {}
         metrics = {}
         completed = True
@@ -308,7 +318,7 @@ class Backend(BaseBackend):
         tests['build/build'] = 'pass' if results['result'] == 'pass' else 'fail'
         logs = self.fetch_url(results['download_url'], 'build.log').text
 
-        return status, completed, metadata, tests, metrics, logs
+        return status, completed, metadata, tests, metrics, logs, attachments
 
     def update_metadata_from_file(self, results, metadata):
         if "download_url" in results:
@@ -327,6 +337,7 @@ class Backend(BaseBackend):
         tests = {}
         metrics = {}
         logs = ''
+        attachments = {}
 
         # Pick up some metadata from results
         metadata_keys = settings.get('TEST_METADATA_KEYS', [])
@@ -361,7 +372,7 @@ class Backend(BaseBackend):
 
             self.add_skip_boot_test(tests, metadata)
 
-            return status, completed, metadata, tests, metrics, logs
+            return status, completed, metadata, tests, metrics, logs, attachments
 
         # Fetch results even if the job fails, but has results
         if results['result'] == 'fail':
@@ -370,12 +381,12 @@ class Backend(BaseBackend):
         elif results['result'] == 'error':
             test_job.failure = 'tuxsuite infrastructure error'
             self.add_skip_boot_test(tests, metadata)
-            return 'Incomplete', completed, metadata, tests, metrics, logs
+            return 'Incomplete', completed, metadata, tests, metrics, logs, attachments
 
         elif results['result'] == 'canceled':
             test_job.failure = 'tuxsuite job canceled'
             self.add_skip_boot_test(tests, metadata)
-            return 'Canceled', completed, metadata, tests, metrics, logs
+            return 'Canceled', completed, metadata, tests, metrics, logs, attachments
 
         # If boot result is unkown, a retry is needed, otherwise, it either passed or failed
         if 'unknown' == results['results']['boot']:
@@ -384,6 +395,13 @@ class Backend(BaseBackend):
         # Retrieve TuxRun log
         logs = self.fetch_url(job_url + '/', 'logs?format=txt').text
 
+        attachment_list = ["reproducer", "tux_plan.yaml"]
+        attachments = {}
+        for name in attachment_list:
+            response = self.fetch_url(job_url + '/', name)
+            if response.ok:
+                attachments[name] = ContentFile(response.content)
+
         # Follow up the chain and retrieve build name
         self.set_build_name(test_job, job_url, results, metadata, settings)
 
@@ -407,7 +425,7 @@ class Backend(BaseBackend):
                     # test_log = self.get_test_log(log_dict, test)
                     tests[test_name] = result
 
-        return status, completed, metadata, tests, metrics, logs
+        return status, completed, metadata, tests, metrics, logs, attachments
 
     def fetch(self, test_job):
         url = self.job_url(test_job)

{squad-1.93.2 → squad-1.93.5}/squad/ci/models.py

@@ -109,7 +109,7 @@ class Backend(models.Model):
             test_job.fetched_at = timezone.now()
             test_job.save()
 
-        status, completed, metadata, tests, metrics, logs = results
+        status, completed, metadata, tests, metrics, logs, attachments = results
 
         if not completed:
             tests = {}
@@ -136,6 +136,7 @@ class Backend(models.Model):
                 metrics_file=json.dumps(metrics),
                 log_file=logs,
                 completed=completed,
+                attachments=attachments,
             )
             test_job.testrun = testrun
         except (DuplicatedTestJob, InvalidMetadata) as exception:

{squad-1.93.2 → squad-1.93.5}/squad/core/tasks/__init__.py

@@ -36,7 +36,7 @@ from squad.core.data import JSONTestDataParser, JSONMetricDataParser
 from squad.core.statistics import geomean
 from squad.core.notification import Notification
 from squad.core.plugins import apply_plugins
-from squad.core.utils import join_name
+from squad.core.utils import join_name, split_dict
 from rest_framework import status
 from jinja2 import TemplateSyntaxError
 from . import exceptions
@@ -46,7 +46,6 @@ from .notification import maybe_notify_project_status
 from .notification import notify_patch_build_created
 from .notification import notify_delayed_report_callback, notify_delayed_report_email
 
-
 test_parser = JSONTestDataParser
 metric_parser = JSONMetricDataParser
 
@@ -213,50 +212,131 @@ def get_suite(test_run, suite_name):
 
 class ParseTestRunData(object):
 
+    @staticmethod
+    def create_suites(project, suites_slugs):
+        # Insert/Get all suites at once, not likely to be a big list
+        SuiteMetadata.objects.bulk_create([
+            SuiteMetadata(
+                kind='suite',
+                suite=suite,
+                name='-',
+            ) for suite in suites_slugs
+        ], ignore_conflicts=True)
+
+        suites_metadata_ids = {
+            m.suite: m.id for m in SuiteMetadata.objects.filter(kind='suite', name='-', suite__in=suites_slugs)
+        }
+
+        Suite.objects.bulk_create([
+            Suite(
+                project=project,
+                slug=suite,
+                metadata_id=suites_metadata_ids[suite],
+            ) for suite in suites_slugs
+        ], ignore_conflicts=True)
+
+        return {s.slug: s.id for s in project.suites.filter(slug__in=suites_slugs)}
+
+    @staticmethod
+    def create_tests_batch(testrun, tests_batch, issues_by_full_name, suites_ids):
+
+        # Create SuiteMetadata in bulk
+        SuiteMetadata.objects.bulk_create([
+            SuiteMetadata(
+                suite=test['suite_slug'],
+                name=test['test_name'],
+                kind='test',
+            ) for test in tests_batch.values()
+        ], ignore_conflicts=True)
+
+        # We need the extra SELECT due to `ignore_conflicts=True` above
+        metadata_ids = {}
+        metadata_names = {}
+        metadatas = SuiteMetadata.objects.filter(
+            kind='test',
+            name__in=[
+                t['test_name'] for t in tests_batch.values()
+            ]
+        ).all()
+
+        for metadata in metadatas:
+            full_name = join_name(metadata.suite, metadata.name)
+            metadata_ids[full_name] = metadata.id
+            metadata_names[metadata.id] = full_name
+
+        # Create tests in batch
+        created_tests = Test.objects.bulk_create([
+            Test(
+                test_run=testrun,
+                suite_id=suites_ids[test['suite_slug']],
+                metadata_id=metadata_ids[full_name],
+                result=test['result'],
+                log=test['log'],
+                has_known_issues=test['has_known_issues'],
+                build_id=testrun.build_id,
+                environment_id=testrun.environment_id,
+            ) for full_name, test in tests_batch.items()
+        ])
+
+        # Attach known issues, if any
+        for test in created_tests:
+            test_full_name = metadata_names[test.metadata_id]
+            test_full_name = join_name(metadata.suite, metadata.name)
+            issues = issues_by_full_name.get(test_full_name, [])
+            if len(issues) > 0:
+                test.known_issues.add(*issues)
+
     @staticmethod
     def __call__(test_run):
         if test_run.data_processed:
             return
 
-        issues = {}
-        for issue in KnownIssue.active_by_environment(test_run.environment):
-            issues.setdefault(issue.test_name, [])
-            issues[issue.test_name].append(issue)
+        project = test_run.build.project
 
         # Issues' test_name should be interpreted as regexes
         # so compile them prior to matching against test names
         # The * character should be replaced by .*?, which is regex for "everything"
+        issues = defaultdict(list)
         issues_regex = {}
+        for issue in KnownIssue.active_by_environment(test_run.environment):
+            issues[issue.test_name].append(issue)
+
         for test_name_regex in issues.keys():
             pattern = re.escape(test_name_regex).replace('\\*', '.*?')
             regex = re.compile(pattern)
             issues_regex[regex] = issues[test_name_regex]
 
+        tests_details = {}
+        suites_slugs = set()
+        issues_by_full_name = {}
         for test in test_parser()(test_run.tests_file):
             # TODO: remove check below when test_name size changes in the schema
             if len(test['test_name']) > 256:
                 continue
-            suite = get_suite(
-                test_run,
-                test['group_name']
-            )
-            metadata, _ = SuiteMetadata.objects.get_or_create(suite=suite.slug, name=test['test_name'], kind='test')
-            full_name = join_name(suite.slug, test['test_name'])
 
-            test_issues = list(itertools.chain(*[issue for regex, issue in issues_regex.items() if regex.match(full_name)]))
+            full_name = join_name(test['group_name'], test['test_name'])
+            test_issues = list(itertools.chain(*[
+                issue for regex, issue in issues_regex.items()
+                if regex.match(full_name)
+            ]))
 
-            test_obj = Test.objects.create(
-                test_run=test_run,
-                suite=suite,
-                metadata=metadata,
-                result=test['pass'],
-                log=test['log'],
-                has_known_issues=bool(test_issues),
-                build=test_run.build,
-                environment=test_run.environment,
-            )
-            for issue in test_issues:
-                test_obj.known_issues.add(issue)
+            suites_slugs.add(test['group_name'])
+            tests_details[full_name] = {
+                'suite_slug': test['group_name'],
+                'test_name': test['test_name'],
+                'result': test['pass'],
+                'log': test['log'],
+                'has_known_issues': bool(test_issues),
+            }
+
+            issues_by_full_name[full_name] = test_issues
+
+        suites_ids = ParseTestRunData.create_suites(project, suites_slugs)
+
+        # Insert tests in batches
+        batch_size = 1000
+        for batch in split_dict(tests_details, batch_size):
+            ParseTestRunData.create_tests_batch(test_run, batch, issues_by_full_name, suites_ids)
 
         for metric in metric_parser()(test_run.metrics_file):
             # TODO: remove check below when test_name size changes in the schema
@@ -534,7 +614,10 @@ def remove_delayed_reports():
 @transaction.atomic
 def cleanup_build(build_id):
     build = Build.objects.get(pk=build_id)
-    BuildPlaceholder.objects.create(project=build.project, version=build.version)
+    build_placeholder, created = BuildPlaceholder.objects.get_or_create(project=build.project, version=build.version)
+    if not created:  # Update build deletion date when placeholder already exists
+        build_placeholder.build_deleted_at = timezone.now()
+        build_placeholder.save()
     build.delete()
 
 

{squad-1.93.2 → squad-1.93.5}/squad/settings.py

@@ -263,7 +263,7 @@ USE_TZ = True
 # http://whitenoise.evans.io/en/stable/django.html
 STATIC_URL = '/static/'
 STATIC_ROOT = os.getenv('SQUAD_STATIC_DIR', os.path.join(DATA_DIR, 'static'))
-STATICFILES_STORAGE = 'whitenoise.storage.CompressedManifestStaticFilesStorage'
+STATICFILES_STORAGE = 'whitenoise.storage.CompressedStaticFilesStorage'
 
 # Squad default storage directory
 MEDIA_ROOT = os.getenv('SQUAD_STORAGE_DIR', 'storage')

squad-1.93.5/squad/version.py

@@ -0,0 +1 @@
+__version__ = '1.93.5'

squad-1.93.5/squad.egg-info/PKG-INFO

@@ -0,0 +1,55 @@
+Metadata-Version: 2.2
+Name: squad
+Version: 1.93.5
+Summary: Software Quality Dashboard
+Home-page: https://github.com/Linaro/squad
+Author: Antonio Terceiro
+Author-email: antonio.terceiro@linaro.org
+License: GPLv3+
+Platform: any
+License-File: COPYING
+Requires-Dist: aiohttp
+Requires-Dist: celery
+Requires-Dist: cryptography
+Requires-Dist: coreapi
+Requires-Dist: django_crispy_forms==1.14.0
+Requires-Dist: Django<5,>=3
+Requires-Dist: django-allauth
+Requires-Dist: django-bootstrap3
+Requires-Dist: django-celery-results
+Requires-Dist: django-cors-headers
+Requires-Dist: django-debug-toolbar
+Requires-Dist: django-simple-history>3.0
+Requires-Dist: django_filter>=2.0
+Requires-Dist: djangorestframework>=3.9.2
+Requires-Dist: djangorestframework-filters>=1.0.0.dev0
+Requires-Dist: drf-extensions
+Requires-Dist: future
+Requires-Dist: gunicorn
+Requires-Dist: importlib-metadata>3
+Requires-Dist: Jinja2==3.0.3
+Requires-Dist: Markdown
+Requires-Dist: msgpack>=0.5.0
+Requires-Dist: psycopg2-binary
+Requires-Dist: python_dateutil
+Requires-Dist: PyYAML>=5.1
+Requires-Dist: PyJWT
+Requires-Dist: pyzmq
+Requires-Dist: requests
+Requires-Dist: setuptools
+Requires-Dist: sqlparse
+Requires-Dist: svgwrite
+Requires-Dist: whitenoise
+Provides-Extra: postgres
+Requires-Dist: psycopg2-binary; extra == "postgres"
+Dynamic: author
+Dynamic: author-email
+Dynamic: description
+Dynamic: home-page
+Dynamic: license
+Dynamic: platform
+Dynamic: provides-extra
+Dynamic: requires-dist
+Dynamic: summary
+
+Software Quality Dashboard

{squad-1.93.2 → squad-1.93.5}/squad.egg-info/entry_points.txt

@@ -4,4 +4,3 @@ squad-admin = squad.manage:main
 squad-listener = squad.run.listener:main
 squad-scheduler = squad.run.scheduler:main
 squad-worker = squad.run.worker:main
-

squad-1.93.2/requirements.txt → squad-1.93.5/squad.egg-info/requires.txt

@@ -3,7 +3,7 @@ celery
 cryptography
 coreapi
 django_crispy_forms==1.14.0
-Django>=3
+Django<5,>=3
 django-allauth
 django-bootstrap3
 django-celery-results
@@ -20,6 +20,7 @@ importlib-metadata>3
 Jinja2==3.0.3
 Markdown
 msgpack>=0.5.0
+psycopg2-binary
 python_dateutil
 PyYAML>=5.1
 PyJWT
@@ -29,3 +30,6 @@ setuptools
 sqlparse
 svgwrite
 whitenoise
+
+[postgres]
+psycopg2-binary

{squad-1.93.2 → squad-1.93.5}/test/ci/backend/test_fake.py

@@ -48,7 +48,7 @@ class FakeBackendTest(TestCase):
     def test_fetch(self):
         job = TestJob.objects.create(backend=self.backend, target=self.project)
         impl = FakeBackend(self.backend)
-        (status, completed, metadata, tests, metrics, logs) = impl.fetch(job)
+        (status, completed, metadata, tests, metrics, logs, attachments) = impl.fetch(job)
         self.assertIsInstance(status, str)
         self.assertIsInstance(completed, bool)
         self.assertIsInstance(metadata, dict)