• 
      

    Fix fetching more than 100 branches on GitLab.

    Review Request #11016 — Created May 7, 2020 and submitted — Latest diff uploaded

    Information

    Review Board
    release-3.0.x

    Reviewers

    The GitLab API v4 support wasn't factoring in pagination for the
    branches API (or the groups API, but that's currently only used on v3).
    This means that if more than 100 branches are defined, only those first
    100 will be shown in the New Review Request page.

    This switches our implementation to use the standard pagination support.

    It also cleans up the cod around checking the Link header to meet our
    coding standards. Note that capitalization has changed for the Link
    key in this code, but in practice the underlying code normalizes key
    lookups, so this doesn't change any logic. It just makes it easier to
    test against.

    Unit tests have been added to check that paginated branches work
    correctly.

    Unit tests pass.

    Commits

    Files