rewrite mocha tests given concurrency issues
- Dominant language
- TypeScript
- Stars
- 3
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
tests have been problematic with phantom caching of objects between tests that should be torn down. Running all tests individual as an example gives us the 401 and 403 in the right spots. however, run the full suite remnant 403's fall into code paths werhere 401 is expected. and vica versa.
So, plan is to remove supertest and rewrite/cleanup the tests as needed.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by running the tests individually and as the full suite to reproduce the incorrect 401 and 403 paths, then inspect the existing Mocha tests and their Supertest usage. Review the test teardown and caching behavior; done means the tests no longer depend on shared state and the full suite reports 401 and 403 responses in the expected cases.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- testing-qa
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100