Figure out usage of `user_count` v/s `total_quantity`
<!-- Please complete the template below as best as you can. Make sure to check if this issue has already been raised by someone else first to avoid duplication.
For each section below, please add screenshots or links or anything that may help visual learners understand the problem better, even if this takes you an extra minute or two this is a great help to some folks.
https://www.learning-styles-online.com/style/visual-spatial/
-->
### Problem
Currently we have 2 methods which are related and it is not clear which one should be used.
- [`user_count`](https://gitlab.com/gitlab-org/customers-gitlab-com/-/blob/64df4d88157e8d7ea455eb6a1b3529e1ee797144/app/models/subscription.rb#L487-489) : returns `main_product&.quantity`
- [`total_quantity`](https://gitlab.com/gitlab-org/customers-gitlab-com/-/blob/64df4d88157e8d7ea455eb6a1b3529e1ee797144/app/models/subscription.rb#L468-473) : loops through all the products that are `gitlab_plan?` and sums the quantity
### Proposal
Revisit both the above methods and check if we need these 2 implementations or if they can be unified.
issue