Fix fetching more than 100 branches on GitLab.

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

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.

Summary ID
Fix fetching more than 100 branches no GitLab.
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.
0ec178bf35430b9ff1f9afdbc75bd03460e761c2
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (884b482)
Loading...