fix(SignalR): ensure access token factory is always called and improve error handling in tests

Author: daniloneto

src/SignalR/clients/ts/signalr/src/AccessTokenHttpClient.ts

@@ -65,7 +65,7 @@ export class AccessTokenHttpClient extends HttpClient {
     }
 
     private async _refreshTokenAndRetry(request: HttpRequest, original: HttpResponse | Error): Promise<HttpResponse> {
-        const newToken = await this._accessTokenFactory();
+        const newToken = await this._accessTokenFactory!();
         if (!newToken) {
             if (original instanceof HttpResponse) {
                 return original;

src/SignalR/clients/ts/signalr/tests/AccessTokenHttpClient.test.ts

@@ -72,8 +72,7 @@ describe("AccessTokenHttpClient", () => {
 
             await client.get("http://example.com/prime");
             try {
-                await client.get("http://example.com/resource");
-                expect.fail("expected to throw");
+                await expect(client.get("http://example.com/resource")).rejects.toThrow(HttpError);
             } catch (e: any) {
                 expect(e).toBeInstanceOf(HttpError);
                 expect(e.statusCode ?? e.status).toBe(status);