A test for this would be awesome. Maybe in the functional tests. For an example see:https://github.com/python-gitlab/python-gitlab/blob/master/tools/functional/api/test_projects.py#L190

Codecov Report

Merging#1437 (fdc46ba) intomaster (09ef8d4) willnot change coverage.
The diff coverage isn/a.

@@           Coverage Diff           @@##           master    #1437   +/-   ##=======================================  Coverage   80.24%   80.24%           =======================================  Files          73       73             Lines        4064     4064           =======================================  Hits         3261     3261             Misses        803      803

Flags with carried forward coverage won't be shown.Click here to find out more.

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update09ef8d4...fdc46ba. Read thecomment docs.

A test for this would be awesome. Maybe in the functional tests. For an example see:https://github.com/python-gitlab/python-gitlab/blob/master/tools/functional/api/test_projects.py#L190

@JohnVillalovos in this case I think we won't be able to test it in functional tests, as it's a premium feature and our CE docker container won't return the attribute.

Edit: I'll merge this for now John, as writing tests for all attributes would get tedious unless we need to test something specific around their behavior (like our list issue), but we should definitely think about how to maintain these hardcoded attributes better.

Since GitLab doesn't have an OpenAPI spec, maybe by periodically scanning the markdown tables from their docs for required/optional args for all endpoints and checking against our tuples, lol