test: Add additional cases for triggerAppConfigUpdate method

packages/backend/src/models/app-auth-client.test.js

@@ -160,21 +160,61 @@ describe('AppAuthClient model', () => {
     });
   });
 
-  it('triggerAppConfigUpdate should trigger an update in related app config', async () => {
+  describe('triggerAppConfigUpdate', () => {
-    await createAppConfig({ key: 'gitlab' });
+    it('should trigger an update in related app config', async () => {
+      await createAppConfig({ key: 'gitlab' });
 
-    const appAuthClient = await createAppAuthClient({
+      const appAuthClient = await createAppAuthClient({
-      appKey: 'gitlab',
+        appKey: 'gitlab',
+      });
+
+      const appConfigBeforeUpdateSpy = vi.spyOn(
+        AppConfig.prototype,
+        '$beforeUpdate'
+      );
+
+      await appAuthClient.triggerAppConfigUpdate();
+
+      expect(appConfigBeforeUpdateSpy).toHaveBeenCalledOnce();
     });
 
-    const appConfigBeforeUpdateSpy = vi.spyOn(
+    it('should update related AppConfig after creating an instance', async () => {
-      AppConfig.prototype,
+      const appConfig = await createAppConfig({
-      '$beforeUpdate'
+        key: 'gitlab',
-    );
+        disabled: false,
+        shared: true,
+      });
 
-    await appAuthClient.triggerAppConfigUpdate();
+      await createAppAuthClient({
+        appKey: 'gitlab',
+        active: true,
+      });
 
-    expect(appConfigBeforeUpdateSpy).toHaveBeenCalledOnce();
+      const refetchedAppConfig = await appConfig.$query();
+
+      expect(refetchedAppConfig.connectionAllowed).toBe(true);
+    });
+
+    it('should update related AppConfig after updating an instance', async () => {
+      const appConfig = await createAppConfig({
+        key: 'gitlab',
+        disabled: false,
+        shared: true,
+      });
+
+      const appAuthClient = await createAppAuthClient({
+        appKey: 'gitlab',
+        active: false,
+      });
+
+      let refetchedAppConfig = await appConfig.$query();
+      expect(refetchedAppConfig.connectionAllowed).toBe(false);
+
+      await appAuthClient.$query().patchAndFetch({ active: true });
+
+      refetchedAppConfig = await appConfig.$query();
+      expect(refetchedAppConfig.connectionAllowed).toBe(true);
+    });
   });
 
   it('$beforeInsert should call AppAuthClient.encryptData', async () => {