Summary
The following packages have no test files and lack unit test coverage. Adding tests for these packages would improve reliability and catch regressions.
Packages missing tests
| Package |
Description |
Priority |
shock-server/controller/node/acl |
ACL endpoint handlers |
High — complex auth logic |
shock-server/controller/node/index |
Index endpoint handlers |
Medium |
shock-server/controller/preauth |
Pre-auth endpoint handlers |
Medium |
shock-server/db |
MongoDB connectivity |
Medium — hard to unit test without mock |
shock-server/location |
Storage location management |
Medium |
shock-server/request |
Request parsing, auth helpers |
High — core request handling |
shock-server/types |
Type request handlers |
Low |
shock-server/versions |
Version info endpoint |
Low |
shock-client |
CLI client utilities |
Low |
Notes
shock-server (main package) is excluded — typically not unit tested
- Some packages (e.g.,
db, location) may require mock infrastructure from shock-server/test/
- Integration tests in
shock-server/integration/ cover some of these code paths indirectly (39 tests passing via Docker Compose)
- The
request and controller/node/acl packages are highest priority since they contain authentication and authorization logic
Current test coverage
- 28 packages have test files and pass locally
- 2 packages (
integration, node) require Docker infrastructure and pass in Docker Compose
- 9 packages listed above have no test files
References
Summary
The following packages have no test files and lack unit test coverage. Adding tests for these packages would improve reliability and catch regressions.
Packages missing tests
shock-server/controller/node/aclshock-server/controller/node/indexshock-server/controller/preauthshock-server/dbshock-server/locationshock-server/requestshock-server/typesshock-server/versionsshock-clientNotes
shock-server(main package) is excluded — typically not unit testeddb,location) may require mock infrastructure fromshock-server/test/shock-server/integration/cover some of these code paths indirectly (39 tests passing via Docker Compose)requestandcontroller/node/aclpackages are highest priority since they contain authentication and authorization logicCurrent test coverage
integration,node) require Docker infrastructure and pass in Docker ComposeReferences
feature/cache-s3-upload-ttlshock-server/test/(mock DB, mock FS, test utilities)