learningequality / learningequality/studio
`DetailsTabView`: Investigate and fix skipped test suite
Open
Nobody has claimed this yet.
P2 - normal
TAG: tech update / debt
TAG: unit tests
- Dominant language
- Python
- Stars
- 191
- Forks
- 307
- Avg merge
- 5d 6h
- Merged PRs (30d)
- 10
Description
Observed behavior
The tests in DetailsTabView are currently being skipped.
Expected behavior
The entire test suite is skipped - see here
User-facing consequences
Potential edge-case bugs may come through in spite of QA testing.
Additional comments
Once this is fixed, the additional metadata fields that have been added will also need to be added to the test suite.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at contentcuration/contentcuration/frontend/channelEdit/components/edit/tests/detailsTabView.spec.js, especially the skipped suite near line 52. Run the DetailsTabView tests and inspect why the suite is skipped. Done means the suite runs successfully and covers the additional metadata fields mentioned in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend, testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100