@nejch ,
Can you check if this is functionally complete?
If it is, I'll start to add tests.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.53%. Comparing base (bcef988) to head (b57fbf2).

@@           Coverage Diff           @@
##             main    #2963   +/-   ##
=======================================
  Coverage   96.53%   96.53%           
=======================================
  Files          94       94           
  Lines        5997     5997           
=======================================
  Hits         5789     5789           
  Misses        208      208           

Thanks for the contribution @ka28kumar I just got back to this.

Yes, this looks good! If you could add a few basic tests/asserts to the existing functional tests, this would be really appreciated! Thanks!

Will add. Thanks!

Did you still want to add a few tests here @ka28kumar? Could just be a simple extension of:

E.g.:

def test_group_service_accounts(group):
    service_account = group.service_accounts.create(
        {"name": "gitlab-service-account", "username": "gitlab-service-account"}
    )
    assert service_account.name == "gitlab-service-account"
    assert service_account.username == "gitlab-service-account"

    service_accounts = group.service_accounts.list()
    assert service_accounts[0] == service_account

    service_account.delete()

Or something like that. Let me know otherwise we can also take over here!

I'm sorry I got occupied with some tasks. Feel free to take over.

Thanks @ka28kumar let's get this merged and I've opened a follow-up for the tests.