Skip to content

Commit

Permalink
feature(remove-android): Remove codec tests (#6751)
Browse files Browse the repository at this point in the history
#### Details

Deprecating Android support will touch hundreds of files, so we're
breaking it up into more manageable units. This PR removes the files
under `src/tests/miscellaneous/codecs`. These tests were added as a
guard against accidentally shipping an Electron bundle that included
proprietary codecs. Now that we no longer ship Electron, we no longer
need the tests.

##### Motivation

Android deprecation feature

##### Context

<!-- Are there any parts that you've intentionally left out-of-scope for
a later PR to handle? -->

<!-- Were there any alternative approaches you considered? What
tradeoffs did you consider? -->


#### Pull request checklist
<!-- If a checklist item is not applicable to this change, write "n/a"
in the checkbox -->
- [n/a] Addresses an existing issue (part of Android deprecation
feature)
- [x] Ran `yarn fastpass`
- [x] Added/updated relevant unit test(s) (and ran `yarn test`)
- [x] Verified code coverage for the changes made. Check coverage report
at: `<rootDir>/test-results/unit/coverage`
- [x] PR title *AND* final merge commit title both start with a semantic
tag (`fix:`, `chore:`, `feat(feature-name):`, `refactor:`). See
`CONTRIBUTING.md`.
- [n/a] (UI changes only) Added screenshots/GIFs to description above
- [n/a] (UI changes only) Verified usability with NVDA/JAWS
  • Loading branch information
DaveTryon committed Jun 22, 2023
1 parent 8e47f95 commit c32c5dc
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 37 deletions.
17 changes: 0 additions & 17 deletions src/tests/miscellaneous/codecs/codecs-test.js

This file was deleted.

Binary file removed src/tests/miscellaneous/codecs/codecs-test.mp3
Binary file not shown.
20 changes: 0 additions & 20 deletions src/tests/miscellaneous/codecs/codecs.html

This file was deleted.

0 comments on commit c32c5dc

Please sign in to comment.