Skip to content

Commit d477188

Browse files
committed
test fixes
1 parent bf30cc0 commit d477188

File tree

10 files changed

+35
-33
lines changed

10 files changed

+35
-33
lines changed

src/common/net/http-client.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,10 @@ export abstract class HttpClient implements HttpClientInterface {
1212
readonly MINIMUM_SLEEP_TIME_IN_MILLISECONDS = 500;
1313
readonly RETRY_STATUS_CODES = [500, 502, 504];
1414

15-
constructor(readonly baseURL: string, readonly options?: RequestInit) {}
15+
constructor(
16+
readonly baseURL: string,
17+
readonly options?: RequestInit,
18+
) {}
1619

1720
/** The HTTP client name used for diagnostics */
1821
getClientName(): string {

src/common/utils/pagination.spec.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ describe('AutoPaginatable', () => {
6464
const resultPromise = paginatable.autoPagination();
6565

6666
// Fast-forward through setTimeout calls
67-
jest.advanceTimersByTimeAsync(250);
67+
await jest.advanceTimersByTimeAsync(350);
6868

6969
const result = await resultPromise;
7070

@@ -114,10 +114,10 @@ describe('AutoPaginatable', () => {
114114

115115
const resultPromise = paginatable.autoPagination();
116116

117-
jest.advanceTimersByTimeAsync(250);
117+
await jest.advanceTimersByTimeAsync(700); // 350ms * 2 for two pages
118118
await resultPromise;
119119

120-
expect(setTimeoutSpy).toHaveBeenCalledWith(expect.any(Function), 250);
120+
expect(setTimeoutSpy).toHaveBeenCalledWith(expect.any(Function), 350);
121121
});
122122

123123
it('passes through additional options to API calls', async () => {

src/directory-sync/directory-sync.spec.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -469,9 +469,8 @@ describe('DirectorySync', () => {
469469
it(`requests a Directory User`, async () => {
470470
fetchOnce(userWithRoleResponse);
471471

472-
const subject = await workos.directorySync.getUser(
473-
'directory_user_456',
474-
);
472+
const subject =
473+
await workos.directorySync.getUser('directory_user_456');
475474

476475
expect(subject).toEqual(userWithRole);
477476
});

src/fga/serializers/check-options.serializer.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,8 @@ const serializeCheckWarrantOptions = (
3636
resource_id: isResourceInterface(warrant.resource)
3737
? warrant.resource.getResourceId()
3838
: warrant.resource.resourceId
39-
? warrant.resource.resourceId
40-
: '',
39+
? warrant.resource.resourceId
40+
: '',
4141
relation: warrant.relation,
4242
subject: isSubject(warrant.subject)
4343
? {

src/fga/serializers/create-resource-options.serializer.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ export const serializeCreateResourceOptions = (
1313
resource_id: isResourceInterface(options.resource)
1414
? options.resource.getResourceId()
1515
: options.resource.resourceId
16-
? options.resource.resourceId
17-
: '',
16+
? options.resource.resourceId
17+
: '',
1818
meta: options.meta,
1919
});

src/fga/serializers/delete-resource-options.serializer.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,6 @@ export const serializeDeleteResourceOptions = (
1313
resource_id: isResourceInterface(options)
1414
? options.getResourceId()
1515
: options.resourceId
16-
? options.resourceId
17-
: '',
16+
? options.resourceId
17+
: '',
1818
});

src/fga/serializers/write-warrant-options.serializer.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ export const serializeWriteWarrantOptions = (
1414
resource_id: isResourceInterface(warrant.resource)
1515
? warrant.resource.getResourceId()
1616
: warrant.resource.resourceId
17-
? warrant.resource.resourceId
18-
: '',
17+
? warrant.resource.resourceId
18+
: '',
1919
relation: warrant.relation,
2020
subject: isSubject(warrant.subject)
2121
? {

src/organizations/organizations.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,9 @@ export class Organizations {
113113

114114
async listOrganizationFeatureFlags(
115115
options: ListOrganizationFeatureFlagsOptions,
116-
): Promise<AutoPaginatable<FeatureFlag>> {
116+
): Promise<
117+
AutoPaginatable<FeatureFlag, ListOrganizationFeatureFlagsOptions>
118+
> {
117119
const { organizationId, ...paginationOptions } = options;
118120

119121
return new AutoPaginatable(
@@ -130,7 +132,7 @@ export class Organizations {
130132
deserializeFeatureFlag,
131133
params,
132134
),
133-
paginationOptions,
135+
options,
134136
);
135137
}
136138
}

src/user-management/user-management.spec.ts

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1205,9 +1205,8 @@ describe('UserManagement', () => {
12051205
describe('getPasswordReset', () => {
12061206
it('sends a Get PaswordReset request', async () => {
12071207
fetchOnce(passwordResetFixture);
1208-
const passwordReset = await workos.userManagement.getPasswordReset(
1209-
passwordResetId,
1210-
);
1208+
const passwordReset =
1209+
await workos.userManagement.getPasswordReset(passwordResetId);
12111210
expect(fetchURL()).toContain(
12121211
`/user_management/password_reset/${passwordResetId}`,
12131212
);
@@ -1695,9 +1694,8 @@ describe('UserManagement', () => {
16951694
describe('getInvitation', () => {
16961695
it('sends a Get Invitation request', async () => {
16971696
fetchOnce(invitationFixture);
1698-
const invitation = await workos.userManagement.getInvitation(
1699-
invitationId,
1700-
);
1697+
const invitation =
1698+
await workos.userManagement.getInvitation(invitationId);
17011699
expect(fetchURL()).toContain(
17021700
`/user_management/invitations/${invitationId}`,
17031701
);
@@ -1711,9 +1709,8 @@ describe('UserManagement', () => {
17111709
describe('findInvitationByToken', () => {
17121710
it('sends a find invitation by token request', async () => {
17131711
fetchOnce(invitationFixture);
1714-
const invitation = await workos.userManagement.findInvitationByToken(
1715-
invitationToken,
1716-
);
1712+
const invitation =
1713+
await workos.userManagement.findInvitationByToken(invitationToken);
17171714
expect(fetchURL()).toContain(
17181715
`/user_management/invitations/by_token/${invitationToken}`,
17191716
);
@@ -1812,9 +1809,8 @@ describe('UserManagement', () => {
18121809
accepted_user_id: 'user_01HGK4K4PXNSG85RNNV0GXYP5W',
18131810
});
18141811

1815-
const response = await workos.userManagement.acceptInvitation(
1816-
invitationId,
1817-
);
1812+
const response =
1813+
await workos.userManagement.acceptInvitation(invitationId);
18181814

18191815
expect(fetchURL()).toContain(
18201816
`/user_management/invitations/${invitationId}/accept`,
@@ -1833,9 +1829,8 @@ describe('UserManagement', () => {
18331829
const invitationId = 'invitation_01H5JQDV7R7ATEYZDEG0W5PRYS';
18341830
fetchOnce(invitationFixture);
18351831

1836-
const response = await workos.userManagement.revokeInvitation(
1837-
invitationId,
1838-
);
1832+
const response =
1833+
await workos.userManagement.revokeInvitation(invitationId);
18391834

18401835
expect(fetchURL()).toContain(
18411836
`/user_management/invitations/${invitationId}/revoke`,

src/workos.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,10 @@ export class WorkOS {
6969
readonly widgets = new Widgets(this);
7070
readonly vault = new Vault(this);
7171

72-
constructor(readonly key?: string, readonly options: WorkOSOptions = {}) {
72+
constructor(
73+
readonly key?: string,
74+
readonly options: WorkOSOptions = {},
75+
) {
7376
if (!key) {
7477
this.key = getEnv('WORKOS_API_KEY');
7578

0 commit comments

Comments
 (0)