Skip to content

Commit 4f1dc04

Browse files
Dhoni77d-goog
andauthored
refactor!: remove DEFAULT_UNIVERSE from BaseExternalClient (#1690)
* chore: remove DEFAULT_UNIVERSE from BaseExternalClient * chore: update unit tests * chore: fix formatting --------- Co-authored-by: Daniel Bankhead <[email protected]>
1 parent 659496b commit 4f1dc04

3 files changed

+3
-11
lines changed

src/auth/baseexternalclient.ts

-5
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,6 @@ const DEFAULT_TOKEN_URL = 'https://sts.{universeDomain}/v1/token';
7373
// eslint-disable-next-line @typescript-eslint/no-var-requires
7474
const pkg = require('../../../package.json');
7575

76-
/**
77-
* For backwards compatibility.
78-
*/
79-
export {DEFAULT_UNIVERSE} from './authclient';
80-
8176
/**
8277
* Shared options used to build {@link ExternalAccountClient} and
8378
* {@link ExternalAccountAuthorizedUserClient}.

test/test.baseexternalclient.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ import {
2323
EXPIRATION_TIME_OFFSET,
2424
BaseExternalAccountClient,
2525
BaseExternalAccountClientOptions,
26-
DEFAULT_UNIVERSE,
2726
} from '../src/auth/baseexternalclient';
2827
import {
2928
OAuthErrorResponse,
@@ -40,7 +39,7 @@ import {
4039
mockStsTokenExchange,
4140
getExpectedExternalAccountMetricsHeaderValue,
4241
} from './externalclienthelper';
43-
import {AuthClientOptions} from '../src/auth/authclient';
42+
import {AuthClientOptions, DEFAULT_UNIVERSE} from '../src/auth/authclient';
4443

4544
nock.disableNetConnect();
4645

test/test.externalaccountauthorizeduserclient.ts

+2-4
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,13 @@ import {
2323
ExternalAccountAuthorizedUserClient,
2424
ExternalAccountAuthorizedUserClientOptions,
2525
} from '../src/auth/externalAccountAuthorizedUserClient';
26-
import {
27-
DEFAULT_UNIVERSE,
28-
EXPIRATION_TIME_OFFSET,
29-
} from '../src/auth/baseexternalclient';
26+
import {EXPIRATION_TIME_OFFSET} from '../src/auth/baseexternalclient';
3027
import {GaxiosError, GaxiosResponse} from 'gaxios';
3128
import {
3229
getErrorFromOAuthErrorResponse,
3330
OAuthErrorResponse,
3431
} from '../src/auth/oauth2common';
32+
import {DEFAULT_UNIVERSE} from '../src/auth/authclient';
3533

3634
nock.disableNetConnect();
3735

0 commit comments

Comments
 (0)