Page MenuHomeSoftware Heritage

Refactor swh.lister.github.utils to swh.core.github.utils
ClosedPublic

Authored by ardumont on May 17 2022, 10:22 AM.

Details

Summary

This also:

  • extracts pytest fixtures needed to cover the GitHubSession code.
  • avoid mutating the inputted credentials passed in constructors
  • rework test docstrings to "it should" format for more concise description

Related to T4232

TODO in other diffs:

  • D7840: Move pytest fixtures into a runtime pytest module (to be reusable in swh.lister)
  • D7870: Branch the use of GitHubSession in the new function to compute the canonical github urls

Diff Detail

Repository
rDCORE Foundations and core functionalities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D7840 (id=28317)

Rebasing onto 60f384d35a...

Current branch diff-target is up to date.
Changes applied before test
commit 6fa229792d57cc0b445293ade8699d06074cd8b9
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon May 16 16:59:04 2022 +0200

    Refactor swh.lister.github.utils to swh.core.github.utils
    
    This also:
    - extracts pytest fixtures needed to cover the GitHubSession code.
    - avoid mutating the inputted credentials passed in constructors
    - rework test docstrings to "it should" format for more concise description
    
    Related to T4232

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

anlambert added a subscriber: anlambert.

Looks good to me.

This revision is now accepted and ready to land.May 17 2022, 11:06 AM