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