fix(GithubMultiOrgEntityProvider): add missing await
Signed-off-by: Phil Kuang <pkuang@factset.com>
This commit is contained in:
@@ -0,0 +1,5 @@
|
||||
---
|
||||
'@backstage/plugin-catalog-backend-module-github': patch
|
||||
---
|
||||
|
||||
Properly handle `async` event callback
|
||||
-2
@@ -1694,7 +1694,6 @@ describe('GithubMultiOrgEntityProvider', () => {
|
||||
},
|
||||
},
|
||||
});
|
||||
await new Promise(process.nextTick);
|
||||
|
||||
expect(entityProviderConnection.applyMutation).toHaveBeenCalledTimes(1);
|
||||
expect(entityProviderConnection.applyMutation).toHaveBeenCalledWith({
|
||||
@@ -1867,7 +1866,6 @@ describe('GithubMultiOrgEntityProvider', () => {
|
||||
},
|
||||
},
|
||||
});
|
||||
await new Promise(process.nextTick);
|
||||
|
||||
expect(entityProviderConnection.applyMutation).toHaveBeenCalledTimes(1);
|
||||
expect(entityProviderConnection.applyMutation).toHaveBeenCalledWith({
|
||||
|
||||
@@ -388,7 +388,10 @@ export class GithubMultiOrgEntityProvider implements EntityProvider {
|
||||
|
||||
// https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#membership
|
||||
if (params.topic.includes('membership')) {
|
||||
this.onMembershipChangedInTeam(eventPayload as MembershipEvent, orgs);
|
||||
await this.onMembershipChangedInTeam(
|
||||
eventPayload as MembershipEvent,
|
||||
orgs,
|
||||
);
|
||||
}
|
||||
|
||||
return;
|
||||
|
||||
Reference in New Issue
Block a user