refactor: changed data-cy to data-test-id

[deltork]

PR Goal?

Refactoring in anticipation of adding tests to studio

Fixes?

n/a

Feedback sought?

sanity check

Priority?

blocking

Tests added?

n/a

How to test?

Run web component tests npx nx run-many --targets=serve-test-data,build,serve,test:once --projects=web-component

Confidence?

High

Version change?

refactor

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
package-lock.json	71% smaller
packages/web-component/cypress/e2e/missing.assets.feedback.cy.js	1% smaller
packages/web-component/cypress/e2e/embedded.cy.js	1% smaller
packages/web-component/cypress/e2e/annotated.cy.js	0% smaller
packages/web-component/cypress/e2e/basic.cy.js	0% smaller
packages/web-component/cypress/e2e/customizations.cy.js	0% smaller
packages/web-component/cypress/e2e/edit.cy.js	0% smaller
packages/web-component/cypress/e2e/settings.cy.js	0% smaller
packages/web-component/cypress/e2e/translated.cy.js	0% smaller
packages/web-component/cypress/support/commands.js	0% smaller
packages/web-component/src/components/read-along-component/read-along.tsx	0% smaller

[github-actions]

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-10-31 19:49 UTC

[joanise]

lgtm

[joanise]

Actually, I approved this, but can you change the commit message? It should be a refactor: changed data-cy to data-test-id, not chore: ...

[joanise]

We tend to overuse chore:, but in this case it really doesn't apply.