test(auth): address review comments on JwtValidator unit tests

[Copilot]

Cleans up packages/auth/src/jwt.test.ts based on PR review feedback.

Changes

• Remove unused import: Drop type Mock from the vitest import line
• Add file header: Add /** Tests for @orch/auth/jwt — JwtValidator unit tests */ per project convention
• Eliminate redundant validate() calls: Refactor "missing issuer" and "provider not found" tests to call validate() once, capture the error, and assert all properties on it — instead of calling it 2–3 times with separate expect(...).rejects chains
• Fix indentation: Align the rethrow OrchestratorError test body to 4-space indent, consistent with the rest of the file

Before (redundant calls):

await expect(validator.validate(validToken)).rejects.toThrow(OrchestratorError);
await expect(validator.validate(validToken)).rejects.toThrow('JWT missing required "iss" claim');
try {
    await validator.validate(validToken); // called a 3rd time just to assert .code
} catch (err: any) {
    expect(err.code).toBe(ErrorCode.TOKEN_INVALID);
}

After (single call):

let error: any;
try {
    await validator.validate(validToken);
} catch (err: any) {
    error = err;
}
expect(error).toBeInstanceOf(OrchestratorError);
expect(error.message).toContain('JWT missing required "iss" claim');
expect(error.code).toBe(ErrorCode.TOKEN_INVALID);

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.