Skip to content

test: add missing test coverage for admin queries and edge cases#498

Merged
lakhansamani merged 2 commits intomainfrom
test/add-missing-test-coverage
Mar 1, 2026
Merged

test: add missing test coverage for admin queries and edge cases#498
lakhansamani merged 2 commits intomainfrom
test/add-missing-test-coverage

Conversation

@lakhansamani
Copy link
Contributor

Summary

  • Added 8 new test files for untested admin queries (meta, users, user, admin_session, verification_requests, webhooks, webhook_logs, email_templates)
  • Added edge-case tests to existing test files:
    • signup_test.go: Signup when EnableSignup=false should fail
    • login_test.go: Login with EnableBasicAuthentication=false, login with revoked user
    • reset_password_test.go: Password mismatch validation
    • update_profile_test.go: Password change flow (old_password, new_password, confirm_new_password)

Test plan

  • Run go vet ./internal/integration_tests/... - passes
  • Run TEST_DBS="sqlite" go test -p 1 -v ./internal/integration_tests/ for full validation

New test files:
- meta_test.go: Test meta query with different config flags
- users_test.go: Test _users admin query with/without auth, pagination
- user_test.go: Test _user admin query by ID, by email, unauthorized
- admin_session_test.go: Test _admin_session query
- verification_requests_test.go: Test _verification_requests admin query
- webhooks_test.go: Test _webhooks list and _webhook single queries
- webhook_logs_test.go: Test _webhook_logs query
- email_templates_test.go: Test _email_templates query

Edge-case tests added to existing files:
- signup_test.go: Test signup when EnableSignup=false
- login_test.go: Test login with basic auth disabled, login with revoked user
- reset_password_test.go: Test password mismatch
- update_profile_test.go: Test password change flow
@lakhansamani lakhansamani merged commit ebefe08 into main Mar 1, 2026
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.

1 participant