Skip to content

Commit 0d0d59e

Browse files
authored
Merge pull request #4334 from cloud-gov/4333-improve-brittle-admin-user-api-test
fix: Improve brittle admin user API test
2 parents 121bca9 + 994521e commit 0d0d59e

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

test/api/admin/requests/user.test.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ describe('Admin - Users API', () => {
2626
Organization.truncate({ force: true, cascade: true }),
2727
OrganizationRole.truncate({ force: true, cascade: true }),
2828
User.truncate({ force: true, cascade: true }),
29+
UAAIdentity.truncate({ force: true, cascade: true }),
2930
])
3031
);
3132

@@ -72,10 +73,10 @@ describe('Admin - Users API', () => {
7273

7374
it('returns all users', async () => {
7475
const user1 = await factory.user();
75-
createUAAIdentity({uaaId: 'user_id_1', email: 'user1@example.com', userId: user1.id });
76+
await createUAAIdentity({uaaId: 'user_id_1', email: 'user1@example.com', userId: user1.id });
7677

7778
const user2 = await factory.user();
78-
createUAAIdentity({uaaId: 'user_id_2', email: 'user2@example.com', userId: user2.id });
79+
await createUAAIdentity({uaaId: 'user_id_2', email: 'user2@example.com', userId: user2.id });
7980

8081
const org1 = await factory.organization.create();
8182
const org2 = await factory.organization.create();
@@ -120,10 +121,10 @@ describe('Admin - Users API', () => {
120121

121122
it('returns all users with UAA identities', async () => {
122123
const user1 = await factory.user();
123-
createUAAIdentity({uaaId: 'user_id_1', email: 'user1@example.com', userId: user1.id });
124+
await createUAAIdentity({uaaId: 'user_id_1', email: 'user1@example.com', userId: user1.id });
124125

125126
const user2 = await factory.user();
126-
createUAAIdentity({uaaId: 'user_id_2', email: 'user2@example.com', userId: user2.id });
127+
await createUAAIdentity({uaaId: 'user_id_2', email: 'user2@example.com', userId: user2.id });
127128

128129
const user3 = await factory.user();
129130
const user4 = await factory.user();
@@ -165,10 +166,10 @@ describe('Admin - Users API', () => {
165166

166167
it('returns all users with UAA identities', async () => {
167168
const user1 = await factory.user();
168-
createUAAIdentity({uaaId: 'user_id_1', email: 'user1@example.com', userId: user1.id });
169+
await createUAAIdentity({uaaId: 'user_id_1', email: 'user1@example.com', userId: user1.id });
169170

170171
const user2 = await factory.user();
171-
createUAAIdentity({uaaId: 'user_id_2', email: 'user2@example.com', userId: user2.id });
172+
await createUAAIdentity({uaaId: 'user_id_2', email: 'user2@example.com', userId: user2.id });
172173

173174
const user3 = await factory.user();
174175
const user4 = await factory.user();

0 commit comments

Comments
 (0)