Skip to content

fix: require auth in k6 baseline load test - #1729

Open
EmeditWeb wants to merge 3 commits into
LabsCrypt:mainfrom
EmeditWeb:fix/loadtest-auth-1500
Open

fix: require auth in k6 baseline load test#1729
EmeditWeb wants to merge 3 commits into
LabsCrypt:mainfrom
EmeditWeb:fix/loadtest-auth-1500

Conversation

@EmeditWeb

Copy link
Copy Markdown

Summary

  • require JWT auth before hitting protected load-test routes
  • fail the k6 run when checks do not pass
  • document required GitHub secret/variable configuration for the workflow

Changes

  • updated to send and optional , with explicit calls for missing credentials and failed checks
  • updated to pass , , and from GitHub secrets/vars and explain how to configure them

Verification

  • workflow yaml ok

Closes #1500

- backend/src/config/env.ts: remove trailing whitespace
- frontend/src/app/components/providers/WalletProvider.tsx: Prettier formatting
- frontend/src/app/hooks/useRepaymentOperation.ts: Prettier formatting
- contracts/loan_manager/src/events.rs: add blank line for doc continuation (clippy::doc_lazy_continuation)
… tests

- Fixed useRepaymentOperation.ts: Initialize queryClient with useQueryClient() hook
  Resolves TS2552 'Cannot find name queryClient' errors at lines 96, 97, 100, 103, 119

- Fixed envValidation.test.ts: Set all required env vars in beforeEach
  Ensures tests run with complete environment setup, prevents false failures
  when running in CI with minimal environment
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.

[Testing] k6 baseline load test hits authenticated endpoints with no credentials and never fails on check errors

1 participant