Skip to content

Fix test configuration for auth tests#232

Open
IanMayo wants to merge 3 commits intomainfrom
hotfix-test-fixes
Open

Fix test configuration for auth tests#232
IanMayo wants to merge 3 commits intomainfrom
hotfix-test-fixes

Conversation

@IanMayo
Copy link
Collaborator

@IanMayo IanMayo commented Nov 30, 2025

Summary

  • Enable auth mode in test configuration with required env vars (AUTH, TOKEN_SECRET, initial user credentials)
  • Add --forceExit to Jest for clean test exit
  • Add .unref() to token cleanup interval to prevent blocking process exit
  • Bump version to 0.8.3 (also syncs swagger.json which was at 0.8.1)

Test plan

  • All 35 unit tests pass
  • No worker process exit warnings

Add AUTH, TOKEN_SECRET, and initial user credentials to test script
so auth-related tests can run properly. Add --forceExit for clean exit.
Prevents the setInterval from blocking process exit during tests.
@thevahidal
Copy link
Owner

thevahidal commented Dec 6, 2025

Hi @IanMayo, Thanks for the PR.
I reviewed these changes in the previous PR, should we merge them separately or just merge the previous one and close this?

@IanMayo
Copy link
Collaborator Author

IanMayo commented Dec 6, 2025

Hello @thevahidal - I saw this as a separate issue, so it seemed tidier to fix this on main first, then the second one just relates to the sqlite dependency.

@thevahidal
Copy link
Owner

I agree @IanMayo, then please apply the comments I asked on the other one, onto this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants