The following discussions from gitlab-ce!30864 should be addressed: - [x] @alexkalderimis started a [discussion](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/30864#note_193852471): > This test became redundant since what it is trying to check is now subsumed in the `web_ide_counter_spec.rb` - i.e. that each total is correctly called and reported. > > Ideally this should be tested at the counter level, and we shouldn't leak internal details into this class. - [ ] @fjsanpedro started a [discussion](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/30864#note_193864039): (+1 comment) > @alexkalderimis I guess we should create a method `totals` in `RedisCounter` since now that method is shared among all the counters, and each one of them should implement it. > > WDYT? Basically we ought to decide what the interface of usage counters should be, and how they should be implemented in the future.