feat(api): add merge request approval state - #1641

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.93%. Comparing base (5a1678f) to head (f41b093).
⚠️ Report is 1518 commits behind head on main.

@@            Coverage Diff             @@
##             main    #1641      +/-   ##
==========================================
+ Coverage   91.90%   91.93%   +0.02%     
==========================================
  Files          74       74              
  Lines        4287     4302      +15     
==========================================
+ Hits         3940     3955      +15     
  Misses        347      347              

The CI failure appears to be unrelated to this change?

Thanks for the PR. It would be great if there can be some tests for this, assuming it is a "Free Tier" feature.

Thanks for the PR. It would be great if there can be some tests for this, assuming it is a "Free Tier" feature.

It's a premium feature since 13.9. I was prepared to add unit tests, but I noticed that "merge request approvals" doesn't have any to begin with.

Thanks for the PR. It would be great if there can be some tests for this, assuming it is a "Free Tier" feature.

It's a premium feature since 13.9. I was prepared to add unit tests, but I noticed that "merge request approvals" doesn't have any to begin with.

Hi @JacobHenner, thanks for this! As @JohnVillalovos said, we won't be able to add functional tests without a license but we can have some unit tests maybe.

Maybe in https://github.com/python-gitlab/python-gitlab/blob/main/tests/unit/objects/test_project_merge_request_approvals.py?
Just a simple get() call and mock response to make sure we keep sending the right request. Thanks again for the super clean PR!