Refactor approvals_list_spec to not use `js-`
<!--IssueSummary start-->
<details>
<summary>
Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards.
</summary>
- [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=233537)
</details>
<!--IssueSummary end-->
`ee/spec/frontend/vue_mr_widget/components/approvals/approvals_list_spec.js`
Update to use the preferred way of querying DOM elements (from `js-` to `data` instead). Will also need to delete all the `js-` classes (but watch out, some are being used by feature specs, so we want to keep those)
https://docs.gitlab.com/ee/development/testing_guide/frontend_testing.html#how-to-query-dom-elements
> It is not recommended that you add .js-* classes just for testing purposes. Only do this if there are no other feasible options available.
## Note
- `js-commented-by` can be removed
- Please check if the other `js-` can also be safely removed
issue