Cleanup

[fisx]

• some haddocks, formatting
• re-arrange helpers
• make API client functions in integration tests more polymorphic (needed for upcoming tests)
• none of these changes should change behavior
• i recomment reading commit by commit

[jschaul]

Failing test:

      GET, PUT [/i]?/teams/{tid}/legalhold - too large:

FAIL

        Exception: No code?

        CallStack (from HasCallStack):

          error, called at test/integration/API/Util.hs:404:40 in main:API.Util

          getInvitationCode, called at test/integration/API/Util.hs:337:23 in main:API.Util

          addUserToTeamWithRole', called at test/integration/API/Util.hs:320:18 in main:API.Util

          addUserToTeamWithRole, called at test/integration/API/Util.hs:313:17 in main:API.Util

          addUserToTeam, called at test/integration/API/Util.hs:109:12 in main:API.Util

          createBindingTeamWithMembers, called at test/integration/API/Teams/LegalHold.hs:536:29 in main:API.Teams.LegalHold

[fisx]

Pretty sure this is unrelated, but I've made an attempt at fixing it under the assumption it's a race condition: c331dea

[fisx]

(couldn't reproduce locally, btw)