fix(btp-utils): return oauth destination

SAP · Jan 9, 2025 · d976179 · d976179
1 parent 9c8bd0b
commit d976179
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 11 deletions.
diff --git a/packages/btp-utils/src/app-studio.ts b/packages/btp-utils/src/app-studio.ts
@@ -238,13 +238,18 @@ async function generateOAuthTokenExchangeDestination(
  *
  * @param destination destination info
  * @param logger Logger
+ * @returns Newly generated OAuth destination
  */
-export async function createBTPOAuthExchangeDestination(destination: Destination, logger?: Logger): Promise<void> {
+export async function createBTPOAuthExchangeDestination(
+    destination: Destination,
+    logger?: Logger
+): Promise<OAuth2Destination> {
     if (!isAppStudio()) {
         throw new Error(`Creating SAP BTP destinations is only supported on SAP Business Application Studio.`);
     }
     const btpDestination = await generateOAuthTokenExchangeDestination(destination, logger);
     await createBTPDestination(btpDestination);
+    return btpDestination;
 }
 
 /**

diff --git a/packages/btp-utils/test/app-studio.test.ts b/packages/btp-utils/test/app-studio.test.ts
@@ -238,15 +238,7 @@ describe('App Studio', () => {
 
         test('generate new SAP BTP destination', async () => {
             process.env[ENV.H2O_URL] = server;
-            let bodyParam;
-            nock(server)
-                .post('/api/createDestination', (body) => {
-                    bodyParam = body;
-                    return true;
-                })
-                .reply(200);
-            await expect(createBTPOAuthExchangeDestination(destination)).resolves.toBeUndefined();
-            expect(bodyParam).toMatchInlineSnapshot(`
+            const result = `
                 Object {
                   "Authentication": "OAuth2UserTokenExchange",
                   "Description": "Destination generated by App Studio for 'my-abap-env', Do not remove.",
@@ -263,7 +255,16 @@ describe('App Studio', () => {
                   "tokenServiceURL": "http://my-server/oauth/token",
                   "tokenServiceURLType": "Dedicated",
                 }
-            `);
+            `;
+            let bodyParam;
+            nock(server)
+                .post('/api/createDestination', (body) => {
+                    bodyParam = body;
+                    return true;
+                })
+                .reply(200);
+            await expect(createBTPOAuthExchangeDestination(destination)).resolves.toMatchInlineSnapshot(result);
+            expect(bodyParam).toMatchInlineSnapshot(result);
         });
 
         test('throw exception if no UAA credentials found', async () => {