Differential D8313

Move flask test server to session scope
ClosedPublic
Actions

Authored by jayeshv on Aug 25 2022, 11:13 AM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDGQL5cf1ac26d9f7: Move flask test server to session scope

Summary

Moving the flask test server to pytest session scope improves the time to
run functional tests substantially. It is safe to do this since GraphQL
deals only with the archive data.

Diff Detail

Repository

rDGQL GraphQL API

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jayeshv created this revision.Aug 25 2022, 11:13 AM

Herald added a reviewer: Reviewers. · View Herald TranscriptAug 25 2022, 11:13 AM

Build is green

Patch application report for D8313 (id=30012)

Rebasing onto 6fb1913ff9...

Current branch diff-target is up to date.

Changes applied before test

commit 5cf1ac26d9f75983829e0c2395d71366a8bfdac4
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Thu Aug 25 11:04:43 2022 +0200

    Move flask test server to session scope
    
    Moving the flask test server to pytest session scope improves the time to
    run functional tests substantially. It is safe to do this since GraphQL
    deals only with the archive data.

See https://jenkins.softwareheritage.org/job/DGQL/job/tests-on-diff/125/ for more details.

Harbormaster completed remote builds in B31061: Diff 30012.Aug 25 2022, 11:15 AM

jayeshv requested review of this revision.Aug 25 2022, 11:15 AM