cicorias / cicorias/jwtproxy

rewrite mocha tests given concurrency issues

Open
#24 0 comments 0 reactions 0 assignees View on GitHub
bug enhancement
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.