Make test pass

Author: cezudas

packages/openops/src/lib/openops-tables/openops-tables-common-properties.ts

@@ -40,10 +40,13 @@ export function openopsTablesDropdownProperty(): any {
 
 export async function getTableFields(
   tableName: string,
-  context: PropertyContext,
   axiosRetryConfig?: IAxiosRetryConfig,
 ): Promise<OpenOpsField[]> {
-  return await getTableFieldsFromContext(tableName, context, axiosRetryConfig);
+  const { token } = await authenticateDefaultUserInOpenOpsTables();
+
+  const tableId = await getTableIdByTableName(tableName as unknown as string);
+
+  return await getFields(tableId, token, false, axiosRetryConfig);
 }
 
 export async function getTableFieldsForMigration(

packages/openops/test/openops-tables/openops-tables-common-properties.test.ts

@@ -1,13 +1,25 @@
 const getTableNames = jest.fn();
 const getTableFieldsFromContextMock = jest.fn();
+const getTableIdByTableName = jest.fn();
+const getFields = jest.fn();
+const authenticateDefaultUserInOpenOpsTables = jest.fn();
+
 jest.mock('../../src/lib/openops-tables/tables', () => {
   return {
     getTableNames: getTableNames,
+    getTableIdByTableName: getTableIdByTableName,
   };
 });
 jest.mock('../../src/lib/openops-tables/fields', () => {
   return {
     getTableFieldsFromContext: getTableFieldsFromContextMock,
+    getFields: getFields,
+  };
+});
+jest.mock('../../src/lib/openops-tables/auth-user', () => {
+  return {
+    authenticateDefaultUserInOpenOpsTables:
+      authenticateDefaultUserInOpenOpsTables,
   };
 });
 
@@ -29,7 +41,7 @@ describe('table property', () => {
   test('should return dropdown with all tables', async () => {
     const mockContext = {
       server: { tablesDatabaseId: 1, tablesDatabaseToken: 'encrypted-token' },
-    } as any;
+    };
     getTableNames.mockResolvedValue(['table1', 'table2']);
 
     const result = await openopsTablesDropdownProperty().options(
@@ -50,7 +62,7 @@ describe('table property', () => {
   test('should handle empty tables', async () => {
     const mockContext = {
       server: { tablesDatabaseId: 1, tablesDatabaseToken: 'encrypted-token' },
-    } as any;
+    };
     getTableNames.mockResolvedValue([]);
 
     const result = await openopsTablesDropdownProperty().options(
@@ -72,10 +84,12 @@ describe('getTableFields', () => {
   });
 
   test('should return the options', async () => {
-    const mockContext = {
-      server: { tablesDatabaseId: 1, tablesDatabaseToken: 'encrypted-token' },
-    } as any;
-    getTableFieldsFromContextMock.mockResolvedValue([
+    authenticateDefaultUserInOpenOpsTables.mockResolvedValue({
+      token: 'mock-token',
+      refresh_token: 'mock-refresh-token',
+    });
+    getTableIdByTableName.mockResolvedValue(123);
+    getFields.mockResolvedValue([
       {
         name: 'field1',
         id: 1,
@@ -94,7 +108,7 @@ describe('getTableFields', () => {
       },
     ]);
 
-    const result = await getTableFields('Opportunity', mockContext);
+    const result = await getTableFields('Opportunity');
 
     expect(result).toMatchObject([
       {
@@ -114,35 +128,28 @@ describe('getTableFields', () => {
         type: 'equal',
       },
     ]);
-    expect(getTableFieldsFromContextMock).toHaveBeenCalledTimes(1);
-    expect(getTableFieldsFromContextMock).toHaveBeenCalledWith(
-      'Opportunity',
-      mockContext,
-      undefined,
-    );
+    expect(authenticateDefaultUserInOpenOpsTables).toHaveBeenCalledTimes(1);
+    expect(getTableIdByTableName).toHaveBeenCalledWith('Opportunity');
+    expect(getFields).toHaveBeenCalledWith(123, 'mock-token', false, undefined);
   });
 
   test('should handle empty fields', async () => {
-    const mockContext = {
-      server: { tablesDatabaseId: 1, tablesDatabaseToken: 'encrypted-token' },
-    } as any;
-    getTableFieldsFromContextMock.mockResolvedValue([]);
+    authenticateDefaultUserInOpenOpsTables.mockResolvedValue({
+      token: 'mock-token',
+      refresh_token: 'mock-refresh-token',
+    });
+    getTableIdByTableName.mockResolvedValue(123);
+    getFields.mockResolvedValue([]);
 
-    const result = await getTableFields('Opportunity', mockContext);
+    const result = await getTableFields('Opportunity');
 
     expect(result).toMatchObject([]);
-    expect(getTableFieldsFromContextMock).toHaveBeenCalledTimes(1);
-    expect(getTableFieldsFromContextMock).toHaveBeenCalledWith(
-      'Opportunity',
-      mockContext,
-      undefined,
-    );
+    expect(authenticateDefaultUserInOpenOpsTables).toHaveBeenCalledTimes(1);
+    expect(getTableIdByTableName).toHaveBeenCalledWith('Opportunity');
+    expect(getFields).toHaveBeenCalledWith(123, 'mock-token', false, undefined);
   });
 
   test('should return our supported properties for the field', async () => {
-    const mockContext = {
-      server: { tablesDatabaseId: 1, tablesDatabaseToken: 'encrypted-token' },
-    } as any;
     const numberField: NumberOpenOpsField = {
       name: 'field1',
       id: 1,
@@ -180,14 +187,19 @@ describe('getTableFields', () => {
       duration_format: 'format',
     };
 
-    getTableFieldsFromContextMock.mockResolvedValue([
+    authenticateDefaultUserInOpenOpsTables.mockResolvedValue({
+      token: 'mock-token',
+      refresh_token: 'mock-refresh-token',
+    });
+    getTableIdByTableName.mockResolvedValue(123);
+    getFields.mockResolvedValue([
       numberField,
       singleSelectField,
       multiSelectField,
       durationField,
     ]);
 
-    const result = await getTableFields('Opportunity', mockContext);
+    const result = await getTableFields('Opportunity');
 
     expect(result).toEqual([
       numberField,