test: correct onRefresh signature in MockAuthClient test-utils

[Copilot]

Affected Package(s)

• @forward-software/react-auth (lib)
• @forward-software/react-auth-google (packages/google-signin)
• Examples
• CI/CD / Repository configuration

Related Issue(s)

Motivation

MockAuthClient and MockAuthClientWithHooks in lib/test/test-utils.tsx declared onRefresh without the required currentTokens first parameter, diverging silently from the AuthClient interface. Anyone using these mocks as a reference would learn the wrong contract.

Description of Changes

• Fixed onRefresh signature in MockAuthClient to match the AuthClient interface:

// Before (wrong — missing currentTokens)
onRefresh(_minValidity?: number): Promise<MockTokens>

// After (correct)
onRefresh(_currentTokens: MockTokens, _minValidity?: number): Promise<MockTokens>

• Applied the same fix to MockAuthClientWithHooks, which extended MockAuthClient but overrode onRefresh with the same incorrect signature.

Breaking Changes

None

How to Test

1. CI Checks: Verify that all automated tests (Vitest) and build steps pass successfully on this PR.
2. Local Verification (Optional):
   • Run pnpm install to install dependencies.
   • Run pnpm --filter @forward-software/react-auth test to run tests for the affected package.
   • Run pnpm --filter @forward-software/react-auth build to verify the build succeeds.
   • Run pnpm --filter @forward-software/react-auth lint to check for linting errors.

Checklist

• My code follows the project's style guidelines
• I have added or updated tests to cover the changes
• I have updated relevant documentation
• All tests are passing locally
• CI checks are passing
• I have reviewed my own code and lock file changes
• I have checked for any potential security implications
• I have verified the changes work as expected
• My commit messages follow Conventional Commits format

Notes for Reviewers

No behavioural change — this is a pure signature correction in test utilities. The actual onRefresh call in AuthClientEnhancements already passes this.tokens as the first argument (auth.tsx:385); the mocks were just misrepresenting the contract.

[Copilot]

Pull request overview

Aligns the MockAuthClient / MockAuthClientWithHooks test utilities with the real AuthClient contract so the mocks accurately reflect how wrapAuthClient invokes onRefresh.

Changes:

• Corrected MockAuthClient.onRefresh to accept (currentTokens, minValidity?).
• Applied the same onRefresh signature correction to MockAuthClientWithHooks.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.