The following discussion from !159557 should be addressed: - [ ] @atevans started a [discussion](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/159557#note_2016763089): > **note:** (non-blocking) I feel like this is mocked out to the point where we're testing that 1 = 1, but I see it's following the pattern from [the can_modify_blob specs](https://gitlab.com/gitlab-org/gitlab/-/blob/acb0bd9e0987824710688633817990793e34af5d/spec/presenters/blob_presenter_spec.rb#L99). I'd feel more comfortable if we had an integration spec for these fields without mocking, like in [the GraphQL blobs request specs](https://gitlab.com/gitlab-org/gitlab/-/blob/master/spec/requests/api/graphql/project/repository/blobs_spec.rb)