Page MenuHomeSoftware Heritage

Add functional tests for release target
ClosedPublic

Authored by jayeshv on Aug 25 2022, 10:51 AM.

Details

Summary

Relase target can be either revision, release, directory or content.
Add extra test data with real DB objects as release target.

Diff Detail

Repository
rDGQL GraphQL API
Branch
release-target-tests
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 31060
Build 48592: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 48591: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D8312 (id=30011)

Rebasing onto 6fb1913ff9...

Current branch diff-target is up to date.
Changes applied before test
commit 21c95068a2ef29e6eb93d68f80da525ad4540efc
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Thu Aug 25 10:43:29 2022 +0200

    Add functional tests for release target
    
    Relase target can be either revision, release, directory or content.
    Add extra test data with real DB objects as release target.

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

What would a query look like when the client wants the SWHID but does not know what type to expect? Could you add that in a test?

Client can request all the possible types in this case and the data requested under the right type will be returned.
https://forge.softwareheritage.org/D8312#216301:~:text=def%20test_get_release_target_unknown(client)%3A

Build is green

Patch application report for D8312 (id=30025)

Rebasing onto 527bf88bdb...

Current branch diff-target is up to date.
Changes applied before test
commit 40c0debad7b9973595fdccccb972c579540a7cbb
Author: Jayesh Velayudhan <jayesh@softwareheritage.org>
Date:   Thu Aug 25 10:43:29 2022 +0200

    Add functional tests for release target
    
    Relase target can be either revision, release, directory or content.
    Add extra test data with real DB objects as release target.

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

This revision is now accepted and ready to land.Aug 25 2022, 2:21 PM
This revision was automatically updated to reflect the committed changes.